bwang
bwang

Reputation: 13

displaying a result in render from a function in REACTJS

I want to render the text 'Results!' and the name of the largestLikeResult from my getLargest() function.

    getLargest() { 
        var largestLikeResult = null
        var largerstLikeNum= 0 
        if(this.props.results!=null){
            //.map goes through every result
            this.props.results.map(i=> {
                console.log(i.name)
                this.state.resultsRef.child(this.replaceAll("."," ",i.name)).once('value',function(snapshot) {
                if(largerstLikeNum<snapshot.val().right)
                {   
                    console.log("new largest in town")
                    largerstLikeNum = snapshot.val().right 
                    largestLikeResult= i.name 
                    console.log(largestLikeResult)
                }
                }) 
            })
            return (
                <div>
                {largestLikeResult}
                </div>
                )


        }
        else {
            return null
        }
    }

    render(){ 
        return (
            <div>
            Results! 
            <h1>{this.getLargest()}</h1>
            </div>  
        ) 
    }  
}
export default DisplayResults

Currently, only Results! shows up on page and the name of the largestLikeResult shows up in the console, not page. Any quick changes I can add to render() to show the value of largestLikeResult?

Thanks in advance!

Upvotes: 1

Views: 403

Answers (1)

Jee Mok
Jee Mok

Reputation: 6556

For a quick change, I think if you change from map to forEach it will work fine:

this.props.results.forEach(i => {

But I'd suggest to refactor getLargest() function into something similar to this:

getLargest() { 
    let largerstLikeNum = 0; 
    const { results } = this.props;
    const { resultsRef } = this.state;

    // Always return early
    if (!results || !Array.isArray(results)) {
      return null;
    }

    return (
      <div>
        {
          results.map(i => {
              return resultsRef.child(this.replaceAll('.', ' ', i.name)).once('value', snapshot => {
                if (largerstLikeNum < snapshot.val().right) {   
                    console.log('new largest in town');
                    largerstLikeNum = snapshot.val().right ;
                    return (
                      <div>{i.name}</div>
                    );
                }
                return null;
              }) 
          })
        }
      </div>
    );
}

Upvotes: 1

Related Questions