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_use_list 전화를 photo.land_uses.list 호출 할 수 있습니다.

그러나 특정 사진에 속하는 범위가 지정된 인스턴스에 대해 호출되는 대신 클래스에 대해 호출되기 때문에 작동하지 않습니다.

내가 생각하는 것을 할 수있는 방법이 있습니까? 더 일반적으로 앱에서 이와 같은 문제에 어떻게 접근합니까? LandUse 모델의 목록 코드를 올바른 방법으로 옮기고 있습니까? 아니면 다른 것을 추천 하시겠습니까?

Answer 1

첫째, 나는 Demeter Law 그 자체를 위반하지 않는다고 생각합니다. 속성에 대해 하나의 메소드를 호출하여 임시 변수를 만든 다음 임시 변수에 대해 작업하는 메소드가 있습니다.

다른 계급에서이 작업을 완전히 수행하는 경우 Demeter의 법칙을 위반하는 것입니다. 예 :

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과 상호 작용하는 방법이클래스에 대해 알고 있다면, 그 사진에는 토지 용도의 문자열을 반환하는 메소드 만 있습니다.

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

나는 레일 애플 리케이션의 앞에 아닙니다 그러나 나는 LandUse 객체

의

복귀와

photo.land_uses 

배열 그래서 그냥 같은 그 배열에 아래로지도를 이동해야 믿습니다

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

원래 다른 모델에 있던 모델입니다. 나는 당신이 맞을지도 모른다고 생각합니다. 따라서 Photo은 LandUse에 대해 너무 많이 알고 있습니다. 따라서 그것을 옮길 것입니다.