Quando dovrei usare o meno BooleanUtils.isTrue (...) e BooleanUtils.isFalse (...)? [chiuso]

Question

Quando dovrei usare o meno BooleanUtils.isTrue (...) e BooleanUtils.isFalse (...)? [chiuso]

#1 da (5 voti)
#2 da (2 voti)
#3 da (1 voti)

5

Informazioni su questa funzione: org.apache.commons.lang3.BooleanUtils.isFalse(Boolean bool) e isTrue simile, il mio lavoro (meno esperto) lo usa per ogni booleano nel codice. Sto cercando di convincerlo che non è una buona pratica, perché rende il codice confuso. Nel mio POV, dovrebbe essere usato SOLO se il booleano da testare è un tipo di non-primitivo Boolean e può essere null . In realtà, anche questo penso che non sia necessario, perché l'implementazione di questa funzione è semplicemente Boolean.TRUE.equals(bool) o Boolean.FALSE.equals(bool) . Ad ogni modo, penso che sia totalmente pazzesco fare qualcosa del tipo:

boolean isReady = true;

if (BooleanUtils.isTrue(isReady)) {
   // ...
}

quando fai freddo semplicemente

if (isReady) {
   // ...
}

o (!isReady) per il contrario.

Il suo unico argomento per usare questo è "è facile da leggere". Non riesco ad accettare questo argomento. Ho sbagliato? Quali argomenti posso usare per convincerlo che in questo caso nessun codice inutile è meglio di un codice inutile?

Grazie ragazzi.

programming-practices java apache

posta Felipe 12.06.2013 - 03:02

fonte

3 risposte

2

1) I bug provengono dal codice. Più codice hai, più bug. Cerca sempre di scrivere meno codice, anche se sono solo alcune parole qua e là.

2) Facile non è il più facile. Qual è più semplice da leggere?

Prova a scendere a compromessi con if(isReady == True) , se davvero desidera un confronto esplicito.

risposta data 12.06.2013 - 03:55

fonte

1

Penso che l'obiettivo dovrebbe essere quello di avere nomi descrittivi delle variabili booleane, come hai mostrato nell'esempio.

risposta data 12.06.2013 - 06:27

fonte

Leggi altre domande sui tag programming-practices java apache

Passare un oggetto tra diversi gestori o una Superclasse che contiene gran parte della logica Stabilire un'API per fornire accesso app / script agli utenti finali a più tipi di database

score 5 · Accepted Answer

È assurdo usare BooleanUtils.isTrue(isReady) quando isReady è un boolean , non un Boolean . Il tuo collega sembra uno di quei ragazzi che pensa che più codice scrive, più produttivo è. Forse dovrebbe andare a trovare un lavoro di programmazione in Ada.