使用 ruby​​ 以 DRY 方式提取哈希中的值

Question

我的应用程序将 json_element 传递给不同的方法，其中的键是不同的，有时是空的。

为了处理它，我一直在使用以下示例代码对提取进行硬编码：

def act_on_ruby_tag(json_element)

begin

  # logger.progname        = __method__
  logger.debug             json_element

  code                   = json_element['CODE']['$'] unless json_element['CODE'].nil?
  predicate              = json_element['PREDICATE']['$'] unless json_element['PREDICATE'].nil?
  replace                = json_element['REPLACE-KEY']['$'] unless json_element['REPLACE-KEY'].nil?
  hash                   = json_element['HASH']['$'] unless json_element['HASH'].nil?

我想消除对值的硬编码，但不太确定如何。

我开始思考如下：

keys = json_element.keys
keys.each do |k|
        set_key = k.downcase
        instance_variable_set("@" + set_key, json_element[k]['$']) unless json_element[k].nil?
      end 

然后在方法的其余部分使用@code。

我本来打算把它变成一个方法，然后替换掉所有这些硬编码的代码。

但我不完全确定这是否是一条好路。

Answer 1

从具有{ code: ... } 之类的方法返回哈希结构几乎总是比设置任意实例变量更好。如果您将它们返回到一致的容器中，调用者就可以更轻松地处理将其交付到正确的位置、存储以供以后使用，或者挑选出他们想要的东西并丢弃其余的。

尝试通过一系列更小、更轻的操作来打破一个大而笨重的步骤也是一个好主意。这使得代码更容易理解：

def extract(json)
  json.reject do |k, v|
    v.nil?
  end.map do |k, v|
    [ k.downcase, v['$'] ]
  end.to_h
end

然后你得到这个：

extract(
  'TEST' => { '$' => 'value' },
  'CODE' => { '$' => 'code' },
  'NULL' => nil
)
# => {"test"=>"value", "code"=>"code"}

如果您想将整个内容作为实例变量持久化，这是一种相当典型的模式，但它会有一个可预测的名称，不受您正在使用的任意 JSON 文档的支配。

另一种方法是将键硬编码为常量，例如：

KEYS = %w[ CODE PREDICATE ... ]

然后改用它，或者更进一步，在 YAML 或 JSON 文件中定义它，您可以读入以进行配置。这实际上取决于这些变化的频率，以及您对输入的不规则性有什么样的期望。

Answer 2

这是一种更简洁的方式来完成原始代码的工作。

code, predicate, replace, hash = json_element.values_at *%w{
  CODE PREDICATE REPLACE-KEY HASH
}.map { |x| x.fetch("$", nil) if x }