2

下面是我为一小段 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
4

2 回答 2

4

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

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

.map也可以使用“pretzel-colon”速记来完成：

@employees.map(&:name).join("\n")
于 2013-02-11T01:43:06.860 回答
1

尽管@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]]
于 2013-02-11T05:34:46.963 回答