Ruby/Rails：創建一個對其子項實例進行操作的類方法？

Question

在我的應用程序，Photo has_and_belong_to_many :land_uses

我在Photo模型這個輔助方法：

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

這在我看來是一個代碼味道（得墨忒耳），但我一直無法弄清楚如何將其移至LandUse模型。我想要做的是一樣的東西：

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

這樣而不是調用photo.land_use_list我可以打電話給photo.land_uses.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 

有關關聯擴展的更多信息，請查看docs。

符合德米特法則的最好方法是做或多或少的你在做什麼，因爲通過在Photo上添加你的方法，這意味着與Photo交互的方法也不需要了解LandUse類，只是該照片有一個方法返回一個土地用途名稱的字符串。

Answer 2

您可以使用：

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

    def to_s 
    self.name 
    end 
end 

希望它能幫助！

Answer 3

我不是一個Rails應用程序的前面，但我相信

photo.land_uses 

與回報LandUse對象

數組所以你只需要向下移動地圖到陣列，如：

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

這是你最初的 - 只是在你的其他模型。我想你可能是對的，這意味着Photo知道太多關於LandUse，因此我會將其移出。