Estaba realizando un método java al cual le paso una matricula de un camion. Este método, cuando encontrase un camión con esa matrícula, iba a apuntar su hora de llegada, e iba a marcar si había llegado a tiempo o no (esta última parte funciona, es meramente explicativa). El problema viene cuando utilizo este método, nunca encuentra el camión con la matricula indicada. Mi código del método es:
public void llegaCamion(String matricula) {
LocalDate date=LocalDate.now();
int dow=date.getDayOfWeek().ordinal();
if(dow==0) {
for(int i=0;i<planificadosMonday.size();i++) {
if(planificadosMonday.get(i).getmatricula()==matricula) {
planificadosMonday.get(i).setHoraDesc(LocalTime.now());
}
}
System.out.println("Lo siento, no hemos encontrado el camión");
}else if(dow==1) {
for(int i=0;i<planificadosTuesday.size();i++) {
if(planificadosTuesday.get(i).getmatricula()==matricula) {
planificadosTuesday.get(i).setHoraDesc(LocalTime.now());
}
}
System.out.println("Lo siento, no hemos encontrado el camión");
}else if(dow==2) {
for(int i=0;i<planificadosWednes.size();i++) {
if(planificadosWednes.get(i).getmatricula()==matricula) {
planificadosWednes.get(i).setHoraDesc(LocalTime.now());
}
}
System.out.println("Lo siento, no hemos encontrado el camión");
}else if(dow==3) {
for(int i=0;i<planificadosThurs.size();i++) {
if(planificadosThurs.get(i).getmatricula()==matricula) {
planificadosThurs.get(i).setHoraDesc(LocalTime.now());
}
}
System.out.println("Lo siento, no hemos encontrado el camión");
}else if(dow==4) {
for(int i=0;i<planificadosFri.size();i++) {
if(planificadosFri.get(i).getmatricula()==matricula) {
planificadosFri.get(i).setHoraDesc(LocalTime.now());;
System.out.println("El camion ha llegado");
}
}
System.out.println("Lo siento, no hemos encontrado el camión");
}else {
System.out.println("Lo siento, no hay camiones asignados el fin de semana");
}
}
Como se puede intuir, es un if anidado que dependiendo del día que sea mira en una lista u otra (los camiones ya están distribuidos por días correctamente en otros métodos, y las listas contienen todos los elementos que deben). El caso, es que en mi main, ponga la matrícula que ponga, no encuentra el camión aunque esté en la lista.
Os paso mi código del main
public static void main(String[]args) throws IOException {
Calendar c=Calendar.getInstance();
LocalDate date=LocalDate.now();
DayOfWeek dow=date.getDayOfWeek();
ListaCamiones l=new ListaCamiones();
if(dow.ordinal()==DayOfWeek.MONDAY.ordinal()) {
l.getPlanif("M");
l.muestraCamionesPlanif("M");
}//SEGUIR CON LOS DEMAS DIAS
else if(dow.ordinal()==DayOfWeek.TUESDAY.ordinal()) {
l.getPlanif("T");
l.muestraCamionesPlanif("T");
}
else if(dow.ordinal()==DayOfWeek.WEDNESDAY.ordinal()){
l.getPlanif("W");
l.muestraCamionesPlanif("W");
}else if(dow.ordinal()==DayOfWeek.THURSDAY.ordinal()) {
l.getPlanif("TH");
l.muestraCamionesPlanif("TH");
}else if(dow.ordinal()==DayOfWeek.FRIDAY.ordinal()) {
l.getPlanif("F");
l.muestraCamionesPlanif("F");
}else {
System.out.println("Error");
}
ArrayList<Camion>cam=(ArrayList<Camion>) l.getPlanificadosFri();
for(int i=0;i<cam.size();i++) {
System.out.println(cam.get(i).getmatricula());
}
l.muestraCamionesPlanif("F");
String matricula="opi1928";
l.llegaCamion(matricula);
}
De hecho, en:
for(int i=0;i<cam.size();i++) {
System.out.println(cam.get(i).getmatricula());
}
consigo ver las matrículas, pero después cuando uso el método llegaCamion() no la encuentra.
==
, sino conequals
. Es muy probable que el problema esté ahí, todas estas comparaciones fallarán:if(planificadosMonday.get(i).getmatricula()==matricula) {
, por tanto, debes escribirlas así todas:if( planificadosMonday.get(i).getmatricula().equals(matricula) ) {
, por otra parte, veo un código demasiado repetitivo. Se podría optimizar usando algo así como arrays o estructuras de datos, pero ese sería ya otro tema.if elseif else
por una estructuraswitch... case
ganarías en claridad y te ahorrarías varias líneas de código.