diff --git a/mmkurssit/src/fxMMKurssit/KurssinlisaysGUIController.java b/mmkurssit/src/fxMMKurssit/KurssinlisaysGUIController.java index a40298e0977d4eaa5566cacab0481a66655d8b07..94caecdf3c8b45fa3d3406d22bba6802c934fd36 100644 --- a/mmkurssit/src/fxMMKurssit/KurssinlisaysGUIController.java +++ b/mmkurssit/src/fxMMKurssit/KurssinlisaysGUIController.java @@ -1,5 +1,5 @@ package fxMMKurssit; -import fi.jyu.mit.fxgui.Dialogs; +//import fi.jyu.mit.fxgui.Dialogs; import fi.jyu.mit.fxgui.ModalControllerInterface; import javafx.application.Platform; //import javafx.event.ActionEvent; @@ -7,9 +7,6 @@ import javafx.fxml.FXML; import javafx.scene.control.Alert; import javafx.scene.control.Button; import javafx.scene.control.TextField; -import mmkurssit.Kurssi; -import mmkurssit.MMKurssit; -import mmkurssit.SailoException; import javafx.scene.control.Alert.AlertType; /** diff --git a/mmkurssit/src/fxMMKurssit/LisaaTapaaminenGUIController.java b/mmkurssit/src/fxMMKurssit/LisaaTapaaminenGUIController.java index 35520c0d8a358d4e8b41ba645b9f404d7bdbf135..fccc0e24b4199f99019a1d4c8261be8dbe6cb59d 100644 --- a/mmkurssit/src/fxMMKurssit/LisaaTapaaminenGUIController.java +++ b/mmkurssit/src/fxMMKurssit/LisaaTapaaminenGUIController.java @@ -1,6 +1,6 @@ package fxMMKurssit; -import fi.jyu.mit.fxgui.CheckBoxChooser; -import fi.jyu.mit.fxgui.ListChooser; +//import fi.jyu.mit.fxgui.CheckBoxChooser; +//import fi.jyu.mit.fxgui.ListChooser; import fi.jyu.mit.fxgui.ModalControllerInterface; //import javafx.event.ActionEvent; import javafx.fxml.FXML; @@ -11,7 +11,6 @@ import javafx.scene.control.Label; import javafx.scene.control.TextArea; import javafx.scene.control.TextField; import javafx.scene.control.Alert.AlertType; -import mmkurssit.Kurssi; /** * @author aichi diff --git a/mmkurssit/src/fxMMKurssit/MMKurssitGUIController.java b/mmkurssit/src/fxMMKurssit/MMKurssitGUIController.java index f7050551793e38815165e85aa3bbd5c4f740582f..320aac582f67c8e79e054420ed3a7b688690d072 100644 --- a/mmkurssit/src/fxMMKurssit/MMKurssitGUIController.java +++ b/mmkurssit/src/fxMMKurssit/MMKurssitGUIController.java @@ -2,9 +2,9 @@ package fxMMKurssit; import fi.jyu.mit.fxgui.ModalController; import fi.jyu.mit.fxgui.CheckBoxChooser; +import fi.jyu.mit.fxgui.Dialogs; import fi.jyu.mit.fxgui.ListChooser; import javafx.application.Platform; -//import javafx.event.ActionEvent; import javafx.fxml.FXML; import javafx.fxml.FXMLLoader; import javafx.scene.Parent; @@ -16,6 +16,7 @@ import javafx.scene.control.TextField; import javafx.stage.Stage; import mmkurssit.Kurssi; import mmkurssit.MMKurssit; +import mmkurssit.SailoException; import mmkurssit.Tapaaminen; @@ -66,10 +67,12 @@ public class MMKurssitGUIController { @FXML void addMeeting() { - ModalController.showModal(LisaaTapaaminenGUIController.class.getResource("LisaaTapaaminenGUIView.fxml"), - "Missä ja Milloin Kurssi", null, ""); + addTapaaminen(); + + /*ModalController.showModal(LisaaTapaaminenGUIController.class.getResource("LisaaTapaaminenGUIView.fxml"), + "Missä ja Milloin Kurssi", null, "");*/ } - + //TapaaminenTiedot jotain outo? Ei toimi sen takia, että renamed afterwards? @FXML void openMeetingInfo() { @@ -86,11 +89,13 @@ public class MMKurssitGUIController { @FXML void addCourse() { - ModalController.showModal(KurssinlisaysGUIController.class.getResource("KurssinlisaysGUIView.fxml"), - "Missä ja Milloin Kurssi", null, ""); + /*ModalController.showModal(KurssinlisaysGUIController.class.getResource("KurssinlisaysGUIView.fxml"), + "Missä ja Milloin Kurssi", null, "");*/ + addKurssi(); } - + + @FXML void openCourseList() { // } @@ -121,15 +126,67 @@ public class MMKurssitGUIController { } //=================================================================================== - /** - * Asetetaan käytettävä mmkurssit + * Asetetaan käytettävä mmkurssit (aktivoitu MMKurssitMain:lla) * @param mmkurssit jota käytetään */ public void setMMKurssit(MMKurssit mmkurssit) { this.mmkurssit = mmkurssit; } + + + //========= + + /** + * Lisätään mmkurssille uusi kurssi + */ + private void addKurssi() { + Kurssi uusi = new Kurssi(); + uusi.rekisteroi(); + uusi.tayta(); + try { + mmkurssit.lisaa(uusi); + } catch (SailoException e) { + Dialogs.showMessageDialog("Ongelmia uuden kurssin luomisessa" + e.getMessage()); + } + haeKurssi(uusi.getID()); + } + + + /** + * Haetaan kurssia + * @param nro kurssin ID, joka valitaan + */ + private void haeKurssi(int nro) { + allCourses.clear();//laitetaan tyhjäksi eka + + int indeksi = 0; + for (int i= 0; i<mmkurssit.getKurssienKpl(); i++) { //käydään läpi kaikki kurssit + Kurssi k = mmkurssit.annaKurssi(i); //otetaan 1 kurssi kerrallaan + if (k.getID() == nro) indeksi = i; + allCourses.add(""+k.getNimi(), k); + } + allCourses.setSelectedIndex(indeksi); + } + + + private void addTapaaminen() { + Tapaaminen uusi = new Tapaaminen(); + uusi.rekisteroi(); + uusi.tayta(); + + mmkurssit.lisaa(uusi); + + haeTapaaminen(uusi.getkID()); +} + + + private void haeTapaaminen(int kurssinID) { + tapaamiset.clear();//laitetaan tyhjäksi eka + + // + } diff --git a/mmkurssit/src/fxMMKurssit/test/MMKurssitMainTest.java b/mmkurssit/src/fxMMKurssit/test/MMKurssitMainTest.java new file mode 100644 index 0000000000000000000000000000000000000000..c9a7f8d43973fa32588ed2676113f8110078dfa8 --- /dev/null +++ b/mmkurssit/src/fxMMKurssit/test/MMKurssitMainTest.java @@ -0,0 +1,15 @@ +package fxMMKurssit.test; +// Generated by ComTest BEGIN +import static org.junit.Assert.*; +import org.junit.*; +import fxMMKurssit.*; +// Generated by ComTest END + +/** + * Test class made by ComTest + * @version 2025.03.18 09:51:04 // Generated by ComTest + * + */ +@SuppressWarnings("all") +public class MMKurssitMainTest { +} \ No newline at end of file diff --git a/mmkurssit/src/mmkurssit/Kurssi.java b/mmkurssit/src/mmkurssit/Kurssi.java index 51754fd08ebf532962e5cfca1e24572b057d4a11..84937d7077b5b2a7e2c173b9e8d914474dcbbb34 100644 --- a/mmkurssit/src/mmkurssit/Kurssi.java +++ b/mmkurssit/src/mmkurssit/Kurssi.java @@ -6,6 +6,8 @@ import fi.jyu.mit.ohj2.Mjonot; import java.io.PrintStream; +import apumetodit.RandomNumero; + /** * - kopioi CRC-kortin vastuut tähän * @author aichinos @@ -79,9 +81,8 @@ public class Kurssi { * </pre> */ public void rekisteroi() { - //this.kurssiID = nextID; - //nextID++; this.setID(nextID); + // = { this.kurssiID = nextID; nextID++; } } @@ -118,7 +119,7 @@ public class Kurssi { * */ public void tayta() { - this.nimi = "Esimerkkikurssi"; //+ RandomNumero.rand(1,100); + this.nimi = "Esimerkkikurssi"+ RandomNumero.rand(1,100); //+ RandomNumero.rand(1,100); this.opeEtunimi = "Vesa"; this.opeSukunimi = "Kokko"; diff --git a/mmkurssit/src/mmkurssit/MMKurssit.java b/mmkurssit/src/mmkurssit/MMKurssit.java index 3d90140cb5b0a62011fc23b1a4d612a1617ce387..82b29acf6c2b290d5b4c37c7be5875d912d499f1 100644 --- a/mmkurssit/src/mmkurssit/MMKurssit.java +++ b/mmkurssit/src/mmkurssit/MMKurssit.java @@ -10,8 +10,8 @@ import java.util.List; */ public class MMKurssit { - private final Kurssit kurssit = new Kurssit(); - private final Tapaamiset tapaamiset = new Tapaamiset(); + private /*final*/ Kurssit kurssit = new Kurssit(); + private /*final*/ Tapaamiset tapaamiset = new Tapaamiset(); //================================================================== diff --git a/mmkurssit/src/mmkurssit/Tapaaminen.java b/mmkurssit/src/mmkurssit/Tapaaminen.java index 27470089d797a06e540d63944d441ddf08c0519c..daa54253122ca32c0b039b3e7fdd39792d77361c 100644 --- a/mmkurssit/src/mmkurssit/Tapaaminen.java +++ b/mmkurssit/src/mmkurssit/Tapaaminen.java @@ -1,6 +1,6 @@ package mmkurssit; -import java.io.OutputStream; +//import java.io.OutputStream; import java.io.PrintStream; import apumetodit.RandomNumero; @@ -25,13 +25,19 @@ public class Tapaaminen { private int kesto = 90;//oletuskena 90min kesto private String paikka = ""; private String muu = ""; - private int random;//testia varten! Ota pois loppuvaiheessa!!! + private int randomNimi;//testia varten! Ota pois loppuvaiheessa!!! private static int nextmID = 1;//next tapaaminen ID //========================================================================= + /** + * Test konstructor, poista lopussa!!! + */ + public Tapaaminen() { + this.kID = 1; + } /** Tapaamisen alustus * @param kurssiID kurssin ID, johon tapaaminen liittyy @@ -45,7 +51,7 @@ public class Tapaaminen { * */ public void tayta() { - this.random = RandomNumero.rand(1,100); + this.randomNimi = RandomNumero.rand(1,100); this.paiva = 16; this.kkausi = 3; this.vuosi = 2025; @@ -62,10 +68,10 @@ public class Tapaaminen { * Tulostetaan tapaamisen tiedot * @param os teitovirta johon tulostetaan */ - public void tulosta(OutputStream os) { +/* public void tulosta(OutputStream os) { //tulsota(new PrintStream(os)); } - + */ /** * Tulostetaan tapaamisen tiedot @@ -73,7 +79,7 @@ public class Tapaaminen { */ public void tulosta(PrintStream out) { out.println(String.format("Coruse ID: %3d", kID)); - out.println(String.format("Meeting ID: %03d", tapaaminenID) + String.format(" randomnro: %3d", random)); + out.println(String.format("Meeting ID: %03d", tapaaminenID) + String.format(" nimiksi: %3d", randomNimi)); out.println(milloin()); out.println("Aloitusaika: " + aloitusaika()); out.println("Loppuu: "+ lopetusaika()); @@ -169,17 +175,19 @@ public class Tapaaminen { * @param args ei käytössä */ public static void main(String[] args) { - - Tapaaminen japani4_1 = new Tapaaminen(1); - Tapaaminen ohjelmointi2_1 = new Tapaaminen(2); - japani4_1.rekisteroi(); - japani4_1.tayta(); + Kurssi japani3 = new Kurssi(); + Kurssi ohjelmointi2 = new Kurssi(); + Tapaaminen japani3_1 = new Tapaaminen(japani3.getID()); + Tapaaminen ohjelmointi2_1 = new Tapaaminen(ohjelmointi2.getID()); + + japani3_1.rekisteroi(); + japani3_1.tayta(); ohjelmointi2_1.rekisteroi(); ohjelmointi2_1.tayta(); - japani4_1.tulosta(System.out); + japani3_1.tulosta(System.out); ohjelmointi2_1.tulosta(System.out); } diff --git a/mmkurssit/src/mmkurssit/test/KurssiTest.java b/mmkurssit/src/mmkurssit/test/KurssiTest.java index 3fdea14d58977a39915aa80ff8e723438dddae95..1643332159ebd80a5e6d51973666edd722108371 100644 --- a/mmkurssit/src/mmkurssit/test/KurssiTest.java +++ b/mmkurssit/src/mmkurssit/test/KurssiTest.java @@ -7,7 +7,7 @@ import mmkurssit.*; /** * Test class made by ComTest - * @version 2025.03.12 10:15:28 // Generated by ComTest + * @version 2025.03.18 09:38:53 // Generated by ComTest * */ @SuppressWarnings("all") @@ -16,26 +16,26 @@ public class KurssiTest { // Generated by ComTest BEGIN - /** testRekisteroi97 */ + /** testRekisteroi70 */ @Test - public void testRekisteroi97() { // Kurssi: 97 + public void testRekisteroi70() { // Kurssi: 70 Kurssi saksa = new Kurssi(); - assertEquals("From: Kurssi line: 99", 0, saksa.getID()); + assertEquals("From: Kurssi line: 72", 0, saksa.getID()); saksa.rekisteroi(); Kurssi ohjelmointi = new Kurssi(); ohjelmointi.rekisteroi(); int n1 = saksa.getID(); int n2 = ohjelmointi.getID(); - assertEquals("From: Kurssi line: 105", n1+1, n2); + assertEquals("From: Kurssi line: 78", n1+1, n2); } // Generated by ComTest END // Generated by ComTest BEGIN - /** testToString156 */ + /** testToString154 */ @Test - public void testToString156() { // Kurssi: 156 + public void testToString154() { // Kurssi: 154 Kurssi hollanti = new Kurssi(); hollanti.parse("1|Esimerkkikurssi|Kokko |Vesa"); - assertEquals("From: Kurssi line: 159", "1|Esimerkkikurssi|Kokko Vesa", hollanti.toString()); + assertEquals("From: Kurssi line: 157", "1|Esimerkkikurssi|Kokko Vesa", hollanti.toString()); } // Generated by ComTest END } \ No newline at end of file diff --git a/mmkurssit/src/mmkurssit/test/KurssitTest.java b/mmkurssit/src/mmkurssit/test/KurssitTest.java index 21355f9f8c463057ad163e659fd5a03829ba77c1..b722f4760cebe8c98a22e8426d276cd0d8bb741c 100644 --- a/mmkurssit/src/mmkurssit/test/KurssitTest.java +++ b/mmkurssit/src/mmkurssit/test/KurssitTest.java @@ -7,7 +7,7 @@ import mmkurssit.*; /** * Test class made by ComTest - * @version 2025.03.10 14:44:08 // Generated by ComTest + * @version 2025.03.18 09:39:05 // Generated by ComTest * */ @SuppressWarnings("all") @@ -17,32 +17,32 @@ public class KurssitTest { // Generated by ComTest BEGIN /** - * testLisaa30 + * testLisaa38 * @throws SailoException when error */ @Test - public void testLisaa30() throws SailoException { // Kurssit: 30 + public void testLisaa38() throws SailoException { // Kurssit: 38 Kurssit kurssit = new Kurssit(); Kurssi test1 = new Kurssi(); Kurssi test2 = new Kurssi(); - assertEquals("From: Kurssit line: 36", 0, kurssit.getKpl()); - kurssit.lisaa(test1); assertEquals("From: Kurssit line: 37", 1, kurssit.getKpl()); - kurssit.lisaa(test2); assertEquals("From: Kurssit line: 38", 2, kurssit.getKpl()); - kurssit.lisaa(test1); assertEquals("From: Kurssit line: 39", 3, kurssit.getKpl()); - assertEquals("From: Kurssit line: 40", test1, kurssit.anna(0)); - assertEquals("From: Kurssit line: 41", test2, kurssit.anna(1)); - assertEquals("From: Kurssit line: 42", test1, kurssit.anna(2)); - assertEquals("From: Kurssit line: 44", false, kurssit.anna(0) == test2); - assertEquals("From: Kurssit line: 45", true, kurssit.anna(0) == test1); + assertEquals("From: Kurssit line: 44", 0, kurssit.getKpl()); + kurssit.lisaa(test1); assertEquals("From: Kurssit line: 45", 1, kurssit.getKpl()); + kurssit.lisaa(test2); assertEquals("From: Kurssit line: 46", 2, kurssit.getKpl()); + kurssit.lisaa(test1); assertEquals("From: Kurssit line: 47", 3, kurssit.getKpl()); + assertEquals("From: Kurssit line: 48", test1, kurssit.anna(0)); + assertEquals("From: Kurssit line: 49", test2, kurssit.anna(1)); + assertEquals("From: Kurssit line: 50", test1, kurssit.anna(2)); + assertEquals("From: Kurssit line: 52", false, kurssit.anna(0) == test2); + assertEquals("From: Kurssit line: 53", true, kurssit.anna(0) == test1); try { - assertEquals("From: Kurssit line: 46", test1, kurssit.anna(3)); - fail("Kurssit: 46 Did not throw IndexOutOfBoundsException"); + assertEquals("From: Kurssit line: 54", test1, kurssit.anna(3)); + fail("Kurssit: 54 Did not throw IndexOutOfBoundsException"); } catch(IndexOutOfBoundsException _e_){ _e_.getMessage(); } - kurssit.lisaa(test1); assertEquals("From: Kurssit line: 48", 4, kurssit.getKpl()); - kurssit.lisaa(test1); assertEquals("From: Kurssit line: 49", 5, kurssit.getKpl()); + kurssit.lisaa(test1); assertEquals("From: Kurssit line: 56", 4, kurssit.getKpl()); + kurssit.lisaa(test1); assertEquals("From: Kurssit line: 57", 5, kurssit.getKpl()); try { kurssit.lisaa(test1); - fail("Kurssit: 50 Did not throw SailoException"); + fail("Kurssit: 58 Did not throw SailoException"); } catch(SailoException _e_){ _e_.getMessage(); } } // Generated by ComTest END } \ No newline at end of file diff --git a/mmkurssit/src/mmkurssit/test/MMKurssitTest.java b/mmkurssit/src/mmkurssit/test/MMKurssitTest.java index e6fa37e64744a624fa5231903415e7d5626b1d27..ad17d70ba2bdd6f01f7d2d30f9116b35515c46de 100644 --- a/mmkurssit/src/mmkurssit/test/MMKurssitTest.java +++ b/mmkurssit/src/mmkurssit/test/MMKurssitTest.java @@ -8,13 +8,14 @@ import mmkurssit.*; /** * Test class made by ComTest - * @version 2025.03.11 15:21:20 // Generated by ComTest + * @version 2025.03.18 09:43:14 // Generated by ComTest * */ @SuppressWarnings("all") public class MMKurssitTest { + // Generated by ComTest BEGIN /** testAnnaKurssi50 */ @Test diff --git a/mmkurssit/src/mmkurssit/test/TapaaminenTest.java b/mmkurssit/src/mmkurssit/test/TapaaminenTest.java index 7bb37de49e0930539be78abcc3bd427502962b79..f4c4ed19de608d6773ce3b4ef325e42feb43dffe 100644 --- a/mmkurssit/src/mmkurssit/test/TapaaminenTest.java +++ b/mmkurssit/src/mmkurssit/test/TapaaminenTest.java @@ -7,7 +7,7 @@ import mmkurssit.*; /** * Test class made by ComTest - * @version 2025.03.10 14:02:45 // Generated by ComTest + * @version 2025.03.18 09:39:16 // Generated by ComTest * */ @SuppressWarnings("all") diff --git a/mmkurssit/src/mmkurssit/test/TapaamisetTest.java b/mmkurssit/src/mmkurssit/test/TapaamisetTest.java index 3385a4713717c395d0b2db804f879a9b531ea3dd..8625f2a89912d64365cfe116354069aa918a9472 100644 --- a/mmkurssit/src/mmkurssit/test/TapaamisetTest.java +++ b/mmkurssit/src/mmkurssit/test/TapaamisetTest.java @@ -8,7 +8,7 @@ import mmkurssit.*; /** * Test class made by ComTest - * @version 2025.03.11 15:16:05 // Generated by ComTest + * @version 2025.03.18 09:39:29 // Generated by ComTest * */ @SuppressWarnings("all")