El método contains()
lo que hace es llamar al método equals()
de la clase contenida en el ArrayList
, en este caso, de la clase Cliente, y lo compara con el objeto cliente que le indiques.
Por lo tanto, en vez de pasarle como parámetro contains(identificacion)
, deberías pasarle el objeto clienteNuevo
.
Quedaría así:
if (listaCliente.contains(clienteNuevo)) {
System.out.println("Identificacion repetida");
}
Aclaración:
Si utilizas el método contains()
sin modificar el método equals()
de la clase Cliente, sólo devolverá true si TODOS sus atributos coinciden (nombre, edad, identificación, presupuesto).
Para personalizar el método equals()
de la clase Cliente, para, por ejemplo, que interprete que dos clientes son iguales si, como mínimo, los identificadores coindicen, e ignorando los demás atributos, deberás hacer un Override.
Tu clase Cliente quedaría así:
class Cliente {
private int identificador;
//el resto de atributos.
//constructor
//getters y setters.
// AQUI EL OVERRIDE DE EQUALS.
@Override
public boolean equals(Cliente cliente) {
// Esto es una forma resumida para hacer un if y un return en la misma linea.
return this.identificador == cliente.getIdentificador();
}
}
Además, el método contains ya retorna un boolean, por lo que es reduntante la comprobación ==true
.
Las demás respuestas también pueden funcionar, pero creo que esta es una forma más limpia de hacerlo.
Un saludo y que te sirva de ayuda!