Rubocop 规则：永远不要将 'do' 与多行 'while 一起使用

Question

我有以下代码

  # colours a random cell with a correct colour
  def colour_random!
    while true do
      col, row = rand(columns), rand(rows)
      cell = self[row,col]
      if cell.empty? then
        cell.should_be_filled? ? cell.colour!(1) : cell.colour!(0)
        break
      end
    end
  end

做什么并不重要，尽管它应该很明显。关键是 Rubocop 给了我一个警告

切勿将 'do' 与多行 'while 一起使用

为什么我不应该这样做？那我该怎么做呢？

Answer 1

while 是一个关键字，所以你不需要传递一个块。没有do..end 它将正常工作。下面的就好了

  def colour_random!
    while true
      col, row = rand(columns), rand(rows)
      cell = self[row,col]
      if cell.empty? then
        cell.should_be_filled? ? cell.colour!(1) : cell.colour!(0)
        break
      end
    end
  end

while 是一个关键字，如果你向它传递一个块，比如do..end，它仍然可以按照你的要求工作，不会抛出任何错误，而只是一个警告。但是，如果您尝试将 Proc 或 Method 对象传递给它，并使用 & 关键字动态尝试将其转换为 block 可能会很危险，就像我们通常做的那样。这意味着

# below code will work as expected just throwing an warning.
x = 2
while x < 2 do
  #code
end

但是如果你尝试像下面这样错误地做

while &block # booom!! error

原因是while 是一个关键字，它不支持任何to_proc 方法来满足您的需要。所以这可能很危险。

Ruby 风格指南还建议Never use while/until condition do for multi-line while/until

我认为原因正如Nobuyoshi Nakada在mailing list中所说的那样

loop 是一个kernel 方法，它接受一个block。 块引入了新的局部变量范围。

  loop do
    a = 1
    break   
  end   
  p a #=> causes NameError

while 没有。

  while 1
    a = 1
    break   
  end
  p a #=> 1

Answer 2

Ruby 实际上有一个while true 的快捷方式：the loop statement。

def colour_random!
  loop do
    col, row = rand(columns), rand(rows)
    cell = self[row,col]
    if cell.empty? then
      cell.should_be_filled? ? cell.colour!(1) : cell.colour!(0)
      break
    end
  end
end