无论输入如何，Ruby while循环都会不断重复

Question

我写了一个简单的猜数字游戏。 但即使我输入正确的数字，它也会一直循环。 请帮忙，谢谢！

puts "Pick a number between 0 - 1000."
user_guess = gets.chomp.to_i

my_num = rand(831)
guess_count = 0

until user_guess == my_num do
  if user_guess == my_num
    guess_count += 1
    puts "you got it!"
  elsif  user_guess <= 830
    guess_count += 1
    puts "higher"
  else user_guess >= 1000
    guess_count += 1
    puts "lower"
  end 
end

puts "You guessed my number in #{guess_count} attempts. Not bad"

Answer 1

向用户询问数字的代码部分在循环之外，因此在检查答案后不会重复。 如果您想在用户猜测错误时要求用户再次猜测，则该代码需要在循环内。

my_num = rand(831)
guess_count = 0
keep_going = true


while keep_going do
  puts "Pick a number between 0 - 1000."
  user_guess = gets.chomp.to_i
  if user_guess == my_num
    guess_count += 1
    puts "you got it!"
    keep_going = false
  elsif  user_guess <= 830
    guess_count += 1
    puts "higher"
  else user_guess >= 1000
    guess_count += 1
    puts "lower"
  end 
end


puts "You guessed my number in #{guess_count} attempts. Not bad"

这段代码仍然有一些错误会阻止游戏正常工作，看看你是否能发现它们是什么。

Answer 2

由于@Tobias 已经回答了您的问题，我想花一些时间来建议您如何使您的代码更像 Ruby。

首先，虽然您可以使用while或until循环，但我建议您主要依赖Kernel#loop方法来编写大多数循环。 这只会导致循环在loop块内继续，直到遇到关键字break ¹ 。 这很像while true或until false （常用于某些语言），但我认为它读起来更好。 更重要的是， loop的使用可以保护其块内的计算免受窥探。 （有关这一点的示例，请参阅下面的其他注意事项部分。）

您还可以通过执行return或exit退出loop的块，但通常您将使用break 。

我的第二个主要建议是，对于此类问题，您使用case statement而不是if/elsif/else/end结构。 让我们首先使用范围来做到这一点。

使用带范围的 case 语句

my_num = rand(831)
guess_count = 0

loop do
  print "Pick a number between 0 and 830: "
  guess_count += 1
  case gets.chomp.to_i
  when my_num
    puts "you got it!"
    break
  when 0..my_num-1
    puts "higher"
  else
    puts "lower"
  end
end

这里有几点需要注意。

• 我使用了print而不是puts因此用户将在与提示相同的行上输入他们的响应。
• 不管用户的响应如何， guess_count都会增加，这样可以在执行 case 语句之前完成。
• 无需将用户的响应 ( gets.chomp.to_i ) 分配给变量。
• case 语句将值与适当的case 相等方法=== 。

关于最后一点，这里我们将一个整数 ( gets.chomp.to_i ) 与另一个整数 ( my_num ) 和范围(0..my_num-1) 。 在第一个实例中，使用Integer#=== ，它等效于Integer#== 。 对于范围，使用Range#===方法。

例如，假设my_num = 100并且gets.chomp.to_i #=> 50 case 语句如下所示。

case 50
when 100
  puts "you got it!"
  break
when 0..99
  puts "higher"
else
  puts "lower"
end

在这里我们发现100 == 50 #=> false和(0..99) === 50 #=> true ，所以显示puts "higher" 。 (0..99) === 50返回true因为整数（在===的右侧）被范围（在左侧）覆盖。 这与50 === (0..90) ，后者大致读作“ (0..99)是50的成员”，因此返回false 。

这里有几个例子，说明如何使用 case 语句，因为它们依赖于三重相等方法。

case obj
when Integer
  obj + 10
when String
  obj.upcase
when Array
  obj.reverse
...
end

case str
when /\A#/
  puts "A comment"
when /\blaunch missiles\b/
  big_red_button.push
...
end 

使用带有宇宙飞船运算符<=>的 case 语句

Ruby 的Array#sort和Enumerable#sort方法使用 spaceship 运算符，但还有其他用途，如 case 语句。 这里我们可以使用Integer#<=>来比较两个整数。

my_num = rand(831)
guess_count = 0

loop do
  print "Pick a number between 0 and 830: "
  case gets.chomp.to_i <=> my_num
  when 0
    puts "you got it!"
    break
  when -1         
    puts "higher"
  else # 1
    puts "lower"
  end
end

在其他应用程序中，宇宙飞船运算符可能用于比较字符串 ( String#<=> )、数组 ( Array#<=> )、 Date对象 ( Date#<=> ) 等。

使用哈希

散列通常可以用作case statements的替代方法。 这里我们可以写如下。

response = { -1=>"higher", 0=>"you got it!", 1=>"lower" }
my_num = rand(831)
guess_count = 0

loop do
  print "Pick a number between 0 and 830: "
  guess = gets.chomp.to_i
  puts response[guess <=> my_num]
  break if guess == my_num
end

这里我们需要两次gets.chomp.to_i的值，所以我将它保存到一个变量中。

其他注意事项

假设我们写如下：

i = 0
while i < 5
  i += 1
  j = i
end
j #=> 5

发现循环后面的j等于5 。

如果我们改为使用loop ：

i = 0
loop do
  i += 1
  j = i
  break if i == 5
end
j #=> NameError (undefined local variable or method 'j')

虽然while和loop都可以访问i ，但是loop将在其块中创建的局部变量的值限制在块中。 那是因为块创建了一个新的scope ，这是很好的编码实践。 while和until不使用块。 我们通常不希望循环后面的代码可以访问在循环中创建的局部变量，这也是loop优先于while和until原因之一。

最后，关键字break也可以与一个参数一起使用，该参数的值由loop返回。 例如：

def m
  i = 0
  loop do
    i += 1
    break 5*i if i == 10
  end
end

m #=> 50

或者

i = 0
n = loop do
  i += 1
  break 5*i if i == 10
end
n #=> 50

^{1. 如果您检查Kernel#loop的文档，您将看到在loop的块内执行break等效于引发StopIteration异常。}