¿comparar objetos en Java?

Question

Estoy usando Java 8, Netbeans 8.2 y JUnit 4.12.

Tengo dos instancias de una clase que quiero comparar para saber si son "iguales", donde sus propiedades tienen los mismos valores. Es decir que si tienen id y nombre, si escribo en el test:

assertEquals(instanceA.getId(), instanceB.getId( ));
assertEquals(instanceA.getName(), instanceB.getName( ));

pasa OK, pero si añado un tercer assert comparándolos directamente:

assertEquals(instanceA, instanceB);

o bien

assertTrue(instanceA.equals(instanceB));

este falla. Creo que porque aunque guardan lo mismo son instancias diferentes (apuntan a diferentes direcciones de memoria). ¿Alguien puede aclararlo? Gracias.

Editado

La pregunta que se sugiere como "duplicado" se centra en comparar el valor de dos Strings y todas las respuestas van en este sentido, excepto esta: https://es.stackoverflow.com/a/42937/20709 que hace una comparación de objetos, obviando esta respuesta, todas las demás soluciones fallan. Mi pregunta se refiere a cómo comparar objetos de acuerdo al valor de sus propiedades y en concreto de cara a implementar test unitarios. Creo que revisando ambas preguntas y sus respuestas pueden verse similitudes pero está claro que son dos preguntas diferentes.

Tal vez no me explique correctamente. Si alguien no lo tiene claro, ¿si es la misma pregunta como tiene una solución completamente diferente?

Comparar dos strings:

String a = "hola";
String b = "hola";
boolean res = a.equals(b);
System.out.println("Comparando strings: " + res); 
// Comparando strings: true

Comparar dos objetos:

class Foo {
    private int id;
    private String name;

    public Foo(int id, String name) {
        this.id = id;
        this.name = name;
    }

} // class

Foo a = new Foo(1, "hola");
Foo b = new Foo(1, "hola");
boolean res = a.equals(b);
System.out.println("Comparando objetos: " + res); 
// Comparando objetos: false

¿Qué ha pasado? Pues eso es lo que estaba preguntando básicamente...

Answer 1

El método equals de Object es simplemente una comparación

public boolean equals(Object obj) {
    return (this == obj);
}

Por lo que si quieres que sean iguales (aún apuntando a diferente localización de memoria) debes sobreescribir el método equals y si ese objeto lo usarás en algun Collection con hash, sobreescribe de una vez el hashCode

En caso de que sea imposible editar la clase por ser clase Legacy, no tengas el source o demás existe la biblioteca de test

Dependencia en Maven

<dependency>
    <groupId>org.unitils</groupId>
    <artifactId>unitils-core</artifactId>
    <version>3.4.2</version>
    <scope>test</scope>
</dependency>

y cambiar en tu código de

assertEquals(instanceA, instanceB);

a

assertReflectionEquals(instanceA, instanceB);

La documentación y explicación viene en su página oficial http://www.unitils.org/tutorial-reflectionassert.html

Answer 2

Tienes que sobreescribir el método

equals(Object obj)

Ahi puedes comparar los atributos que tu estimes necesarios para determinar de que dos objetos son iguales.

Lo que hace Java por defecto es comparar la referencia para determinar si dos objetos son iguales.

Dale un vistazo en la documentación para ver las propiedades que tiene que cubrir tu método equals.

https://docs.oracle.com/javase/7/docs/api/java/lang/Object.html#equals(java.lang.Object)

Answer 3

Una alternativa para no tocar el código de las clases del proyecto y poder usar JUnit sin tener que hacer múltiples asserts es sobreescribir el método equals() dentro de una nueva clase, creada sólo para las pruebas.

Creo una clase XxxMock que extiende de la clase Xxx a testear y la situó junto a las clases con los test, en esta clase XxxMock sólo se añaden los constructores necesarios y el método equals() sobreescrito.

Así para testear la siguiente clase:

public class Foo {

    private String id;
    private String name;
    private int age;


    public Foo(String id, String name, int age) {
        this.id   = id;
        this.name = name;
        this.age  = age;
    }

    // Setters and Getters
    // ...

} // class

Antes necesitaba un assert para comprobar cada una de las propiedades:

assertEquals(expResult.getId(),   result.getId( ));
assertEquals(expResult.getName(), result.getName( ));
assertEquals(expResult.getDescription(), result.getDescription( ));

Ahora, creo la siguiente clase y la situó con las clases que contienen las pruebas:

public class FooMock extends Foo {

    public FooMock(String id, String name, int age) {
        super(id, name, age);
    }


    /**
     *
     * @param obj
     * @return
     */
    @Override
    public boolean equals(Object obj) {
        if (obj == null) {
            return false;
        }

        Foo foo = (Foo) obj;

        return (
            ((this.getId() == null && foo.getId() == null) ||
                this.getId().equals(foo.getId( ))) &&
            ((this.getName() == null && foo.getName() == null) ||
                this.getName().equals(foo.getName( ))) &&
            this.getAge() == foo.getAge( ));
    }

} // class

Ahora "expResult" deberá ser una instancia de FooMock en lugar de una de Foo y sólo será necesario un assert:

assertTrue(expResult.equals(result));

Editado:

Se comprueba si obj es nulo dentro de equals().