Java-Como conseguir que un elemento no se repita en un ArrayList?
Puedes hacer uso del método contains de la clase ArrayList
, en la cual servirá para saber si el objeto existe en la lista. Este método retornará true
si el objeto se encuentra en el ArrayList
, de lo contrario, devuelve false
.
Sin embargo, deberás sobreescribir (realizar cierta modificación a un método heredado de una clase, en este caso de Object) el método equals en la clase Empleado
, debido a que, el método contains
internamente invocará este método para poder comparar los objetos.
Ahora vamos a imaginar que la clase Empleado
fue definido de esta manera y adicionalmente añadiremos el método equals
al final de la clase:
class Empleado
{
private String nombre;
private String sueldo;
public Empleado(String nombre, String sueldo)
{
this.nombre = nombre;
this.sueldo = sueldo;
}
public String getNombre(){ return nombre; }
public String getSueldo(){ return sueldo; }
@Override
public boolean equals(Object o)
{
if(o == null) return false;
Empleado emp = (Empleado)o;
//Se compara los atributos de ambos objetos
return (sueldo.equalsIgnoreCase(emp.sueldo)) || (nombre.equalsIgnoreCase(emp.nombre));
}
}
Por último solo faltaría invocar el método contains
antes del método add
. Es decir, solo vamos añadir un objeto en la lista, siempre y cuando, el objeto no se encuentra en el ArrayList
.
//si la respuesta es s volvera a preguntar nombre y suelo sino dirá gracias
if (respuesta.equalsIgnoreCase("s"))
{
System.out.println("Cual es tu nombre?");
nombre = lector.nextLine();
System.out.println("Cual es tu sueldo?");
sueldo = lector.nextLine();
//Creamos el objeto de tipo Empleado
Empleado emp = new Empleado(nombre, sueldo);
//Verificamos si el objeto se encuentra en la lista y si es verdad, lo añadimos en la lista
if(empleados.contains(emp) == false)
empleados.add(emp);
}
Y de este modo, nunca tendrás una información repetida.
También había código redundante, porqué primero pides información por primera vez y después lo vuelves hacer en el do-while, cuando en realidad todo puede ir en el bucle.
El código sin la redundancia quedaría de esta forma:
public class Program {
public static void main(String[] args) {
//llamamos al metodo
getEmpleados();
}
public static void getEmpleados() {
//creamos el array list de empleados
ArrayList <Empleado> empleados = new ArrayList<Empleado>();
String nombre;
String sueldo;
String respuesta;
Scanner lector = new Scanner(System.in);
//empezamos un bucle mientras el usuario desee insertar mas usuarios
do {
System.out.println("Cual es tu nombre?");
nombre = lector.nextLine();
System.out.println("Cual es tu sueldo?");
sueldo = lector.nextLine();
//Creamos el objeto de tipo Empleado
Empleado emp = new Empleado(nombre, sueldo);
//Verificamos si el objeto se encuentra en la lista y si es verdad, lo añadimos en la lista
if(empleados.contains(emp) == false)
empleados.add(emp);
System.out.println("Quieres insertar los datos de algun empleado mas?(S/N)");
respuesta = lector.nextLine();
if(respuesta.equalsIgnoreCase("n"))
{
System.out.println("Gracias");
break;//rompemos el ciclo do-while
}
}while(true); //mientras la respuesta sea s
//Se imprime la lista
for(Empleado emp : empleados)
{
System.out.println("Nombre: " + emp.getSueldo() + "\nSueldo: " + emp.getNombre());
}
}
}
EDIT:
Esta condicional no tiene sentido:
if (e.getNombre() == e.getNombre() && e.getSueldo() == e.getSueldo() || e.getNombre() == e.getNombre() || e.getSueldo() == e.getSueldo() )
Porqué básicamente estarías comparando los atributos de un mismo objeto y por lo tanto, la condición siempre dará como resultado un true
y esa es la razón del porqué no te funciona.