結果が同じになってしまうので分けれるようにしたい

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
  if player_hand < 0 || 2 < player_hand then
  puts "無効な値です"
  end
  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
 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 @win
        puts "#{@win}"
     return false
     elsif @lose
        puts "#{@lose}"
     end
     return false
  end
  if player_direction != program_direction
   puts "#{@draw}"
  end
  return true
 end
 
 
while 
 result1 = JankenAcchimuite.new("あいこ","あっち向いて","あっち向いて")
 next_game = true
  while next_game
  next_game = result1.janken
  end
 result2 = JankenAcchimuite.new("もう一度ジャンケンを行います","負けました","勝ちました")
 if result2.acchimuite_hoi == false
  if @win
   puts "勝ちました"
  elsif @lose
   puts"負けました"
  end
  break
 end
end

Answer 1

じゃんけんにどっちが勝ったかという情報を、向き一致時に使っていないからですね。
@winの意味を途中で忘れてしまっているようです。思い出しましょう。

また、class JankenAcchimuiteを閉じるendが抜けているようです。

Comments

[diego]

回答ありがとうございます。
if result2.acchimuite_hoi == false
こちらの文のことでしょうか？

if result2.acchimuite_hoi == @win,@lose
にしてみたのですが動かなかったです。
どうすれば情報を持たせられますか？

[otn]

自分で考えて書いたコードでは無いのでしょうか？

[diego]

要所要所コピペしながら作ったので理解が浅いと思います…

[otn]

なるほど。
・じゃんけんでどっちが勝ったかはどこで判断しているか？
・判断した結果（どっちが買ったか）は、変数に保存しているか？保存していないならする
・じゃんけんで勝った結果を知る必要があるのはどこか？
・そこで先ほど保存した、じゃんけん結果を参照する
という方針で。

あと、それぞれの変数に何が入っているか（何のために存在する変数か）を把握できていないようなので、
変数一覧表（変数名とそこに格納する値の意味＝存在目的）を書きましょう。

[diego]

変数に保存というのが分からず、検索してみても引っかかりませんでした。勝ち負けの区別をどうつければいいのでしょうか？　
返り値を使うべきですか？

[otn]

案1：
じゃんけんメソッドの返り値は、今は「アイコかどうか」の2値だが、これを人間勝ち、人間負け、アイコ の3種類の値を返すようにして、メイン側でそれを変数に代入。
あっち向いてメソッドに、引数を追加して、それを渡して、あっち向いてメソッドでは、向きが一致した場合に、その引数を見て勝ちが負けか表示する。

案2：
返り値や引数を使わず、勝ち負け保存用インスタンス変数にじゃんけんメソッドで代入して、あっちむいてメソッドでそれを参照する。
（インスタンス変数を途中で書き換えることを忌避する考え方も有り）