diff --git a/src/HT/fx/Karkelot.java b/src/HT/fx/Karkelot.java index 5f2bea5f15c910b274cd24515706cf22049cbec7..a86bb5399be40cf9b44cb63ed44424ea2ecf1b2a 100644 --- a/src/HT/fx/Karkelot.java +++ b/src/HT/fx/Karkelot.java @@ -158,6 +158,10 @@ public class Karkelot { pelaajat.poistaPelaaja(pel); } + public int etsiNimella(String nimi) { + return pelaajat.etsiNimella(nimi); + } + /** * Testiohjelma kerhosta diff --git a/src/HT/fx/PadelMainGUIController.java b/src/HT/fx/PadelMainGUIController.java index 77b21fc7a3b27fd5e045fe26c3194852b7b3ef8b..9a4003378c9fb3c8eaff4bb56c0e0c35ce973dd8 100644 --- a/src/HT/fx/PadelMainGUIController.java +++ b/src/HT/fx/PadelMainGUIController.java @@ -61,7 +61,7 @@ public class PadelMainGUIController { TextField haluttu =(TextField) NimiVbox.getChildren().get(m); String muokattavannimi = haluttu.getText(); - muokattava = etsiNimella(muokattavannimi); + muokattava = karkelo.etsiNimella(muokattavannimi); muokkaaPelaaja(null, "kanat"); @@ -264,28 +264,6 @@ 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 diff --git a/src/HT/fx/Pelaajat.java b/src/HT/fx/Pelaajat.java index 5afa87ea9559be369153a7dd6217978862380475..c881979b409e9c408a9c8463c411b52fc4dadeb7 100644 --- a/src/HT/fx/Pelaajat.java +++ b/src/HT/fx/Pelaajat.java @@ -50,6 +50,8 @@ public class Pelaajat { * pelaajat.anna(2) == aku1 === false; * pelaajat.anna(2) == aku2 === true; * pelaajat.poistaPelaaja(aku3); pelaajat.getLkm() === 2; + * aku1.setNimi("Teppo"); + * pelaajat.etsiNimella("Teppo") === 1; * </pre> */ public void lisaa(Pelaaja pelaaja) throws SailoException { @@ -116,6 +118,28 @@ public class Pelaajat { } } + /** + * Etsii tietyn nimisen pelaajan id: + * @param nimi ketä etsitään + * @return löydetyn pelaajan id + */ + public int etsiNimella(String nimi) { + Pelaaja pelaajat[] = alkiot; + int j=0; + + while(j<lkm) { + + if(pelaajat[j].getNimi().equals(nimi)){ + + return pelaajat[j].getTunnusNro(); + } + + j++; + } + + return 0; + } + /** * @return pelaajat listana */ diff --git a/src/HT/fx/test/PadelMainGUIControllerTest.java b/src/HT/fx/test/PadelMainGUIControllerTest.java new file mode 100644 index 0000000000000000000000000000000000000000..e74b2cd750b5dedb36e8647c94a878ac560c38fd --- /dev/null +++ b/src/HT/fx/test/PadelMainGUIControllerTest.java @@ -0,0 +1,33 @@ +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 13:01:00 // Generated by ComTest + * + */ +@SuppressWarnings({ "all" }) +public class PadelMainGUIControllerTest { + + + + // Generated by ComTest BEGIN + /** testEtsiNimella272 */ + @Test + public void testEtsiNimella272() { // PadelMainGUIController: 272 + Pelaaja aku1 = new Pelaaja(), aku2 = new Pelaaja(), aku3 = new Pelaaja(), aku4 = new Pelaaja(); + aku1.rekisteroi(); + aku2.rekisteroi(); + aku3.rekisteroi(); + aku4.rekisteroi(); + aku1.setNimi("Teppo1"); + aku2.setNimi("Teppo2"); + aku3.setNimi("Teppo3"); + aku4.setNimi("Teppo4"); + assertEquals("From: PadelMainGUIController line: 282", 2, etsiNimella("Teppo2")); + } // 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 new file mode 100644 index 0000000000000000000000000000000000000000..a284e737d63edcd8f0b4ddffe9c8453ea238ae1a --- /dev/null +++ b/src/HT/fx/test/PelaajatTest.java @@ -0,0 +1,42 @@ +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 13:09:09 // 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()); + aku1.setNimi("Teppo"); + assertEquals("From: Pelaajat line: 54", 1, pelaajat.etsiNimella("Teppo")); + } // Generated by ComTest END +} \ No newline at end of file