ジャンケンがループしない

Question

コード
``puts "あっち向いてホイを始めます"
puts "ジャンケン・・・"
class JankenAcchimuite
 def initialize (draw ="あいこ",lose ="負け",win ="勝ち")
   @draw = draw
   @lose = lose
   @win = win
 end
 
 def janken
  #実装する
  puts "[0]グー,[1]チョキ,[2]パー"
  player_hand = gets.to_i
  program_hand = rand(3)

  jankens = ["グー","チョキ","パー"]
  puts "あなたの手:#{jankens[player_hand]},相手の手:#{jankens[program_hand]}"

  if player_hand == program_hand
   puts "#{@draw}"
   return true
  elsif  (player_hand == 0 && program_hand == 2 ) ||( player_hand == 1 && program_hand == 0 )||( player_hand == 2 && program_hand == 1)
   puts "#{@lose}"
   return false
  elsif  (player_hand == 0 && program_hand == 1 ) ||( player_hand == 1 && program_hand == 2 )||( player_hand == 2 && program_hand == 0)
   puts "#{@win}"
   return false
  end
 
 next_game = true

 while next_game
  next_game = janken
 end
 
 end
 
 def acchimuite_hoi 
   puts "0(上)1(下)2(左)3(右)"
  player_direction = gets.to_i
  program_direction = rand(4)
  directions = ["上","下","左","右"]
   puts "あなた：#{directions[player_direction]} 相手：#{directions[program_direction]}"

  if player_direction == program_direction
     if @lose
        puts "#{@lose}"
     elsif @win
        puts "#{@win}"
     end
  else
      puts "#{@draw}"
      
  end
 end

 result1 = JankenAcchimuite.new("あいこ","あっち向いて","あっち向いて")
 puts result1.janken
 result2 = JankenAcchimuite.new("もう一度ジャンケンを行います","負けました","勝ちました")
 puts result2.acchimuite_hoi
end


、、、
whileを使用しtrue,falseで戻しているつもりなのですが、ジャンケンがループせずにそのままあっち向いてホイに行ってしまいます。お願いします。

Answer 1

そもそもwhile next_gameループに行く前に結果に関わらすreturnしてるからループしません。
またそれを修正したとしてもwhile内でjankenメソッドを呼び出していないのでやはり意図通りループしません。
なおかつ適切にフラグ変数を設定していないので無限ループになっています。

Comments

[diego]

returnはtrue,falseを返しているだけではないのでしょうか？

[RiaFeed]

返すと同時に実行場所もjankenメソッドを呼び出した場所に戻ってしまうので、
現在提示のコードではwhile next_gameと書いている行が実行されることはありません。

[diego]

申し訳ありません知識不足で上手く理解できているのかが分かりません。
returnよりも先にwhile next_gameが読み込まれれば大丈夫ということでしょうか？

[RiaFeed]

そもそも意図を推測する(あいこの時は決着つくまでじゃんけんを繰り返す？)感じではそもそもループの位置が間違っていると思います。
jankenメソッドの中でjankenメソッドを呼び出すなら再帰呼び出しすればいいのでwhileでループする必要がありません
(とはいえ個人的に再帰は非推奨なのでよくわからなかったらスルーして)
ループするならresult1.jankenの結果をputsせずに変数に入れてその変数を基にループさせる感じがいいのかな。
(本当はloop doにしたほうがいいけど更なる混乱を招きそうなのでこんな感じで)

result1 = JankenAcchimuite.new("あいこ","あっち向いて","あっち向いて")
next_game = result1.janken
while next_game
next_game = result1.janken
end

[diego]

すいません遅くなりました。
呼び出し位置が下にあるのに先にループしていたということですね…
分かりやすくありがとうございます。

Answer 2

ジャンケンがループしない、んではなくて、ループはしてるんだけどその中で実行していない、ってことですね

next_game = janken

Comments

[diego]

実行してしてないとはどういったことでしょうか？