>> Java-Como conseguir que un elemento no se repita en un ArrayList? Puedes hacer uso del método [contains][1] 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][2] (realizar cierta modificación a un método heredado de una clase, en este caso de [Object][3]) el método [equals][3] 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. [1]: https://docs.oracle.com/javase/7/docs/api/java/util/ArrayList.html#contains(java.lang.Object) [2]: https://docs.oracle.com/javase/tutorial/java/IandI/override.html [3]: https://docs.oracle.com/javase/7/docs/api/java/lang/Object.html