19

我对 Ruby 有点陌生，虽然我发现它是一种非常直观的语言，但我很难理解隐式返回值的行为方式。

我正在开发一个小程序来 grep Tomcat 日志并从相关数据生成管道分隔的 CSV 文件。这是我用来从日志条目生成行的简化示例。

class LineMatcher
  class << self
    def match(line, regex)
      output = ""
      line.scan(regex).each do |matched|
        output << matched.join("|") << "\n"
      end
      return output
    end        
  end
end


puts LineMatcher.match("00:00:13,207 06/18 INFO  stateLogger - TerminationRequest[accountId=AccountId@66679198[accountNumber=0951714636005,srNumber=20]",
                       /^(\d{2}:\d{2}:\d{2},\d{3}).*?(\d{2}\/\d{2}).*?\[accountNumber=(\d*?),srNumber=(\d*?)\]/)

当我运行此代码时，我会返回以下内容，这是显式返回输出值时所期望的。

00:00:13,207|06/18|0951714636005|20

但是，如果我将 LineMatcher 更改为以下内容并且不明确返回输出：

    class LineMatcher
      class << self
        def match(line, regex)
          output = ""
          line.scan(regex).each do |matched|
            output << matched.join("|") << "\n"
          end
        end        
      end
    end

然后我得到以下结果：

00:00:13,207
06/18
0951714636005
20

显然，这不是我们想要的结果。感觉我应该能够摆脱输出变量，但不清楚返回值来自哪里。此外，欢迎任何其他关于可读性的建议/改进。

4

2 回答 2

25

ruby 中的任何语句都返回最后一个计算表达式的值。您需要了解最常用方法的实现和行为，才能准确了解您的程序将如何运行。

#each返回您迭代的集合。也就是说，以下代码将返回 line.scan(regexp) 的值。

line.scan(regex).each do |matched|
  output << matched.join("|") << "\n"
end

如果要返回执行的结果，可以使用map，它的作用是each但返回修改后的集合。

class LineMatcher
  class << self
    def match(line, regex)
      line.scan(regex).map do |matched|
        matched.join("|")
      end.join("\n") # remember the final join
    end        
  end
end

根据您的具体情况，您可以使用几种有用的方法。在这个你可能想要使用inject，除非返回的结果数量scan很高（处理数组然后合并它们比处理单个字符串更有效）。

class LineMatcher
  class << self
    def match(line, regex)
      line.scan(regex).inject("") do |output, matched|
        output << matched.join("|") << "\n"
      end
    end        
  end
end
于 2009-06-30T15:25:49.287 回答
14

在 ruby​​ 中，方法的返回值是最后一条语句返回的值。您也可以选择有明确的回报。

在您的示例中，第一个代码段返回 string output。然而，第二个片段返回each方法返回的值（现在是最后一个 stmt），结果是一个匹配数组。

irb(main):014:0> "StackOverflow Meta".scan(/[aeiou]\w/).each do |match|
irb(main):015:1* s << match
irb(main):016:1> end
=> ["ac", "er", "ow", "et"]

更新：但是，这仍然不能在一行中解释您的输出。我认为这是一个格式错误，它应该在不同的行上打印每个匹配项，因为这就是puts打印数组的方式。一点代码可以比我解释得更好..

irb(main):003:0> one_to_three = (1..3).to_a
=> [1, 2, 3]
irb(main):004:0> puts one_to_three
1
2
3
=> nil

就我个人而言，我发现您的显式返回方法更具可读性（在这种情况下）

于 2009-06-30T15:10:43.047 回答