ケースステートメントでメソッドを呼び出すことができますruby

Question

caseステートメントでループを実行するのがいいかどうかを知りたいと思います。レスポンス（ユーザ入力）が不正な入力であれば、メソッドを呼び出すことはできますか？より良い方法がありますか？

def game_type # Selecting humans or computers as players 
    puts "Please select game type by number:" 
    puts "1 - human vs human" 
    puts "2 - computer vs computer" 
    puts "3 - human vs computer" 
    response = gets.chomp.to_i 

    case response 
    when 1 
     puts "human vs human" 
    when 2 
     puts "computer vs computer" 
    when 3 
     puts "human vs computer" 
    else 
     game_type 
    end 
    end 

Answer 1

再帰的に呼び出すメソッドは非常に悪いことです。再帰呼び出しごとに、コールスタックに追加の深さが追加され、深さが予測されず、パフォーマンスが悪いと予測できないためです。可能であれば、再帰は避けるべきです。この場合、再帰呼び出しを必要としないループを使用します。

def game_type 
    loop do 
    puts <<~_ 
     Please select game type by number: 
     1 - human vs human 
     2 - computer vs computer 
     3 - human vs computer 
    _ 
    response = gets.to_i 
    case response 
    when 1 
     puts "human vs human" 
     break 
    when 2 
     puts "computer vs computer" 
     break 
    when 3 
     puts "human vs computer" 
     break 
    end 
    end 
end 

Answer 2

この種の相互作用のための標準的なパターンは、文句を言い、別のループのためにあなたを送り返すelse支店を持つことです。

Rubyを実行するときに注意する1つのことは、多くの繰り返しを避けるために、このようなものを通常の構造で定義してください。 「自分を繰り返さない」（DRY）の原則は非常に重要です。それを乾かしてください！この場合はそう

：

GAME_TYPE_LIST = [ 
    "Human vs. Human", 
    "Computer vs. Computer", 
    "Human vs. Computer" 
] 

# Convert this list to a hash indexed by 1, 2, 3, etc. 
GAME_TYPES = Hash[GAME_TYPE_LIST.each_with_index.map { |type, i| [ i + 1, type ] }] 

将来的にはより多くの種類を追加することがかなり容易になり、加えても、あなたのメニューを提示する超簡単：

同様に

GAME_TYPES.each do |i, type| 
    puts '%d - %s' % [ i, type ] 
end 

、それが来ます選択する時間：

game_type = nil 

begin 
    puts "Enter type:" 

    game_type = gets.chomp.to_i 
end while (!GAME_TYPES[game_type]) 

これは、あなたが正しいことを繰り返してくれるでしょう。

Answer 3

私は、これはcase文でループを行うには大丈夫な方法であるかどうかを知りたいです。レスポンス（ユーザ入力）が不正な入力であれば、メソッドを呼び出すことはできますか？

ながらでき澤が言ったように、それはパフォーマンス上の理由のために、Rubyで良い習慣ではない、それを行います。

良い方法はありますか？むしろ使用再帰より

、loopを使用します。

GAME_TYPES = { 
    1 => "human vs computer", 
    2 => "computer vs computer", 
    3 => "human vs human" 
} 

def game_type 
    loop do  
    puts <<~___ 
     Please select game type by number: 
     1 - human vs human 
     2 - computer vs computer 
     3 - human vs computer 
    ___ 
    selection = gets.to_i 
    if game_type_str = GAME_TYPES[selection] 
     puts game_type_str 
     break 
    end 
    end 
end 

（私も辞書ではなく、コードがはるかに簡潔breakの束を持つよりますcase文を、使用するようにコードを変更ステートメント）。