ruby - 在自身内部调用方法是不好的做法吗？

Question

我正在创建一个短程序来计算一组给定数字的平均值。这是一个程序，其基本结构首先是在 TI-83 上使用 goto 语句制定的，所以我有点不确定对这类事情的正确做法。这是有问题的代码:

$sum     = 0
$counter = 0

def average
  puts "Input another number, or \"end\" to end."
  input = gets.chomp
  if input == "end"
    puts $counter
    puts ($sum / $counter).to_f
  else
    $sum += input.to_f
    $counter += 1
    puts $counter
    average #Here the method is called again, repeating the cycle.
  end
end

我不确定还有什么办法可以做到这一点，因为像这样构建代码的要点是它可以处理不确定数量的输入，因此它具有重复性。

Answer 1

在具有所谓“尾调用优化”特征的语言中，这种结构是主要的循环方式。然而，Ruby 没有；递归(重新输入一个已经输入的函数)很常用，但不能作为简单循环的一般替代品，如本例所示。

这里，如 pvg 所述，while(或等效地，until)循环更好:

sum = 0
counter = 0
puts "Input a number, or \"end\" to end."
input = gets.chomp
until input == "end"
  sum += input.to_f
  counter += 1
  puts "Input another number, or \"end\" to end."
  input = gets.chomp
end
puts sum / counter

或者一个带有break的无限循环:

sum = 0
counter = 0
loop do
  puts "Input a number, or \"end\" to end."
  input = gets.chomp
  break if input == "end"
  sum += input.to_f
  counter += 1
end
puts sum / counter

但是你也可以使用更多的 Rubyish 方式:

puts 'Input numbers, or "end" to end.'
array = STDIN.each_line.lazy
  .map(&:chomp)
  .take_while { |line| line != "end" }
  .map(&:to_f)
  .to_a
puts array.inject(&:+) / array.size

或者更难看但内存效率更高:

puts 'Input numbers, or "end" to end.'
sum, count = *STDIN.each_line.lazy
  .map(&:chomp)
  .take_while { |line| line != "end" }
  .inject([0, 0]) { |memo, x|
    [memo[0] + x.to_f, memo[1] + 1]
  }
puts sum / count