Estoy haciendo un ejercicio de java sobre objetos insertados en un ArrayList. Pongo trozos de código a ver si consigo expresar el error:
Objeto :
public class Articulo {
private String codigo;
private String descripcion;
private short existencias;
...
ArrayList:
public class Almacen {
private ArrayList <Articulo> almacenPrincipal = new ArrayList();
Iterator <Articulo> itr;
//método para cargar 4 artículos en el almacén
public void cargarAlmacen(){
//se crean los 4 objetos de tipo artículo
Articulo articulo1 = new Articulo();
Articulo articulo2 = new Articulo("AA.123A", "tornillos", (short)9);
Articulo articulo3 = new Articulo("BB.111B", "tuercas");
Articulo articulo4 = new Articulo(articulo1);
//se modifican los artículos
articulo3.setExistencias((short)500);
articulo4.setCodigo("CC.222C");
articulo4.setDescripcion("arandelas");
articulo4.setExistencias((short)500);
//se añaden los 4 artículos al arraylist almacenPrincipal
almacenPrincipal.add (articulo1);
almacenPrincipal.add (articulo2);
almacenPrincipal.add (articulo3);
almacenPrincipal.add (articulo4);
}
//método para realizar la carga de un nuevo artículo en el almacén
public void cargarArticulo(){
System.out.println ("\nIntroduzca código de articulo: ");
String codigo = codigoVal();
if (!comprobarRepetidos(codigo)){
System.out.println ("\nIntroduzca descripción del articulo: ");
String descripcion = descripcionVal();
System.out.println ("\nIntroduzca cantidad de existencias: ");
short existencias = existenciasVal();
Articulo articuloNuevo = new Articulo (codigo, descripcion, existencias);
almacenPrincipal.add (articuloNuevo);
}
}
//método para validar si el código introducido es conforme:
//empieza por dos letras mayúsculas + punto + entre 1 y 3 dígitos numéricos + mínimo 1 letra mayúscula o minúscula
public String codigoVal(){
boolean codigoError=true;
String codigo;
Pattern patron=Pattern.compile("^([A-Z]{2})([\\.]{1})(\\d{1,3})([a-zA-Z]{1,})$");
do{
codigo = teclado.nextLine();
Matcher m=patron.matcher(codigo);
if (m.matches()){
codigoError=false;
}else{
System.err.println ("El código no es válido. Vuelva a intentarlo: ");
}
}while (codigoError);
return codigo;
}
//método para buscar un artículo
public boolean buscarArticulo(){
itr = almacenPrincipal.iterator();
String codigoBuscado = codigoVal();
while (itr.hasNext()){
Articulo articuloBuscado = itr.next();
if(articuloBuscado.getCodigo().equals(codigoBuscado)){
System.out.println ("Los datos del artículo son: \n"+articuloBuscado.mostrarDatos());
return true;
}
}
return false;
}
...
Clase Principal(Main):
System.out.println("\nMenú Artículos");
System.out.println("1. Añadir artículo.");
System.out.println("2. Buscar artículo.");
System.out.println("3. Eliminar artículo.");
System.out.println("4. Visualizar almacén.");
System.out.println("5. Salir.");
System.out.println("\nIntroduzca nº de opción: ");
opcion = teclado.nextByte();
switch (opcion){
case 1:
almacenPrincipal.cargarArticulo();
break;
case 2:
System.out.println("Introduzca código a buscar: ");
if (!almacenPrincipal.buscarArticulo()){
System.out.println ("El artículo no existe.");
}
break;
case 3:
System.out.println("Introduzca código a eliminar: ");
almacenPrincipal.borrarArticulo();
break;
case 4:
almacenPrincipal.visualizarAlmacen();
break;
case 5:
System.out.println ("Fin del programa. Adiós.");
break;
default:
System.err.println ("El nº de opción no es válido. Vuelva a intentarlo.");
}
Si meto opción 2, busca el artículo bien, sin problema.
Si luego meto la opción 1, el nuevo artículo lo carga bien.
Si después meto otra vez la opción 2, antes de pedirme el código del artículo a buscar me sale mensaje de error de que "el artículo no existe", y ya después me deja buscar el artículo, y lo encuentra y lo muestra. La búsqueda es con iterador porque lo exige el enunciado.
No sé por qué, si ejecuto antes la opción 1 que la 2 me sale ese mensaje de error antes de dejarme meter el código:
Es mi primer mensaje. Espero que se entienda bien la pregunta. Muchas a gracias a tod@s por la ayuda.
Un saludo, chache ;)
Scanner
suele dar problemas al introducir un dato y pulsarEnter
, ya que lo almacena como un salto de línea en el buffer\n
y hay que limpiarlo con la instrucciónnextLine();
.