javascript vuejs object array returning only one item in the loop

Question

I am trying to iterate over an array of objects instead its returning only one item here is the code:

setAll(){
  var result =this.cart;
  for (var key in result) {
    var obj = result[key];
  }
  return obj.price;
}

and the test data

 [ { "id": 5, "price": 3200, "quantity": 8, "name": "juice" }, { "id": 6, 
  "price": 300, "quantity": 6, "name": "rice" }, { "id": 8, "price": "100", 
  "quantity": 1, "name": "water" }, { "id": 7, "price": "4500", "quantity": 
   1, "name": "meat" } ]

What is the end result you want here? Do you want to return an array of all of the objects' prices? like [3200, 300, 100, 4500]? — kingdaro
– kingdaro, Commented Feb 16, 2018 at 13:38
It'll be much easier to answer this with a bit more information; nothing apart from your title tells us that you're using Vue, for example. If you provide a bit more information about what you're trying to do, it'll be easier for other to help you achieve that. — rwold
– rwold, Commented Feb 16, 2018 at 13:42
You should use prices = cart.map(({price}) => +price); to get all prices from the cart array. — Hassan Imam
– Hassan Imam, Commented Feb 16, 2018 at 14:00

Yserbius · Accepted Answer · 2018-02-16 13:40:11Z

1

You are iterating through the array and only returning the last object. This code:

for (var key in result) {
   var obj = result[key];
}

does nothing other than set obj to the last item in the list.

answered Feb 16, 2018 at 13:40

Yserbius

1,40413 silver badges19 bronze badges

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

Comments

rwold · Accepted Answer · 2018-02-16 13:42:38Z

0

Right now object will be set to its value from the last iteration through the loop, and you will return the price of that object only.

edited Feb 16, 2018 at 13:42

answered Feb 16, 2018 at 13:40

rwold

2,4861 gold badge17 silver badges22 bronze badges

3 Comments

Hassan Imam Over a year ago

This should be comment

rwold Over a year ago

I thought about that, but it does also contain the information in the answer above it.

rwold Over a year ago

Ok, extracted a comment from the answer.

Igor Ognichenko · Accepted Answer · 2018-02-16 18:38:14Z

0

your function replace to

setAll(){
  return this.cart.map(({ price }) => +price);
}

i am write jsfiddle example for your question

edited Feb 16, 2018 at 18:38

answered Feb 16, 2018 at 14:25

Igor Ognichenko

2762 silver badges14 bronze badges

3 Comments

Vincent Kituyi Over a year ago

using this function I am getting babel-loader error

Igor Ognichenko Over a year ago

screenshot working result monosnap.com/file/sRFX1SzfateAf0ISy0clfUSIxy9YMX

GGO Over a year ago

@IgorOgnichenko edit your answer adding your jsfiddle instead of commenting. Try to make a readable, simple and complete answer

Vincent Kituyi · Accepted Answer · 2018-02-19 04:58:33Z

0

This worked for me:

          setAll(){
          var result =this.cart;
          var res=Object.keys(result).map(function(key){
            return parseInt(result[key].price);
          });
          return res;

         },

answered Feb 19, 2018 at 4:58

Vincent Kituyi

1193 silver badges17 bronze badges

Collectives™ on Stack Overflow

javascript vuejs object array returning only one item in the loop

4 Answers 4

Comments

3 Comments

3 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

3 Comments

3 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related