2017-07-06 2 views
0

Ich möchte mehr Informationen onclick zeigen, aber ich habe verschiedene Divs zum Anzeigen/Verbergen. So. Ich mache es so: Der Staat ist:Zeige mehr Informationen in React

this.state = { 
     showMore: false, 
     showRewards: false, 
     fees: false, 
     showFees: false 
    }; 

Einer der divs:

     <div className='some'> 
          <h3 >Rewards <i className="icon-im-rewards"></i></h3> 
            <h5 className="showMoreDetails" onClick={() => this.show('rewards') }>{ showRewards ? '-' : '+' } { showRewards ? 'Show Less' : 'Show More' }</h5> 
          { showRewards && detail.rewards.reward.map((name, i) => 
            <p key={i}> { name }</p> 
          ) 
          } 
         </div> 

Die Funktion:

Show (name) { let {Showmore , showRewards, showFees} = this.state;

if (name === 'hightlights' && showMore === false) { 
    this.setState({ 
     showMore: true 
    }); 
} else { 
    this.setState({ 
     showMore: false 
    }); 
} 

if (name === 'rewards' && showRewards === false) { 
    this.setState({ 
     showRewards: true 
    }); 
} else { 
    this.setState({ 
     showRewards: false 
    }); 
} 

if (name === 'fees' && showFees === false) { 
    this.setState({ 
     showFees: true 
    }); 
} else { 
    this.setState({ 
     showFees: false 
    }); 
} 

Dies ist zu lang, hässlich und nicht richtig funktioniert (ich kann nur eine erweiterte div zu einem Zeitpunkt, sehen) Wie es zu verbessern?

Antwort

2

Dies ist eine bessere und eine sauberere Weg:

function show (name) { 

    let { showMore, showRewards, showFees } = this.state; 
    showMore = name === 'hightlights' && showMore === false; 
    showRewards = name === 'rewards' && showRewards === false; 
    showFees = name === 'fees' && showFees === false; 

    this.setState({ 
    showFees, 
    showRewards, 
    showMore 
    }); 
+0

es ist viel besser, aber ich kann immer noch nur ein zu einer Zeit sehen –

0

Es könnte einfacher sein, wenn Sie uns die Struktur Detail & Belohnungen zeigen konnte. Ich denke, du mappst hier ein einzelnes Element, deshalb druckt es ein einzelnes Element. Allerdings versuchen diese Zeile zu ändern,

detail.rewards.reward.map((name, i) => <p key={i}> { name }</p>) 

mit folgendem Code ersetzen:

detail.rewards.map((reward, i) => <p key={i}> { reward.name }</p>) 
Verwandte Themen