我的“退货”声明不起作用

Question

我有这个方法，它应该从常量数组​​中生成随机选择。但是该方法不会返回结果。事实上它不会返回我尝试的任何东西。

代码如下：

COLORS = ["B", "G", "R", "O", "Y", "P"]

class Computer
  attr_reader :random_code

  def initialize
    @random_code = secret_code
  end

  def secret_code

    sample_code = []

    sample_code << COLORS.sample(4)

    return sample_code

  end

  def add(a, b)
    return a + b
  end


end


c = Computer.new

c.add(5, 10)    

添加“add”方法只是为了测试它，但它也不起作用。当我说它不起作用时，我的意思是它在控制台中没有显示任何内容。

Answer 1

正如@BroiSatse 指出的那样，问题在于您没有告诉 Ruby 将任何内容打印到控制台。

以下是我将如何修复和简化您的代码：

class Computer
  COLORS = ["B", "G", "R", "O", "Y", "P"]

  attr_reader :random_code

  def initialize
    @random_code = secret_code
  end

  def secret_code
    COLORS.sample(4)
  end

  def add(a, b)
    a + b
  end
end

c = Computer.new
puts c.add(5, 10)

还有一些值得注意的事情：

1. 如果COLORS 是您的计算机类固有的东西，您可以将其拉入类并使其成为类常量。您可以在类中按原样引用COLORS，但请记住，如果您想在类定义之外的任何地方引用COLORS，则必须将其称为Computer::COLORS才能访问它.你现在的方式也很好，这只是一个小的代码组织。

2. 您的 secret_code 函数可以简化为 COLORS.sample(4)。在 Ruby 中，您不必使用 return 关键字，并且在不需要时将其省略是惯用的。函数定义中的最后一个值是返回的值。另外，由于Array#sample已经返回了一个数组，所以不需要定义一个空数组[]并添加到它。