From 9c59af3953eda58ee99f8069c2c85a8c87c70e95 Mon Sep 17 00:00:00 2001
From: Thomas Rubini <74205383+ThomasRubini@users.noreply.github.com>
Date: Tue, 7 Jun 2022 22:48:17 +0200
Subject: [PATCH] cleanup

---
 src/main/java/fr/univ_amu/iut/AppMain.java    |  6 +--
 .../java/fr/univ_amu/iut/fp/Controller.java   | 45 +++++--------------
 2 files changed, 14 insertions(+), 37 deletions(-)

diff --git a/src/main/java/fr/univ_amu/iut/AppMain.java b/src/main/java/fr/univ_amu/iut/AppMain.java
index 4a4d3a9..e4ebcd5 100644
--- a/src/main/java/fr/univ_amu/iut/AppMain.java
+++ b/src/main/java/fr/univ_amu/iut/AppMain.java
@@ -22,19 +22,19 @@ public class AppMain extends Application {
         testMode = res!=null&&res.equals("true");
 
 
-        ScreenController.addScreen("Acceuil",FXMLLoader.load(getClass().getResource("/fr/univ_amu/iut/fp/fp.fxml")));
+        ScreenController.addScreen("Accueil",FXMLLoader.load(getClass().getResource("/fr/univ_amu/iut/fp/fp.fxml")));
         
         //TODO Ajouter les pages d'admin pour l'ajout des usages
         //ScreenController.addScreen("admin1",pane);
         //ScreenController.addScreen("admin2",pane);
         
-        ScreenController.activate("Acceuil");
+        ScreenController.activate("Accueil");
         stage.setTitle("Carte des académie");
         stage.setScene(scene);
         stage.show();
     }
 
-    public static void main(String[] args) throws InterruptedException {
+    public static void main(String[] args) {
         launch(args);
     }
 }
diff --git a/src/main/java/fr/univ_amu/iut/fp/Controller.java b/src/main/java/fr/univ_amu/iut/fp/Controller.java
index dd05f56..26003ee 100644
--- a/src/main/java/fr/univ_amu/iut/fp/Controller.java
+++ b/src/main/java/fr/univ_amu/iut/fp/Controller.java
@@ -2,6 +2,7 @@ package fr.univ_amu.iut.fp;
 
 import fr.univ_amu.iut.AppMain;
 import fr.univ_amu.iut.Donnees;
+import fr.univ_amu.iut.dao.DAO;
 import fr.univ_amu.iut.dao.DAODiscipline;
 import fr.univ_amu.iut.dao.DAOThematique;
 import fr.univ_amu.iut.dao.DAOUsage;
@@ -80,7 +81,7 @@ public class Controller implements Initializable {
     }
 
     private Button initButton(Object obj,int x,int y){
-        String nom = "";
+        String nom;
 
         EventHandler<ActionEvent> actionHandler;
         EventHandler<MouseEvent> onPressHandler;
@@ -99,15 +100,11 @@ public class Controller implements Initializable {
         if (obj instanceof Discipline) {
             Discipline disciplineActuelle = (Discipline) obj;
             nom = disciplineActuelle.getNom();
-            actionHandler = event -> {
-                Donnees.setDisciplineSelectionee(disciplineActuelle);
-            };
+            actionHandler = event -> Donnees.setDisciplineSelectionee(disciplineActuelle);
         } else {
             Thematique thematiqueActuelle = (Thematique) obj;
             nom = thematiqueActuelle.getNom();
-            actionHandler = event -> {
-                Donnees.setThematiqueSelectionee(thematiqueActuelle);
-            };
+            actionHandler = event -> Donnees.setThematiqueSelectionee(thematiqueActuelle);
         }
 
 
@@ -157,31 +154,15 @@ public class Controller implements Initializable {
         return bt;
     }
 
-    private Void placeButtonDiscipline() {
-        List<Discipline> disciplines = daoDiscipline.findAll();
-        discipline.setMinHeight(disciplines.size() * 65);
-        for (int i = 0; i < disciplines.size(); ++i) {
+    private void placeButtons(List<?> list, AnchorPane anchorPane) {
+        anchorPane.setMinHeight(list.size() * 65);
+        for (int i = 0; i < list.size(); ++i) {
             if (i % 2 == 0) {
-                discipline.getChildren().add(initButton(disciplines.get(i), 38, i / 2 * 130));
+                anchorPane.getChildren().add(initButton(list.get(i), 38, i / 2 * 130));
             } else {
-                discipline.getChildren().add(initButton(disciplines.get(i), 332, i / 2 * 130));
+                anchorPane.getChildren().add(initButton(list.get(i), 332, i / 2 * 130));
             }
         }
-
-        return null;
-    }
-
-    private Void placeButtonThematique() {
-        List<Thematique> thematiques = daoThematique.findAll();
-        thematique.setMinHeight(thematiques.size() * 65);
-        for (int i = 0; i < thematiques.size(); ++i) {
-            if (i % 2 == 0) {
-                thematique.getChildren().add(initButton(thematiques.get(i), 38, i / 2 * 130));
-            } else {
-                thematique.getChildren().add(initButton(thematiques.get(i), 332, i / 2 * 130));
-            }
-        }
-        return null;
     }
 
     @Override
@@ -195,13 +176,9 @@ public class Controller implements Initializable {
         daoThematique = daoFactory.createDAOThematique();
         daoUsage = daoFactory.createDAOUsage();
 
-        // a chaque fois
+        placeButtons(daoThematique.findAll(), thematique);
+        placeButtons(daoDiscipline.findAll(), discipline);
 
-        List<Discipline> d = daoDiscipline.findAll();
-        List<Thematique> t = daoThematique.findAll();
-
-        placeButtonThematique();
-        placeButtonDiscipline();
         EventHandler<ActionEvent> handleRechercheTextuelle = event ->{
 
             Donnees.setUsagesObtenus(daoUsage.findByNamePart(barreDeRecherche.getText()));