修饰符形式的三元和救援的简短替代

Question

加载文件时，我曾经做过一些变体：

first_file = "#{__dir__}/first_file.txt"
first_data = File.exist?(first_file) ? File.readlines(first_file) : []

second_file = "#{__dir__}/second_file.yaml"
second_data = File.exist?(second_file) ? YAML.load_file(second_file) : {}

这对我来说总是有点不对劲。需要检查文件是否存在是需要存在的东西，但我真正关心的是之后发生的事情。

最终我发现了内联rescue，并开始这样做：

first_file = "#{__dir__}/first_file.txt"
first_data = File.readlines(firs_file) rescue []

second_file = "#{__dir__}/second_file.yaml"
second_data = YAML.load_file(second_file) rescue {}

这对我来说更清楚——尝试做你真正想要的事情，如果你不能做其他事情，那就是你想要的后备。它比三进制更短更清晰。

但我在任何地方都重复了对avoid rescue in modifier form 的相同建议。我理解为什么会这样：它可以隐藏我们没有预测到的错误。

如果我可以将Errno::ENOENT 定义为内联表单中rescue 的错误，我会的。但据我所知，inline rescue 无法指定要从中拯救的错误。

那么有什么选择呢？我能做什么像内联rescue一样简短明了，但又没有缺点？

Answer 1

如果我真的担心这些行的可读性，我会创建一些辅助方法。

def read_with_fallback(filename, fallback)
  File.exist?(filename) ? yield(File.open(filename)) : fallback
end

def safe_read_lines(filename)
  read_with_fallback(first_file, []) {|file| file.readlines }
end

def safe_read_yaml(filename)
  read_with_fallback(second_file, {}) {|file| YAML.parse(file.read) }
end

first_file = "#{__dir__}/first_file.txt"
first_data = safe_read_lines(first_file)

second_file = "#{__dir__}/second_file.yaml"
second_data = safe_read_yaml(second_file)

但是，如果我只有其中的一两个，并且不必在其他所有方法中重复它们，那么我就不会费心了。您的第一个表单“足够好”。