replace object in array base on index

Question

So, Im using react and I need to keep adding objects to an array of objects (object may have the same index, thats why I check for label and index). When the object that I want to add has the same label property as one that already is in that array, it should replace the previous object. So, lets say, only one object for each label. What I have works until I work with more then one label. When I do so, the array accepts more than one objects for each label...

 if (this.state.thumbnailsAtivas.some(thumbnail => {
     thumbnail.index === textura.index
 }) && this.state.thumbnailsAtivas.some(thumbnail => {
     thumbnail.label === textura.label
 })) {
     console.log("already in array");
 }
 else if (this.state.thumbnailsAtivas.some(thumbnail => thumbnail.label === textura.label)) {
     console.log("label already with item");
     this.state.thumbnailsAtivas.some((thumbnail, index) => {
         const tempData = (this.state.thumbnailsAtivas).slice(0);
         tempData[index] = textura;
         this.setState({thumbnailsAtivas: tempData})
     })
 } else {
     this.setState({thumbnailsAtivas: [...this.state.thumbnailsAtivas, textura]},);
 }

@PaulMcloughlin I dont have this online but I can pastbin this code. Hard to make it work outside my project, since Im getting dynamic data by axios — Nelson Silva
– Nelson Silva, Commented Apr 18, 2018 at 21:46
@NelsonSilva I forgot to mention findIndex earlier in the other question. Sorry! — ibrahim mahrir
– ibrahim mahrir, Commented Apr 18, 2018 at 23:02

ibrahim mahrir · Accepted Answer · 2018-04-18 23:04:03Z

3

You can use another Array function called findIndex which have the same usage as some but returns a result like indexOf does (returns the index of the element in an array or -1 if no element matches):

let index = this.state.thumbnailsAtivas.findIndex(
    thumbnail => thumbnail.label === textura.label
);

if(index !== -1) {
    this.state.thumbnailsAtivas[index] = yourNewObject;
}

Note: To optimise your code a little bit, you could get rid of the call to some and use findIndex (once) for both checking existence and finding the index.

edited Apr 18, 2018 at 23:04

answered Apr 18, 2018 at 22:57

ibrahim mahrir

31.8k5 gold badges50 silver badges78 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

replace object in array base on index

1 Answer 1

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related