ハッシュオブジェクト（連想配列）を引数として扱う問題

Question

●問題
ある映画のハッシュを定義し、格納されている「title」(タイトル）・「genre」(ジャンル)・「year」(公開年)の三つの要素の中から一つを取り出すプログラムを作成してください。

●定義する変数
①movie = {"title" => "ハリーポッター", "genre" => "ファンタジー", "year" => "2001年"}
②ユーザーが入力するキーを、getsメソッドを利用し定義しましょう

●模範解答

def movie_info(movie, data)
    puts "#{movie[data]}"
  end

  movie = {"title" => "ハリーポッター", "genre" => "ファンタジー", "year" => "2001年"}

  puts "以下から一つを選んで入力してください。
    ・title
    ・genre
    ・year"

  info = gets.chomp

  movie_info(movie, info)

模範解答ではハッシュのキーに文字列を利用していますが、私は以下のようにシンボルを利用して書きました。

def movie_select(movie, imput)
  puts "#{movie[imput]}"
end

movie = {:title => "ハリーポッター", :genle => "ファンタジー", :year => "2001年"}

puts "以下から一つを選んで入力してください。
  ・title
  ・genre
  ・year"

imput = gets.chomp

movie_select(movie, imput)

この書き方だと、出力した際に

MBP-Kei:Desktop keita$ ruby sample.rb
以下から一つを選んで入力してください。
  ・title
  ・genre
  ・year
title

と、最後の文が空白になってしまいます。

原因として
・メソッド内の#{movie[imput]の記述がおかしい
ということを検討したのですが、解決に至りませんでした。

ハッシュ内でシンボルオブジェクトを利用してこの問題解決に望む場合、メソッド内などではどのように記述してあげればいいのでしょうか？

Comments

[argius]

Rubyタグの追加をお願いします。

Answer 1

動作についての説明は otn さんの説明の通りです。

勝手ながら Kei227さんの回答コードをさらに、書き換えてみました。

def movie_select(movie, key)
  puts "#{key}: #{movie[key.to_sym]}"
end

movie = {
  title: 'ハリーポッター',
  genre: 'ファンタジー',
  year:  '2001年'
}.freeze

puts "以下から一つを選んで入力してください。
  ・title
  ・genre
  ・year"

key = gets.chomp

movie_select(movie, key)

# for test
movie_select(movie, :title)
movie_select(movie, 'title')
movie_select(movie, 'genre')
movie_select(movie, 'year')
movie_select(movie, 'xxx')

最後の数行はテスト用です。
実行例：

$ ruby 3.rb 
以下から一つを選んで入力してください。
  ・title
  ・genre
  ・year
year
year: 2001年
title: ハリーポッター
title: ハリーポッター
genre: ファンタジー
year: 2001年
xxx: 

書き換えポイント：

• movie_select() に文字列、シンボルのどちらを渡しても動作するようにした。
• movie を変更できないようにした。
• ハッシュの値の書き方を "key" => "val" でなく、 key: 'val' にした。

rubocop という ruby コードのチェックツールがあります。(web などで調べてみてください)
↑のコードは このツールでチェックさせても警告は出ません。

Comments

[Kei227]

movie_select() に文字列、シンボルのどちらを渡しても動作するようにした。

どこを変更することで上記が適応されたのでしょうか？

また、ハッシュの書き方は『key: 'val'』のような書き方が簡潔でいいと思うのですが、この書き方をする上での注意点などはありますでしょうか？ 　

[katoy]

def movie_select(movie, key)
puts "#{key}: #{movie[key.to_sym]}"
end
の部分に注目してください。
movie[key.to_sym]　としています。
key が "title" なら key.to_sym は :title になります。
key が :title なら key.to_sym .は :title そもままです。

ハッシュの書き方については
http://qiita.com/YusukeHigaki/items/18db85aa9cba3cb42569
を参照してください。

rubocop では、 => 形式のハッシュ記法は警告が出ます。
(文字列を " で囲む記述も警告が出ます。 ' で囲むのが好ましいです。)

Answer 2

:titleと"title"は異なりますので、movie["title"]は存在しません。
キーがシンボルなら、入力した文字列をシンボルに変換しないといけません。

input = gets.chomp.to_sym

movie_select(movie, input)