Problema con il metodo equals

[Xert]

Salve ,
sono uno studente universitario di ingegneria informatica ed essendo alle prime armi non sono molto pratico del linguaggio java.
Oggi mi sono appena imbattuto in un problemino con il metodo equals.


Persona p1 = newPersona("Mario", "Rossi");
Persona p2 = new Persona("Mario", "Rossi");
System.out.println(p1.equals(p2));

Questa porzione di codice dovrebbe restituire true ma restituisce false.
Ciò non accade per oggetti di tipo String oppure Rectangle.

Qualcuno sa spiegarmi come risolvere il problema ?
Grazie in anticipo.

[andbin]

Questa porzione di codice dovrebbe restituire true ma restituisce false.
Ciò non accade per oggetti di tipo String oppure Rectangle.

La tua classe Persona deve ridefinire il metodo public boolean equals(Object o), altrimenti resta solo il equals "ereditato" da Object .... che si basa solo sulla identità (operatore == ) degli oggetti.

[Xert]

Si ho provato a ridefinire usando HashCode() and equals() ma lo stesso non funziona

[andbin]

Si ho provato a ridefinire usando HashCode() and equals() ma lo stesso non funziona

hashCode() serve tipicamente (e quasi esclusivamente) nelle collezioni map o set basate su hash-table (HashMap e simili). Se ridefinisci equals o hashCode, "dovresti" ridefinire anche l'altro (c'è un contratto ben preciso tra i due).

Comunque senza vedere cosa hai scritto .... non so che dire ...

[Xert]

Spero di non aver scritto stupidaggini


public class Persona {

public String nome;
public String cognome;


public Persona(String nome, String cognome) {
// TODO Auto-generated constructor stub
}

// TODO Auto-generated method stub

Persona p1 = new Persona("Mario", "Rossi");
Persona p2 = new Persona("Mario", "Rossi");




public boolean equals(Object p2){
Persona other = (Persona) p2;
boolean x= nome.equals(other.nome);
return x;
}



}

[andbin]

Spero di non aver scritto stupidaggini

Se è questo che hai scritto .... mancano un po' di cose. Tecnicamente, il equals è corretto. Ma generalmente non si fa così, cioè non si fa un cast diretto e basta (equals non deve schiantarsi lanciando ClassCastException, invece bisognerebbe testare prima il tipo con l'operatore instanceof)
Il tuo equals comunque confronta solo i nomi ... e oltretutto direttamente sui campi che sono pure public (non è proprio il massimo questo ...).

Il vero punto che non va è che il costruttore è vuoto, non fa nulla, quindi i campi non vengono inizializzati se fai solo new Persona( .... ). E se i campi restano a null ... allora sì che il equals si schianta, e con NullPointerException.

E non ho comunque capito cosa ci fanno (e che senso hanno) i Persona p1/p2 lì in mezzo ...

[Xert]

Io vorrei confrontare il contenuto dei due oggetti mediante l equals e farmi restituire il valore true.
potresti correggermi tu il codice?

[andbin]

Io vorrei confrontare il contenuto dei due oggetti mediante l equals e farmi restituire il valore true.
potresti correggermi tu il codice?

Fai la classe Persona così:
- campi nome/cognome private
- costruttore che riceve nome/cognome e li assegna ai campi
- metodi getter/setter public
- metodo equals che:
a) testa il Object usando instanceof, es. if (obj instanceof Persona)
b) Se è un Persona, fai il cast e confronti i nomi e cognomi tra i due oggetti (e ovviamente restituisci true/false). Usa i getter per prendere i valori.
c) altrimenti restituisci false.

Se pensi sia difficile ... è più facile di quello che pensi.

[Xert]

public class Persona {

private String nome;
private String cognome;



// TODO Auto-generated method stub






public String getNome() {
return nome;
}


public void setNome(String nome) {
this.nome = nome;
}


public String getCognome() {
return cognome;
}


public void setCognome(String cognome) {
this.cognome = cognome;
}




public boolean equals(Object obj){
if(obj instanceof Persona){
Persona other = (Persona) obj ;
return nome.equals(other.nome) && cognome.equals(other.cognome);

}
else {return false;}
}
}


----------------------------------------------------------------------------------------------------------------------------------------------------

public class main {


public static void main(String[] args) {
// TODO Auto-generated method stub
Persona p3 = new Persona();
p3.setCognome("russo");
p3.setNome("pasquale");
Persona p2 = new Persona();
p2.setCognome("russo");
p2.setNome("pasquale");
boolean x = p3.equals(p2);
System.out.print(x);
}


}



Ho appena risolto senza l'ausilio del costruttore grazie per l'aiuto.
Comunque io volevo semplicemente fare un confronto tra due oggetti con parametri differenti passati al costruttore.
Sai spiegarmi il perché non mi funziona così?
Grazie ancora

[Santino83_02]

Battuta: Xert ti prego almeno non dire che studi ingegneria informatica!

Serio:

se questa è la tua classe:

public class Persona {

public String nome;
public String cognome;


public Persona(String nome, String cognome) {
// TODO Auto-generated constructor stub
}
//.... other methods
}

mi spieghi che cosa dovrebbe farci il costruttore con le variabili che passi? Invece

public class Persona{

private String nome;
private String cognome;

public Persona(String nome, String cognome){
this.nome = nome;
this.cognome = cognome;
}

//getters

//equals and hashcode
}

allora si che poi ti funziona l'equals