Migliorie UX

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,17 +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 {
@@ -102,8 +172,13 @@ 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 {
-                    annoImmatricolazione = Integer.parseInt(sdf.format(annoImmatricolazioneDate));
+                    ritorno = Integer.parseInt(sdf.format(annoImmatricolazioneDate));
                 }
             } catch (ParseException _) {
                 System.out.println("Errore nella lettura della data.");
@@ -112,17 +187,20 @@ public class MainGarage {
             }
         } while (error);
 
-        System.out.print("Inserisci il tipo di alimentazione: ");
-        tipoAlimentazione = sc.nextLine().trim();
+        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;
@@ -134,23 +212,21 @@ public class MainGarage {
             }
         } while (error);
 
-        do {
-            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();
+        return ritorno;
+    }
+
+    static int capacitaCarico() {
+        boolean error;
+        int ritorno = 0;
 
-            switch (scelta) {
-                case "furgone":
         do {
             error = false;
             try {
                 System.out.print("Inserire la capacità di carico: ");
-                            capacitaCarico = sc.nextInt();
+                ritorno = sc.nextInt();
+                sc.nextLine();
 
-                            if (capacitaCarico <= 0) {
+                if (ritorno <= 0) {
                     System.out.println("Errore: la capacità di carico deve essere un numero positivo.");
                     pausa();
                     error = true;
@@ -161,16 +237,21 @@ public class MainGarage {
                 error = true;
             }
         } while (error);
-                    veicolo = new Furgone(annoImmatricolazione, marca, tipoAlimentazione, cilindrata, capacitaCarico);
-                    break;
-                case "automobile":
+
+        return ritorno;
+    }
+
+    static int numPorte() {
+        boolean error;
+        int ritorno = 0;
         do {
             error = false;
             try {
                 System.out.print("Inserire il numero di porte: ");
-                            numPorte = sc.nextInt();
+                ritorno = sc.nextInt();
+                sc.nextLine();
 
-                            if (numPorte <= 2) {
+                if (ritorno < 2) {
                     System.out.println(
                             "Errore: il numero di porte deve essere un numero positivo e non può essere inferiore a 2.");
                     pausa();
@@ -182,23 +263,26 @@ public class MainGarage {
                 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();
+
+        return ritorno;
+    }
+
+    static int numTempiMotore() {
+        boolean error;
+        int ritorno = 0;
 
         do {
             error = false;
 
             try {
-                            System.out.println("Opzioni possibli");
+                System.out.println("Opzioni possibili");
                 System.out.println("2");
                 System.out.println("4");
-                            System.out.print("Inserisci il numero di tempi del motore: ");
-                            numTempiMotore = sc.nextInt();
+                System.out.print("Inserisci il numero di tempi del motore tra le opzioni possibili: ");
+                ritorno = sc.nextInt();
+                sc.nextLine();
 
-                            switch (numTempiMotore) {
+                switch (ritorno) {
                     case 2, 4:
                         break;
                     default:
@@ -214,23 +298,7 @@ public class MainGarage {
             }
         } while (error);
 
-                    veicolo = new Motocicletta(annoImmatricolazione, marca, tipoAlimentazione, cilindrata, tipologia,
-                            numTempiMotore);
-                    break;
-                default:
-                    System.out.println("Opzione non valida.");
-                    pausa();
-                    error = true;
-                    break;
-            }
-        } 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.");
-        }
+        return ritorno;
     }
 
     static void estraiVeicolo() {