Using search in react js

Question

I want to filter content based on the input typed, I don't know where did I go wrong. Whenever I typed my input it just doesn't filter the content instead of that it is showing all the content there.

  function searchingFor(searchingTerm) {
        return function(x){
            console.log("searching",x);
            return x.name.toLowerCase().includes(searchingTerm.toLowerCase())|| searchingTerm;
        }
    }

    class Main extends React.Component{

        componentWillMount(){
            this.props.fetchTopicsTableContent(this.state.sortBy,'ASC',0,this.props.match.params.Occupation).then(result=> (this.setState({rows:result.payload.data})))
            this.props.countTableContent(this.props.match.params.Occupation).then(result=>(this.setState({count:result.payload})));
        }

        constructor(props){
            super(props);

            this.state={
                searchTerm:"",
                rows:""
            }
        }

    searchHandler(e){
            this.setState({searchTerm:e.target.value})
            // console.log("rows",this.state.rows)
            {this.state.rows.filter(searchingFor(this.state.searchTerm)).map(item=>{
                console.log(item);
                 // this.setState({rows:item})
            })}
        }

    render(){
        return(
            <form>
               <input type="text"
                  value={this.state.searchTerm}
                  onChange={this.searchHandler.bind(this)}
                />
            </form>);}}

why are you running map? and are you sure searchingFor() returns true results ? — Sakhi Mansoor
– Sakhi Mansoor, Commented Sep 21, 2018 at 12:36
just use parameter directly filter(searchingFor(e.target.value) — xadm
– xadm, Commented Sep 21, 2018 at 12:59

ltd9938 · Accepted Answer · 2018-09-21 13:11:05Z

1

this.setState({searchTerm:e.target.value}) #setState is async operation and if you want to complete action how this operations will finish.

You should use callback function as for the 2d argument: this.setState({searchTerm:e.target.value}, () => {...here is your code})

edited Sep 21, 2018 at 13:11

ltd9938

1,4541 gold badge17 silver badges29 bronze badges

answered Sep 21, 2018 at 12:30

RomanRem

264 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Gowtham Koushik Over a year ago

Yes you are correct, when I combine this and the comment which has been verified, this works. Thanks for the comment

Gowtham Koushik Over a year ago

When I try to use setState under this, error is showing up. Like I want the item there to store it in a state called rows. If you look up there you can see the comment inside the filter function. It is not working.

Hitakshi Mahajan · Accepted Answer · 2018-09-21 13:17:55Z

1

The JavaScript operator || is defined to return the left value if it evaluates to a truthy value, otherwise the right value instead of returning true itself. So try replacing this

return x.name.toLowerCase().includes(searchingTerm.toLowerCase())|| searchingTerm;

with

return x.name.toLowerCase().includes(searchingTerm.toLowerCase())|| false;

answered Sep 21, 2018 at 13:17

Hitakshi Mahajan

614 bronze badges

Comments

Chandan Bhandari · Accepted Answer · 2018-09-26 07:07:36Z

0

your searchingFor() function is wrong. Make the provided changes:

this.state.rows
.filter(item => item.name.toLowerCase().indexOf(searchingTerm.toLowerCase()) > -1)
.foreach(item => console.log(item));

You can use the spread operator to save the data

edited Sep 26, 2018 at 7:07

answered Sep 21, 2018 at 13:01

Chandan Bhandari

2172 silver badges4 bronze badges

Collectives™ on Stack Overflow

Using search in react js

3 Answers 3

2 Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related