From 64e66cfca9ec7147404b8e1ac0f33801fd971f92 Mon Sep 17 00:00:00 2001
From: Lahtinen Konsta Aleksi <konsta.a.lahtinen@student.jyu.fi>
Date: Thu, 20 Mar 2025 18:02:53 +0200
Subject: [PATCH] tallennusta paranneltu ja koodista tehty leuttavampaa
---
Ohj2HT/src/Luokat/Kalenteri.java | 1 +
Ohj2HT/src/Luokat/Reissut.java | 2 -
.../fxKalenteri/KalenteriGUIController.java | 53 ++++---------------
.../LisaaReissuGUIController.java | 3 +-
4 files changed, 11 insertions(+), 48 deletions(-)
diff --git a/Ohj2HT/src/Luokat/Kalenteri.java b/Ohj2HT/src/Luokat/Kalenteri.java
index 34525d3..72d0aa1 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 fd88606..fcc0756 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 0bd207c..6979690 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 5da608d..e1efeb7 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
--
GitLab