Dubbio su un metodo equals

[Markus85]

Ciao a tutti.

Mi sono imbattuto in un esercizio svolto in cui si fa l'override del metodo equals per confrontare due oggetti di tipo "Memoria".

Ogni oggetto Memoria contiene degli oggetti Variabile.

Non ho capito bene come il metodo verifica l'uguaglianza dei due oggetti:

@Override
public boolean equals(Object o){
if(o==this) return true;
if(o==null) return false;
if(!(o instanceof Memoria)) return false;
Memoria m = (Memoria)o;
for(Variabile v : this)
 if(!m.contains(v)) return false;
 for(Variabile v : m)
  if(!this.contains(v)) return false;
return true;

il metodo contains itera l'oggetti this per vedere se contiene la variaile v.

Questo equals non fa il confronto due volte?

Cioè basterebbe fare l confronto una sola volta :

@Override
public boolean equals(Object o){
if(o==this) return true;
if(o==null) return false;
if(!(o instanceof Memoria)) return false;
Memoria m = (Memoria)o;
for(Variabile v : this)
 if(!m.contains(v)) return false;
return true;

O forse ho interpretato male io?

Aiutatemi

CIAO

[valia]

io dico di no, perché controlli due cose differenti:
Pensa di avere due insiemi A e B.I due sono uguali se e solo se tutto A è in B e tutto B è in A.
Il fatto che tutto A sia in B non assicura il contrario.
Stai testando quello

[andbin]

Originariamente inviato da Markus85
Ogni oggetto Memoria contiene degli oggetti Variabile.

for(Variabile v : this)

Dovresti chiarire, perlomeno a me, perché stai facendo un "for-each" sul this.
La tua classe è Iterable? Se sì, iterabile di che cosa? E poi "contiene degli oggetti Variabile", quanti e come (e se centra con Iterable)?

Senza capire questo, è un po' difficile dire se è ok o no.

[Andrea1979]

a parte che senza parentesi grafe quel codice è disgustosamente illegibile, comunque il concetto è questo: considera Memoria m1 e m2

m1 contiene
Variabile v1, v2, v3

m2 contiene
Variabile v1, v2, v3, v4

se tu verifichi solo se il set di Variabile di m1 sia contenuto in m2 per dire se m1 e m2 sono uguali otterrai true. Ma m2 contiene anche un Variabile, v4, non contenuto in m1. Ergo, bisogna verificare che anche tutto quanto contenuto in m2 sia contenuto in m1.

[valia]

si io ho dato per scontato che le operazioni fossero valide e che ci fossero problemi a capire perché fai la doppia iterazione

sulle Collection io faccio un controllo diverso

boolean ret = true; 
List A ;
List B; 

if (A.size() == B.size()){
//con buona probabilità sono uguali
 for (Object a1 : A){
    if (!b.contains(a1) ){
        ret = false; 
        break;  
   }
 }
}else {
  ret = false; 
}
return ret;

[Markus85]

Giusto!

Grazie per l'aiuto ragazzi!

CIAO

[andbin]

Io però non ho ancora capito perché facevi/fai il for-each sul this. Giusto per curiosità mia ... eh!