Chiaro con una speciale istruzione condizionale

Question

Ti mostrerò una piccola parte della mia app in cui mi sto chiedendo quale sia il modo corretto di mettere un condizionale su cui sto lavorando. Se entrambi i modi vi mostrerò ragione, vorrei che tu mi dica le conseguenze/avversità

if ((some.thing === '' || 0) || (some.how === '' || 0)) { 
     //something is going on here 
} 

è così che ho finora, c'è qualcosa di brutto con esso?

o dovrebbe essere meglio così:

if ((some.thing === '' || some.thing === 0) || (some.how === '' || some.how === 0)) { 
     //something is going on here 
} 

quindi quali sono i vostri suggerimenti? è lo stesso risultato alla fine?

EDIT

Aggiunta di un altro modo:

if (some.thing === '' || some.thing === 0 || some.how === '' || some.how === 0) { 
     //something is going on here 
} 

Answer 1

L'operatore ha || precedenza sugli operatori di confronto.

Quindi some.thing === '' || 0 corrisponde a (some.thing === '') || (0). It:

• valuterà a 0 se some.thing === '' è false o
• valuterà a true se some.thing === '' è true.

Guardate questo esempio (come in esecuzione in una console JavaScript):

> some = { thing: 0 } 
    Object { thing: 0 } 
> some.thing === '' 
    false 
> some.thing === '' || 0 // this is like false || 0 
    0 
> some = { thing: '' } 
    Object { thing: "" } 
> some.thing === '' 
    true 
> some.thing === '' || 0 // this is like true || 0 
    true 

Preferisco qualcosa come l'ultima espressione che hai scritto.

EDIT: realtà sia '' e 0 sono falsy in JavaScript, in modo da poter semplicemente scrivere l'espressione completa come:

if (!some.thing || !some.how) { 
    //something is going on here 
} 

Answer 2

|| non è come IN clausola di SQL. Fondamentalmente questo è quello che state facendo:

someBoolean || 0 

così non è mai confronta con l'uguaglianza some.thing type-safe con il Number0.

Answer 3

Questa parte sembra pesce a me:

some.thing === '' || 0 

Vuoi dire questo, invece:

some.thing === '' || some.thing === 0 

0 è sempre falsy, quindi l'espressione some.thing === '' || 0 è sempre equivalente a some.thing === ''

EDIT

è necessario l'espressione finale:

(some.thing === '' || some.thing === 0 || some.how === '' || some.how === 0) 

Answer 4

Sul primo esempio, si sono fondamentalmente facendo questo:

if ((condition1 || condition2) || (condition3 || condition4)) 

Ora condition1 e la condizione 3 può essere vera o falsa, a seconda di ciò che è contenuto di some.thing e some.how, ma le condizioni 2 e 4 saranno sempre falsa (0 è falso), quindi, in pratica quello che stai dicendo è:

if (condition1 || condition3) 

tua seconda d esempio:

if (condition1 || condition2 || condition3 || condition4) 

tuo terzo esempio, sembra più simile a questo:

if ((condition1 || condition2) || (condition3 || condition4)) 

E ora tutti possono essere false o vere (per entrambe le seconde e terze esempi).

Tenete a mente, però, se per esempio some.thing è '' mentre some.how è 'qualcosa' l'intero insieme di condizioni sarà vero.

Answer 5

Utilizzare il secondo snippet di codice. Il primo frammento di codice non è valido, perché

if ((some.thing === '' || 0) || (some.how === '' || 0)) { 
     //something is going on here 
} 

pugno valutare some.thing === '' che può essere vero o falso allora restituire questo risultato o falso, in modo che il codice è finalmente

if (some.thing === '' || some.how === '') { 
     //something is going on here 
}