我该如何改进这个ruby代码和我对Hashes的使用？

Question

这里的要点是用于浏览数组docfiles和返回两个阵列（temporary_file_paths和temporary_file_names）。

我决定返回一个哈希，但我觉得我可以摆脱2个临时数组，但我不知道怎么...

def self.foobar docfiles 
     temporary_information = Hash.new 
     temporary_file_paths = [] 
     temporary_file_names = [] 
     docfiles.each do |docfile| 
      if File.exist? docfile.path 
      temporary_file_paths << "new_path" 
      temporary_file_names << "something_else" 
      end 
     end 
     temporary_information[:file_paths] = temporary_file_paths 
     temporary_information[:file_names] = temporary_file_names 
     return temporary_information 
    end 

Answer 1

这里有很多解决方案。

返回双精度值。

def self.foobar(docfiles) 
    temporary_file_paths = [] 
    temporary_file_names = [] 
    docfiles.each do |docfile| 
    if File.exist? docfile.path 
     temporary_file_paths << new_path 
     temporary_file_names << something_else 
    end 
    end 
    [temporary_file_paths, temporary_file_names] 
end 

paths, names = Class.foo(...) 

使用收集。

def self.foobar(docfiles) 
    docfiles.map do |docfile| 
    File.exist?(docfile.path) ? [new_path, something_else] : nil 
    end.compact 
end 

paths, names = Class.foo(...) 

使用注射（如果你想有一个哈希）

def self.foobar(docfiles) 
    docfiles.inject({ :file_paths => [], :file_names => []}) do |all, docfile| 
    if File.exist?(docfile.path) 
     all[:file_paths] << new_path 
     all[:file_names] << something_else 
    end 
    all 
    end 
end 

所有的解决方案上面不改变主方法的逻辑。 我不太喜欢使用数组/散列来代替对象，所以当执行需要多次细化时，通常最终会将哈希转换为对象。

TemporaryFile = Struct.new(:path, :something_else) 

def self.foobar docfiles 
    docfiles.map do |docfile| 
    if File.exist?(docfile.path) 
     TemporaryFile.new(new_path, something_else) 
    end 
    end.compact 
end 

另外，我不知道其他something的意思，但如果它的东西，你可以从new_path得到的，那么你可以使用懒人执行。

TemporaryFile = Struct.new(:path) do 
    def something_else 
    # ... 
    end 
end 

def self.foobar docfiles 
    docfiles.map do |docfile| 
    TemporaryFile.new(new_path) if File.exist?(docfile.path) 
    end.compact 
end 

Answer 2

是的，就用它们来代替临时哈希：

def self.foobar(docfiles) 
    temporary_information = { :file_paths => [], :file_names => [] } 
    docfiles.each do |docfile| 
     if File.exist? docfile.path 
     temporary_information[:file_paths] << new_path 
     temporary_information[:file_names] << something_else 
     end 
    end 
    return temporary_information 
end 

Answer 3

这听起来像你可能会去约一个尴尬的方式解决，但这里是你的一个简化版本。

def self.foobar docfiles 
    temporary_information = Hash.new 
    temporary_information[:file_paths] = [] 
    temporary_information[:file_names] = [] 

    docfiles.each do |docfile| 
     if File.exist? docfile.path 
     temporary_information[:file_paths] << new_path 
     temporary_information[:file_names] << something_else 
     end 
    end 

    return temporary_information 
end 

另外，正如Alex建议的那样，您可以使用inject。下面是一个工作示例向您展示一个缩短的版本，我写之前，我看到Alex的职位:-)

['test', 'test2', 'test3'].inject({}) { |result,element| { :file_paths => result[:file_paths].to_s + element, :file_names => result[:file_names].to_s + element } } 

Answer 4

你能避免使用临时数组，像这样：

def self.foobar docfiles 
    temporary_information = {:file_paths => [], :file_names => []} 
    docfiles.each do |docfile| 
    if File.exist? docfile.path 
     temporary_information[:file_paths] << new_path 
     temporary_information[:file_names] << something_else 
    end 
    end 
    return temporary_information 
end 

您还可以乘坐这是更进一步和使用inject：

def self.foobar docfiles 
    docfiles.inject({:file_paths => [], :file_names => []}) do |temp_info,docfile| 
    if File.exist? docfile.path 
     temp_info[:file_paths] << new_path 
     temp_info[:file_names] << something_else 
     temp_info 
    end 
    end 
end 

这可能会稍微干净，或不。我喜欢inject，但由于我不认为在速度或效率方面有任何实质性差异，所以这可能只是一个偏好问题。