From 3b998a981d5f2c26c42cba26ce1d9f7957a2f78a Mon Sep 17 00:00:00 2001 From: La Programmatrice Verde Date: Wed, 26 Nov 2025 00:09:08 +0100 Subject: [PATCH] Refactoring --- src/fatturascontrini/FatturaScontrini.java | 91 +++++++++++----------- 1 file changed, 46 insertions(+), 45 deletions(-) diff --git a/src/fatturascontrini/FatturaScontrini.java b/src/fatturascontrini/FatturaScontrini.java index 492e1f2..9412b95 100644 --- a/src/fatturascontrini/FatturaScontrini.java +++ b/src/fatturascontrini/FatturaScontrini.java @@ -35,6 +35,7 @@ public class FatturaScontrini { static final String PATH_PROGRAM_DATA = "./src/fatturascontrini/programData.txt"; static final String ERRORE_DEFAULT = "Errore: opzione non valida."; static final String ERRORE_FILE = "Errore nella lettura del file: "; + static final String ERRORE_MENU_VUOTO = "Errore: il menu è vuoto."; static int codiceScontrino = getCodice(0); public static void main(String[] args) { @@ -58,7 +59,7 @@ public class FatturaScontrini { break; case 1: if (getFileSize(PATH_MENU) < 1) { - System.out.println("Errore: il menu è vuoto."); + System.out.println(); } else { stampaMenu(); } @@ -66,7 +67,7 @@ public class FatturaScontrini { break; case 2: if (getFileSize(PATH_MENU) < 1) { - System.out.println("Errore: il menu è vuoto."); + System.out.println(ERRORE_MENU_VUOTO); } else { ordinazione = placeOrdinazione(ordinazione); } @@ -127,7 +128,7 @@ public class FatturaScontrini { } static ArrayList ordinazione() { - ArrayList ritorno = new ArrayList<>(); + ArrayList ordinazione = new ArrayList<>(); boolean exit; boolean error; boolean isDuplicate = false; @@ -137,13 +138,13 @@ public class FatturaScontrini { exit = true; int[] ordine = { selezioneMenu(), quantita() }; - isDuplicate = isOrdinazioneDuplicata(ritorno, ordine); + isDuplicate = isOrdinazioneDuplicata(ordinazione, ordine); if (!isDuplicate) { - ritorno.add(ordine); + ordinazione.add(ordine); } - System.out.println(ordinazioneToString(ritorno)); + System.out.println(ordinazioneToString(ordinazione)); do { error = false; @@ -151,12 +152,12 @@ public class FatturaScontrini { scelta = sc.nextLine().trim().toLowerCase(); switch (scelta) { case "s": - ritorno = cancellaInOrdinazione(ritorno); + ordinazione = cancellaInOrdinazione(ordinazione); exit = false; break; case "n": System.out.println("Stampa dello scontrino. . ."); - pagare(ritorno); + pagare(ordinazione); break; default: System.out.println(ERRORE_DEFAULT); @@ -167,12 +168,12 @@ public class FatturaScontrini { } while (error); } while (!exit); - return ritorno; + return ordinazione; } static int selezioneMenu() { boolean error; - int ritorno = -1; + int voceSelezionata = -1; do { error = false; @@ -181,10 +182,10 @@ public class FatturaScontrini { System.out.print("Scelta: "); try { - ritorno = sc.nextInt(); + voceSelezionata = sc.nextInt(); sc.nextLine(); - if (ritorno < 0 || ritorno >= getFileSize(PATH_MENU)) { + if (voceSelezionata < 0 || voceSelezionata >= getFileSize(PATH_MENU)) { System.out.println(ERRORE_DEFAULT); pausa(); error = true; @@ -196,7 +197,7 @@ public class FatturaScontrini { } } while (error); - return ritorno; + return voceSelezionata; } static void stampaMenu() { @@ -215,33 +216,33 @@ public class FatturaScontrini { } static int getFileSize(String path) { - int i = 0; + int dimensioneFile = 0; try (BufferedReader br = new BufferedReader(new FileReader(path))) { String riga = br.readLine(); while (riga != null) { - i++; + dimensioneFile++; riga = br.readLine(); } } catch (IOException e) { System.out.println(ERRORE_FILE + e.getMessage()); } - return i; + return dimensioneFile; } static int quantita() { boolean error; - int ritorno = -1; + int quantita = -1; do { error = false; System.out.print("Inserire la quantità da acquistare: "); try { - ritorno = sc.nextInt(); + quantita = sc.nextInt(); sc.nextLine(); - if (ritorno <= 0) { + if (quantita <= 0) { System.out.println("Errore: non è possibile selezionare una quantità minore di zero."); pausa(); error = true; @@ -253,17 +254,17 @@ public class FatturaScontrini { } } while (error); - return ritorno; + return quantita; } static boolean isOrdinazioneDuplicata(ArrayList ordinazione, int[] ordine) { - boolean ritorno = false; + boolean isOrdinazioneDuplicata = false; boolean error; String scelta; for (int[] ordini : ordinazione) { if (ordini[0] == ordine[0]) { - ritorno = true; + isOrdinazioneDuplicata = true; do { error = false; System.out @@ -287,7 +288,7 @@ public class FatturaScontrini { } } - return ritorno; + return isOrdinazioneDuplicata; } static String ordinazioneToString(ArrayList ordinazione) { @@ -322,19 +323,19 @@ public class FatturaScontrini { } static String[][] tabellaNomePrezzo() { - String[][] ritorno = new String[2][getFileSize(PATH_MENU)]; + String[][] tabellaNomePrezzo = new String[2][getFileSize(PATH_MENU)]; try (BufferedReader br = new BufferedReader(new FileReader(PATH_MENU))) { String riga = br.readLine(); for (int i = 0; i < getFileSize(PATH_MENU); i++) { - ritorno[0][i] = riga.split(":")[0]; - ritorno[1][i] = riga.split(":")[1].trim(); + tabellaNomePrezzo[0][i] = riga.split(":")[0]; + tabellaNomePrezzo[1][i] = riga.split(":")[1].trim(); riga = br.readLine(); } } catch (IOException e) { System.out.println(ERRORE_FILE + e.getMessage()); } - return ritorno; + return tabellaNomePrezzo; } static ArrayList cancellaInOrdinazione(ArrayList ordinazione) { @@ -480,7 +481,7 @@ public class FatturaScontrini { break; case 1: if (getFileSize(PATH_MENU) < 1) { - System.out.println("Errore: il menu è vuoto."); + System.out.println(ERRORE_MENU_VUOTO); } else { stampaMenu(); } @@ -493,7 +494,7 @@ public class FatturaScontrini { break; case 3: if (getFileSize(PATH_MENU) < 1) { - System.out.println("Errore: il menu è vuoto."); + System.out.println(ERRORE_MENU_VUOTO); } else { rimuoviVoceMenu(); System.out.println("Piatto rimosso con successo."); @@ -618,17 +619,17 @@ public class FatturaScontrini { static double prezzo() { boolean error; - double ritorno = -1; + double prezzo = -1; do { error = false; System.out.print("Inserire il prezzo da acquistare: "); try { - ritorno = sc.nextDouble(); + prezzo = sc.nextDouble(); sc.nextLine(); - if (ritorno <= 0) { + if (prezzo <= 0) { System.out.println("Errore: non è possibile impostare un prezzo minore di zero."); pausa(); error = true; @@ -640,7 +641,7 @@ public class FatturaScontrini { } } while (error); - return ritorno; + return prezzo; } static void rimuoviVoceMenu() { @@ -663,19 +664,19 @@ public class FatturaScontrini { } static ArrayList fileToStringArray(String path) { - ArrayList ritorno = new ArrayList<>(); + ArrayList fileAsStringArray = new ArrayList<>(); try (BufferedReader br = new BufferedReader(new FileReader(path))) { String riga = br.readLine(); while (riga != null) { - ritorno.add(riga); + fileAsStringArray.add(riga); riga = br.readLine(); } } catch (IOException e) { System.out.println(ERRORE_FILE + e.getMessage()); } - return ritorno; + return fileAsStringArray; } static void fattura() { @@ -714,35 +715,35 @@ public class FatturaScontrini { static ArrayList trovaScontriniPerData(String data) { ArrayList scontrini = trovaScontrini(); - ArrayList ritorno = new ArrayList<>(); + ArrayList listaScontriniPerData = new ArrayList<>(); for (File scontrino : scontrini) { if (scontrino.getName().contains(data)) { - ritorno.add(scontrino); + listaScontriniPerData.add(scontrino); } } - return ritorno; + return listaScontriniPerData; } static ArrayList trovaScontrini() { - ArrayList ritorno = new ArrayList<>(); + ArrayList listaScontrini = new ArrayList<>(); File cartellaScontrini = new File(PATH_SCONTRINI); File[] listaFile = cartellaScontrini.listFiles(); if (listaFile != null) { for (int i = 0; i < listaFile.length; i++) { if (listaFile[i].isFile() && listaFile[i].getName().split("_")[0].equals("scontrino")) { - ritorno.add(listaFile[i]); + listaScontrini.add(listaFile[i]); } } } - return ritorno; + return listaScontrini; } static double calcolaTotaleScontrini(ArrayList scontrini) { - double ritorno = 0; + double totaleScontrini = 0; for (File scontrino : scontrini) { try (BufferedReader br = new BufferedReader(new FileReader(scontrino))) { String riga = br.readLine(); @@ -751,12 +752,12 @@ public class FatturaScontrini { riga = br.readLine(); } while (!riga.contains("Totale: ")); - ritorno += Double.parseDouble(riga.split(":")[1]); + totaleScontrini += Double.parseDouble(riga.split(":")[1]); } catch (IOException e) { System.out.println(ERRORE_FILE + e.getMessage()); } } - return ritorno; + return totaleScontrini; } static void stampaFattura(Fattura fattura, String data) {