Co to jest Rails Way? (czytelność vs suszenie)

Question

Mam model użytkownika, który ma metody głosowania. Chcę pisać metody proxy do głosowania.

Jest to czytelny sposób:

def vote_up item 
    return false unless can? :vote, item 
    vote item, :up 
end 

def vote_down item 
    return false unless can? :vote, item 
    vote item, :down 
end 

I to jest suchy sposób:

%w(up down).each do |vtype| 
    define_method "vote_#{vtype}" do |item| 
    return false unless can? :vote, item 
    vote item, vtype.to_sym 
    end 
end 

Który z nich jest lepszy i dlaczego?

Answer 1

wyłącznie dlatego OP wydawała się podoba mój komentarz, włożę go jako odpowiedź:

Osobiście rozważa masz 2 sposoby tylko tutaj, i to jest mało prawdopodobne, że kiedykolwiek dodać więcej (vote_sideways vote_diagonally?) Po prostu pójdę w czytelny sposób. Jeśli mógłbyś mieć wiele, wiele więcej, chciałbym pójść w sposób DRY (ponieważ staje się łatwo rozszerzalny) z czytelnym komentarzem do wyjaśnienia innym programistom (lub sobie później!).

Answer 2

Ani (przepraszam).

def vote_count(item,vtype) 
    return false unless can? :vote, item 
    vote item, vtype 
end 

Powodzenia

Answer 3

IMHO, w tym przypadku, czytelność przebija suche. Szybko skanuje i łatwo go zgadnąć. Powiedziawszy to, jeśli zaczniesz dodawać rodzaje głosów, drugie podejście może być bardziej elastyczne. YMMV.

Answer 4

Obie.

Jestem z Anilem; po prostu przekazujmy typ - meta-programowanie to jako pierwsza instancja jest paskudna.

To powiedziawszy, I am fanem wygodnych metod - ale powinny one wywoływać rodzajową metodę z typem.

Dzięki temu generowana metoda jest zwięzła - prawdziwa praca jest wykonywana w sposób ogólny, ale użytkownik interfejsu API wciąż otrzymuje te same metody wygody.