2017-08-09 91 views
0

我想要做一個表單,可以添加/刪除行通過點擊添加/刪除按鈕。我現在的問題是,刪除按鈕總是可以刪除最後一行,而不是刪除按鈕所在的行。當我使用數組刪除操作符時,它可以刪除所需的行。然而,這種方法並沒有真正刪除數組中的對象,並且可能很難在for循環中進行檢查。我想知道我在哪裏做錯了?更新 - 動態添加/刪除行不按預期工作

更新時間: 1.增加的jsfiddle https://jsfiddle.net/69z2wepo/84246/ 2.保存下拉值母公司的狀態

constructor(props){ 
    super(props) 
    this.state = { 
    dropdown: [ 
     {first: "true", hideAdd: "false", result: ""} 
    ] 
    } 
    this.addRow = this.addRow.bind(this); 
    this.removeRow = this.removeRow.bind(this); 
    this.onChange = this.onChange.bind(this); 
} 

onChange = (key, value) => { 
    let oldArray = JSON.parse(JSON.stringify(this.state.dropdown)); 
    oldArray[key]['result'] = value; 
    this.setState({dropdown:oldArray}, function(){ 
     console.log(this.state.dropdown); 
     //console.log(this.state.dropdown.length) 
    }) 
} 

removeRow = (e, key) => { 
    e.preventDefault(); 

    let oldArray = JSON.parse(JSON.stringify(this.state.dropdown)); 

    oldArray.slice(); 
    oldArray.splice(key, 1); 

    for (let i=0; i<oldArray.length; i++){ 

     if (i==0){ 
      oldArray[i]['first'] = 'true'; 
     } 

     if (oldArray.length==1){ 
      oldArray[i]['hideAdd'] = 'false'; 
     }else{ 
      oldArray[i]['hideAdd'] = 'true'; 
      if (i == oldArray.length-1){ 
       oldArray[i]['hideAdd'] = 'false'; 
      } 
     } 

    } 

    this.setState({dropdown:oldArray}, function(){ 
     console.log(this.state.dropdown); 
     //console.log(oldArray); 
     //console.log(oldArray.length); 
    }) 

} 
render() { 
     return (
      <Grid.Column> 
       <Grid style={comStyles().gridWidth}> 
        <Grid.Row> 
         <Grid.Column width={4} textAlign='left' style={comStyles().lineHeight}>Then</Grid.Column> 
         <Grid.Column width={12}> 
          { 
          this.state.dropdown.map((item, index) => (
           <RulesThenDropdown default={item.result} onChange={this.onChange} add={this.addRow} remove={this.removeRow} id={index} key={index} hideAdd={item.hideAdd} first={item.first} /> 
          )) 
         } 
         </Grid.Column> 
        </Grid.Row> 
       </Grid> 
      </Grid.Column> 
     ) 
    } 

而以下是從子組件的代碼

render() { 
    const Options = thenOptions; 

    let extra = ""; 

    const removeBtn = <button onClick={(e,m)=>this.props.remove(e,this.props.id)} className="circular ui icon button"><i className="icon minus"></i></button> 
    const addBtn = <button onClick={(e,m)=>this.props.add(e,this.props.id)} className="circular ui icon button"><i className="icon plus"></i></button> 

    if(this.props.first==="false"){ 
     if(this.props.hideAdd=="true"){ 
      extra = removeBtn; 
     }else{ 
      extra = <div>{addBtn}{removeBtn}</div>; 
     } 
    }else if(this.props.first==="true"){ 
     if(this.props.hideAdd!="true"){ 
      extra = addBtn; 
     }else{ 
      extra = removeBtn; 
     } 
    } 

    return (
     <div style={comStyles().buttonsMainWrapper}> 
      <Form.Dropdown 
       placeholder='Then' 
       fluid 
       selection 
       options={Options} 
       defaultValue = {this.props.result} 
       onChange={(e,{ value })=>this.props.onChange(this.props.id, value)} 
      /> 
      <div style={comStyles().buttonsGroup}> 
      { 
       extra 
      } 
      </div> 
     </div> 
    ) 
} 
+0

一個很好的替代陣列中的端部,這個問題是這行'oldArray.splice(key,1);'?您是否嘗試過'console.log'並檢查前後的'oldArray'值? –

+0

嗨Firice,是的,數組似乎是正確的 – HUNG

+0

'splice'和'delete'之間'oldArray'的結果是一樣的嗎?如果在'oldArray = oldArray.filter(n => n)'之後使用'delete'並清理'oldArray'' –

回答

1

問題是你'直接修改組件狀態,而不是使用this.setState()更新不可變副本。

罪魁禍首是這一行:

let oldArray = this.state.dropdown; 

不復制的狀態,而是獲得參考,所以現在既oldArray和this.state.dropdown都指向存儲器中的相同的結構。

當您隨後進行切片,拼接和更新oldArray時,您打破了有關狀態可變性的React組件合同(請參閱https://facebook.github.io/react/docs/state-and-lifecycle.html#do-not-modify-state-directly)。

要解決這個問題,你需要深克隆this.state.dropdown像這樣:

let oldArray = JSON.parse(JSON.stringify(this.state.dropdown)) 

(見https://stackoverflow.com/a/5344074/501217瞭解詳細信息)

+0

不幸的是,行爲仍然是一樣的:( – HUNG

+0

)您還需要分配oldArray.slice()和oldArray.splice()的結果。與array.push和array.pop不同,它們不會對數組產生副作用,而是返回一個新數組,因爲您已經深度複製了oldArray,所以可以跳過slice語句並將'oldArray.splice(...)'更改爲'oldArray = oldArray.splice(...)' –

+0

對不起,我仍然有點困惑。let oldArray = JSON.parse(JSON.stringify(this.state。落下)); 我首先這樣做,那麼你的意思是這裏的oldArray已經被深度複製了,然後我可以oldArray.splice(key,1)並繼續for循環來更新某個值,最後setState?現在還是一樣,我不確定我是否誤解了任何步驟 – HUNG

1

我想添加更多@Kim Burgaard答案,那Immutable List是在state

dropdown: List([{first: "true", hideAdd: "false", id: -1}])