Welcome to OGeek Q&A Community for programmer and developer-Open, Learning and Share
Welcome To Ask or Share your Answers For Others

Categories

0 votes
563 views
in Technique[技术] by (71.8m points)

reactjs - Warning: Each child in a list should have a unique "key" prop. Check the render method of `Body`. yet i already have key

i am new to react and i have build simple value incrementing and removing project but i got Each child in a list should have a unique "key" prop error in my console. I don't know from where i am getting this error although i have already mentioned key in my div. someone please tell me from where i am getting this error

import React from 'react'

class Body extends React.Component{

constructor(props){
    super(props)
    this.state = { 
         counters:[
        {id:1, value:0},
        {id:2, value:0},
        {id:3, value:0},
        {id:4, value:0},
    ],};
   
}

render(){
    return(
        <>
        <nav className="navbar navbar-light bg-light">
            <a className="navbar-brand" href="/#">Navbar
            <span className='badge badge-pill badge-secondary m-3'>{this.state.counters.filter(counter => counter.value > 0).length}</span>
            </a>
        </nav>

        <div className='container'>
            <button className='btn btn-primary btn-sm' onClick={()=>this.handleReset()}> Reset</button>
            {this.state.counters.map((counter)=>{
                return(
                    <>
                    <div key={counter.id}>
                        <span className={this.changeClass(counter)}>{counter.value ===0 ? 'zero':counter.value}</span>
                        <button className='btn btn-secondary' onClick={()=> this.handleIncrement(counter)} >Increment</button>
                        <button className='btn btn-danger m-2' onClick ={()=> this.handleDelete(counter)} >Delete</button>
                    </div>
                    
                    </>
                    
                )
            })}
        </div>

        </>

    )
}


changeClass(counter){
    let classes = 'btn btn-sm m-2 btn-'
    classes += counter.value === 0 ? 'warning': 'primary'
    return classes
}

handleIncrement(counter){
    
    const counters = [...this.state.counters]
    const index = counters.indexOf(counter)
    counters[index].value ++
    this.setState({counters})
   
}

handleDelete(counter){
     const counters = this.state.counters.filter(c=> c.id !== counter.id )
    this.setState({counters})
}


handleReset(){
    
   const counters = this.state.counters.map(counter=>{
        counter.value = 0
        return counter
    } )
    this.setState({counters})
      
}


 }


export default Body

与恶龙缠斗过久,自身亦成为恶龙;凝视深渊过久,深渊将回以凝视…
Welcome To Ask or Share your Answers For Others

1 Reply

0 votes
by (71.8m points)

The issue lies in the ghost tags, they are at the highest level, so they are expected to retain the key.

if you do need the ghost tags to exist you could instead use <React.Fragment key={...your key}> as opposed to ghost tags

import React from 'react'

class Body extends React.Component{

constructor(props){
    super(props)
    this.state = { 
         counters:[
        {id:1, value:0},
        {id:2, value:0},
        {id:3, value:0},
        {id:4, value:0},
    ],};
   
}

render(){
    return(
        <>
        <nav className="navbar navbar-light bg-light">
            <a className="navbar-brand" href="/#">Navbar
            <span className='badge badge-pill badge-secondary m-3'>{this.state.counters.filter(counter => counter.value > 0).length}</span>
            </a>
        </nav>

        <div className='container'>
            <button className='btn btn-primary btn-sm' onClick={()=>this.handleReset()}> Reset</button>
            {this.state.counters.map((counter)=>{
                return(
                    <div key={counter.id>
                    
                        <span className={this.changeClass(counter)}>{counter.value ===0 ? 'zero':counter.value}</span>
                        <button className='btn btn-secondary' onClick={()=> this.handleIncrement(counter)} >Increment</button>
                        <button className='btn btn-danger m-2' onClick ={()=> this.handleDelete(counter)} >Delete</button>
                    </div>
                    
                 
                    
                )
            })}
        </div>

        </>

    )
}


changeClass(counter){
    let classes = 'btn btn-sm m-2 btn-'
    classes += counter.value === 0 ? 'warning': 'primary'
    return classes
}

handleIncrement(counter){
    
    const counters = [...this.state.counters]
    const index = counters.indexOf(counter)
    counters[index].value ++
    this.setState({counters})
   
}

handleDelete(counter){
     const counters = this.state.counters.filter(c=> c.id !== counter.id )
    this.setState({counters})
}


handleReset(){
    
   const counters = this.state.counters.map(counter=>{
        counter.value = 0
        return counter
    } )
    this.setState({counters})
      
}


 }


export default Body

与恶龙缠斗过久,自身亦成为恶龙;凝视深渊过久,深渊将回以凝视…
OGeek|极客中国-欢迎来到极客的世界,一个免费开放的程序员编程交流平台!开放,进步,分享!让技术改变生活,让极客改变未来! Welcome to OGeek Q&A Community for programmer and developer-Open, Learning and Share
Click Here to Ask a Question

...