From 35d2a6e6e165df7c398275211e0026a0367d39d4 Mon Sep 17 00:00:00 2001 From: joonatan <joonatan.a.nikkila@student.jyu.fi> Date: Wed, 24 Apr 2024 15:59:57 +0300 Subject: [PATCH] Pelaajan poistoa edistetty, ei toimi --- Pelaajatiedot | 4 +-- src/HT/fx/LisaaPelaajaGUIController.java | 8 +++--- src/HT/fx/PadelLisaaOtteluGUIController.java | 4 +-- src/HT/fx/PadelMainGUIController.java | 26 +++++++++++--------- src/HT/fx/Pelaaja.java | 5 ++++ src/HT/fx/Pelaajat.java | 19 ++++++++------ 6 files changed, 37 insertions(+), 29 deletions(-) diff --git a/Pelaajatiedot b/Pelaajatiedot index 9636226..3dc3573 100644 --- a/Pelaajatiedot +++ b/Pelaajatiedot @@ -4,6 +4,4 @@ 4|Riku|2001-04-01|970|riku@gmail.com|0 5|Jussi|2003-04-01|1030|jussi@luukku.fi|1 6|Tuomas|2002-03-27|1030|tuomas@luukku.fi|0 -7|Levy|2002-04-06|990|levy@edu.muurame.fi|1 -8|Pepe|1996-09-03|990|pepe@ilomantsinponnistus.fi|1 -9|Samu|2002-09-10|980|samu@padelx.com|0 +7|Pepe|1996-09-03|990|pepe@ilomantsinponnistus.fi|1 diff --git a/src/HT/fx/LisaaPelaajaGUIController.java b/src/HT/fx/LisaaPelaajaGUIController.java index 1c59779..53a920a 100644 --- a/src/HT/fx/LisaaPelaajaGUIController.java +++ b/src/HT/fx/LisaaPelaajaGUIController.java @@ -39,13 +39,11 @@ public class LisaaPelaajaGUIController implements ModalControllerInterface<Strin @FXML private Button OKButton; @FXML void handleOk() { - - + kasitteleOk(); Stage stage = (Stage) OKButton.getScene().getWindow(); stage.close(); - - + } @FXML void handleAlusta() { @@ -56,6 +54,8 @@ public class LisaaPelaajaGUIController implements ModalControllerInterface<Strin if(muok>0) { Pelaaja pel = PadelMainGUIController.getKarkelo().annaPelaaja(muok); PadelMainGUIController.getKarkelo().poistaPelaaja(pel); + + Stage stage = (Stage) OKButton.getScene().getWindow(); stage.close(); diff --git a/src/HT/fx/PadelLisaaOtteluGUIController.java b/src/HT/fx/PadelLisaaOtteluGUIController.java index 95a6796..3267fc0 100644 --- a/src/HT/fx/PadelLisaaOtteluGUIController.java +++ b/src/HT/fx/PadelLisaaOtteluGUIController.java @@ -54,7 +54,7 @@ public class PadelLisaaOtteluGUIController implements ModalControllerInterface<S alusta(menuOsallistuja4); alusta(menuVoittaja1); alusta(menuVoittaja2); - Dialogs.showMessageDialog("Ei osata vielä"); + } if(muok>0) { @@ -67,7 +67,7 @@ public class PadelLisaaOtteluGUIController implements ModalControllerInterface<S Ottelu ott = PadelMainGUIController.getKarkelo().noudaOttelu(muok); PadelMainGUIController.getKarkelo().poistaOttelu(ott); - + // Suljetaan ikkuna Stage stage = (Stage) OKButton.getScene().getWindow(); diff --git a/src/HT/fx/PadelMainGUIController.java b/src/HT/fx/PadelMainGUIController.java index 8c7e0e2..4e5dad0 100644 --- a/src/HT/fx/PadelMainGUIController.java +++ b/src/HT/fx/PadelMainGUIController.java @@ -163,14 +163,13 @@ public class PadelMainGUIController { int m = 1; int n = 1; - System.out.println(karkelo.getPelaajia()+" "+ Pelaaja.getSeuraavaNro()); + for (Node lapsi : NimiVbox.getChildren()) { if (lapsi instanceof TextField) { TextField tf = (TextField) lapsi; - - tf.setText(karkelo.annaPelaaja(i).getNimi()); + if(karkelo.annaPelaaja(i)!=null) tf.setText(karkelo.annaPelaaja(i).getNimi()); i++; } } @@ -178,7 +177,7 @@ public class PadelMainGUIController { for (Node lapsi : IkaVbox.getChildren()) { if (lapsi instanceof TextField) { TextField tf = (TextField) lapsi; - tf.setText(String.valueOf(karkelo.annaPelaaja(j).getIka())); + if(karkelo.annaPelaaja(j)!=null) tf.setText(String.valueOf(karkelo.annaPelaaja(j).getIka())); j++; } } @@ -187,7 +186,7 @@ public class PadelMainGUIController { if (lapsi instanceof TextField) { TextField tf = (TextField) lapsi; - tf.setText(String.valueOf(karkelo.annaPelaaja(n).getRating())); + if(karkelo.annaPelaaja(n)!=null) tf.setText(String.valueOf(karkelo.annaPelaaja(n).getRating())); n++; } } @@ -196,7 +195,7 @@ public class PadelMainGUIController { if (lapsi instanceof TextField) { TextField tf = (TextField) lapsi; - tf.setText(karkelo.annaPelaaja(k).getSahkoposti()); + if(karkelo.annaPelaaja(k)!=null) tf.setText(karkelo.annaPelaaja(k).getSahkoposti()); k++; } } @@ -204,13 +203,16 @@ public class PadelMainGUIController { for (Node lapsi : AutoVbox.getChildren()) { if (lapsi instanceof TextField) { TextField tf = (TextField) lapsi; - if(karkelo.annaPelaaja(m).getAuto()) { - tf.setText("Kyllä"); - m++; - }else { - tf.setText("Ei"); - m++; + if(karkelo.annaPelaaja(m)!=null) { + if(karkelo.annaPelaaja(m).getAuto()) { + tf.setText("Kyllä"); + + }else { + tf.setText("Ei"); + + } } + m++; } } } diff --git a/src/HT/fx/Pelaaja.java b/src/HT/fx/Pelaaja.java index 8831514..c1c4b45 100644 --- a/src/HT/fx/Pelaaja.java +++ b/src/HT/fx/Pelaaja.java @@ -239,6 +239,11 @@ public class Pelaaja { return seuraavaNro; } + public void setTunnusNro(int i) { + tunnusnro=i; + + } + diff --git a/src/HT/fx/Pelaajat.java b/src/HT/fx/Pelaajat.java index e98577f..f41f11e 100644 --- a/src/HT/fx/Pelaajat.java +++ b/src/HT/fx/Pelaajat.java @@ -82,10 +82,12 @@ public class Pelaajat { Pelaaja palautettava = null; int m = 0; while(m<alkiot.length) { - if(alkiot[m].getTunnusNro() == i) { - palautettava = alkiot[m]; - break; - } + if(alkiot[m]!=null) { + if(alkiot[m].getTunnusNro() == i) { + palautettava = alkiot[m]; + break; + } + } m++; } @@ -135,10 +137,11 @@ public class Pelaajat { try (PrintStream fo = new PrintStream(new FileOutputStream("Pelaajatiedot", false))) { int i = 1; while ( i<=PadelMainGUIController.getKarkelo().getPelaajia()) { - - fo.print(PadelMainGUIController.getKarkelo().annaPelaaja(i).getTiedot()+ "\n"); + if(PadelMainGUIController.getKarkelo().annaPelaaja(i)!=null) { + fo.print(PadelMainGUIController.getKarkelo().annaPelaaja(i).getTiedot()+ "\n"); + + } i++; - } //System.out.println("Heureka ! "); } catch(FileNotFoundException ex) { @@ -199,7 +202,7 @@ public class Pelaajat { } i++; } - lkm--; + //lkm--; alkiot = alkiot2; } -- GitLab