Ridefinizione equals

[cavalieredimarte]

Salve a tutti!
Devo svolgere un programma dove è necessaria la ridefinizione in overloading e overriding del metodo equals, x la sorgente si di codice che ho postato sono corretti i metodi?

public class Calcolatore implements Serializable {
private String nome;
private int primo;
private int secondo;
private int terzo;
private static final long serialVersionUID=45L;
public Calcolatore (int primo, int secondo,int terzo,String nome)
{
this.primo=primo;
this.secondo=secondo;
this.terzo=terzo;
this.nome=nome;

}

public boolean equals (Object obj)
{
if (obj istanceof Calcolatore)
Calcolatore k = (Calcolatore) obj;
return (k.getNome().compareTo(nome)==0) && (k.getPrimo==primo)....
}
else return false;
}
public boolean equals (Calcolatore k)
{
return (k.getNome().compareTo(nome)==0) && (k.getPrimo==primo)....
}

[banryu79]

Io di solito faccio così:

	public boolean equals (Object obj)
	{
	    if (obj istanceof Calcolatore)
	    {
	        Calcolatore k = (Calcolatore) obj;
	        if((k.getNome().compareTo(nome)==0) && (k.getPrimo==primo)...)
	        {
	             return true;
	        }
	    }
	    
	    return false;
	}

Basta un metodo unico, che prende come argomento un Object: per me è più comodo avere un'unico metodo così ho un solo punto nel codice dove leggere.

Inoltre, sempre per motivi di leggibilità, preferisco procedere con degli if annidati che vanno a verificare la verità dell'uguaglianza e alla fine della catena torno true; altrimenti si esce con false.



P.S.: dato che "nome" è di tipo String per fare il confronto potresti sfruttare il metodo .equals di String, così:

...
       f(k.getNome().equals(nome) && k.getPrimo==primo)...)
       {
            return true;
       }
...

che è un po' più leggibile