From e16e58524ed9ad85d9567556c6462afb2f3ee2be Mon Sep 17 00:00:00 2001
From: thmustap <tuomas.mustaparta@gmail.com>
Date: Thu, 9 May 2024 12:50:43 +0300
Subject: [PATCH] =?UTF-8?q?Testit=20luotu=20luokkiin=20Ottelu,=20Ottelut?=
=?UTF-8?q?=20sek=C3=A4=20Pelaaja?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
---
Pelaajatiedot | 8 +++---
src/HT/fx/Karkelot.java | 36 ++++++++++++-----------
src/HT/fx/Ottelu.java | 13 ++++++++-
src/HT/fx/Ottelut.java | 10 +++++++
src/HT/fx/PadelMainGUIController.java | 35 ++++++++++++++++++++---
src/HT/fx/Pelaaja.java | 9 ++++++
src/HT/fx/test/OtteluTest.java | 15 ++++++++--
src/HT/fx/test/OttelutTest.java | 17 ++++++++++-
src/HT/fx/test/PelaajaTest.java | 16 +++++++++--
src/HT/fx/test/PelaajatTest.java | 41 ---------------------------
10 files changed, 128 insertions(+), 72 deletions(-)
delete mode 100644 src/HT/fx/test/PelaajatTest.java
diff --git a/Pelaajatiedot b/Pelaajatiedot
index 876264a..10aec37 100644
--- a/Pelaajatiedot
+++ b/Pelaajatiedot
@@ -1,6 +1,6 @@
-1|Rasse|2002-04-01|1010|rasse@gmail.com|1
2|Joonatan|1998-02-24|1030|joonatan@gmail.com|1
-3|Tommi|2002-04-01|920|tommi@gmail.com|0
-4|Riku|2002-03-27|970|R@luukku.fi|0
5|Tuomas|2002-03-27|1030|tuomas@luukku.fi|0
-6|Petri|1996-09-04|1030|pepe@ilomantsinponnistus.fi|1
+6|Petriiiii|1996-09-04|1030|pepe@ilomantsinponnistus.fi|1
+1|Rasse|2002-04-01|1010|rasse@gmail.com|1
+4|Riku|2002-03-27|970|R@luukku.fi|0
+3|Tommi|2002-04-01|920|tommi@gmail.com|0
diff --git a/src/HT/fx/Karkelot.java b/src/HT/fx/Karkelot.java
index af0769d..5f2bea5 100644
--- a/src/HT/fx/Karkelot.java
+++ b/src/HT/fx/Karkelot.java
@@ -138,6 +138,26 @@ public class Karkelot {
pelaajat.talleta();
ottelut.talleta();
}
+
+
+ /**
+ * kutsuu Ottelut-luokan poistaOttelu-metodia
+ * @param ott ottelu
+ */
+ public void poistaOttelu(Ottelu ott) {
+ ottelut.poistaOttelu(ott);
+
+ }
+
+
+ /**
+ * kutsuu Pelaajat-luokan poistaPelaaja-metodia
+ * @param pel pelaaja
+ */
+ public void poistaPelaaja(Pelaaja pel) {
+ pelaajat.poistaPelaaja(pel);
+ }
+
/**
* Testiohjelma kerhosta
@@ -170,20 +190,4 @@ public class Karkelot {
}
}
- /**
- * kutsuu Ottelut-luokan poistaOttelu-metodia
- * @param ott ottelu
- */
- public void poistaOttelu(Ottelu ott) {
- ottelut.poistaOttelu(ott);
-
- }
-
- /**
- * kutsuu Pelaajat-luokan poistaPelaaja-metodia
- * @param pel pelaaja
- */
- public void poistaPelaaja(Pelaaja pel) {
- pelaajat.poistaPelaaja(pel);
- }
}
diff --git a/src/HT/fx/Ottelu.java b/src/HT/fx/Ottelu.java
index 2f88960..bdb9c93 100644
--- a/src/HT/fx/Ottelu.java
+++ b/src/HT/fx/Ottelu.java
@@ -63,7 +63,7 @@ public class Ottelu {
*/
public Ottelu(int idNro, LocalDate paivamaara, int pelaajaNro1, int pelaajaNro2, int pelaajaNro3, int pelaajaNro4, int voittajaNro1, int voittajaNro2, String lopputulos ) {
this.idNro = idNro;
- this.paivamaara=paivamaara;
+ this.paivamaara = paivamaara;
this.pelaajaNro1 = pelaajaNro1;
this.pelaajaNro2 = pelaajaNro2;
this.pelaajaNro3 = pelaajaNro3;
@@ -110,6 +110,7 @@ public class Ottelu {
* @return uusi tunnusnumero
* @example
* <pre name="test">
+ * #import java.time.LocalDate;
* Ottelu ottelu1 = new Ottelu();
* Ottelu ottelu2 = new Ottelu();
* ottelu1.rekisteroi();
@@ -117,6 +118,16 @@ public class Ottelu {
* int n1 = ottelu1.getIdNro();
* int n2 = ottelu2.getIdNro();
* n1 === n2-n1;
+ * Pelaaja aku1 = new Pelaaja(), aku2 = new Pelaaja(), aku3 = new Pelaaja(), aku4 = new Pelaaja();
+ * aku1.rekisteroi();
+ * aku2.rekisteroi();
+ * aku3.rekisteroi();
+ * aku4.rekisteroi();
+ * ottelu1.setpelaajaNro1(1); ottelu1.setpelaajaNro2(2); ottelu1.setpelaajaNro3(3); ottelu1.setpelaajaNro4(4);
+ * ottelu1.setvoittajaNro1(1); ottelu1.setvoittajaNro2(2);
+ * ottelu1.setPvm(LocalDate.parse("2001-01-01"));
+ * ottelu1.setLopputulos("6-0");
+ * ottelu1.getTiedot() === "1|1|2|3|4|1|2|2001-01-01|6-0";
* </pre>
*/
public int rekisteroi() {
diff --git a/src/HT/fx/Ottelut.java b/src/HT/fx/Ottelut.java
index 81855e7..052f19e 100644
--- a/src/HT/fx/Ottelut.java
+++ b/src/HT/fx/Ottelut.java
@@ -30,6 +30,16 @@ public class Ottelut implements Iterable<Ottelu> {
/**
* @param ott ottelu
+ * @example
+ * <pre name="test">
+ * Ottelut ottelut = new Ottelut();
+ * Ottelu ottelu1 = new Ottelu(), ottelu2 = new Ottelu();
+ * ottelut.getLkm() === 0;
+ * ottelut.lisaa(ottelu1); ottelut.getLkm() === 1;
+ * ottelut.lisaa(ottelu2); ottelut.getLkm() === 2;
+ * ottelut.haeOttelu(1) === ottelu2;
+ * ottelut.poistaOttelu(ottelu2); ottelut.getLkm() === 1;
+ * </pre>
*/
public void lisaa(Ottelu ott) {
alkiot.add(ott);
diff --git a/src/HT/fx/PadelMainGUIController.java b/src/HT/fx/PadelMainGUIController.java
index ea12eb5..77b21fc 100644
--- a/src/HT/fx/PadelMainGUIController.java
+++ b/src/HT/fx/PadelMainGUIController.java
@@ -58,7 +58,12 @@ public class PadelMainGUIController {
@Override
public void handle(MouseEvent arg0) {
int m = boksi.getChildren().indexOf(uusiNappi);
- muokkaaPelaaja(null, "kanat", m);
+ TextField haluttu =(TextField) NimiVbox.getChildren().get(m);
+ String muokattavannimi = haluttu.getText();
+
+ muokattava = etsiNimella(muokattavannimi);
+
+ muokkaaPelaaja(null, "kanat");
poistaKaikkiKentat();
@@ -130,11 +135,10 @@ public class PadelMainGUIController {
/**
* @param modalityStage -
* @param oletus -
- * @param m luku
* @return controller
*/
- public static String muokkaaPelaaja(Stage modalityStage, String oletus, int m) {
- muokattava = m;
+ public static String muokkaaPelaaja(Stage modalityStage, String oletus) {
+ //muokattava = m;
return ModalController.showModal(
LisaaPelaajaGUIController.class.getResource("LisaaPelaajaGUIView.fxml"),
"Lisää pelaaja", modalityStage, oletus);
@@ -260,6 +264,28 @@ public void poistaKaikkiKentat() {
}
+ /**
+ * Etsii tietyn nimisen pelaajan id:
+ * @param nimi ketä etsitään
+ * @return löydetyn pelaajan id
+ */
+ public int etsiNimella(String nimi) {
+ Pelaaja pelaajat[] = getKarkelo().getPelaajat();
+ int j=0;
+
+ while(j<getKarkelo().getPelaajia()) {
+
+ if(pelaajat[j].getNimi().equals(nimi)){
+
+ return pelaajat[j].getTunnusNro();
+ }
+
+ j++;
+ }
+
+ return 0;
+ }
+
/**
* järjestää pelaaja taulukon pelaajien ratingin mukaan
@@ -280,6 +306,7 @@ public void poistaKaikkiKentat() {
}
return pelaajat;
}
+
/**
diff --git a/src/HT/fx/Pelaaja.java b/src/HT/fx/Pelaaja.java
index 6e0a65c..73ef82a 100644
--- a/src/HT/fx/Pelaaja.java
+++ b/src/HT/fx/Pelaaja.java
@@ -166,6 +166,7 @@ public class Pelaaja {
* @return jäsenen uusi tunnusNro
* @example
* <pre name="test">
+ * #import java.time.LocalDate;
* Pelaaja aku1 = new Pelaaja();
* aku1.getTunnusNro() === 0;
* aku1.rekisteroi();
@@ -174,6 +175,14 @@ public class Pelaaja {
* int n1 = aku1.getTunnusNro();
* int n2 = aku2.getTunnusNro();
* n1 === n2-1;
+ * aku1.setRating(1050);
+ * aku1.getRating() === 1050;
+ * aku1.setNimi("Teppo");
+ * aku1.getNimi() === "Teppo";
+ * aku1.setSyntaika(LocalDate.parse("2001-01-01"));
+ * aku1.setSahkoposti("aku1@gmail.com");
+ * aku1.getSahkoposti() === "aku1@gmail.com";
+ * aku1.getTiedot() === "1|Teppo|2001-01-01|1050|aku1@gmail.com|0";
* </pre>
*/
public int rekisteroi() {
diff --git a/src/HT/fx/test/OtteluTest.java b/src/HT/fx/test/OtteluTest.java
index 03cbb8a..d918fe3 100644
--- a/src/HT/fx/test/OtteluTest.java
+++ b/src/HT/fx/test/OtteluTest.java
@@ -1,5 +1,6 @@
package HT.fx.test;
// Generated by ComTest BEGIN
+import java.time.LocalDate;
import static org.junit.Assert.*;
import org.junit.*;
import HT.fx.*;
@@ -7,7 +8,7 @@ import HT.fx.*;
/**
* Test class made by ComTest
- * @version 2024.04.29 12:13:03 // Generated by ComTest
+ * @version 2024.05.09 12:45:46 // Generated by ComTest
*
*/
@SuppressWarnings({ "all" })
@@ -25,6 +26,16 @@ public class OtteluTest {
ottelu2.rekisteroi();
int n1 = ottelu1.getIdNro();
int n2 = ottelu2.getIdNro();
- assertEquals("From: Ottelu line: 119", n2-n1, n1);
+ assertEquals("From: Ottelu line: 120", n2-n1, n1);
+ Pelaaja aku1 = new Pelaaja(), aku2 = new Pelaaja(), aku3 = new Pelaaja(), aku4 = new Pelaaja();
+ aku1.rekisteroi();
+ aku2.rekisteroi();
+ aku3.rekisteroi();
+ aku4.rekisteroi();
+ ottelu1.setpelaajaNro1(1); ottelu1.setpelaajaNro2(2); ottelu1.setpelaajaNro3(3); ottelu1.setpelaajaNro4(4);
+ ottelu1.setvoittajaNro1(1); ottelu1.setvoittajaNro2(2);
+ ottelu1.setPvm(LocalDate.parse("2001-01-01"));
+ ottelu1.setLopputulos("6-0");
+ assertEquals("From: Ottelu line: 130", "1|1|2|3|4|1|2|2001-01-01|6-0", ottelu1.getTiedot());
} // Generated by ComTest END
}
\ No newline at end of file
diff --git a/src/HT/fx/test/OttelutTest.java b/src/HT/fx/test/OttelutTest.java
index 19fb300..2a89b91 100644
--- a/src/HT/fx/test/OttelutTest.java
+++ b/src/HT/fx/test/OttelutTest.java
@@ -7,9 +7,24 @@ import HT.fx.*;
/**
* Test class made by ComTest
- * @version 2024.04.29 12:13:20 // Generated by ComTest
+ * @version 2024.05.09 12:43:19 // Generated by ComTest
*
*/
@SuppressWarnings({ "all" })
public class OttelutTest {
+
+
+
+ // Generated by ComTest BEGIN
+ /** testLisaa34 */
+ @Test
+ public void testLisaa34() { // Ottelut: 34
+ Ottelut ottelut = new Ottelut();
+ Ottelu ottelu1 = new Ottelu(), ottelu2 = new Ottelu();
+ assertEquals("From: Ottelut line: 37", 0, ottelut.getLkm());
+ ottelut.lisaa(ottelu1); assertEquals("From: Ottelut line: 38", 1, ottelut.getLkm());
+ ottelut.lisaa(ottelu2); assertEquals("From: Ottelut line: 39", 2, ottelut.getLkm());
+ assertEquals("From: Ottelut line: 40", ottelu2, ottelut.haeOttelu(1));
+ ottelut.poistaOttelu(ottelu2); assertEquals("From: Ottelut line: 41", 1, ottelut.getLkm());
+ } // Generated by ComTest END
}
\ No newline at end of file
diff --git a/src/HT/fx/test/PelaajaTest.java b/src/HT/fx/test/PelaajaTest.java
index 162291b..222ff40 100644
--- a/src/HT/fx/test/PelaajaTest.java
+++ b/src/HT/fx/test/PelaajaTest.java
@@ -1,13 +1,15 @@
package HT.fx.test;
// Generated by ComTest BEGIN
+import java.time.LocalDate;
import static org.junit.Assert.*;
import org.junit.*;
import HT.fx.*;
// Generated by ComTest END
+import HT.fx.Pelaaja;
/**
* Test class made by ComTest
- * @version 2024.04.29 12:13:32 // Generated by ComTest
+ * @version 2024.05.09 12:19:33 // Generated by ComTest
*
*/
@SuppressWarnings({ "all" })
@@ -20,12 +22,20 @@ public class PelaajaTest {
@Test
public void testRekisteroi168() { // Pelaaja: 168
Pelaaja aku1 = new Pelaaja();
- assertEquals("From: Pelaaja line: 170", 0, aku1.getTunnusNro());
+ assertEquals("From: Pelaaja line: 171", 0, aku1.getTunnusNro());
aku1.rekisteroi();
Pelaaja aku2 = new Pelaaja();
aku2.rekisteroi();
int n1 = aku1.getTunnusNro();
int n2 = aku2.getTunnusNro();
- assertEquals("From: Pelaaja line: 176", n2-1, n1);
+ assertEquals("From: Pelaaja line: 177", n2-1, n1);
+ aku1.setRating(1050);
+ assertEquals("From: Pelaaja line: 179", 1050, aku1.getRating());
+ aku1.setNimi("Teppo");
+ assertEquals("From: Pelaaja line: 181", "Teppo", aku1.getNimi());
+ aku1.setSyntaika(LocalDate.parse("2001-01-01"));
+ aku1.setSahkoposti("aku1@gmail.com");
+ assertEquals("From: Pelaaja line: 184", "aku1@gmail.com", aku1.getSahkoposti());
+ assertEquals("From: Pelaaja line: 185", "1|Teppo|2001-01-01|1050|aku1@gmail.com|0", aku1.getTiedot());
} // Generated by ComTest END
}
\ No newline at end of file
diff --git a/src/HT/fx/test/PelaajatTest.java b/src/HT/fx/test/PelaajatTest.java
deleted file mode 100644
index 8ef3de7..0000000
--- a/src/HT/fx/test/PelaajatTest.java
+++ /dev/null
@@ -1,41 +0,0 @@
-package HT.fx.test;
-// Generated by ComTest BEGIN
-import static org.junit.Assert.*;
-import org.junit.*;
-import HT.fx.*;
-// Generated by ComTest END
-
-/**
- * Test class made by ComTest
- * @version 2024.05.09 11:54:44 // Generated by ComTest
- *
- */
-@SuppressWarnings({ "all" })
-public class PelaajatTest {
-
-
-
- // Generated by ComTest BEGIN
- /**
- * testLisaa36
- * @throws SailoException when error
- */
- @Test
- public void testLisaa36() throws SailoException { // Pelaajat: 36
- Pelaajat pelaajat = new Pelaajat();
- Pelaaja aku1 = new Pelaaja(), aku2 = new Pelaaja(), aku3 = new Pelaaja();
- aku1.rekisteroi();
- aku2.rekisteroi();
- aku3.rekisteroi();
- assertEquals("From: Pelaajat line: 43", 0, pelaajat.getLkm());
- pelaajat.lisaa(aku1); assertEquals("From: Pelaajat line: 44", 1, pelaajat.getLkm());
- pelaajat.lisaa(aku2); assertEquals("From: Pelaajat line: 45", 2, pelaajat.getLkm());
- pelaajat.lisaa(aku3); assertEquals("From: Pelaajat line: 46", 3, pelaajat.getLkm());
- assertEquals("From: Pelaajat line: 47", aku1, pelaajat.anna(1));
- assertEquals("From: Pelaajat line: 48", aku2, pelaajat.anna(2));
- assertEquals("From: Pelaajat line: 49", aku3, pelaajat.anna(3));
- assertEquals("From: Pelaajat line: 50", false, pelaajat.anna(2) == aku1);
- assertEquals("From: Pelaajat line: 51", true, pelaajat.anna(2) == aku2);
- pelaajat.poistaPelaaja(aku3); assertEquals("From: Pelaajat line: 52", 2, pelaajat.getLkm());
- } // Generated by ComTest END
-}
\ No newline at end of file
--
GitLab