Metodo equals ed ereditarietà tra classi

[gaten]

Ragazzi io ho una classe operaio (classe Op) in cui ho i seguenti attributi:
nome (String) e salario (short), mentre un operaio specializzato
(classe OpSp, sottoclasse di Op), in aggiunta possiede una specializzazione (riferimento ad un oggetto di tipo Specialty).

Devo implementare(usando equals) questo criterio di uguaglianza:
Due operai semplici sono uguali se hanno lo stesso nome. Due operai specializzati, in più, devono avere anche la stessa specializzazione (nel senso di ==). Un operaio semplice non è mai uguale ad un operaio specializzato.

Io avevo pensato di mettere sicuramente in Op equals in questo modo:

public class Op {
   String nome;
   short salario;

   public Op {...}

   public boolean equals(Object o) {
     if (!(o instanceof Op) return false;
     Op operaio = (Op)o;
     
     return nome.equals(operaio.nome);
   }
}

public class OpSp extends Op {
 // qui come ridefinisco equals ?
}

poichè nome è di tipo String, facendo nome.equals(operaio.nome), richiama equals di String confrontanto se i nomi sono uguali, giusto?

Come potrei fare riguardo la specializzazione, salvo restando che ho fatto bene per la classe Op.

[Zorgatone]

public class Op {
   String nome;
   short salario;

   public Op {...}

   public boolean equals(Object o) {
     if (!(o instanceof Op) return false;
     Op operaio = (Op)o;
     
     return nome.equals(operaio.nome);
   }
}

public class OpSp extends Op {
   Speciality specializzazione;

   public OpSp {...}

   @Override
   public boolean equals(Object o) {
     if (!(o instanceof OpSp) return false;
     OpSp operaio = (OpSp)o;
     
     return nome.equals(operaio.nome);
   }
}

[gaten]

Io avevo pensato ad una cosa del genere, magari qualcuno che acconsente o specifica eventuali errori:

public class Op {
   String nome;
   short salario;

   public Op {...}

   public boolean equals(Object o) {
     if (o.getClass() != Op.class) return false;
     Op operaio = (Op)o;
     
     return nome.equals(operaio.nome);
   }
}

public class OpSp extends Op {
   Speciality spec;

   public boolean equals(Object o) {
     if (o.getClass() != OpSp.class) return false;
     
     if ( !(super.equals(o) ) return false;

     OpSp operaio = (OpSp)o;    

     return ( spec == operaio.spec);
   }
}

Grazie anticipatamente.

[Alex'87]

ps1: Op e OpSp sono nomi orrendi, cerca di migliorare la leggibilità.
ps2: nell'equals non hai controllato il caso "null" e il caso "this", ad esempio:

@Override
public boolean equals(Object other) 
{
    if (other == null || other.getClass() != Op.class)
    {
        return false;
    }

    if (other == this) 
    {
        return true;
    }

    Op op = (Op) other;
    return nome.equals(op.nome);
}

[gaten]

Alex riguardo all'equals da mettere in OperaioSpecializzato?

L'opraio specializzato oltre ad avere lo stesso nome deve avere la stessa specializzazione in termini di (==).
Come lo faresti?

[Alex'87]

Originariamente inviato da gaten
Alex riguardo all'equals da mettere in OperaioSpecializzato?

L'opraio specializzato oltre ad avere lo stesso nome deve avere la stessa specializzazione in termini di (==).
Come lo faresti?

Con un AND?

OpSp opSp = (OpSp) other;
return nome.equals(opSp.nome) && spec == opSp.spec;

Mi pare manchino un po' le basi...

ps3: i campi con visiblità di default sono da evitare: mettili private (o protected visto che c'è l'ereditarietà in mezzo...) e metti getter e setter.

[gaten]

Sono tutti private i campi, tranne l'attributo nome(protected) in Operaio, poichè dev'essere ereditato da OperaioSpecializzato.

Qualora invece il criterio di uguaglianza è legato al fatto che un operaio semplice è uguale ad uno specializzato se hanno lo stesso nome e la specializzazione di quest'ultimo è null, come andrebbero ridefiniti gli equals.

In questo caso è inevitabile usare il metodo getClass in entrambi gli equals poichè dice che gli operai semplici non sono mai uguali ad operai specializzati. Giusto?