Ruby / Rails：创建一个对其子实例进行操作的类方法？

Question

在我的应用程序中，Photo has_and_belong_to_many :land_uses

我在Photo模型中有这个辅助方法：

def land_use_list
  land_uses.map(&:name).join(', ')
end

这让我觉得有代码味道（demeter），但我无法做到找出如何将其移动到 LandUse 模型。我想做的是这样的：

class LandUse < ActiveRecord::Base
  ...
  def self.list
    self.map(&:name).join(', ')
  end
  ...
end

这样我就可以调用 photo.land_uses.list 而不是调用 photo.land_use_list 。

但这不起作用，因为它是针对类调用的，而不是针对属于特定照片的作用域实例调用的。

有办法实现我的想法吗？更一般地说，您如何在应用程序中解决此类问题？将列表代码移至 LandUse 模型是否是正确的方法，或者您会推荐不同的方法吗？

Answer 1

首先，我认为这本身并不违反德墨忒尔定律。您有一个对象上的方法，该方法调用属性上的一个方法来创建临时变量，然后对该临时变量进行操作。

如果你是从完全不同的阶层做这件事，那就违反了德墨忒尔法则。例如

class User
  def names_of_lands_ive_known
    photos.map(:land_uses).map(:name).join ', '
  end
end

，事实上，它只是很好的信息隐藏。但是，如果您希望能够编写 photo.land_uses.names，您可以向关联添加扩展来执行您想要的操作。

class Photo
  has_and_belong_to_many :land_uses do
    def names_as_list_string
      all.map(:name).join ', '
    end
  end
end

有关关联扩展的更多信息，请查看文档。

遵守德米特定律的最佳方法是或多或少地做你正在做的事情，因为通过在 Photo 上添加你的方法，这意味着与 Photo< 交互的方法/code>，也不需要了解 LandUse 类，只需该照片有一个返回土地用途名称字符串的方法即可。

Answer 2

我不在 Rails 应用程序前面，但我相信

photo.land_uses

会返回一个 LandUse 对象数组

，因此您只需将地图向下移动到该数组，例如：

photo.land_uses.map(&:name).join(', ')

这就是您最初拥有的 - 就在您的其他型号。我认为你可能是对的，这意味着 Photo 对 LandUse 了解太多，因此我会将其移出。

Answer 3

您可以使用：

class LandUse
  def self.list_for_photo(id)
    LandUse.find_by_photo_id(id).join(', ')
  end

  def to_s
    self.name
  end
end

希望有帮助！