können Sie eine Methode in einer Case-Anweisung aufrufen Ruby

Question

Ich möchte wissen, ob dies eine gute Möglichkeit ist, eine Schleife in der case-Anweisung zu tun. Ist es in Ordnung, die Methode zurückzurufen, wenn die Antwort (Benutzereingabe) eine schlechte Eingabe ist? Gibt es einen besseren Weg?

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

Das rekursive Aufrufen der Methode ist sehr schlecht, da für jeden rekursiven Aufruf eine zusätzliche Tiefe zum Aufruf-Stack hinzugefügt wird und Sie nicht vorhersagen können, wie tief sie sein wird, was die Performance beeinträchtigt. Im Allgemeinen sollte Rekursion möglichst vermieden werden. Verwenden Sie in diesem Fall eine Schleife, für die kein rekursiver Aufruf erforderlich ist.

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

Das Standardmuster für diese Art der Interaktion ist eine else Zweig zu haben, die für eine andere Schleife beschwert und sendet Ihnen zurück.

Eine Sache zu beachten bei Ruby ist, sollten Sie versuchen, solche Dinge in einer regelmäßigen Struktur zu definieren, um eine Menge Wiederholungen zu vermeiden. Das Prinzip "Do not Repair Yourself" (DRY) ist sehr wichtig. Halte es trocken!

Also in diesem Fall:

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 ] }] 

, dass es ziemlich einfach macht mehr Arten in der Zukunft hinzuzufügen, und auch super einfach Ihr Menü zu präsentieren:

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

Ebenso, wenn es darum geht, Zeit zu wählen:

game_type = nil 

begin 
    puts "Enter type:" 

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

Das wird Schleifen Sie util Sie erhalten es richtig.

Answer 3

Ich möchte wissen, ob dies eine Ordnung Art und Weise ist eine Schleife in der Case-Anweisung zu tun. Ist es in Ordnung, die Methode zurückzurufen, wenn die Antwort (Benutzereingabe) eine schlechte Eingabe ist?

Während Sie kann tun, als Sawa gesagt, das ist keine gute Praxis in Ruby, aus Gründen, die Leistung bezogen.

Gibt es einen besseren Weg?

Anstatt Verwendung Rekursion verwenden 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 

(Ich habe auch Ihren Code ein Wörterbuch eher als eine case Anweisung zu verwenden, die den Code viel prägnanter macht als ein Bündel von break mit Aussagen).