Iterate all the property in object, find and replace

Question

I have this Javascript object generated from my code and player object is

const player = {
   cards: []
}

My question is how do I loop through all my card in player object and replace all the "A" faceValue to 1? Using findIndex or splice?

player: Object
cards: Array(2)
0: Array(1)
0: card {suit: "spades", face: "7", faceValue: 7}
length: 1
__proto__: Array(0)
1: Array(1)
0: card {suit: "clubs", face: "A", faceValue: 11}
length: 1
1: Array(2)
0: card {suit: "spades", face: "A, faceValue: 11}
length: 1

write a loop, check if face is == A, if it is, set faceValue to 1 — Ronnie
– Ronnie, Commented May 2, 2019 at 23:58
It's unclear what the structure of your data is (from the log output), so it's difficult to suggest a way to iterate over that structure. — Felix Kling
– Felix Kling, Commented May 2, 2019 at 23:58
developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/… — messerbill
– messerbill, Commented May 2, 2019 at 23:58

Rannie Aguilar Peralta · Accepted Answer · 2019-05-03 00:30:31Z

1

You can use Array map your cards object to update its value

player.cards = player.cards.map(card => {
    if (card.face === 'A') card.faceValue = 1;
    return card;
});

Update: It is better to use forEach if we are not generating new values

player.cards.forEach(card => {
    if (card.face === 'A') card.faceValue = 1;
});

edited May 3, 2019 at 0:30

Rannie Aguilar Peralta

1,77817 silver badges19 bronze badges

answered May 3, 2019 at 0:00

Jairo Malanay

1,3479 silver badges13 bronze badges

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

3 Comments

Felix Kling Over a year ago

No need to use .map if you mutate the objects (i.e. not generating new values). Just use .forEach then.

Steve Over a year ago

any example of .forEach to find and replace?

messerbill Over a year ago

yes @FelixKling is right but mutating arrays is mostly a bad practice. I'd create a new one like const myCards = player.cards.map(....) and if needed assign that array to the player afterwards. Imho this is a better pattern, even if it may be slower

Collectives™ on Stack Overflow

Iterate all the property in object, find and replace

1 Answer 1

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related