Java - Stile di codifica: quali sono i contro e i pro del confronto di stringhe di stile "ABC" .equals ("SOMESTRING")?

Question

Permettetemi innanzitutto di inizio con un codice di esempio ...

String password = ""; 

if("PIRATE".equals(password)) { 
    // Do something 
} 

vedere qui, il String costante o letterale (qualunque cosa) "pirata" viene utilizzato per verificare l'uguaglianza delle due stringhe. Mentre ...

String password = ""; 
if(password.equals("PIRATE")) { 
    // Do something 
} 

questo funziona anche esattamente come il codice precedente.

Oggigiorno, vedo molto del primo stile "STRING_LITERAL" .equals (STRING_OBJECT), mentre le persone Java inviano il codice.

Quindi la mia domanda è, Da dove viene questo stile? e È meglio del secondo stile in alcun modo?

In effetti trovo il secondo stile più logico del primo, perché?!

consente di dare un requisito del genere, se l'utente fornito-password è uguale al "pirata" poi dare il permesso di quell'utente

quando si avvia l'attuazione del requisito di cui sopra,

String userProvidedPassword = getPaswordFromUser(); 

if(userProvidedPassword.equals("PIRATE")) { 
// Allow him 
} 

Non è più logico di "PIRATE" .equals (userProvidedPassword);?! Basti pensare che ...

mi corregga se sbaglio .. Grazie ..

---

EDIT: Scusate, this question non è venuto nella mia ricerca precedente, e risponde alla mia domanda perfettamente. Anche grazie a tutti quelli che hanno dato una mano qui ..

Answer 1

Quando si scrive password.equals("PIRATE") si è quasi chiedendo un NullPointerException, dove ci sono possibilità per password forza null.

Si tratta di evitare NullPointerException.

if("PIRATE".equals(password)) { 
    // Do something 
} 

Evita NullPointerException

dove come

if(password.equals("PIRATE")) { 
    // Do something 
} 

thrwos si NullPointerException se password è null.

Tuttavia, Personalmente ritengo che questo sia strano nel mezzo del codice. e preferisco sempre scrivere

if(password !=null && password.equals("PIRATE")){ 

//do something 

} 

Answer 2

"PIRATE".equals(password) non può dare come risultato NullPointerException.

mentre

password.equals("PIRATE") lancerà una NullPointerException se password sembra essere null.

Pertanto l'uso del primo è incoraggiato: è più sicuro.

Answer 3

L'unico inconveniente è davvero un problema di stile. Dicendo "PIRATE" .equals (password) si chiama Yoda Condition. Tuttavia, come hanno già affermato le persone, è più sicuro usare quel tipo di condizione (quindi continuerei a usarlo).