Migliorie UX

src/garage/Garage.java

@@ -14,21 +14,25 @@ public class Garage {
     public static final int DIMENSIONE_GARAGE = 15;
     VeicoloAMotore[] veicoli = new VeicoloAMotore[DIMENSIONE_GARAGE];
 
-    public int immettiNuovoVeicolo(VeicoloAMotore v) {
+    public int immettiNuovoVeicolo(VeicoloAMotore v) throws ArrayIndexOutOfBoundsException {
         int ritorno = trovaPosizioneLibera();
         veicoli[ritorno] = v;
         return ritorno;
     }
-    
+
     public VeicoloAMotore estraiVeicolo(int posto) {
         VeicoloAMotore ritorno = this.veicoli[posto];
         this.veicoli[posto] = null;
         return ritorno;
     }
-  
+
     public void stampaSituazionePosti() {
-        for (VeicoloAMotore veicoloAMotore : veicoli) {
-            System.out.println(veicoloAMotore.toString());
+        for (int i = 0; i < this.veicoli.length; i++) {
+            if (this.veicoli[i] != null) {
+                System.out.println("Posizione " + i + ":\n" + this.veicoli[i].toString());
+            } else {
+                System.out.println("La posizione " + i + " è vuota.");
+            }
         }
     }
 

src/garage/MainGarage.java

@@ -23,6 +23,7 @@ public class MainGarage {
 
     static Scanner sc = new Scanner(System.in);
     static Garage garage = new Garage();
+    static final String ERRORE_OPZIONE = "Opzione non valida.";
 
     public static void main() {
         int scelta = -1;
@@ -55,7 +56,7 @@ public class MainGarage {
                         pausa();
                         break;
                     default:
-                        System.out.println("Opzione non valida.");
+                        System.out.println(ERRORE_OPZIONE);
                         pausa();
                         break;
                 }
@@ -81,16 +82,86 @@ public class MainGarage {
         String tipologia;
         int numTempiMotore = 0;
 
-        SimpleDateFormat sdf = new SimpleDateFormat("yyyy");
-        Date annoImmatricolazioneDate;
-        boolean error;
-        Date now = new Date();
-        String scelta;
+        boolean error = false;
+        int scelta;
         VeicoloAMotore veicolo = null;
+        int posizioneOccupata;
 
         System.out.print("Inserisci la marca: ");
         marca = sc.nextLine().trim();
 
+        annoImmatricolazione = annoImmatricolazione();
+
+        System.out.print("Inserisci il tipo di alimentazione: ");
+        tipoAlimentazione = sc.nextLine().trim();
+
+        cilindrata = cilindrata();
+
+        do {
+            try {
+                error = false;
+                System.out.println("Opzioni possibili:");
+                System.out.println("1. Furgone");
+                System.out.println("2. Automobile");
+                System.out.println("3. Motocicletta");
+                System.out.print("Inserire la tipologia di veicolo tra le opzioni sopra: ");
+                scelta = sc.nextInt();
+                sc.nextLine();
+
+                switch (scelta) {
+                    case 1:
+                        capacitaCarico = capacitaCarico();
+                        veicolo = new Furgone(annoImmatricolazione, marca, tipoAlimentazione, cilindrata,
+                                capacitaCarico);
+                        break;
+                    case 2:
+                        numPorte = numPorte();
+                        veicolo = new Automobile(annoImmatricolazione, marca, tipoAlimentazione, cilindrata, numPorte);
+                        break;
+                    case 3:
+                        System.out.print("Inserire la tipologia della motocicletta: ");
+                        tipologia = sc.nextLine();
+
+                        numTempiMotore = numTempiMotore();
+                        veicolo = new Motocicletta(annoImmatricolazione, marca, tipoAlimentazione, cilindrata,
+                                tipologia,
+                                numTempiMotore);
+                        break;
+                    default:
+                        System.out.println(ERRORE_OPZIONE);
+                        pausa();
+                        error = true;
+                        break;
+                }
+            } catch (InputMismatchException _) {
+                System.out.println(ERRORE_OPZIONE);
+                pausa();
+                error = true;
+            }
+        } while (error);
+
+        try {
+            posizioneOccupata = garage.immettiNuovoVeicolo(veicolo);
+            System.out.println("Veicolo inserito con successo alla posizione " + posizioneOccupata + ".");
+        } catch (ArrayIndexOutOfBoundsException _) {
+            System.out.println("Errore: non ci sono posti disponibili, riprovare.");
+        }
+    }
+
+    static int annoImmatricolazione() {
+        boolean error;
+        Date annoImmatricolazioneDate;
+        SimpleDateFormat sdf = new SimpleDateFormat("yyyy");
+        Date now = new Date();
+        String annoIniziale = "1900";
+        Date dataIniziale = null;
+        try {
+            dataIniziale = sdf.parse(annoIniziale);
+        } catch (ParseException _) {
+            System.out.println("Errore non previsto nell'inizializzazione della data.");
+        }
+        int ritorno = 0;
+
         do {
             error = false;
             try {
@@ -101,8 +172,14 @@ public class MainGarage {
                     System.out.println("Errore: il veicolo non può esser stato immatricolato nel futuro.");
                     pausa();
                     error = true;
+                } else if (annoImmatricolazioneDate.before(dataIniziale)) {
+                    System.out.println(
+                            "Errore: il veicolo non può esser stato immatricolato prima del " + annoIniziale + ".");
+                    pausa();
+                    error = true;
+                } else {
+                    ritorno = Integer.parseInt(sdf.format(annoImmatricolazioneDate));
                 }
-                annoImmatricolazione = Integer.parseInt(annoImmatricolazioneDate.toString());
             } catch (ParseException _) {
                 System.out.println("Errore nella lettura della data.");
                 pausa();
@@ -110,17 +187,20 @@ public class MainGarage {
             }
         } while (error);
 
-        System.out.print("Inserisci il tipo di alimentazione: ");
-        tipoAlimentazione = sc.nextLine().trim();
-        // TODO: controlli sull'alimentazione (niente veicoli alimentati a scoregge)
+        return ritorno;
+    }
 
+    static int cilindrata() {
+        boolean error;
+        int ritorno = 0;
         do {
             error = false;
             try {
                 System.out.print("Inserire la cilindrata: ");
-                cilindrata = sc.nextInt();
+                ritorno = sc.nextInt();
+                sc.nextLine();
 
-                if (cilindrata <= 0) {
+                if (ritorno <= 0) {
                     System.out.println("Errore: la cilindrata deve essere un numero positivo.");
                     pausa();
                     error = true;
@@ -132,100 +212,93 @@ public class MainGarage {
             }
         } while (error);
 
+        return ritorno;
+    }
+
+    static int capacitaCarico() {
+        boolean error;
+        int ritorno = 0;
+
         do {
             error = false;
-            System.out.println("Opzioni possibili:");
-            System.out.println("Furgone");
-            System.out.println("Automobile");
-            System.out.println("Motocicletta");
-            System.out.print("Inserire la tipologia di veicolo: ");
-            scelta = sc.nextLine().trim().toLowerCase();
+            try {
+                System.out.print("Inserire la capacità di carico: ");
+                ritorno = sc.nextInt();
+                sc.nextLine();
 
-            switch (scelta) {
-                case "furgone":
-                    do {
-                        error = false;
-                        try {
-                            System.out.print("Inserire la capacità di carico: ");
-                            capacitaCarico = sc.nextInt();
-
-                            if (capacitaCarico <= 0) {
-                                System.out.println("Errore: la capacità di carico deve essere un numero positivo.");
-                                pausa();
-                                error = true;
-                            }
-                        } catch (InputMismatchException _) {
-                            System.out.println("Errore nella lettura della capacità di carico.");
-                            pausa();
-                            error = true;
-                        }
-                    } while (error);
-                    veicolo = new Furgone(annoImmatricolazione, marca, tipoAlimentazione, cilindrata, capacitaCarico);
-                    break;
-                case "automobile":
-                    do {
-                        error = false;
-                        try {
-                            System.out.print("Inserire il numero di porte: ");
-                            numPorte = sc.nextInt();
-
-                            if (numPorte <= 2) {
-                                System.out.println(
-                                        "Errore: il numero di porte deve essere un numero positivo e non può essere inferiore a 2.");
-                                pausa();
-                                error = true;
-                            }
-                        } catch (InputMismatchException _) {
-                            System.out.println("Errore nella lettura del numero di porte.");
-                            pausa();
-                            error = true;
-                        }
-                    } while (error);
-                    veicolo = new Automobile(annoImmatricolazione, marca, tipoAlimentazione, cilindrata, numPorte);
-                    break;
-                case "motocicletta":
-                    System.out.print("Inserire la tipologia della motocicletta: ");
-                    tipologia = sc.nextLine();
-
-                    do {
-                        error = false;
-
-                        try {
-                            System.out.println("Opzioni possibli");
-                            System.out.println("2");
-                            System.out.println("4");
-                            System.out.print("Inserisci il numero di tempi del motore: ");
-                            numTempiMotore = sc.nextInt();
-
-                            switch (numTempiMotore) {
-                                case 2, 4:
-                                    break;
-                                default:
-                                    System.out.println("Errore: opzione non valida.");
-                                    pausa();
-                                    error = true;
-                                    break;
-                            }
-                        } catch (InputMismatchException _) {
-                            System.out.println("Errore nella lettura del numero di tempi del motore.");
-                            pausa();
-                            error = true;
-                        }
-                    } while (error);
-
-                    veicolo = new Motocicletta(annoImmatricolazione, marca, tipoAlimentazione, cilindrata, tipologia,
-                            numTempiMotore);
-                    break;
-                default:
-                    System.out.println("Opzione non valida.");
+                if (ritorno <= 0) {
+                    System.out.println("Errore: la capacità di carico deve essere un numero positivo.");
                     pausa();
                     error = true;
-                    break;
+                }
+            } catch (InputMismatchException _) {
+                System.out.println("Errore nella lettura della capacità di carico.");
+                pausa();
+                error = true;
             }
         } while (error);
 
-        garage.immettiNuovoVeicolo(veicolo);
-        System.out.println("Veicolo inserito con successo.");
+        return ritorno;
+    }
+
+    static int numPorte() {
+        boolean error;
+        int ritorno = 0;
+        do {
+            error = false;
+            try {
+                System.out.print("Inserire il numero di porte: ");
+                ritorno = sc.nextInt();
+                sc.nextLine();
+
+                if (ritorno < 2) {
+                    System.out.println(
+                            "Errore: il numero di porte deve essere un numero positivo e non può essere inferiore a 2.");
+                    pausa();
+                    error = true;
+                }
+            } catch (InputMismatchException _) {
+                System.out.println("Errore nella lettura del numero di porte.");
+                pausa();
+                error = true;
+            }
+        } while (error);
+
+        return ritorno;
+    }
+
+    static int numTempiMotore() {
+        boolean error;
+        int ritorno = 0;
+
+        do {
+            error = false;
+
+            try {
+                System.out.println("Opzioni possibili");
+                System.out.println("2");
+                System.out.println("4");
+                System.out.print("Inserisci il numero di tempi del motore tra le opzioni possibili: ");
+                ritorno = sc.nextInt();
+                sc.nextLine();
+
+                switch (ritorno) {
+                    case 2, 4:
+                        break;
+                    default:
+                        System.out.println("Errore: opzione non valida.");
+                        pausa();
+                        error = true;
+                        break;
+                }
+            } catch (InputMismatchException _) {
+                System.out.println("Errore nella lettura del numero di tempi del motore.");
+                pausa();
+                error = true;
+            }
+        } while (error);
+
+        return ritorno;
     }
 
     static void estraiVeicolo() {
@@ -235,32 +308,28 @@ public class MainGarage {
 
         do {
             error = false;
+            try {
+                System.out.print("Inserire la posizione: ");
+                posizione = sc.nextInt();
 
-            do {
-                try {
-                    System.out.print("Inserire la posizione: ");
-                    posizione = sc.nextInt();
-
-                    if (posizione <= 0 || posizione >= Garage.DIMENSIONE_GARAGE) {
-                        System.out.println("Errore: la posizione non esiste.");
-                        pausa();
-                        error = true;
-                    }
-                } catch (InputMismatchException _) {
-                    System.out.println("Errore nella lettura della posizione.");
+                if (posizione < 0 || posizione >= Garage.DIMENSIONE_GARAGE) {
+                    System.out.println("Errore: la posizione non esiste.");
                     pausa();
                     error = true;
                 }
-            } while (error);
-
-            veicoloEstratto = garage.estraiVeicolo(posizione);
-            if (veicoloEstratto != null) {
-                System.out.println(veicoloEstratto.toString());
-            } else {
-                System.out.println("Errore: la posizione indicata è vuota.");
+            } catch (InputMismatchException _) {
+                System.out.println("Errore nella lettura della posizione.");
                 pausa();
                 error = true;
             }
         } while (error);
+
+        veicoloEstratto = garage.estraiVeicolo(posizione);
+        if (veicoloEstratto != null) {
+            System.out.println("Veicolo estratto:");
+            System.out.println(veicoloEstratto.toString());
+        } else {
+            System.out.println("Errore: la posizione indicata è vuota.");
+        }
     }
 }