È una cattiva pratica utilizzare l'assegnazione di riferimento in PHP quando si gestiscono variabili potenzialmente indefinite?

Question

Ho pensato di utilizzare l'assegnazione di riferimento come scorciatoia per gestire variabili potenzialmente non definite.

In altre parole, invece di:

$foo = isset($this->blah['something']['else']) ? $this->blah['something']['else'] : null; 
if (!is_null($foo) && ...){ 
    //do something with $foo 
} 

avrei potuto fare questo:

$foo = &$this->blah['something']['else']; 
if (!is_null($foo) && ...){ 
    //do something with $foo 
} 

Sembra semplice, giusto? A causa del modo in cui PHP gestisce l'assegnazione per riferimento, non devo preoccuparmi di $ this-> blah ['something'] ['else'] in fase di definizione, perché se non esiste verrà creato automaticamente e impostato a NULL.

Questa strategia è disapprovata?

Answer 1

Se si desidera solo per verificare se una variabile è impostata e non nullo, allora è meglio di prova:

if (! empty($this->blah['something']['else'])) {...} 

In questo modo si evita la creazione di due riferimenti ad un valore che non poteva anche non esistere.

A mio parere, ogni volta che si crea un nuovo riferimento allo stesso valore, il codice diventa più difficile da comprendere a colpo d'occhio.

Se avete bisogno di zero non vuota, quindi si crea meglio una funzione globale come questo:

function eempty(& $var) { 
    return empty($var) && $var !== 0; 
} 

Answer 2

Personalmente penso che potrebbe essere essere più convenzionale, ma assegnare un valore di riferimento come quello mi sembra un po 'ridondante. Mi limiterei a usare l'operatore ternario più comunemente usato nella tua situazione. Spesso è molto più chiaro da capire. Certo, dovresti seguire la tua semantica personale il più vicino possibile.