Skip to main content
se añadió 1 carácter en el cuerpo
Origen Enlace
MrDave1999
  • 7.6k
  • 1
  • 11
  • 22
Origen Enlace
MrDave1999
  • 7.6k
  • 1
  • 11
  • 22

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.