使用红宝石触发器作为滤波器可以减少kludgy？

Question

为了获得部分文本，我在触发器前面使用了一个true if kludge：

desired_portion_lines = text.each_line.find_all do |line|
  true if line =~ /start_regex/ .. line =~ /finish_regex/
end
desired_portion = desired_portion_lines.join

如果我删除了true if位，它会抱怨

范围的错误值（ArgumentError）

有可能减少它的价值，或者我应该这样做

desired_portion_lines = ""
text.each_line do |line|
  desired_portion_lines << line if line =~ /start_regex/ .. line =~ /finish_regex/
end

或者是否有更好的方法不使用枚举？

Answer 1

如果你是逐行做的，我的偏好是这样的

line =~ /finish_regex/ && p=0
line =~ /start_regex/ && p=1
puts line if p

如果你有一个字符串。 我会用拆分

mystring.split(/finish_regex/).each do |item|
  if item[/start_regex/] 
     puts item.split(/start_regex/)[-1]
  end
end

Answer 2

我认为

desired_portion_lines = ""
text.each_line do |line|
  desired_portion_lines << line if line =~ /start_regex/ .. line =~ /finish_regex/
end

完全可以接受。 ..运算符非常强大，但很多人都没有使用它，可能是因为它们不了解它的作用。 可能它看起来很奇怪或尴尬，因为你不习惯使用它，但它会在你身上成长。 在处理文本文件中的行范围时，它在Perl中非常常见，这是我第一次遇到它，并最终使用它很多。

我唯一不同的做法是添加一些括号，以便在逻辑上将逻辑测试彼此分开，并从行的其余部分：

desired_portion_lines = ""
text.each_line do |line|
  desired_portion_lines << line if ( (line =~ /start_regex/) .. (line =~ /finish_regex/) )
end

Ruby（和Per​​l）编码器似乎厌恶使用括号，但我认为它们对于在视觉上分离逻辑测试很有用。 对我来说，这是一个可读性，并且，通过扩展，它是一个维护的东西。

我能想到的另一件事可能会有所帮助，那就是将desired_portion_lines更改为一个数组，然后将选定的行推到它上面。 目前，使用desired_portion_lines << line追加到字符串，每次都要改变它。 推送数组然后加入其元素以构建字符串可能会更快。

回到第一个例子。 我没有对此进行测试，但我认为您可以将其简化为：

desired_portion = text.each_line.find_all { |line| line =~ /start_regex/ .. line =~ /finish_regex/ }.join

使用触发器迭代文件中所有行的唯一缺点是，如果启动模式可以多次出现，则会将每个找到的块添加到desired_portion 。

Answer 3

true if使用!!() （触发器属于括号之间!!() ，则可以通过替换true if来保存三个字符。