Qual è il modo corretto per verificare la presenza di oggetti in Rails/Ruby?

Question

Ho molti modelli e relazioni. A causa di questo fatto, c'è sacco di chiamate in views/controllori, che assomigliano a questo:

@object.something.with_something.value 

Alcuni parte della catena può finire per essere pari a zero, che è perfettamente ok. Qual è il modo corretto/pulito/veloce per verificare l'esistenza dell'oggetto terminale?

chiama qualcosa come:

@object.something.with_something.value if defined? @object.something.with_something.value 

considerato ok?

Answer 1

In modo nativo, è necessario utilizzare l'operatore && (non defined?), ma questo può diventare molto dettagliato, molto rapidamente.

Così, invece di fare questo:

(@object && @object.something && @object.something.with_something && 
    @object.something.with_something.value) 

È possibile fare questo quando ActiveSupport è presente:

@object.try(:something).try(:with_something).try(:value) 

o installare il invocation construction kit e utilizzare i suoi strumenti di valutazione custodito:

Ick::Maybe.belongs_to YourClass 
maybe(@object) { |obj| obj.something.with_something.value } 

Answer 2

what.you.are.doing è talvolta denominato "Train wreck". Viene anche descritto come una violazione della legge di Demetra.

Detto questo, penso che ci sia qualcosa chiamato "andand" che può aiutare con quello che stai facendo.

Answer 3

È meglio organizzare il resto del codice per vedere questo problema al massimo l'ultimo oggetto in una catena.

defined? non farà quello che vuoi. Qualcosa può essere defined? e nil allo stesso tempo.

Quando il problema è limitato a l'ultimo attributo in una catena di riferimenti:

@object.something.with_something.value if @object.something.with_something 

potrei approfittare dei fatti che:

nil.to_a => [] 
nil.to_s => '' 
nil.to_f => 0.0 
nil.to_i => 0 

Quindi, se si sa che qualcosa sta nil o Array, spesso è possibile scrivere codice migliore senza alcun condizionale scrivendo qualcosa del tipo:

something.to_a.each do |e| 
    . . . 

Answer 4

Un'altra opzione è utilizzare Null Object pattern per garantire che nessuno di questi oggetti sia sempre nullo. Probabilmente, se il tuo codice sta per accedere a catena in questo modo, è necessario definire sempre .