Ruby on Rails: Each Loop, Separate resulting hash with comma within an array

Question

I'm trying to create a hash using an each loop as follows:

[
  hash = session[:cart].each do |product|
  price = product[0].price.to_i*100
  {
    name: product[0].name, description: product[0].description, quantity: product[1], amount: product[0].price
  }, #Is it possible to add a comma here? Doing this normally causes an error
  end
]

Which should hopefully produce this Output

Output

 [
   {name: "Hellow", description: "Many Hellows", quantity: 1, price: 1000},
   {name: "Hellow", description: "Many Hellows", quantity: 1, price: 1000}
 ]

In your hash, you wrote: "amount: price: product[0].price". Is that a typo? — Tiago Farias
– Tiago Farias, Commented Jul 21, 2014 at 19:22
oh yes, that's a typo, let me correct it. i'm trying to use activemerchant and it seems to only accept the multiple hashes if there's a comma separating the hashes D: ... which is the point really since all of these will be in an array — Kishe
– Kishe, Commented Jul 21, 2014 at 19:26

user229044 · Accepted Answer · 2014-07-21 19:31:33Z

3

You want .map, not .each. Each iterates. Map translates one thing into another.

[
  session[:cart].map do |product|
    price = product[0].price.to_i*100
    {
      name: product[0].name,
      description: product[0].description,
      quantity: product[1],
      amount: product[0].price
    } 
  end
]

edited Jul 21, 2014 at 19:31

answered Jul 21, 2014 at 19:18

user229044♦

241k41 gold badges347 silver badges350 bronze badges

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

6 Comments

Kishe Over a year ago

Changing it to map seem to still seem to cause syntax error, unexpected ',', expecting keyword_end ... any ideas ? :O

user229044 Over a year ago

@Kishe No comma. That comma after the last } has no meaning and is syntactically invalid.

Kishe Over a year ago

Bonus Question: What if they need to be separated within an array as the edit above. (I'm really sorry i didn't mention this earlier)

user229044 Over a year ago

@Kishe Then get rid of the hash = part, and wrap the whole thing in []. Please stop changing your question. If you have a new question, ask a new question, don't continually change the meaning of your old question.

Kishe Over a year ago

@maegar Yessir. sorry about that.

|

Collectives™ on Stack Overflow

Ruby on Rails: Each Loop, Separate resulting hash with comma within an array

1 Answer 1

6 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

6 Comments

Your Answer

Sign up or log in

Post as a guest

Related