lisaatulos tehty toimivaksi ja loput parannukset tehty!

data/premierliigatulokset.csv

@@ -1965,5 +1965,3 @@ Man City,Fulham,1.24,11.11,6.85,H
 West Ham,Ipswich,1.8,4.25,3.96,H
 Everton,Newcastle,3.15,2.23,3.62,D
 Aston Villa,Man United,2.21,3.06,3.82,D
-Chelsea,Nott'm Forest,1.48,6.32,4.73,D
-Brighton,Tottenham,2.9,2.23,4.01,H

src/LisaaTulos.java

@@ -2,12 +2,12 @@ import javafx.fxml.FXML;
 import javafx.fxml.FXMLLoader;
 import javafx.scene.Parent;
 import javafx.scene.Scene;
+import javafx.scene.control.Alert;
 import javafx.scene.control.Button;
 import javafx.scene.control.ComboBox;
 import javafx.scene.control.TextArea;
 import javafx.stage.Stage;
-import liiga.Joukkue;
-
+import java.io.FileWriter;
 import java.util.List;
 import java.io.IOException;
 import java.nio.file.Files;
@@ -22,6 +22,10 @@ public class LisaaTulos {
     @FXML
     private Button etusivulleNappi;
 
+    /// Nappi jota painamalla uusi tulos tallentuu csv tiedostoon
+    @FXML
+    private Button lisaatulosnappi;
+
     /// Kenttä jossa valitaan kotijoukkue tiputusvalikosta
     @FXML
     private ComboBox<String> valitseKotijoukkue;
@@ -53,9 +57,9 @@ public class LisaaTulos {
     public String kotijoukkue;
     public String vierasjoukkue;
     public String lopputulos;
-    public double kotijoukkueenKerroin;
-    public double vierasjoukkueenKerroin;
-    public double tasapelinKerroin;
+    public String kotijoukkueenKerroin;
+    public String vierasjoukkueenKerroin;
+    public String tasapelinKerroin;
     public String valittuLiiga;
 
     /// Alustaa napit ja lisää vaihtoehdot tiputusvalikkoihin.
@@ -71,6 +75,18 @@ public class LisaaTulos {
         }
     });
 
+    // Aluksi kotijoukkue,vierasjoukkue, lopputulos ja liigan valinta on null
+    valitseKotijoukkue.setValue(null);
+    valitseVierasjoukkue.setValue(null);
+    valitseLopputulos.setValue(null);
+    valitseLiigaNappi.setValue(null);
+
+    // Aluksi kertoimet ovat null
+    homekerroin.setText("");
+    awaykerroin.setText("");
+    drawkerroin.setText("");
+
+
     // Lisää liigan valintanappiin arvot
     valitseLiigaNappi.getItems().addAll("Bundesliiga", "Valioliiga");
 
@@ -84,11 +100,9 @@ public class LisaaTulos {
     valitseLiigaNappi.setOnAction(event -> {
         valittuLiiga = valitseLiigaNappi.getValue();
 
-
     valitseKotijoukkue.getItems().clear();
     valitseVierasjoukkue.getItems().clear();
 
-
     if ("Bundesliiga".equals(valittuLiiga)) {
             valitseKotijoukkue.getItems().addAll("Borussia M.Gladbach", "Augsburg", "Arminia Bielefeld", "VfB Stuttgart", "Union Berlin",
                 "Wolfsburg", "Borussia Dortmund", "Mainz 05", "FC Cologne", "RB Leipzig",
@@ -110,15 +124,25 @@ public class LisaaTulos {
     }
     });
 
-    // Aluksi kotijoukkue,vierasjoukkue ja lopputulos on null
-    valitseKotijoukkue.setValue(null);
-    valitseVierasjoukkue.setValue(null);
-    valitseLopputulos.setValue(null);
+    valitseKotijoukkue.setOnAction(event -> {
+        kotijoukkue = valitseKotijoukkue.getValue();
+    });
+
+    valitseVierasjoukkue.setOnAction(event -> {
+        vierasjoukkue = valitseVierasjoukkue.getValue();
+    });
+
+    valitseLopputulos.setOnAction(event -> {
+        lopputulos = valitseLopputulos.getValue();
+    });
 
-    // Aluksi kertoimet ovat nollassa
-    kotijoukkueenKerroin = 0.0;
-    vierasjoukkueenKerroin = 0.0;
-    tasapelinKerroin = 0.0;
+    lisaatulosnappi.setOnAction(event -> {
+        try {
+            LisaaTulosLuetteloon();
+        } catch (IOException e) {
+            throw new RuntimeException(e);
+        }
+    });
 
     }
 
@@ -134,19 +158,52 @@ public class LisaaTulos {
 
     @FXML
     /// Lisää käyttäjän antaman ottelun luetteloon (data -kansiossa)
-    private void LisaaTulosLuetteloon() throws IOException{
+    public void LisaaTulosLuetteloon() throws IOException{
+
+    // Kertoimet saadaan käyttäjän syötteestä
+    kotijoukkueenKerroin = homekerroin.getText();
+    vierasjoukkueenKerroin = awaykerroin.getText();
+    tasapelinKerroin = drawkerroin.getText();
+
+    String csv;
+
+    if(Objects.equals(valittuLiiga, "Bundesliiga")){
+        csv = "data/bundesliigatulokset.csv";
+    }
+    else {
+        csv = "data/premierliigatulokset.csv";
+    }
+
+    String lisattavaRivi = kotijoukkue + "," + vierasjoukkue + "," + kotijoukkueenKerroin + "," + vierasjoukkueenKerroin + "," + tasapelinKerroin + "," + lopputulos;
 
-    String kotijoukkueenID;
-    String vierasjoukkueenID;
+    FileWriter tiedosto = new FileWriter(csv, true);
+    tiedosto.append(lisattavaRivi);
+    tiedosto.append("\n");
+    tiedosto.flush();
+
+    // Kertoo käyttäjälle että tulos lisätty
+    Alert ilmoitus = new Alert(Alert.AlertType.INFORMATION);
+    ilmoitus.setTitle(null);
+    ilmoitus.setHeaderText(null);
+    ilmoitus.setContentText("Tulos on lisätty onnistuneesti! Voit nyt lisätä uuden tuloksen halutessasi");
+    ilmoitus.showAndWait();
+
+    // Alustetaan tuloksen lisäämiseen tarvittavat kentät
+    valitseKotijoukkue.setValue(null);
+    valitseVierasjoukkue.setValue(null);
+    valitseLopputulos.setValue(null);
+    valitseLiigaNappi.setValue(null);
+    homekerroin.setText("");
+    awaykerroin.setText("");
+    drawkerroin.setText("");
 
     }
 
-    /*
     /// Etsii joukkueen ID:n oikeasta tiedostosta nimen perusteella
-    public int EtsiJoukkuettaVastaavaId(Joukkue joukkue) throws IOException {
+    public int EtsiJoukkuettaVastaavaId(String joukkue) throws IOException {
 
-        Path premierID = Paths.get("ohj2/data/premierID.csv");
-        Path bundesID = Paths.get("ohj2/data/bundesID.csv");
+        Path premierID = Paths.get("data/premierID.csv");
+        Path bundesID = Paths.get("data/bundesID.csv");
 
         List<String> tiedosto;
 
@@ -172,9 +229,4 @@ public class LisaaTulos {
         return joukkueenId;
     }
 
-    public void main(String[] args) throws IOException {
-        int i = EtsiJoukkuettaVastaavaId("Chelsea");
-        System.out.println(i);
-    }
-    */
 }

src/lisaaTulos.fxml

@@ -26,7 +26,7 @@
             <TextArea fx:id="awaykerroin" layoutX="232.0" layoutY="192.0" prefHeight="39.0" prefWidth="179.0" promptText="vierasjoukkueen kerroin (x.xx)" />
             <TextArea fx:id="drawkerroin" layoutX="429.0" layoutY="192.0" prefHeight="39.0" prefWidth="163.0" promptText="tasapelin kerroin" />
             <Button fx:id="etusivulleNappi" layoutY="1.0" mnemonicParsing="false" prefHeight="36.0" prefWidth="89.0" text="ETUSIVULLE" />
-            <Button layoutX="158.0" layoutY="298.0" mnemonicParsing="false" prefHeight="58.0" prefWidth="292.0" text="LISÄÄ" />
+            <Button fx:id= "lisaatulosnappi" layoutX="158.0" layoutY="298.0" mnemonicParsing="false" prefHeight="58.0" prefWidth="292.0" text="LISÄÄ" />
             <ComboBox fx:id="valitseLiigaNappi" layoutX="246.0" layoutY="67.0" prefWidth="150.0" promptText="valitse liiga" />
 
          </children>