diff --git a/Ohj2HT/src/Luokat/Kalenteri.java b/Ohj2HT/src/Luokat/Kalenteri.java index 34525d3a0710b8ade590a9bb12b3c98faef27a61..72d0aa1c9425868af05f3b96990f010e0a1a5729 100644 --- a/Ohj2HT/src/Luokat/Kalenteri.java +++ b/Ohj2HT/src/Luokat/Kalenteri.java @@ -18,6 +18,7 @@ public class Kalenteri { vieheTulkki = new Tulkki(dir + "/../vieheTyypit.dat"); kokoTulkki = new Tulkki(dir + "/../koot.dat"); this.reissut = new Reissut(lajit,kokoTulkki, vieheTulkki); + reissut.sort(); vieheLaskuri = new Laskuri(dir +"/../vieheLaskuri.dat",vieheTulkki); kalaLaskuri = new Laskuri(dir +"/../kalaLaskuri.dat",lajit); } diff --git a/Ohj2HT/src/Luokat/Reissut.java b/Ohj2HT/src/Luokat/Reissut.java index fd886063c4b8c53315c8ac6da57a8ac22e19a641..fcc0756f61c031756019f62a3f7aa5e3de541d8e 100644 --- a/Ohj2HT/src/Luokat/Reissut.java +++ b/Ohj2HT/src/Luokat/Reissut.java @@ -90,9 +90,7 @@ public class Reissut { } public void sort() { - reissut.sort(Comparator.comparing(Reissu::getPaivamaara).reversed()); - } diff --git a/Ohj2HT/src/fxKalenteri/KalenteriGUIController.java b/Ohj2HT/src/fxKalenteri/KalenteriGUIController.java index 0bd207c06500547c9cd99647b4f72e9c88e7f8c9..6979690f9224c531e6452ad82b5e899daf1e5400 100644 --- a/Ohj2HT/src/fxKalenteri/KalenteriGUIController.java +++ b/Ohj2HT/src/fxKalenteri/KalenteriGUIController.java @@ -63,28 +63,22 @@ public class KalenteriGUIController implements Initializable { private void LisaaReissu(){ ModalController.showModal(KalenteriMain.class.getResource("/fxLisaaReissu/LisaaReissuGUIView.fxml"), "Lisää reissu", null,kalenteri); - - - kirjoitaLisaaReissu(); + kirjoitaLisaaReissu(); tekstienPaivitys(); } /** - * kirjoittaa reissut reissut listaan. + * kirjoittaa reissut näkyville. */ - - public void kirjoitaLisaaReissu() { Reissut s = kalenteri.getReissut(); s.sort(); - if (s != null) { reissutLista.getItems().clear(); this.Saaliisi.getItems().clear(); for (Reissu reissu : s.getReissut()) { reissutLista.add(reissu.getPvmString() + " " + reissu.getSijainti(),reissu); - } } } @@ -110,33 +104,11 @@ public class KalenteriGUIController implements Initializable { private void tallenna() { Dialogs.showMessageDialog("Tallennetetaan! Mutta ei toimi vielä"); kirjoitaLisaaReissu(); - - - if ( reissumaara.get() - aluksi > 0){ - TiedostonHiplailija t = new TiedostonHiplailija(System.getProperty("user.dir") + "/../reissut.dat"); - TiedostonHiplailija sa = new TiedostonHiplailija(System.getProperty("user.dir")+ "/../saaliit.dat"); - System.out.println(System.getProperty("user.dir")); - Tulkki lajit = new Tulkki(System.getProperty("user.dir") + "/../lajit.dat"); - Tulkki vieheTulkki = new Tulkki(System.getProperty("user.dir") + "/../vieheTyypit.dat"); - Tulkki kokoTulkki = new Tulkki(System.getProperty("user.dir") + "/../koot.dat"); - - kalenteri.getReissut().sort(); - for (int i = aluksi; i < reissumaara.get(); i++){ - System.out.println(kalenteri.getReissut().getReissut().get(i)); - t.kirjoita(kalenteri.getReissut().getReissut().get(i).toString()); - - for (Saalis s : kalenteri.getReissut().getReissut().get(i).getSaaliit()) { - sa.kirjoita(s.tiedotTallenusMuodossa(lajit,vieheTulkki,kokoTulkki)); - - } - + kalenteri.getReissut().sort(); if ( reissumaara.get() - aluksi > 0){ if (kalenteri.tallenna(aluksi, reissumaara.get())) { - Dialogs.showMessageDialog("Tallennus onnistui!"); - aluksi = reissumaara.get(); - - } - } + Dialogs.showMessageDialog("Tallennus onnistui!"); + aluksi = reissumaara.get(); } } @@ -178,21 +150,18 @@ public class KalenteriGUIController implements Initializable { // Poistaminen - if (valittuReissu.getpoistetaanko()) { - + if (valittuReissu.getpoistetaanko()) { System.out.println("Ennen poistoa " + kalenteri.getReissut().getReissut().size()); System.out.println(valittuReissu.getpoistetaanko()); kalenteri.getReissut().removeReissu(); System.out.println("Jälkeen " + kalenteri.getReissut().getReissut().size()); List<Reissu> r = kalenteri.getReissut().getReissut(); - + kalenteri.getReissut().sort(); if (r != null) { reissutLista.getItems().clear(); this.Saaliisi.getItems().clear(); - for (Reissu reissu : r) { - reissutLista.add(reissu.getPaivamaara() + " " + reissu.getSijainti(),reissu); - } + kirjoitaLisaaReissu(); tekstienPaivitys(); } @@ -214,11 +183,7 @@ public class KalenteriGUIController implements Initializable { List<Reissu> r = kalenteri.getReissut().getReissut(); aluksi = r.size(); reissutLista.getItems().clear(); - if (r != null) { - for (Reissu reissu : r) { - reissutLista.add(reissu.getPaivamaara() +" " + reissu.getSijainti(),reissu); - } - } + kirjoitaLisaaReissu(); this.reissuLkm.textProperty().bind(reissumaara.asString()); this.saalisLkm.textProperty().bind(saalismaara.asString()); this.lempiViehe.textProperty().bind(parasViehe); diff --git a/Ohj2HT/src/fxLisaaReissu/LisaaReissuGUIController.java b/Ohj2HT/src/fxLisaaReissu/LisaaReissuGUIController.java index 5da608d07c78ff9234a3093496b491aa83d89714..e1efeb799733e6406e2671494b090a9668b88761 100644 --- a/Ohj2HT/src/fxLisaaReissu/LisaaReissuGUIController.java +++ b/Ohj2HT/src/fxLisaaReissu/LisaaReissuGUIController.java @@ -121,8 +121,7 @@ public class LisaaReissuGUIController implements Initializable, ModalControllerI @Override public Kalenteri getResult() { - - return new Kalenteri() ; + return kale; } @Override