プログラムの変更をする方法

Question

def aisatsu(name)
  time = Time.now  
  case time.hour
  when 5..10  
    return "#{name}さん、おはようございます!"
  when 11..17 
    return "#{name}さん、こんにちは!"
  else            return "#{name}さん、こんばんは!"
  end
end

result = aisatsu(ARGV[0])
printf("%s %d時です。\n", result, time.hour)　

Answer 1

time = Time.nowで作成したtimeという変数は、aisatsuメソッドの中だけで有効なローカル変数です。メソッドの外で参照しようとしても、「そんな変数無い」というエラーになります。

返り値で返せば良いかと思います。

def aisatsu(name)
  time = Time.now  # 現在時を取得
  case time.hour
  when 5..10  # 5時から10時まで
    return "#{name}さん、おはようございます!", time.hour
  when 11..17 # 11時から17時まで
    return "#{name}さん、こんにちは!", time.hour
  else        # それ以外
    return "#{name}さん、こんばんは!", time.hour
  end
end

result, hour = aisatsu(ARGV[0])
printf("%s %d時です。\n", result, hour)

Comments

[退会済みユーザー]

回答ありがとうございますm(_ _"m)
そんな風にもできるんですね！教えていただきありがとうございます(o_ _)ｏ))

Answer 2

関数の中で定義した変数（ローカル変数）は、その関数の中でしか有効ではありません。

で、 %d時です。ってのもaisatuの中に含んでしまえばいいのでは

Comments

[退会済みユーザー]

回答ありがとうございますm(_ _)m
そういうやり方もあるんですね。とても勉強になりました＜(_ _)＞

Answer 3

トップレベルメソッド(関数っぽいやつ)のaisatsuの定義中に使われる変数timeは、有効範囲が定義の終わりまでだからです。

つまり、定義の外側で実行するprintfからは、そのtimeの値は使えません。別の変数として扱われるので、未定義だよーというエラーが発生します。

別の変数として、timeにTime.nowの値を設定してもいいのですが、一か所でしか使われていないので、直接hourをくっつけて、

-printf("%s %d時です。\n", result, time.hour)
+printf("%s %d時です。\n", result, Time.now.hour)

とでも書けばいいのでは。

Comments

[退会済みユーザー]

ご丁寧な解説ありがとうございます！とても助かりました????