[JAVA] Problema metodo

[Mrmorrison]

Salve a tutti, allora avrei una domanda, sto facendo un programmino con una simulazione di chat, vorrei crearmi due metodi (helpers) privati che verifichino la presenza o meno di un oggetto ( nel mio caso un utente) all interno di una lista che gestisce gli utenti collegati. Questi metodi controllano sia se il sender è già presente e sia se il receiver è già presente in modo tale da sfruttarli in un altro metodo che si occuperà dell accesso dell utente.

So che la domanda è stupida ma non riesco a capire perchè il valore mi torna sempre false.
Perdonatemi è da poco che studio java.

public boolean verifySender(User sender){

 if( utentiInChat.contains(sender)){

 }
 return false; 

}

[minomic]

Perchè dentro all'if non c'è nulla, quindi arriva lì, esegue l'istruzione nulla (non fa niente) e poi passa a return false.
Dovrebbe essere sufficiente scrivere così:

public boolean verifySender(User sender){

 if( utentiInChat.contains(sender)){
    return true;
 }
 return false; 

}

o anche

public boolean verifySender(User sender){

 return utentiInChat.contains(sender);

}

[Alex'87]

Oltre a quanto detto da minomic aggiungo: User ridefinisce correttamente il metodo equals?

[Mrmorrison]

Sono un pollo! Grazie della risposta!

[valia]

Originariamente inviato da Alex'87
Oltre a quanto detto da minomic aggiungo: User ridefinisce correttamente il metodo equals?

una nota: oltre ad equals, ricorda hashcode (e il buon LeleFt se ha tempo e voglia ci ricorda il perché bisogna farlo). In soldoni: due oggetti sono uguali se e solo se equals torna true e gli hashcode sono uguali.
Bisogna ricordarsi di implementare tali metodi qualora si lavori con le collection.

Altra soluzione è l'uso di mappe, sfruttando get(chiave) che torna null se l'elemento non è presente, l'elemento se ti serve (e lavori su tipi che in genere sono primitivi, già ottimizzati per ricerche di questo tipo)

[LeleFT]

Originariamente inviato da valia
una nota: oltre ad equals, ricorda hashcode (e il buon LeleFt se ha tempo e voglia ci ricorda il perché bisogna farlo). In soldoni: due oggetti sono uguali se e solo se equals torna true e gli hashcode sono uguali.

Beh, l'hai già detto tu.
La documentazione di equals() (in Object) è chiara:

Note that it is generally necessary to override the hashCode method whenever this method ( equals() ) is overridden, so as to maintain the general contract for the hashCode method, which states that equal objects must have equal hash codes.

Che ricalca quanto esplicitato nella documentazione di hashCode():

If two objects are equal according to the equals(Object) method, then calling the hashCode method on each of the two objects must produce the same integer result.

Ciao.

[valia]

Originariamente inviato da LeleFT
Beh, l'hai già detto tu.

noooo io intendevo la teoria dietro
non la versione corta della javadoc

Vabbé cmq riassunto molto riassunto abbiamo detto tutto (stiamo andando OT, abbiate pazienza il caldo dà alla testa )