Poner un ID unico a cada objerto creado

Question

Tengo una serie de jugadores con una serie de atributos, entre ellos, codigoJugador. El codigoJugador es un ID que debe ser ínico y aumentar con cada nuevo jugador creado. Mi problema es que sé definir el atributo (no sé si está bien) pero no sé cómo hacer para que ese ID se modifique cada vez que se cree un objeto jugador. También tengo otra clase equipo que lleva un array de jugadores.

Lo he definido como String, pero me vale como int que empiece en 0 y aumente progresivamente cada vez que se cree un objeto.

public class JugadorRugby {

    private String nombreJugador;
    private byte dorsal;
    private LocalDate fechaNacimiento;
    private String paisNacimiento;
    private double salario;
    private String demarcacion;
    private boolean comunitario;
    private String codigoJugador;

    public JugadorRugby(String nombreJugador, byte dorsal, LocalDate fechaNacimiento, String paisNacimiento, double salario, String demarcacion, boolean comunitario) {
        this.setNombreJugador(nombreJugador);
        this.setDorsal(dorsal);
        this.fechaNacimiento = fechaNacimiento;
        this.paisNacimiento = paisNacimiento;
        this.setSalario(salario);
        this.setDemarcacion(demarcacion);
        this.comunitario = comunitario;
        this.codigoJugador=generarCodigoJugador();
    }

    public JugadorRugby() {
        this.setNombreJugador("");
        this.setDorsal((byte)0);
        this.fechaNacimiento = LocalDate.of(2019, 1, 1);
        this.paisNacimiento = "España";
        this.setSalario(0);
        this.setDemarcacion("Defender");
        this.comunitario = true;
        this.codigoJugador=generarCodigoJugador();
    }

}

Answer 1

Si no es obligatorio que sea un secuencial numérico y te vale con un string como dices, la mejor manera de obtener id's únicos es mediante la clase UUID:

public string generarCodigoJugador(){
    UUID uuid = UUID.randomUUID();
    return uuid.toString();
}

public JugadorRugby() {
    this.setNombreJugador("");
    this.setDorsal((byte)0);
    this.fechaNacimiento = LocalDate.of(2019, 1, 1);
    this.paisNacimiento = "España";
    this.setSalario(0);
    this.setDemarcacion("Defender");
    this.comunitario = true;
    this.codigoJugador=generarCodigoJugador();
}

Answer 2

Esto es lo que yo le implementaria, creo que es mejor esta opcion que con un String, un atributo compartido para la clase que se genere conforme crees objetos. Se crean consecutivamente. Se inicializa a cero y a partir de ahi se generaran automaticamente. Mucho mas sencillo.

public class JugadorRugby {
    private String nombreJugador;
    private byte dorsal;
    private LocalDate fechaNacimiento;
    private String paisNacimiento;
    private double salario;
    private String demarcacion;
    private boolean comunitario;
    private int codigoJugador;
    public static int conCodigoJugador = 0;

   public JugadorRugby(String nombreJugador, byte dorsal, LocalDate fechaNacimiento, String paisNacimiento, double salario, String demarcacion, boolean comunitario) {
        this.setNombreJugador(nombreJugador);
        this.setDorsal(dorsal);
        this.fechaNacimiento = fechaNacimiento;
        this.paisNacimiento = paisNacimiento;
        this.setSalario(salario);
        this.setDemarcacion(demarcacion);
        this.comunitario = comunitario;
        this.codigoJugador=conCodigoJugador++;
    }

   public JugadorRugby() {
        this.setNombreJugador("");
        this.setDorsal((byte)0);
        this.fechaNacimiento = LocalDate.of(2019, 1, 1);
        this.paisNacimiento = "España";
        this.setSalario(0);
        this.setDemarcacion("Defender");
        this.comunitario = true;
        this.codigoJugador=conCodigoJugador++;
    }

}

Si necesitas una solución threadSafe debes utilizar algo así:

private static AtomicInteger idCodigoJugador = new AtomicInteger(0);

public int getNextId() {
    return idCodigoJugador.incrementAndGet();
}
public JugadorRugby() {
    // ...
    this.codigoJugador = getNextId();
}