2017-05-24 34 views
1

我正在嘗試構建React計算器。到目前爲止,只有當我的第二個數字的長度等於1時,它纔有效。如果它的數字更多,它就會中斷。下面是代碼:反應計算器

class Calculator extends Component { 
    constructor(props) { 
     super(props); 
     this.state = { 
     current: 0, 
     total: 0, 
     operator: '' 
     }; 
    } 
    handleType = (e) =>{ 
    const current = (this.state.current == 0 || this.state.operator != '' || this.state.current == this.state.total) ? '' : this.state.current 
    this.setState({ 
     current: parseInt(current + e.target.attributes.getNamedItem('data-filter').value) 
    }); 
    } 
calculate = (sign, number) => { 
    const total = this.state.total; 
    console.log(total); 
    switch(sign){ 
    case "-": 
     return total - number; 
     break; 
    case "+": 
     return total + number; 
     break; 
    case "*": 
     return total * number; 
     break; 
    case "/": 
     return total/number; 
     break; 
    default: 
     return 0; 
    } 
}; 

handleAction = (e) =>{ 
    const operator = e.target.attributes.getNamedItem('data-filter').value; 
    console.log(operator); 
    this.setState({ 
    total: this.state.current, 
    operator: operator 
    }); 
    console.log(this.state); 
} 
getResult =() =>{ 
    this.setState({ 
    current: this.calculate(this.state.operator, this.state.current) 
    }); 
} 

    render() { 
    return (
     <div className="Calculator text-center"> 
      <h2>Result: {this.state.current} </h2> 
      <div className="row"> 
       <button data-filter="7" onClick={this.handleType}>7</button> 
       <button data-filter="8" onClick={this.handleType} >8</button> 
       <button data-filter="9" onClick={this.handleType} >9</button> 
       <button data-filter="+" onClick ={this.handleAction} >+</button> 
      </div> 
      <div className="row"> 
       <button data-filter="4" onClick={this.handleType}>4</button> 
       <button data-filter="5" onClick={this.handleType}>5</button> 
       <button data-filter="6" onClick={this.handleType}>6</button> 
       <button data-filter="-" onClick ={this.handleAction} >-</button> 
      </div> 
      <div className="row"> 
       <button data-filter="1" onClick={this.handleType}>1</button> 
       <button data-filter="2" onClick={this.handleType}>2</button> 
       <button data-filter="3" onClick={this.handleType}>3</button> 
       <button data-filter="*" onClick ={this.handleAction} >*</button> 
       <button data-filter="/" onClick ={this.handleAction} >/</button> 
      </div> 
      <div className="row"> 
       <button>0</button> 
       <button>save</button> 
       <button>cancel</button> 
       <button onClick={this.getResult}>=</button> 
      </div> 

     </div> 
    ); 
    } 
} 

我明白,這個問題是我的條件:

const current = (this.state.current == 0 || this.state.operator != '' || this.state.current == this.state.total) ? '' : this.state.current 

它工作正常的第一個號碼,當操作者=「」,但一旦我有一個運營商也不會請允許我我的號碼比9

這裏更大的是codepen演示http://codepen.io/polinaz/pen/PmLoKJ?editors=0010

任何想法如何解決呢?謝謝:)

+0

我想演示的鏈接是錯誤 – Sheng

+0

對不起這個!現在更新它 – Polina

回答

1

我做了一些在你的代碼的更改:

如狀態更新可以是異步的(https://facebook.github.io/react/docs/state-and-lifecycle.html#state-updates-may-be-asynchronous),這是非常不好的做法按照您使用的方式使用之前的狀態值:

this.setState({ 
    current: parseInt(current + e.target.attributes.getNamedItem('data-filter').value) 
}); 

我已經改變了許多這些成這種格式:

this.setState((prevState) => { 
    const current = prevState.current; 
    return {current: parseInt(current + value)}; 
}); 

我感動e.target.attributes.getNamedItem('data-filter').value代碼到它自己的變量,由於event-pooling

handleType (e) { 
    const value = e.target.attributes.getNamedItem('data-filter').value; 

你所面臨的主要缺陷是,該操作後,被選中(通過handleAction),您的操作員屬性被填充。這意味着在你的handleType,你做的檢查:this.state.operator != ''

const current = (this.state.current == 0 || this.state.operator != '' || this.state.current == this.state.total) ? '' : this.state.current 

這意味着你不顧先前選擇的當前值。將其設置爲「」,然後將其附加到當前值。

class Calculator extends React.Component { 
 
    constructor(props) { 
 
     super(props); 
 
     this.state = { 
 
     current: 0, 
 
     total: 0, 
 
     operator: '' 
 
     }; 
 
    this.handleType = this.handleType.bind(this); 
 
    this.calculate = this.calculate.bind(this); 
 
    this.handleAction = this.handleAction.bind(this); 
 
    this. getResult = this.getResult.bind(this); 
 
    
 
    } 
 
    
 
    handleType (e) { 
 
    const value = e.target.attributes.getNamedItem('data-filter').value; 
 

 
    this.setState((prevState) => { 
 
     const current = (this.state.current == 0 || this.state.current == this.state.total) ? '' : this.state.current; 
 
     return {current: parseInt(current + value)}; 
 
    }); 
 
     
 
    } 
 
calculate (sign, number) { 
 
    const total = this.state.total; 
 
    console.log(total); 
 
    switch(sign){ 
 
    case "-": 
 
     return total - number; 
 
     break; 
 
    case "+": 
 
     return total + number; 
 
     break; 
 
    case "*": 
 
     return total * number; 
 
     break; 
 
    case "/": 
 
     return total/number; 
 
     break; 
 
    default: 
 
     return 0; 
 
    } 
 
}; 
 

 
handleAction (e) { 
 
    const operator = e.target.attributes.getNamedItem('data-filter').value; 
 
    
 
    this.setState((prevState) => { 
 
    return { 
 
    total: prevState.current, 
 
    operator: operator, 
 
    current: 0 
 
    } 
 
    }); 
 
    
 
    console.log(this.state); 
 
} 
 
getResult() { 
 
    this.setState({ 
 
    current: this.calculate(this.state.operator, this.state.current) 
 
    }); 
 
} 
 

 
    render() { 
 
    return (
 
     <div className="Calculator text-center"> 
 
      <h2>Result: {this.state.current} </h2> 
 
      <div className="row"> 
 
       <button data-filter="7" onClick={this.handleType}>7</button> 
 
       <button data-filter="8" onClick={this.handleType} >8</button> 
 
       <button data-filter="9" onClick={this.handleType} >9</button> 
 
       <button data-filter="+" onClick ={this.handleAction} >+</button> 
 
      </div> 
 
      <div className="row"> 
 
       <button data-filter="4" onClick={this.handleType}>4</button> 
 
       <button data-filter="5" onClick={this.handleType}>5</button> 
 
       <button data-filter="6" onClick={this.handleType}>6</button> 
 
       <button data-filter="-" onClick ={this.handleAction} >-</button> 
 
      </div> 
 
      <div className="row"> 
 
       <button data-filter="1" onClick={this.handleType}>1</button> 
 
       <button data-filter="2" onClick={this.handleType}>2</button> 
 
       <button data-filter="3" onClick={this.handleType}>3</button> 
 
       <button data-filter="*" onClick ={this.handleAction} >*</button> 
 
       <button data-filter="/" onClick ={this.handleAction} >/</button> 
 
      </div> 
 
      <div className="row"> 
 
       <button data-filter="0" onClick={this.handleType}>0</button> 
 
       <button>save</button> 
 
       <button>cancel</button> 
 
       <button onClick={this.getResult}>=</button> 
 
      </div> 
 

 
     </div> 
 
    ); 
 
    } 
 
} 
 

 
ReactDOM.render(
 
    <Calculator/>, 
 
    document.getElementById('root') 
 
);
<script src="https://cdnjs.cloudflare.com/ajax/libs/react/15.1.0/react.min.js"></script> 
 
<script src="https://cdnjs.cloudflare.com/ajax/libs/react/15.1.0/react-dom.min.js"></script> 
 

 
<div id='root'></div>

+0

非常感謝您的解釋!現在對我來說更加清晰! – Polina

1

做了一些更改,請參閱片段。

工作片段:

class Calculator extends React.Component { 
 
    constructor(props) { 
 
     super(props); 
 
     this.state = { 
 
     current: '', 
 
     total: 0, 
 
     operator: '' 
 
     }; 
 
    } 
 

 
    handleType = (e) =>{ 
 
    const current = (this.state.current == 0 || this.state.operator != '' || this.state.current == this.state.total) ? '' : this.state.current 
 
    this.setState({ 
 
     current: this.state.current + e.target.attributes.getNamedItem('data-filter').value 
 
    }); 
 
    } 
 

 
    calculate() { 
 
    const total = parseInt(this.state.total); 
 
    const current = parseInt(this.state.current); 
 
    const sign = this.state.operator; 
 
    switch(sign){ 
 
     case "-": 
 
     return total - current; 
 
      break; 
 
     case "+": 
 
      return total + current; 
 
      break; 
 
     case "*": 
 
      return total * current; 
 
      break; 
 
     case "/": 
 
      return total/current; 
 
      break; 
 
     default: 
 
      return 0; 
 
    } 
 
    }; 
 

 
    handleAction = (e) =>{ 
 
    const operator = e.target.attributes.getNamedItem('data-filter').value; 
 
    this.setState({ 
 
     total: this.state.current, 
 
     current: '', 
 
     operator: operator 
 
    }); 
 
    } 
 

 
    getResult =() => { 
 
    this.setState({ 
 
     total: this.calculate(), 
 
     current: '' 
 
    }); 
 
    } 
 
    
 
    cancel =() => { 
 
    this.setState({ 
 
     total: '', 
 
     current: '', 
 
     operator: '' 
 
    }) 
 
    } 
 

 
    render() { 
 
    return (
 
     <div className="Calculator text-center"> 
 
      Number: {this.state.current} 
 
      <h2>Result: {this.state.total} </h2> 
 
      <div className="row"> 
 
       <button data-filter="7" onClick={this.handleType}>7</button> 
 
       <button data-filter="8" onClick={this.handleType} >8</button> 
 
       <button data-filter="9" onClick={this.handleType} >9</button> 
 
       <button data-filter="+" onClick ={this.handleAction} >+</button> 
 
      </div> 
 
      <div className="row"> 
 
       <button data-filter="4" onClick={this.handleType}>4</button> 
 
       <button data-filter="5" onClick={this.handleType}>5</button> 
 
       <button data-filter="6" onClick={this.handleType}>6</button> 
 
       <button data-filter="-" onClick ={this.handleAction} >-</button> 
 
      </div> 
 
      <div className="row"> 
 
       <button data-filter="1" onClick={this.handleType}>1</button> 
 
       <button data-filter="2" onClick={this.handleType}>2</button> 
 
       <button data-filter="3" onClick={this.handleType}>3</button> 
 
       <button data-filter="*" onClick ={this.handleAction} >*</button> 
 
       <button data-filter="/" onClick ={this.handleAction} >/</button> 
 
      </div> 
 
      <div className="row"> 
 
       <button data-filter="0" onClick={this.handleType}>0</button> 
 
       <button>save</button> 
 
       <button onClick={this.cancel}>cancel</button> 
 
       <button onClick={this.getResult}>=</button> 
 
      </div> 
 

 
     </div> 
 
    ); 
 
    } 
 
} 
 

 
ReactDOM.render(<Calculator/>, document.getElementById('app'))
<script src="https://cdnjs.cloudflare.com/ajax/libs/react/15.1.0/react.min.js"></script> 
 
<script src="https://cdnjs.cloudflare.com/ajax/libs/react/15.1.0/react-dom.min.js"></script> 
 

 
<div id='app'>

+0

非常感謝你! – Polina