oggetto! = Verifica nulla

Question

Questa verifica è se l'oggetto passato null o no è OK o dovrei usare un tipo di metodo equals()?

public void addCard(Card card) throws IllegalArgumentException { 
     if (card != null){ 
      cardList.add(card); 
     } else { 
      throw new IllegalArgumentException(); 
     } 
    } 

Answer 1

preferisco farlo in questo modo:

/** 
* Adds a card to the deck. 
* 
* @param the card to add. 
* @throws IllegalArgumentException if the card is null. 
*/ 
public void addCard(final Card card) 
{ 
    if(card == null) 
    { 
     throw new IllegalArgumentException("card must not be null"); 
    } 

    cardList.add(card); 
} 

1. è meno codice per leggere
2. separa la condizione di errore dalla condizione di attesa (nessun altro = senza trattino)
3. nessuno ha bisogno di vedere cosa è in linea X per vedere che la variabile della carta era nullo = meno tempo per le persone che cercano di scoprire cosa hanno fatto di sbagliato.
4. non c'è bisogno di ingombrare il codice con inutili lanci dichiarazioni - si dovrebbe Javadoc, però con una clausola @throws

Oltre a questo, si sta facendo nel modo giusto, a mio parere.

Answer 2

Come si sta confrontando il riferimento è necessario utilizzare !=. L'utilizzo di equals genererà un'eccezione.

Answer 3

Se si desidera verificare se le schede aggiunte non sono null, il codice funzionerà correttamente. Devi solo assicurarti di gestire lo IllegalArgumentException quando chiami addCard.

Answer 4

Questo è corretto, ma personalmente lo strutturerei al contrario. E 'comune per convalidare argomenti all'inizio del metodo, e poi usarli per il resto del metodo sapendo che sono corretto:

public void addCard(Card card) { 
    if (card == null) { 
     throw new IllegalArgumentException("Attempt to add null card"); 
    } 
    cardList.add(card); 
} 

Il vantaggio di fare tutti i test argomento up-front è che se un viene passato un argomento non valido, si genera un'eccezione prima che si verifichino effetti collaterali - piuttosto che a metà del metodo, che potrebbe lasciare l'oggetto in uno stato non valido. Naturalmente in questo caso non importa, ma io favorire la coerenza qui :)

Si noti che non c'è bisogno di dichiarare IllegalArgumentException - è una sottoclasse di RuntimeException, che significa che è deselezionata (non è necessario di dichiararla).

Answer 5

È possibile utilizzare la classe commons-langValidate. Si rende il codice più breve e più semplice:

public void addCard(Card card) { 
    Validate.notNull(card); 
    cardList.add(card); 
} 

si getterà un IllegalArgumentException con un messaggio di default (è possibile passare un messaggio personalizzato come secondo argomento, se vi piace)

Answer 6

Effective Java da Josh Blosh raccomanda il seguente:

/** 
* ads a card to card list. 
* 
* @param card card to add. Can not be null 
* 
*/ 
public void addCard(Card card) { 
     if (card == null) { 
      throw new NullPointerException("Card can not be null") 
     } 
     cardList.add(card); 
    } 

Quindi, in primo luogo non si dichiara il RuntimeException. In secondo luogo lanci una NullPointerException perché il tuo argomento non è semplicemente errato - è nullo. In terzo luogo, si specificano gli argomenti in javadoc, possono essere nulli o meno.