From e9636cff163bf73e14ba441eefb507ab3cd04816 Mon Sep 17 00:00:00 2001
From: joonatan <joonatan.a.nikkila@student.jyu.fi>
Date: Thu, 14 Mar 2024 10:31:52 +0200
Subject: [PATCH] tulostus tehty

---
 src/HT/fx/Karkelot.java                      |  5 ++
 src/HT/fx/LisaaPelaajaGUIController.java     |  2 +-
 src/HT/fx/OtteluIkkunaGUIController.java     |  5 ++
 src/HT/fx/PadelLisaaOtteluGUIController.java | 94 ++++++++++----------
 src/HT/fx/PadelMainGUIController.java        |  2 +-
 src/HT/fx/Pelaaja.java                       | 20 +++++
 src/HT/fx/Pelaajat.java                      |  5 ++
 7 files changed, 85 insertions(+), 48 deletions(-)

diff --git a/src/HT/fx/Karkelot.java b/src/HT/fx/Karkelot.java
index bec1e80..babd476 100644
--- a/src/HT/fx/Karkelot.java
+++ b/src/HT/fx/Karkelot.java
@@ -64,6 +64,11 @@ public class Karkelot {
         return ottelut.annaOttelut(pelaaja.getTunnusNro());
     }
     
+    /**
+     * Hakee i:nnen ottelun ottelut oliolta
+     * @param i monesko ottelu haetaan
+     * @return ottelun
+     */
     public Ottelu noudaOttelu(int i) {
         return ottelut.haeOttelu(i);
     }
diff --git a/src/HT/fx/LisaaPelaajaGUIController.java b/src/HT/fx/LisaaPelaajaGUIController.java
index 83a467b..79ddfab 100644
--- a/src/HT/fx/LisaaPelaajaGUIController.java
+++ b/src/HT/fx/LisaaPelaajaGUIController.java
@@ -19,7 +19,7 @@ import HT.fx.Karkelot;
 import HT.fx.SailoException;
 
 /**
- * @author joonatan
+ * @author joonatan, tuomas
  * @version 31.1.2024
  *
  */
diff --git a/src/HT/fx/OtteluIkkunaGUIController.java b/src/HT/fx/OtteluIkkunaGUIController.java
index 34e1c54..4ee4e0e 100644
--- a/src/HT/fx/OtteluIkkunaGUIController.java
+++ b/src/HT/fx/OtteluIkkunaGUIController.java
@@ -9,6 +9,11 @@ import javafx.scene.control.TextField;
 import javafx.scene.input.MouseEvent;
 import javafx.stage.Stage;
 
+/**
+ * @author joonatan, tuomas
+ * @version 13.3.2024
+ *
+ */
 public class OtteluIkkunaGUIController implements ModalControllerInterface<String> {
     
     
diff --git a/src/HT/fx/PadelLisaaOtteluGUIController.java b/src/HT/fx/PadelLisaaOtteluGUIController.java
index 10607b2..32090b6 100644
--- a/src/HT/fx/PadelLisaaOtteluGUIController.java
+++ b/src/HT/fx/PadelLisaaOtteluGUIController.java
@@ -12,44 +12,71 @@ import javafx.scene.control.TextField;
 import javafx.scene.input.MouseEvent;
 import javafx.stage.Stage;
 
+/**
+ * @author joonatan, tuomas
+ * @version 13.3.2024
+ *
+ */
 public class PadelLisaaOtteluGUIController implements ModalControllerInterface<String> {
     
-    @FXML
-    private Button OKButton;
+    @FXML private Button OKButton;
 
-    @FXML
-    private ComboBoxChooser<String> menuOsallistuja1;
+    @FXML private ComboBoxChooser<String> menuOsallistuja1;
 
-    @FXML
-    private ComboBoxChooser<String> menuOsallistuja2;
+    @FXML private ComboBoxChooser<String> menuOsallistuja2;
 
-    @FXML
-    private ComboBoxChooser<String> menuOsallistuja3;
+    @FXML private ComboBoxChooser<String> menuOsallistuja3;
 
-    @FXML
-    private ComboBoxChooser<String> menuOsallistuja4;
+    @FXML private ComboBoxChooser<String> menuOsallistuja4;
 
-    @FXML
-    private ComboBoxChooser<String> menuVoittaja1;
+    @FXML private ComboBoxChooser<String> menuVoittaja1;
 
-    @FXML
-    private ComboBoxChooser<String> menuVoittaja2;
+    @FXML private ComboBoxChooser<String> menuVoittaja2;
 
-    @FXML
-    private DatePicker otteluPvm;
+    @FXML private DatePicker otteluPvm;
 
-    @FXML
-    private TextField textLopputulos;
+    @FXML private TextField textLopputulos;
 
     @FXML void handleOk() {
-        
-        
         kasitteleOk();
         Stage stage = (Stage) OKButton.getScene().getWindow();
         stage.close();
+    }
 
-}
+    /**
+     * Tällä hetkellä ei poista mitään, vaan toimii käytännössä päivitä nappina
+     */
+    @FXML void handlePoista() {
+        alusta(menuOsallistuja1);
+        alusta(menuOsallistuja2);
+        alusta(menuOsallistuja3);
+        alusta(menuOsallistuja4);
+        alusta(menuVoittaja1);
+        alusta(menuVoittaja2);
+        Dialogs.showMessageDialog("Ei osata vielä");
+    }
 
+    //=======================================================
+    
+    /**
+     * Lisää vaihtoehdot parametrina annetulle valintamenulle
+     * @param menu mihin menuun lisätään
+     */
+    private void alusta(ComboBoxChooser<String> menu) {
+        menu.clear();
+        int i = 0;
+        
+        int lukumaara = PadelMainGUIController.getKarkelo().getPelaajia();
+        while(i< lukumaara) {
+            menu.add(PadelMainGUIController.getKarkelo().annaPelaaja(i).getNimi(), null);
+            i++;
+        }
+    }
+    
+    /**
+     * Käsittelee ok napin. Hakee jokaisen valitun indeksin ja tekee 
+     * näistä ottelun main controllerin karkeloon
+     */
     public void kasitteleOk() {
         //String pvm = datesyntaika.getValue().format(DateTimeFormatter.ofPattern("yyyy-MM-dd"));
         int PelaajaNro1 = menuOsallistuja1.getSelectedIndex();
@@ -67,18 +94,6 @@ public class PadelLisaaOtteluGUIController implements ModalControllerInterface<S
     
     }
     
-
-    
-    @FXML void handlePoista() {
-        alusta(menuOsallistuja1);
-        alusta(menuOsallistuja2);
-        alusta(menuOsallistuja3);
-        alusta(menuOsallistuja4);
-        alusta(menuVoittaja1);
-        alusta(menuVoittaja2);
-        Dialogs.showMessageDialog("Ei osata vielä");
-    }
-
     @Override
     public String getResult() {
         // TODO Auto-generated method stub
@@ -96,17 +111,4 @@ public class PadelLisaaOtteluGUIController implements ModalControllerInterface<S
         // TODO Auto-generated method stub
         
     }
-
-    //=======================================================
-    
-    private void alusta(ComboBoxChooser<String> menu) {
-        menu.clear();
-        int i = 0;
-        
-        int lukumaara = PadelMainGUIController.getKarkelo().getPelaajia();
-        while(i< lukumaara) {
-            menu.add(PadelMainGUIController.getKarkelo().annaPelaaja(i).getNimi(), null);
-            i++;
-        }
-    }
 }
diff --git a/src/HT/fx/PadelMainGUIController.java b/src/HT/fx/PadelMainGUIController.java
index b301a96..8cc87a9 100644
--- a/src/HT/fx/PadelMainGUIController.java
+++ b/src/HT/fx/PadelMainGUIController.java
@@ -20,7 +20,7 @@ import HT.fx.SailoException;
 
 
 /**
- * @author joonatan
+ * @author joonatan tuomas
  * @version 31.1.2024
  *
  */
diff --git a/src/HT/fx/Pelaaja.java b/src/HT/fx/Pelaaja.java
index 19c3054..2df33e7 100644
--- a/src/HT/fx/Pelaaja.java
+++ b/src/HT/fx/Pelaaja.java
@@ -5,6 +5,11 @@ import java.io.PrintStream;
 import java.time.LocalDate;
 import java.time.Period;
 
+/**
+ * @author joonatan, tuomas
+ * @version 13.3.2024
+ *
+ */
 public class Pelaaja {
     
     
@@ -30,6 +35,9 @@ public class Pelaaja {
         this.auto = auto;
     }
     
+    /**
+     * Muodostaja tyhjän pelaajan luomiseen
+     */
     public Pelaaja() {
         // TODO Auto-generated constructor stub
     }
@@ -63,19 +71,31 @@ public class Pelaaja {
         tulosta(new PrintStream(os));
     }
     
+    /**
+     * @return palauttaa pelaajan nimen
+     */
     public String getNimi() {
         return nimi;
     }
     
+    /**
+     * @return palauttaa pelaajan iän
+     */
     public int getIka() {
         return Period.between(syntymaaika, LocalDate.now()).getYears();  
         
     }
     
+    /**
+     * @return palauttaa pelaajan sähköpostin
+     */
     public String getSahkoposti() {
         return sahkoposti;
     }
     
+    /**
+     * @return palauttaa tiedon onko pelaajalla auto, true = on auto
+     */
     public boolean getAuto() {
         return auto;
     }
diff --git a/src/HT/fx/Pelaajat.java b/src/HT/fx/Pelaajat.java
index 116647e..18636b3 100644
--- a/src/HT/fx/Pelaajat.java
+++ b/src/HT/fx/Pelaajat.java
@@ -1,5 +1,10 @@
 package HT.fx;
 
+/**
+ * @author joonatan, tuomas
+ * @version 13.3.2024
+ *
+ */
 public class Pelaajat {
     private static final int Max_Pelaajat   = 9;
     private int lkm           = 0;
-- 
GitLab