Rubyでハッシュからeach文でうまく値を表示させる方法

Question

##実現したいこと

ハッシュからeach文でうまく値を表示できるようにしたいです。

##問題点
現在、Rubyのクラスとインスタンスの練習で、Moive_reviewというクラスを作って
@moviesというhashのインスタンス変数を定義しています。
show_reviewというインスタンスメソッドで,
@moviesに投稿されたハッシュのデータを、うまく表示させたいのですが、できません。
keyとvalueがtitleと入力した値、starと入力した数字のようになってしまい。
each文の中で、タイトルとstarをputsのところにうまく表示するためには
どのようなコードに変えればいいのでしょうか？

##現在のコード

class Movie_review

  def initialize
    @movies={}
  end

  def post
    title=gets.chomp
    star=gets.to_i
    @movies[:title]=title
    @movies[:star]=star
  end

  def show_review
    @movies.each do |key,value|
    puts "タイトル#{key}"
    puts "五段階評価#{value}"
    end
  end
  
end


movie_review=Movie_review.new

while true do

  puts "番号を入力してください\n1 投稿\n2 レビューを表示"
  input=gets.to_i

  if input==1
    movie_review.post
  elsif input==2
    movie_review.show_review
  else
    exit
  end
  
end

Answer 1

多分、あなたが欲しいのはハッシュじゃなくて、配列の配列なんじゃないかな

def initialize
  @movies = []
end

def post
  title=gets.chomp
  star=gets.to_i
  @movies << [title, star]
end

---

別解

# ハッシュでやる場合
def post
  title=gets.chomp
  star=gets.to_i
  @movies[title] = star
end

# ハッシュの配列でやる場合
def initialize
  @movies = []
end

def post
  title=gets.chomp
  star=gets.to_i
  @movies << {title: title, star: star}
end

def show_review
  @movies.each do |movie|
    puts "タイトル#{movie[:title]}"
    puts "五段階評価#{movie[:star]}"
  end
end

Comments

[yoshi751]

asmさん、回答ありがとうございます！
配列にした場合、適切に動作するようになりました！

ハッシュで試して見たのですが、
Traceback (most recent call last):
3: from class.rb:34:in `<main>'
2: from class.rb:15:in `show_review'
1: from class.rb:15:in `each'
class.rb:16:in `block in show_review': no implicit conversion of Symbol into Integer (TypeError)

このようなエラーが出てしまいました。
このエラーの原因はどこかわかりますでしょうか？

```
class Movie_review

def initialize
@movies={}
end

def post
title=gets.chomp
star=gets.to_i
@movies = {title:title,star:star}
end

def show_review
p @movies
@movies.each do |movie|
puts "タイトル#{movie[:title]}"
puts "五段階評価#{movie[:star]}"
end
end

end

movie_review=Movie_review.new

while true do

puts "番号を入力してください\n1 投稿\n2 レビューを表示"
input=gets.to_i

if input==1
movie_review.post
elsif input==2
movie_review.show_review
else
exit
end

end
```

[asm]

それだと、each使う必要ないですね

def show_review
puts "タイトル#{@movies[:title]}"
puts "五段階評価#{@movies[:star]}"
end

でいいです。

Answer 2

Hashでやるなら```
def post
title=gets.chomp
star=gets.to_i
@movies[title]=star
end

ですね、それでもともとの　def show_review　で表示できます。

＞class.rb:16:in `block in show_review': no implicit conversion of Symbol into Integer (TypeError)

のエラーがでたのは、`@movies.each do |movie|` を誤解しているからです
Hashの each は each_pair とおなじです。
つまり、@movies.each do |key,value|` と同じ。　これを movie 一つで受けているので movieには　[key,value] が入っています。配列です。
配列[:title]なので、シンボルは整数に出来ない　というエラーとなります。