我可以用鸭子打字改进这个方法吗？

希望我没有误解"鸭子打字"的含义,但从我所读到的,它意味着我应该根据对象如何响应方法而不是它的类型/类来编写代码.

这是代码:

def convert_hash(hash)
  if hash.keys.all? { |k| k.is_a?(Integer) }
    return hash
  elsif hash.keys.all? { |k| k.is_a?(Property) }
    new_hash = {}
    hash.each_pair {|k,v| new_hash[k.id] = v}
    return new_hash
  else
    raise "Custom attribute keys should be ID's or Property objects"
  end
end

我想要的是确保我最终得到一个散列,其中键是一个表示ActiveRecord对象的ID的整数.我并不特别喜欢两次迭代哈希键all?以确定是否需要获取ID.

当然,我也会接受任何其他改进此代码的建议:)

如何编写此方法应取决于您是否期望在正常程序执行过程中抛出异常.如果您想要一个可读的异常消息,因为最终用户可能会看到它,那么手动抛出一个是有意义的.否则,我会做这样的事情:

def convert(hash)
    new_hash = {}
    hash.each_pair { |k,v| new_hash[ k.is_a?(Integer) ? k : k.id ] = v }
    return new_hash
end

这将完成同样的事情,如果数组键没有id字段,你仍然会得到一个例外.更好的是,这会使用更多的鸭子类型,因为现在任何具有id字段的东西都是可以接受的,这比明确检查某个属性的东西要好.这使您的代码更加灵活,尤其是在单元测试时.

我们仍然对整数对象进行了明确的检查,但这种偶然的特殊情况通常是可以接受的,尤其是在检查内置数据类型时.