Question

下面是我为一小段Ruby代码编写的单元测试。

subject { AutoShop.new 'Mr. Fix It', 'WA987654321', Employee.new('Sue', 'Owner', 0) }

it '#employee_list returns list of employee names' do
  # setup
  input = [Employee.new('Lou', 'Receptionist', 90_000)]
  subject.append_employees input
  subject.employee_list.should eq "Sue\nLou"
end

以下是该课程的代码。但是，我无法在employee_list方法中正确显示名称。我是Ruby的新手，所以任何建议都会非常感激。

class AutoShop < Business

    attr_accessor :employees

  def initialize(name, tax_id, employee)
    super(name, tax_id)
    @employees = []
    @employees << employee
  end   

  def append_employees(input)
    input.map { |x| @employees << x  }  
  end

  def employee_list
    @employees.map {|x| x.name}
  end   

end

Answer 1

根据您的单元测试，您的employee_list方法应输出由换行符\n分隔的每个名称。因此，添加.join("\n")以从数组中生成字符串。

def employee_list
  @employees.map {|x| x.name }.join("\n")
end

.map也可以用“椒盐卷饼结肠”速记来完成：

@employees.map(&:name).join("\n")

Answer 2

虽然@MichaelBerkowski已经回答了你的主要问题，但我会注意你的append_employees方法看起来有点奇怪。您可能会使用Array#each，而不是Array#map。虽然@employees变量不受该更改的影响，但返回值会有所不同（不必要地吞噬您的内存）。比较：

2.0.0-preview2 :001 > input = [1, 2, 3, 4]
 => [1, 2, 3, 4] 
2.0.0-preview2 :002 > e = [] ; input.each { |x| e << x  }
 => [1, 2, 3, 4] 
2.0.0-preview2 :003 > e = [] ; input.map { |x| e << x  }
 => [[1, 2, 3, 4], [1, 2, 3, 4], [1, 2, 3, 4], [1, 2, 3, 4]]

从数组中拉出名称并在每个后添加换行符

2 个答案: