kommentointia

Ohj2HT/src/Luokat/Reissu.java

@@ -175,7 +175,7 @@ private boolean poistetaanko;
     	this.keli = t[3];
         int tagi = Integer.parseInt(t[0]);
     	this.reissunTag = tagi;
-        SeuraavanReissunTag = tagi+1;
+        SeuraavanReissunTag++;
         this.poistetaanko = false;
     }
     

Ohj2HT/src/fxKalenteri/KalenteriGUIController.java

@@ -39,6 +39,7 @@ public class KalenteriGUIController implements Initializable {
     private int aluksi;
     private boolean onkoPoistettu = false;
     private ObservableList<StringAndObject<String>> lista;
+
     @FXML private void handleTallenna() {
         tallenna();
     }
@@ -95,10 +96,6 @@ public class KalenteriGUIController implements Initializable {
 
     /**
      * Päivitetään laskureiden tekstejä vastaamaan lukumääriä
-     * @example <pre name="test">
-     * Kalenteri k = new Kalenteri();
-     * k.
-     * </pre>
      */
     private void tekstienPaivitys() {
         Platform.runLater(()-> Saaliisi.setItems(FXCollections.observableArrayList(kalenteri.getKalaLaskuri().getArvotAsList())));
@@ -200,15 +197,20 @@ public class KalenteriGUIController implements Initializable {
     @Override
     public void initialize(URL url, ResourceBundle resourceBundle) {
         this.kalenteri = new Kalenteri();
-         List<Reissu> r = kalenteri.getReissut().getReissut();
-         aluksi = r.size();
+        //luodaan uusi kalenteri olio
+
+        List<Reissu> r = kalenteri.getReissut().getReissut();
+        aluksi = r.size();
+        //taas otetaan selville monta reissua oli ohjelman käynnistyessä
         reissutLista.getItems().clear();
         kirjoitaReissut();
-         this.reissuLkm.textProperty().bind(reissumaara.asString());
-         this.saalisLkm.textProperty().bind(saalismaara.asString());
-         this.lempiViehe.textProperty().bind(parasViehe);
-         Saaliisi.setItems(FXCollections.observableArrayList(kalenteri.getKalaLaskuri().getArvotAsList()));
-         tekstienPaivitys();
+        //kirjoitetaan reissut näkyville
+        this.reissuLkm.textProperty().bind(reissumaara.asString());
+        this.saalisLkm.textProperty().bind(saalismaara.asString());
+        this.lempiViehe.textProperty().bind(parasViehe);
+        Saaliisi.setItems(FXCollections.observableArrayList(kalenteri.getKalaLaskuri().getArvotAsList()));
+        tekstienPaivitys();
+        //Alustetaan kaikki laskurit ja niiden tekstit
     }
     
     

Ohj2HT/src/fxKaljoja/KalojaGUIController.java

@@ -39,6 +39,8 @@ public class KalojaGUIController implements Initializable, ModalControllerInterf
 		koko.setItems(FXCollections.observableArrayList("sintti", "keskikokoinen", "iso", "kalavalhe"));
 		TiedostonHiplailija lajeja = new TiedostonHiplailija(System.getProperty("user.dir") + "/../lajit.dat");
 	   TiedostonHiplailija tyypit = new TiedostonHiplailija(System.getProperty("user.dir") + "/../vieheTyypit.dat");
+	   //Asetetaan laatikoiden sisältö suoraan tiedostosta
+
 		System.out.println("Lukija luotu!");
 		List<String> ap = lajeja.lueRivit();
 		String[] apuri = new String[ap.size()];
@@ -50,15 +52,18 @@ public class KalojaGUIController implements Initializable, ModalControllerInterf
 		for (String s : tyypit.lueRivit()) {
 			tyyppilista.add(s.split("\\|")[1]);
 		}
+		//Tässä samalla kokeillaa DynamicComboBoxin sisällön asetusta listasta ja taulukosta
 		vieheet.asetaSisalto(tyyppilista);
 		lajeja.sulje();
 		tyypit.sulje();
+		//Suljetaan tiedoston käsittelijät, jotta vapautuu resurssit
 		vanhatLajit = lajit.getItems().size();
 		vanhatTyypit = vieheet.getItems().size();
+		//otetaan talteen tiedot laatikoiden sisällön määristä (tarvitaan myöhemmin)
    }
 
 	 @FXML private void handlePoista(){
-		 Poista();
+		 ModalController.closeStage(koko);
 	 }
 
 	@FXML private void handleTallenna() {
@@ -70,6 +75,7 @@ public class KalojaGUIController implements Initializable, ModalControllerInterf
 	 private void Tallenna() {
 	   ObservableList<String> uudet = lajit.getSisalto();
 	   int erotus = uudet.size() - vanhatLajit;
+	   //Tässä katsotaan luotiinko uusia lajeja valinta laatikkoon, jos luotiin niin tallennetaan uudet valinnat
 	   if (erotus > 0){
 		   TiedostonHiplailija t = new TiedostonHiplailija(System.getProperty("user.dir") + "/../lajit.dat",true);
 		   for (int i = vanhatLajit; i < uudet.size(); i++){
@@ -79,6 +85,7 @@ public class KalojaGUIController implements Initializable, ModalControllerInterf
 	   }
 	   String kokoString = koko.getSelectionModel().getSelectedItem();
 	   String lajiString = lajit.getSelectionModel().getSelectedItem();
+	   //Varmistutaan, että käyttäjä on kertonut ainakin saaliin lajin ja koon, jotta voidaan tallentaa
 	   if(kokoString != null && (lajiString != null && !lajiString.isEmpty())){
 		   Saalis s = new Saalis(kokoString, lajiString, vieheet.getSelectionModel().getSelectedItem(),vieheVarit.getText());
 		   saalis.add(s);
@@ -89,11 +96,6 @@ public class KalojaGUIController implements Initializable, ModalControllerInterf
 		 ModalController.closeStage(koko);
 	 }
 	 
-	
-
-	private void Poista() {
-		Dialogs.showMessageDialog("poistutaan! Mutta ei toimi vielä");
-	}
 
 	@Override
 	public List<Saalis> getResult() {

Ohj2HT/src/fxaloitusIkkuna/aloitusIkkunaGUIController.java

@@ -107,6 +107,7 @@ public class aloitusIkkunaGUIController implements ModalControllerInterface<Bool
 
     public void handleTakaisin() {
         suljetaanko.set(true);
+        //tämä boolean property valvoo suljetaanko koko sovellus aloitusnäytöstä
         ModalController.closeStage(pyorivaKala);
     }
 }

reissut.dat

 1|17.11.2025|Vuontee|aurinkoinen
-5|16.11.2025|Tikkakoski|sateinen
-2|11.05.2025|Laukaa|aurinkoinen
+2|16.11.2025|Tikkakoski|sateinen
+3|11.05.2025|Laukaa|aurinkoinen
 4|20.03.2025|Keljo|aurinkoinen
-4|19.03.2025|Rovaniemi|Aurinkoinen
+5|19.03.2025|Rovaniemi|Aurinkoinen
 6|18.03.2025|Muurame|Aurinkoinen
-3|12.03.2024|Laukaa|Pilvinen
+7|12.03.2024|Laukaa|Pilvinen

saaliit.dat

 5|1|3|1|3|sininen
 6|2|1|1|1|hopea
-7|5|1|5|1|kupari
 2|1|3|2|2|Fire tiger
 3|2|1|2|1|hopea
+4|4|1|3|1|vihreä
+5|6|3|4|1|kuparinen
 5|5|1|4|3|musta
-5|6|3|4|1|harmaa
-5|5|1|4|3|musta
-5|6|3|4|1|harmaa
+7|5|1|5|1|kupari
 8|3|2|6|1|hopeinen
-4|4|1|3|1|vihreä
-4|4|1|3|1|vihreä