Llevo varios días atascada con este código ya casi lo termino pero he llegado a un punto donde no se muy bien como continuar. Mi código es un menú de un garaje de coches donde hay una lista de coches para registrarlos en el garaje y otra para registrar las reparaciones.
Funcionamiento del menú --> Se insertan coches en el listado una vez ingresados los coches podemos ver el listado de coches que tenemos en el taller, si no hemos insertado ningún coche avisa que no tenemos coches ingresados, a continuación podemos marcar como reparados usando la matrícula, nos pregunta que coche deseamos reparar y le indicamos la matrícula ( aquí es donde estoy atascada, ya que quiero quitar un coche de la lista de coches y pasarla a reparados, pero me añade en reparados y no me la borra con el remove de la lista anterior), también tenemos otra lista que muestra los coches reparados o si no hay muestra un mensaje, finalmente una opción para retirar los coches de nuestra lista de reparados, como que ya han salido del garaje y la ultima para salir del menú y finalizar el programa.
En el caso de retirar los coches con el remove de mi garaje me pasa exactamente igual que en el caaso de querer eliminar coches de mi registro inicial, no encuentro que debo de estar haciendo mal.
Comparto las 2 clases que tengo.
Main
import java.util.ArrayList;
import java.util.Scanner;
public class Main {
//creamos el ArrayList de Coche en el main para que lo cojan todos los metodos
static ArrayList <Coche> listadoCoches = new ArrayList<Coche>();
static ArrayList <Coche> listadoReparados = new ArrayList<Coche>();
public static void main(String[] args) {
int preguntaMenu;
System.out.println("***Bienvenido al programa de registro de coches***");
Scanner lector = new Scanner(System.in);
do {
System.out.println("Tiene "+listadoCoches.size()+" coches en su taller");
System.out.println("Que operacion desea realizar? Introduzca una opcion:\n1.Registrar un coche\n2.Listado coches"
+ "\n3.Marcar como reparado\n4.Listado coches reparados\n5.Retirar Vehiculo\n6.Salir");
preguntaMenu = lector.nextInt();
if(preguntaMenu == 1){
registrarCoche();
}
else if (preguntaMenu == 2){
listadoCoches();
}
else if (preguntaMenu == 3){
menuReparados();
}
else if (preguntaMenu == 4){
listadoReparados();
}
else if (preguntaMenu == 5){
retirarVehiculo();
}
} while (preguntaMenu != 6);
System.out.println("Fin");
}
public static void registrarCoche() {
String model;
String color;
String marca;
int nPuertas;
String matricula;
String cilindrada;
int fechaMatricula;
String respuesta;
Scanner lector = new Scanner(System.in);
//empezamos un bucle mientras el usuario desee insertar mas usuarios
do {
System.out.println("Quiere registrar un coche?(S/N)");
respuesta = lector.next();
//si la respuesta es s volvera a preguntar nombre y suelo sino dira gracias
if (respuesta.equalsIgnoreCase("s")) {
System.out.println("Indique la matricula:");
matricula = lector.next();
System.out.println("Indique el anyo de matriculacion:(solo el anyo)");
fechaMatricula = lector.nextInt();
System.out.println("Cual es el modelo del coche?");
model = lector.next();
System.out.println("Indique su color:");
color = lector.next();
System.out.println("Indique la Marca del vehiculo");
marca = lector.next();
System.out.println("Introduzca la cilindrada del vehiculo: ");
cilindrada = lector.next();
System.out.println("Cuantas puertas tiene el vehiculo?");
nPuertas = lector.nextInt();
listadoCoches.add(new Coche(model,color,marca,nPuertas,matricula,cilindrada,fechaMatricula));
}
}while(respuesta.equalsIgnoreCase("s")); //mientras la respuesta sea s
}
public static void listadoCoches() {
//imprimimos el listado de coches el hashset es recorrido por un for
System.out.println("\n**Coches pendientes de reparacion**");
for(Coche c: listadoCoches) {
System.out.println("\n"+c.mostrarInfo());
}
System.out.println("\n************************\n");
}
public static void menuReparados() {
String matricula = null;
System.out.println("Tiene "+listadoCoches.size()+" coches pendientes de reparacion");
Scanner lector = new Scanner(System.in);
do {
if(listadoCoches.size() > 0 ) {
System.out.println("Introduzca la matricula del coche que desea reparar\nIntroduzca Salir para finalizar");
matricula = lector.nextLine();
if(matricula.equalsIgnoreCase("salir"))
{
System.out.println("Gracias!");
}else {
listadoReparados.add(new Coche(matricula));
//int pos = listadoCoches.indexOf(matricula);
listadoCoches.remove(matricula);
}
}
else {
System.out.println("****************************\nNo hay ningun coche para reparar\n****************************");
}
}while(!matricula.equalsIgnoreCase("salir"));
}
public static void listadoReparados() {
if(listadoReparados.size() > 0 ) {
System.out.println("Estos son los coches que ya tiene reparados");
for(Coche reparado: listadoReparados) {
System.out.println("\n"+reparado.mostrarInfoMat());
}
}
else {
System.out.println("****************************\nNo hay ningun coche reparado\n****************************");
}
}
public static void retirarVehiculo() {
String matricula = null;
System.out.println("Tiene "+listadoReparados.size()+" coches reparados");
Scanner lector = new Scanner(System.in);
do {
if(listadoReparados.size() > 0 ) {
System.out.println("Introduzca la matricula del coche que desea retirar del taller\nIntroduzca Salir para finalizar");
matricula = lector.nextLine();
if(matricula.equalsIgnoreCase("salir"))
{
System.out.println("Gracias!");
}else {
listadoReparados.remove(new Coche(matricula));
}
}
else {
System.out.println("****************************\nNo hay ningun coche para retirar\n****************************");
}
}while(!matricula.equalsIgnoreCase("salir"));
}
}
Objeto Coche
public class Coche {
//variables
private String model;
private String color;
private String marca;
private int nPuertas;
private String matricula;
private String cilindrada;
private int fechaMatricula;
//constructor prefabricado
public Coche(String matricula) {
this.matricula = matricula;
}
//constructor
public Coche(String model, String color, String marca, int nPuertas, String matricula, String cilindrada,
int fechaMatricula) {
this.model = model;
this.color = color;
this.marca = marca;
this.nPuertas = nPuertas;
this.matricula = matricula;
this.cilindrada = cilindrada;
this.fechaMatricula = fechaMatricula;
}
//getters y setters
public String getModel() {
return model;
}
public void setModel(String model) {
this.model = model;
}
public String getColor() {
return color;
}
public void setColor(String color) {
this.color = color;
}
public String getMarca() {
return marca;
}
public void setMarca(String marca) {
this.marca = marca;
}
public int getnPuertas() {
return nPuertas;
}
public void setnPuertas(int nPuertas) {
this.nPuertas = nPuertas;
}
public String getMatricula() {
return matricula;
}
public void setMatricula(String matricula) {
this.matricula = matricula;
}
public String getCilindrada() {
return cilindrada;
}
public void setCilindrada(String cilindrada) {
this.cilindrada = cilindrada;
}
public int getFechaMatricula() {
return fechaMatricula;
}
public void setFechaMatricula(int fechaMatricula) {
this.fechaMatricula = fechaMatricula;
}
//metodo mostrar info
public String mostrarInfo() {
return "Matricula: "+matricula+"\nFecha de matriculacion: "+fechaMatricula+"\nModelo: "+model+"\nColor: "+color+"\nMarca: "+marca+
"\nCilindrada: "+cilindrada+"\nNumero de Puertas: "+nPuertas;
}
//metodo mostrar info
public String mostrarInfoMat() {
return "Matricula: "+matricula;
}
}
En el caso de coche escribí dos constructores ya que en reparados no me pide que incluya nada mas solo que la matrícula.
¿Alguna idea? Creo que el fallo se debe a los dos constructores del objeto coche que no tienen que ver entre sí pero no logro descifrarlo, ni como arreglarlo