Wie zu loopen und nach Benutzereingabe zu fragen

Question

Ich brauche Hilfe mit Schleifen.

Ich versuche, die Eingabe vom Benutzer zu erfassen und die Kosten eines Hotels für die Planung eines Urlaubs zu bestimmen.

Ich habe ein Problem beim Durchlaufen der Eingabe, so dass der Benutzer ein anderes Hotel auswählen und das Programm beenden kann, wenn der Benutzer keine der Tastenanschläge auswählt ("A", "a", "B", "b", " C "," c "," D "oder" d ") und berechnen dann die Gesamtkosten für Hotels.

kam ich mit diesem so weit nach oben:

puts choice = gets.chomp.downcase 
puts "For how many nights?" 
num=gets.chomp.to_i 
puts "Okay...any other hotels?" 
puts choice = gets.chomp.downcase 

#Hotel Prices 

#Hotel A 
if (choice== "a" or choice=="A") 
cost_2= (num/3)*500 + (num%3)*200 
end 

#Hotel B 
if (choice=="b" or choice=="B") 
cost_3= num*250 
end 

#Hotel C 
if (choice=="c" or choice=="C") 
cost_4 = (num/3)*700 + (num%3)*300 
end 

#Hotel D 
if (choice== "d" or choice=="D") 
    cost_5= num*500 
    end 

Hotel Program

Answer 1

Es gibt mehrere mögliche Verbesserungen, um Ihren Code, aber ich werde diese Antwort so einfach und direkt halten, wie ich kann.

Verwenden Sie ein loop und ein einzelner if ... else if ... else ... end Blockcode, anstatt mehrere separate if Aussagen:

loop do 
    if (choice== "a" or choice=="A") 
    cost_2= (num/3)*500 + (num%3)*200 
    #Hotel B 
    elsif (choice=="b" or choice=="B") 
    cost_3= num*250 
    #Hotel C 
    elsif (choice=="c" or choice=="C") 
    cost_4 = (num/3)*700 + (num%3)*300 
    #Hotel D 
    elsif (choice== "d" or choice=="D") 
    cost_5= num*500 
    else 
    puts "Final cost is: [...]" 
    break 
    end 
end 

Man könnte auch sagen, so etwas wie "Type 'done' when you have finished" einen klareren Ausgang betrachten Umsetzung - und ‚ignorieren‘ jede andere Eingabe ist das nicht a/b/c/d/done. Mit anderen Worten, etwas entlang der Linien von:

# ... 
elsif (choice== "d" or choice=="D") 
    cost_5= num*500 
elsif choice == 'done' 
    puts "Final cost is: [...]" 
    break 
else 
    puts 'Unknown option' 
end 
# ... 

Answer 2

Wickeln Sie Ihren Code in einer while-Schleife,

continue = true 
while continue 
    ... 
    if choice == "done" 
    continue = false 
    end 
end 

Offensichtlich ist das Ende Fall kann alles sein, nicht nur "fertig".

Aus Gründen der Fertigstellung, eine elegante Lösung für dieses Problem wäre, wie etwas sein:

loop do 
    puts choice = gets.chomp.downcase 
    puts "For how many nights?" 
    num=gets.chomp.to_i 
    puts "Okay...any other hotels?" 
    puts choice = gets.chomp.downcase 

    selections = { 
    a: (num/3)*500 + (num%3)*200, 
    b: num*250, 
    c: (num/3)*700 + (num%3)*300, 
    d: (num*500) 
    } 

    if selections[choice.to_sym] 
    cost += selections[choice.to_sym] 
    else 
    break 
    end 
end 

Answer 3

Dies ist nicht getestet, aber es ist die Grundidee für das, was ich verwenden würde:

loop do 

    cost = nil 
    loop do 

    puts 'Which hotel? (A, B, C, D)' 
    hotel = gets.chomp.downcase 

    puts 'For how many nights?' 
    days = gets.to_i 

    cost = if hotel == 'a' 
      (days/3) * 500 + (days % 3) * 200 
      elsif hotel == 'b' 
      days * 250 
      elsif hotel == 'c' 
      (days/3) * 700 + (days % 3) * 300 
      elsif hotel == 'd' 
      days * 500 
      else 
      puts 'Unknown hotel.' 
      end 

    break if cost 

    end 

    puts cost 

    puts 'Okay...any other hotels? (y/n)' 

    break unless gets.downcase.start_with?('y')  
end 

Statt if/elsif/else/end gekettet würde ich eine case Anweisung:

cost = case hotel 
     when 'a' 
     (days/3) * 500 + (days % 3) * 200 
     when 'b' 
     days * 250 
     when 'c' 
     (days/3) * 700 + (days % 3) * 300 
     when 'd' 
     days * 500 
     else 
     puts 'Unknown hotel.' 
     end