Welcome to the Treehouse Community
Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community! While you're at it, check out some resources Treehouse students have shared here.
Looking to learn something new?
Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.
Start your free trialLaurence kite
11,768 PointsJavaScript React Basics
HI I am new to React and find it fun but confusing at times.
In a video we were asked to wire up a function to handle a click event for removing a guest from an array that contains a list of invited guests to a party.
I eventually got it working but found a problem which is best explained by adding the code, so below is non working code. Then below that the working code. The important bit is the handleRemove
const GuestList = (props) =>
<ul>
{props.guests
.filter((guest) => !props.isFiltered || guest.isConfirmed)
.map((guest, index)=>
<Guest key={index} name ={guest.name}
isConfirmed ={guest.isConfirmed}
isEditing = {guest.isEditing}
handleConfirmation ={()=> props.toggleConfirmationAt(index)}
handleToggleEditing ={()=> props.toggleEditingnAt(index)}
setName ={(text) => props.setNameAt(text,index)}
handleRemove = {() => props.removeGuestAt(index)}/>
)}
</ul>;
Note how the handleRemove has the index passed to it, which is on the right handside props.removeGuestAt(index)}.
which I understand to be the value(key) associated with each invited guest.
When I first failed at wiring this up I put the index on both sides which I will paste below:
const GuestList = (props) =>
<ul>
{props.guests
.filter((guest) => !props.isFiltered || guest.isConfirmed)
.map((guest, index)=>
<Guest key={index} name ={guest.name}
isConfirmed ={guest.isConfirmed}
isEditing = {guest.isEditing}
handleConfirmation ={()=> props.toggleConfirmationAt(index)}
handleToggleEditing ={()=> props.toggleEditingnAt(index)}
setName ={(text) => props.setNameAt(text,index)}
handleRemove = {(index) => props.removeGuestAt(index)}/>
)}
</ul>;
As you can see the rest of the code is the same, but this one mistake means when you click the remove button in the form nothing works. Also you can not log out the index to the console. So it is not getting back up the tree to the removeGuestAt(index) ?
Although it is passed index it is not the right one, but no error occurs to say what index is this, it just doesn't do as you want. I understand that handleRemove will catch the click() event but not index... So what index is this? What is the index on the left that is then passed to props.removGuestAt(index)
Laurence kite
11,768 PointsLaurence kite
11,768 PointsThis bit needs to be edited it reads:
I eventually got it working but found a problem which is best explained by adding the code, so below is non working code. Then below that the working code. The important bit is the handleRemove
I should have wrote:
Below is the working code..
The next bit is the non working code.