Tu còdigo es casì correcto, pero tienes que sacar la declaraciòn del ArrayList donde se almacenaran los valores y no dentro del mètodo codigoExistente()
, ya que si lo agregas dentro, el array siempre se estarìa inicializando, lo correcto es:
private static ArrayList<Integer> arrayCodigos = new ArrayList();
static boolean codigoExistente(int codigo){
...
...
...
}
De esta forma iteras sobre el array y si no existe el elemento lo agregas, si existe retornarìa un valor true
:
private static ArrayList<Integer> arrayCodigos = new ArrayList();
static boolean codigoExistente(int codigo){
boolean result = false;
//Itera array para buscar si el còdigo existe.
for(int i=0; i<arrayCodigos.size(); i++){
if(codigo==arrayCodigos.get(i)){
result = true;
}
}
//Determina si no encontro el elemento en el array, lo agrega
if(result == false){
arrayCodigos.add(codigo);
}
return result;
}
Puedes probar tu mètodo, ejemplo:
System.out.println("codigo 1 existente? : " + codigoExistente(1));
System.out.println("codigo 2 existente? : " + codigoExistente(2));
System.out.println("codigo 3 existente? : " + codigoExistente(3));
System.out.println("codigo 1 existente? : " + codigoExistente(1));
System.out.println("codigo 2 existente? : " + codigoExistente(2));
System.out.println("codigo 5 existente? : " + codigoExistente(5));
obtendrìas como salida:
codigo 1 existente? : false
codigo 2 existente? : false
codigo 3 existente? : false
codigo 1 existente? : true
codigo 2 existente? : true
codigo 5 existente? : false