Changing value in list of objects

Question

I have a list of objects and I need to loop thru the list and change a value from true to false. Shouldn't a simple for loop do the trick? Am I missing something?

var list = [
  { color: 'blue', 'taste': 'sour', 'available': true },
  { color: 'yellow', 'taste': 'bitter', 'available': false },
  { color: 'red', 'taste': 'sweet', 'available': false },
  { color: 'green', 'taste': 'umami', 'available': false }
]

for(var i = 0; i < list.length; i++){
  if(list[i].available === true){
    list[i].available === false;
  }
}

When I return the list though it's giving me the list as it was first captured. Am I using the wrong loop or is it something else?

Purely a style thing, but why is color an unquoted key when all of your other keys are quoted? Inconsistencies like that have a way of hiding errors... — Jared Smith
– Jared Smith, Commented May 31, 2016 at 18:52
list[i].available === false is performing a comparison. You need to do an assignment: list[i].available = false — Mike Cluck
– Mike Cluck, Commented May 31, 2016 at 18:52
Oh that's just a typo. I was just trying to quickly mock up a bunch of objects in an array. — Alex Marple
– Alex Marple, Commented May 31, 2016 at 18:54

rishabh dev · Accepted Answer · 2016-05-31 18:51:24Z

3

var list = [
  { color: 'blue', 'taste': 'sour', 'available': true },
  { color: 'yellow', 'taste': 'bitter', 'available': false },
  { color: 'red', 'taste': 'sweet', 'available': false },
  { color: 'green', 'taste': 'umami', 'available': false }
]

for(var i = 0; i < list.length; i++){
  if(list[i].available === true){
   // you were not modifying here, just comparing
    list[i].available = false;
  }
}

answered May 31, 2016 at 18:51

rishabh dev

1,7631 gold badge16 silver badges28 bronze badges

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

4 Comments

Anthony Forloney Over a year ago

Mostly nitpicking here but the comparison to === true is not necessary.

blamb Over a year ago

that about solves that question, @Alex Marple i think that was your issue, please mark accepted answer if your good. no point necessary here for anyone (except rishab dev) since the user made a mistake, this is how answers get bloated. Or im going to guess the first 3 upvotes are rishabh dev? :-)

Alex Marple Over a year ago

Ahhh yes. I knew I was missing something troublingly simple. Thank you. Fixed. I will accept this in 10 minutes when it lets me.

blamb Over a year ago

omg now someone downvoted you. dude what the heck is happening to the stack crowd here? lol, thanks for recanting that one..

Rohìt Jíndal · Accepted Answer · 2016-05-31 19:38:45Z

1

You are doing comparision using list[i].available === false.You need to do assign false into list[i].available.So, try this list[i].available = false.

answered May 31, 2016 at 19:38

Rohìt Jíndal

27.4k16 gold badges80 silver badges133 bronze badges

Collectives™ on Stack Overflow

Changing value in list of objects

2 Answers 2

4 Comments

Comments

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

4 Comments

Comments

Related