Estoy haciendo un programa el cual me dice si un vehículo está en pico y placa(no puede salir a la calle), esto lo hago con un switch. El problema viene a la hora de ejecutar el programa, puesto que al hacerlo, el método en el cual verifico esto, siempre me devuelve true, cuando en ocasiones me debería devolver un false.
Caso de prueba
Ejemplo: si la placa es CAM129, me debería devolver false.
Notas
las listas que cree, son respecto a cada dia de la semana, ya que cada dia tiene dos numeros los cuales me definen si el vehiculo esta o no en pico y placa.Está en pico y placa si el último número de la placa es uno de esos dos. También agrego que al intentar hacerlo manualmente, es decir, sin las listas, colocando los números tal cual, funciona correctamente, el problema es que requiero que esten en una lista.
Código fuente
Este es el código:
public class Restriction {
List<Integer>monday = new ArrayList<>();
List<Integer>tuesday = new ArrayList<>();
List<Integer>wednesday = new ArrayList<>();
List<Integer>thursday = new ArrayList<>();
List<Integer>friday = new ArrayList<>();
public void picoPlaca(){
monday.add(6,9);
tuesday.add(5,7);
wednesday.add(1,4);
thursday.add(8,0);
friday.add(3,2);
}
public boolean restrictionCar(Vehicle vehicle){
vehicle.day = vehicle.getDayOfWeek(new Date()); /*vehicle.day tiene el dia de la semana*/
int numCar = vehicle.getPlate().charAt(5); /*numCar tiene el ultimo numero de placa*/
switch(vehicle.day){
case "lunes" -> {
for(int i=0;i<monday.size();i++){
if(numCar == monday.get(i)){
return false;
}
}
return true;
}
case "martes" -> {
for(int i=0;i<tuesday.size();i++){
if(numCar == tuesday.get(i)){
return false;
}
}
return true;
}
case "miercoles" -> {
for(int i=0;i<wednesday.size();i++){
if(numCar == wednesday.get(i)){
return false;
}
}
return true;
}
case "jueves" -> {
for(int i=0;i<thursday.size();i++){
if(numCar == thursday.get(i)){
return false;
}
}
return true;
}
case "viernes" -> {
for(int i=0;i<friday.size();i++){
if(numCar == friday.get(i)){
return false;
}
}
return true;
}
case "sabado" -> {
return true;
}
case "domingo" -> {
return true;
}
default -> {
return false;
}
}
}
public boolean restrictionMotorcycle(Vehicle vehicle){
char numMotorcycle = vehicle.getPlate().charAt(3);
vehicle.day = vehicle.getDayOfWeek(new Date());
switch(vehicle.day){
case "lunes" -> {
for(int i=0;i<monday.size();i++){
if(numMotorcycle == monday.get(i)){
return false;
}
}
return true;
}
case "martes" -> {
for(int i=0;i<tuesday.size();i++){
if(numMotorcycle == tuesday.get(i)){
return false;
}
}
return true;
}
case "miercoles" -> {
for(int i=0;i<wednesday.size();i++){
if(numMotorcycle == wednesday.get(i)){
return false;
}
}
return true;
}
case "jueves" -> {
for(int i=0;i<thursday.size();i++){
if(numMotorcycle == thursday.get(i)){
return false;
}
}
return true;
}
case "viernes" -> {
for(int i=0;i<friday.size();i++){
if(numMotorcycle == friday.get(i)){
return false;
}
}
return true;
}
case "sabado" -> {
return true;
}
case "domingo" -> {
return true;
}
default -> {
return false;
}
}
}
}
public class Vehicle {
private String plate;
private String vehicleType;
private Date dateEntry;
public String day;
public String getDayOfWeek(Date date) {
DateFormat formatter = new SimpleDateFormat("EEEE");
return formatter.format(date);
}
9
. Yo haría que ambos fueran Integer antes de comparar cosas.