Update README.md

Signed-off-by: laurimaaninka <lauri.maaninka@gmail.com>

.gitignore

@@ -11,7 +11,8 @@ out/
 .idea/jarRepositories.xml
 .idea/compiler.xml
 .idea/libraries/
-.idea/dataSources
+.idea/dataSources.xml
+.idea/db-forest-config.xml
 *.iws
 *.iml
 *.ipr

.idea/inspectionProfiles/Project_Default.xml

@@ -2,7 +2,7 @@
   <profile version="1.0">
     <option name="myName" value="Project Default" />
     <inspection_tool class="ExtractMethodRecommender" enabled="true" level="WARNING" enabled_by_default="true">
-      <option name="minLength" value="951" />
+      <option name="minLength" value="735" />
     </inspection_tool>
     <inspection_tool class="GrazieInspection" enabled="false" level="GRAMMAR_ERROR" enabled_by_default="false" />
     <inspection_tool class="LanguageDetectionInspection" enabled="false" level="WARNING" enabled_by_default="false" />
@@ -12,10 +12,5 @@
       <option name="processLiterals" value="true" />
       <option name="processComments" value="true" />
     </inspection_tool>
-    <inspection_tool class="UnusedAssignment" enabled="false" level="WARNING" enabled_by_default="false">
-      <option name="REPORT_PREFIX_EXPRESSIONS" value="false" />
-      <option name="REPORT_POSTFIX_EXPRESSIONS" value="true" />
-      <option name="REPORT_REDUNDANT_INITIALIZER" value="true" />
-    </inspection_tool>
   </profile>
 </component>

.idea/runConfigurations/fi_lpam_Main.xml

@@ -1,5 +1,5 @@
 <component name="ProjectRunConfigurationManager">
-  <configuration default="false" name="Main" type="Application" factoryName="Application" nameIsGenerated="true">
+  <configuration default="false" name="fi.lpam.Main" type="Application" factoryName="Application" nameIsGenerated="true">
     <option name="MAIN_CLASS_NAME" value="fi.lpam.Main" />
     <module name="Ruokamanageri" />
     <option name="PROGRAM_PARAMETERS" value="DEV" />

.idea/sqldialects.xml

@@ -0,0 +1,6 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<project version="4">
+  <component name="SqlDialectMappings">
+    <file url="PROJECT" dialect="SQLite" />
+  </component>
+</project>

CHANGELOG.md

@@ -0,0 +1,7 @@
+# 1.1.0
+- Kuljetuslistaa luodessa kuljetusrivin nollaaminen/palauttaminen oletusmääriin.
+- Kuljetuslistojen ja raporttien tulostamismekanismin läpikäynti
+- Kierroshallinnassa asiakasta raahattaessa placeholderin fiksaus
+
+# 1.0.0
+Ohjelman kieli vaihdettu Pythonista Javaan, sekä tietokanta pickle-tiedostosta SQLite tietokantaan

LICENSE

@@ -1,202 +0,0 @@
-                                 Apache License
-                           Version 2.0, January 2004
-                        http://www.apache.org/licenses/
-
-   TERMS AND CONDITIONS FOR USE, REPRODUCTION, AND DISTRIBUTION
-
-   1. Definitions.
-
-      "License" shall mean the terms and conditions for use, reproduction,
-      and distribution as defined by Sections 1 through 9 of this document.
-
-      "Licensor" shall mean the copyright owner or entity authorized by
-      the copyright owner that is granting the License.
-
-      "Legal Entity" shall mean the union of the acting entity and all
-      other entities that control, are controlled by, or are under common
-      control with that entity. For the purposes of this definition,
-      "control" means (i) the power, direct or indirect, to cause the
-      direction or management of such entity, whether by contract or
-      otherwise, or (ii) ownership of fifty percent (50%) or more of the
-      outstanding shares, or (iii) beneficial ownership of such entity.
-
-      "You" (or "Your") shall mean an individual or Legal Entity
-      exercising permissions granted by this License.
-
-      "Source" form shall mean the preferred form for making modifications,
-      including but not limited to software source code, documentation
-      source, and configuration files.
-
-      "Object" form shall mean any form resulting from mechanical
-      transformation or translation of a Source form, including but
-      not limited to compiled object code, generated documentation,
-      and conversions to other media types.
-
-      "Work" shall mean the work of authorship, whether in Source or
-      Object form, made available under the License, as indicated by a
-      copyright notice that is included in or attached to the work
-      (an example is provided in the Appendix below).
-
-      "Derivative Works" shall mean any work, whether in Source or Object
-      form, that is based on (or derived from) the Work and for which the
-      editorial revisions, annotations, elaborations, or other modifications
-      represent, as a whole, an original work of authorship. For the purposes
-      of this License, Derivative Works shall not include works that remain
-      separable from, or merely link (or bind by name) to the interfaces of,
-      the Work and Derivative Works thereof.
-
-      "Contribution" shall mean any work of authorship, including
-      the original version of the Work and any modifications or additions
-      to that Work or Derivative Works thereof, that is intentionally
-      submitted to Licensor for inclusion in the Work by the copyright owner
-      or by an individual or Legal Entity authorized to submit on behalf of
-      the copyright owner. For the purposes of this definition, "submitted"
-      means any form of electronic, verbal, or written communication sent
-      to the Licensor or its representatives, including but not limited to
-      communication on electronic mailing lists, source code control systems,
-      and issue tracking systems that are managed by, or on behalf of, the
-      Licensor for the purpose of discussing and improving the Work, but
-      excluding communication that is conspicuously marked or otherwise
-      designated in writing by the copyright owner as "Not a Contribution."
-
-      "Contributor" shall mean Licensor and any individual or Legal Entity
-      on behalf of whom a Contribution has been received by Licensor and
-      subsequently incorporated within the Work.
-
-   2. Grant of Copyright License. Subject to the terms and conditions of
-      this License, each Contributor hereby grants to You a perpetual,
-      worldwide, non-exclusive, no-charge, royalty-free, irrevocable
-      copyright license to reproduce, prepare Derivative Works of,
-      publicly display, publicly perform, sublicense, and distribute the
-      Work and such Derivative Works in Source or Object form.
-
-   3. Grant of Patent License. Subject to the terms and conditions of
-      this License, each Contributor hereby grants to You a perpetual,
-      worldwide, non-exclusive, no-charge, royalty-free, irrevocable
-      (except as stated in this section) patent license to make, have made,
-      use, offer to sell, sell, import, and otherwise transfer the Work,
-      where such license applies only to those patent claims licensable
-      by such Contributor that are necessarily infringed by their
-      Contribution(s) alone or by combination of their Contribution(s)
-      with the Work to which such Contribution(s) was submitted. If You
-      institute patent litigation against any entity (including a
-      cross-claim or counterclaim in a lawsuit) alleging that the Work
-      or a Contribution incorporated within the Work constitutes direct
-      or contributory patent infringement, then any patent licenses
-      granted to You under this License for that Work shall terminate
-      as of the date such litigation is filed.
-
-   4. Redistribution. You may reproduce and distribute copies of the
-      Work or Derivative Works thereof in any medium, with or without
-      modifications, and in Source or Object form, provided that You
-      meet the following conditions:
-
-      (a) You must give any other recipients of the Work or
-          Derivative Works a copy of this License; and
-
-      (b) You must cause any modified files to carry prominent notices
-          stating that You changed the files; and
-
-      (c) You must retain, in the Source form of any Derivative Works
-          that You distribute, all copyright, patent, trademark, and
-          attribution notices from the Source form of the Work,
-          excluding those notices that do not pertain to any part of
-          the Derivative Works; and
-
-      (d) If the Work includes a "NOTICE" text file as part of its
-          distribution, then any Derivative Works that You distribute must
-          include a readable copy of the attribution notices contained
-          within such NOTICE file, excluding those notices that do not
-          pertain to any part of the Derivative Works, in at least one
-          of the following places: within a NOTICE text file distributed
-          as part of the Derivative Works; within the Source form or
-          documentation, if provided along with the Derivative Works; or,
-          within a display generated by the Derivative Works, if and
-          wherever such third-party notices normally appear. The contents
-          of the NOTICE file are for informational purposes only and
-          do not modify the License. You may add Your own attribution
-          notices within Derivative Works that You distribute, alongside
-          or as an addendum to the NOTICE text from the Work, provided
-          that such additional attribution notices cannot be construed
-          as modifying the License.
-
-      You may add Your own copyright statement to Your modifications and
-      may provide additional or different license terms and conditions
-      for use, reproduction, or distribution of Your modifications, or
-      for any such Derivative Works as a whole, provided Your use,
-      reproduction, and distribution of the Work otherwise complies with
-      the conditions stated in this License.
-
-   5. Submission of Contributions. Unless You explicitly state otherwise,
-      any Contribution intentionally submitted for inclusion in the Work
-      by You to the Licensor shall be under the terms and conditions of
-      this License, without any additional terms or conditions.
-      Notwithstanding the above, nothing herein shall supersede or modify
-      the terms of any separate license agreement you may have executed
-      with Licensor regarding such Contributions.
-
-   6. Trademarks. This License does not grant permission to use the trade
-      names, trademarks, service marks, or product names of the Licensor,
-      except as required for reasonable and customary use in describing the
-      origin of the Work and reproducing the content of the NOTICE file.
-
-   7. Disclaimer of Warranty. Unless required by applicable law or
-      agreed to in writing, Licensor provides the Work (and each
-      Contributor provides its Contributions) on an "AS IS" BASIS,
-      WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or
-      implied, including, without limitation, any warranties or conditions
-      of TITLE, NON-INFRINGEMENT, MERCHANTABILITY, or FITNESS FOR A
-      PARTICULAR PURPOSE. You are solely responsible for determining the
-      appropriateness of using or redistributing the Work and assume any
-      risks associated with Your exercise of permissions under this License.
-
-   8. Limitation of Liability. In no event and under no legal theory,
-      whether in tort (including negligence), contract, or otherwise,
-      unless required by applicable law (such as deliberate and grossly
-      negligent acts) or agreed to in writing, shall any Contributor be
-      liable to You for damages, including any direct, indirect, special,
-      incidental, or consequential damages of any character arising as a
-      result of this License or out of the use or inability to use the
-      Work (including but not limited to damages for loss of goodwill,
-      work stoppage, computer failure or malfunction, or any and all
-      other commercial damages or losses), even if such Contributor
-      has been advised of the possibility of such damages.
-
-   9. Accepting Warranty or Additional Liability. While redistributing
-      the Work or Derivative Works thereof, You may choose to offer,
-      and charge a fee for, acceptance of support, warranty, indemnity,
-      or other liability obligations and/or rights consistent with this
-      License. However, in accepting such obligations, You may act only
-      on Your own behalf and on Your sole responsibility, not on behalf
-      of any other Contributor, and only if You agree to indemnify,
-      defend, and hold each Contributor harmless for any liability
-      incurred by, or claims asserted against, such Contributor by reason
-      of your accepting any such warranty or additional liability.
-
-   END OF TERMS AND CONDITIONS
-
-   APPENDIX: How to apply the Apache License to your work.
-
-      To apply the Apache License to your work, attach the following
-      boilerplate notice, with the fields enclosed by brackets "[]"
-      replaced with your own identifying information. (Don't include
-      the brackets!)  The text should be enclosed in the appropriate
-      comment syntax for the file format. We also recommend that a
-      file or class name and description of purpose be included on the
-      same "printed page" as the copyright notice for easier
-      identification within third-party archives.
-
-   Copyright 2026 Lauri Maaninka
-
-   Licensed under the Apache License, Version 2.0 (the "License");
-   you may not use this file except in compliance with the License.
-   You may obtain a copy of the License at
-
-       http://www.apache.org/licenses/LICENSE-2.0
-
-   Unless required by applicable law or agreed to in writing, software
-   distributed under the License is distributed on an "AS IS" BASIS,
-   WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-   See the License for the specific language governing permissions and
-   limitations under the License.
-   

README.md

@@ -1,6 +1,4 @@
-# [LICENSE](LICENSE)
+## [Versiohistoria](CHANGELOG.md)
-
-# [Versiohistoria](src/main/resources/CHANGELOG.md)
 
 # Kuvaus 
 Pienen kotiateriapalvelun käyttöön rakennettu ohjelma sisältäen seuraavat ominaisuudet
@@ -9,19 +7,19 @@ Pienen kotiateriapalvelun käyttöön rakennettu ohjelma sisältäen seuraavat o
 - Kuljetuslistojen luominen
 - Kuljetettujen määrien raportointi
 
-# Toteutus
+## Toteutus
 - SQLite tietokanta
 - JavaFX GUI
 - Java 25
 - Maven
 
-# Käyttöönotto
+## Käyttöönotto
 - Vaatimukset
-  - Windows (Linux melkein täydellisellä yhteensopivuudella)
+  - Windows
   - Java 25+
-  - Usean tietokoneen käyttöön OneDrive -tili (samanaikaisuus ei tuettu)
+  - Usean tietokoneen käyttöön OneDrive -tili
-- Lataa haluamasi ohjelmaversio julkaisuista
+- Lataa haluamasi julkaisuversio lähdekoodista
-- Aja .JAR tiedosto
+- Maven `package` komennolla projektista luodaan .jar tiedosto
-- Tietokanta sijaitsee osoitteessa `~/OneDrive/Kuljetusruokasovellus/tietokanta.db`
+- .jar tiedosto sisältää kaikki riippuvuudet
-- Asetukset sijaitsee osoitteessa `~/OneDrive/Kuljetusruokasovellus/asetukset.properties`
+- Tietokantatiedosto asuu osoitteessa `$KOTIKANSIO/OneDrive/Kuljetusruokasovellus/tietokanta.db`
 - Ohjelma luo tietokannan, mikäli sitä ei löydy.

pom.xml

@@ -1,9 +1,9 @@
 <project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
   xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 https://maven.apache.org/maven-v4_0_0.xsd">
     <modelVersion>4.0.0</modelVersion>
-    <groupId>fi.lpam.ruokamanageri</groupId>
+    <groupId>fi.lpam</groupId>
     <artifactId>Ruokamanageri</artifactId>
-    <version>3.2.1</version>
+    <version>1.0</version>
     <properties>
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
         <maven.compiler.source>25</maven.compiler.source>
@@ -43,41 +43,29 @@
         <dependency>
             <groupId>com.dlsc.gemsfx</groupId>
             <artifactId>gemsfx</artifactId>
-            <version>4.0.5</version>
+            <version>3.10.1</version>
             <scope>compile</scope>
         </dependency>
         <dependency>
             <groupId>org.apache.pdfbox</groupId>
             <artifactId>pdfbox</artifactId>
-            <version>3.0.7</version>
+            <version>3.0.6</version>
             <scope>compile</scope>
         </dependency>
         <dependency>
             <groupId>commons-logging</groupId>
             <artifactId>commons-logging</artifactId>
-            <version>1.3.6</version>
+            <version>1.3.5</version>
         </dependency>
         <dependency>
             <groupId>org.xerial</groupId>
             <artifactId>sqlite-jdbc</artifactId>
-            <version>3.53.1.0</version>
+            <version>3.51.2.0</version>
             <scope>compile</scope>
         </dependency>
-        <dependency>
-            <groupId>one.jpro.platform</groupId>
-            <artifactId>jpro-mdfx</artifactId>
-            <version>0.6.2</version>
-        </dependency>
     </dependencies>
 
     <build>
-        <resources>
-            <resource>
-                <directory>src/main/resources</directory>
-                <filtering>true</filtering>
-            </resource>
-        </resources>
-
         <plugins>
             <plugin>
                 <groupId>org.apache.maven.plugins</groupId>

src/main/java/fi/lpam/App.java

@@ -1,56 +1,61 @@
 package fi.lpam;
 import fi.lpam.dataluokat.Tietokanta;
-import fi.lpam.gui.*;
+import fi.lpam.gui.Asiakashallinta;
-import fi.lpam.gui.elementit.MuutosIlmoitus;
+import fi.lpam.gui.Kierroshallinta;
-import fi.lpam.gui.elementit.TabPohja;
+import fi.lpam.gui.KuljetusRaportit;
+import fi.lpam.gui.KuljetusListat;
 import javafx.application.Application;
 import javafx.geometry.Side;
 import javafx.scene.Scene;
+import javafx.scene.control.Tab;
 import javafx.scene.control.TabPane;
 import javafx.scene.image.Image;
 import javafx.stage.Stage;
 
+import java.io.IOException;
 import java.util.Objects;
 
 @SuppressWarnings("InstantiationOfUtilityClass")
 public class App extends Application {
-    TabPohja asiakasHallinta, kierrostenHallinta, kuljetusListat, kuljetetut, asetukset;
-
     static void main() {
         launch();
     }
 
     @Override
-    public void start(Stage primaryStage) {
+    public void start(Stage primaryStage) throws IOException {
         new Tietokanta(Main.dev);
+        TabPane root = new TabPane();
+        root.setSide(Side.LEFT);
+        root.setTabMinWidth(50);
+        root.setTabMinHeight(35);
+        root.setTabMaxHeight(50);
 
-        StringBuilder title = new StringBuilder();
+        Tab asiakasHallinta, kierrostenHallinta, kuljetusListat, kuljetetut;
-        title.append("Ruokamanageri");
+        asiakasHallinta = new Tab("Asiakashallinta", new Asiakashallinta());
-        title.append(" v").append(Main.properties.getProperty("version"));
+        kierrostenHallinta = new Tab("Kierrostenhallinta");
-        if (Main.dev) title.append(" KEHITYS");
+        kuljetusListat  = new Tab("Kuljetuslistat");
-        primaryStage.setTitle(title.toString());
+        kuljetetut = new Tab("Kuljetetut");
+        root.getTabs().addAll(asiakasHallinta, kierrostenHallinta, kuljetusListat, kuljetetut);
 
-        primaryStage.setMaximized(true);
+        root.getSelectionModel().selectedItemProperty().addListener((observable, oldValue, newValue) -> {
+            switch (newValue.getText()) {
+                case "Asiakashallinta": newValue.setContent(new Asiakashallinta());     break;
+                case "Kierrostenhallinta": newValue.setContent(new Kierroshallinta());  break;
+                case "Kuljetuslistat": newValue.setContent(new KuljetusListat());       break;
+                case "Kuljetetut": newValue.setContent(new KuljetusRaportit());         break;
+            }
+        });
 
-        primaryStage.getIcons().add(new Image(Objects.requireNonNull(Main.class.getClassLoader().getResourceAsStream("kuljetusruokalaatikko.jpg"))));
+        for (Tab tab : root.getTabs()) {
+            tab.setClosable(false);
+            tab.setStyle("-fx-font-size: 18px; -fx-font-weight: bold; -fx-font-family: Open Sans");
+        }
 
-        TabPane tabPane = new TabPane();
+        Scene scene = new Scene(root);
-        tabPane.setSide(Side.LEFT);
-        tabPane.setTabMinWidth(50);
-        tabPane.setTabMinHeight(Double.parseDouble(Main.properties.getProperty("buttonFonttiKoko"))*2);
-        tabPane.setTabMaxHeight(Double.parseDouble(Main.properties.getProperty("buttonFonttiKoko"))*2);
-
-        asiakasHallinta = new Asiakashallinta();
-        kierrostenHallinta = new Kierroshallinta();
-        kuljetusListat  = new KuljetusListat();
-        kuljetetut = new KuljetusRaportit();
-        asetukset = new Asetukset();
-        tabPane.getTabs().addAll(asiakasHallinta, kierrostenHallinta, kuljetusListat, kuljetetut, asetukset);
-
-        Scene scene = new Scene(tabPane);
         primaryStage.setScene(scene);
+        primaryStage.setTitle("Ruokamanageri");
+        primaryStage.setMaximized(true);
         primaryStage.show();
-
+        primaryStage.getIcons().add(new Image(Objects.requireNonNull(Main.class.getResourceAsStream("kuljetusruokalaatikko.jpg"))));
-        if (!Main.properties.getProperty("version").equals(Main.properties.getProperty("viimeisinVersio", "0.0.0"))) new MuutosIlmoitus();
     }
 }

src/main/java/fi/lpam/Main.java

@@ -1,16 +1,7 @@
 package fi.lpam;
 
-import org.apache.commons.lang3.SystemUtils;
-
-import java.io.*;
-import java.nio.file.Path;
-import java.util.Properties;
-
 public class Main {
     public static boolean dev;
-    public static Properties properties;
-
-    public static final File asetustiedosto = Path.of(SystemUtils.getUserHome().getAbsolutePath(), "OneDrive", "Kuljetusruokasovellus", "asetukset.properties").toFile();
 
     static void main(String[] args) {
         //noinspection RedundantIfStatement
@@ -19,25 +10,6 @@ public class Main {
         } else {
             dev = false;
         }
-
-        try {
-            Properties defaults = new Properties();
-            defaults.load(Main.class.getClassLoader().getResourceAsStream("defaults.properties"));
-            properties = new Properties(defaults);
-            properties.load(new FileInputStream(asetustiedosto));
-        }
-        catch (IOException e) {
-            //noinspection CallToPrintStackTrace
-            e.printStackTrace();
-        }
-
         App.main();
-
-        try (OutputStream os = new FileOutputStream(asetustiedosto)) {
-            properties.store(os, "");
-        } catch (IOException e) {
-            //noinspection CallToPrintStackTrace
-            e.printStackTrace();
-        }
     }
 }

src/main/java/fi/lpam/dataluokat/Asiakas.java

@@ -3,24 +3,18 @@ package fi.lpam.dataluokat;
 import fi.lpam.gui.virheDialogit.SiirtoVirhe;
 
 import java.sql.*;
-import java.time.DayOfWeek;
-import java.time.LocalDate;
 import java.util.ArrayList;
-import java.util.HashMap;
 
 @SuppressWarnings("DuplicatedCode")
 public class Asiakas implements TietokantaOlio {
     private int id = -1;
     private String nimi;
     private String osoite;
-    private int maanantaiKpl, tiistaiKpl, keskiviikkoKpl, torstaiKpl, perjantaiKpl, lauantaiKpl, sunnuntaiKpl;
+    private int maanantaiKpl, tiistaiKpl, keskiviikkoKpl, torstaiKpl, perjantaiKpl;
     private int salaatit, pääruoat, jälkiruoat;
-    private int dinnerMaanantaiKpl, dinnerTiistaiKpl, dinnerKeskiviikkoKpl, dinnerTorstaiKpl, dinnerPerjantaiKpl, dinnerLauantaiKpl, dinnerSunnuntaiKpl;
-    private int dinnerSalaatit, dinnerPääruoat, dinnerJälkiruoat;
     private String erityisruokavalio;
     private String avainTieto;
     private String lisätiedot;
-    private String laskutusOsoite, yhteyshenkilönNimi, yhteyshenkilönPuhelinnumero, yhteyshenkilönSähköposti;
 
     /**
      * Tyhjä konstruktori uusien luomista varten
@@ -38,37 +32,22 @@ public class Asiakas implements TietokantaOlio {
 
     /**
      * Kun luetaan asiakas tietokannasta
+     * @param id asiakkaan id
+     * @param nimi asiakkaan nimi
+     * @param osoite osoite
+     * @param maanantaiKpl kuljetettava määrä
+     * @param tiistaiKpl kuljetettava määrä
+     * @param keskiviikkoKpl kuljetettava määrä
+     * @param torstaiKpl kuljetettava määrä
+     * @param perjantaiKpl kuljetettava määrä
+     * @param salaatit kuljetettava määrä
+     * @param pääruoat kuljetettava määrä
+     * @param jälkiruoat kuljetettava määrä
+     * @param erityisruokavalio ...
+     * @param avainTieto ...
+     * @param lisätiedot mahdolliset lisätilaukset/tiedot
      */
-    private Asiakas(int id,
+    private Asiakas(int id, String nimi, String osoite, int maanantaiKpl, int tiistaiKpl, int keskiviikkoKpl, int torstaiKpl, int perjantaiKpl, int salaatit, int pääruoat, int jälkiruoat, String erityisruokavalio, String avainTieto, String lisätiedot) {
-                    String nimi,
-                    String osoite,
-                    int maanantaiKpl,
-                    int tiistaiKpl,
-                    int keskiviikkoKpl,
-                    int torstaiKpl,
-                    int perjantaiKpl,
-                    int salaatit,
-                    int pääruoat,
-                    int jälkiruoat,
-                    String erityisruokavalio,
-                    String avainTieto,
-                    String lisätiedot,
-                    String laskutusOsoite,
-                    String yhteyshenkilönNimi,
-                    String yhteyshenkilönPuhelinnumero,
-                    String yhteyshenkilönSähköposti,
-                    int lauantaiKpl,
-                    int sunnuntaiKpl,
-                    int dinnerMaanantaiKpl,
-                    int dinnerTiistaiKpl,
-                    int dinnerKeskiviikkoKpl,
-                    int dinnerTorstaiKpl,
-                    int dinnerPerjantaiKpl,
-                    int dinnerLauantaiKpl,
-                    int dinnerSunnuntaiKpl,
-                    int dinnerSalaatit,
-                    int dinnerPääruoat,
-                    int dinnerJälkiruoat) {
         this.id = id;
         this.nimi = nimi;
         this.osoite = osoite;
@@ -77,28 +56,43 @@ public class Asiakas implements TietokantaOlio {
         this.keskiviikkoKpl = keskiviikkoKpl;
         this.torstaiKpl = torstaiKpl;
         this.perjantaiKpl = perjantaiKpl;
-        this.lauantaiKpl = lauantaiKpl;
-        this.sunnuntaiKpl = sunnuntaiKpl;
         this.salaatit = salaatit;
         this.pääruoat = pääruoat;
         this.jälkiruoat = jälkiruoat;
         this.erityisruokavalio = erityisruokavalio;
         this.avainTieto = avainTieto;
         this.lisätiedot = lisätiedot;
-        this.laskutusOsoite = laskutusOsoite;
+    }
-        this.yhteyshenkilönNimi = yhteyshenkilönNimi;
+
-        this.yhteyshenkilönPuhelinnumero = yhteyshenkilönPuhelinnumero;
+    /**
-        this.yhteyshenkilönSähköposti = yhteyshenkilönSähköposti;
+     * @param nimi asiakkaan nimi
-        this.dinnerMaanantaiKpl = dinnerMaanantaiKpl;
+     * @param osoite osoite
-        this.dinnerTiistaiKpl = dinnerTiistaiKpl;
+     * @param maanantaiKpl kuljetettava määrä
-        this.dinnerKeskiviikkoKpl = dinnerKeskiviikkoKpl;
+     * @param tiistaiKpl kuljetettava määrä
-        this.dinnerTorstaiKpl = dinnerTorstaiKpl;
+     * @param keskiviikkoKpl kuljetettava määrä
-        this.dinnerPerjantaiKpl = dinnerPerjantaiKpl;
+     * @param torstaiKpl kuljetettava määrä
-        this.dinnerLauantaiKpl = dinnerLauantaiKpl;
+     * @param perjantaiKpl kuljetettava määrä
-        this.dinnerSunnuntaiKpl = dinnerSunnuntaiKpl;
+     * @param salaatit kuljetettava määrä
-        this.dinnerSalaatit = dinnerSalaatit;
+     * @param pääruoat kuljetettava määrä
-        this.dinnerPääruoat = dinnerPääruoat;
+     * @param jälkiruoat kuljetettava määrä
-        this.dinnerJälkiruoat = dinnerJälkiruoat;
+     * @param erityisruokavalio ...
+     * @param avainTieto ...
+     * @param lisätiedot mahdolliset lisätilaukset/tiedot
+     */
+    public Asiakas(String nimi, String osoite, int maanantaiKpl, int tiistaiKpl, int keskiviikkoKpl, int torstaiKpl, int perjantaiKpl, int salaatit, int pääruoat, int jälkiruoat, String erityisruokavalio, String avainTieto, String lisätiedot) {
+        this.nimi = nimi;
+        this.osoite = osoite;
+        this.maanantaiKpl = maanantaiKpl;
+        this.tiistaiKpl = tiistaiKpl;
+        this.keskiviikkoKpl = keskiviikkoKpl;
+        this.torstaiKpl = torstaiKpl;
+        this.perjantaiKpl = perjantaiKpl;
+        this.salaatit = salaatit;
+        this.pääruoat = pääruoat;
+        this.jälkiruoat = jälkiruoat;
+        this.erityisruokavalio = erityisruokavalio;
+        this.avainTieto = avainTieto;
+        this.lisätiedot = lisätiedot;
     }
 
     /**
@@ -118,7 +112,7 @@ public class Asiakas implements TietokantaOlio {
     }
 
     /**
-     * Hakee tietokannasta asiakasidn perusteella asiakkaat ja palauttaa ne annetussa järjestyksessä
+     * Hakee tietokannasta asiakasidn perusteella asiakkaat
      * @param IDt haettavat asiakkkaat
      * @return Haetut asiakkaat tai null
      */
@@ -129,17 +123,10 @@ public class Asiakas implements TietokantaOlio {
             sbIDt.append(", ").append(IDt[i]);
         }
         try (Connection tietokanta = Tietokanta.haeYhteys()) {
-            String sql = "select * from asiakkaat where id in (" + sbIDt + ")";
+            String sql = "select * from asiakkaat where id in (" + sbIDt + ") order by instr('" + sbIDt + "', id)";
             Statement stmt = tietokanta.createStatement();
             ResultSet rs = stmt.executeQuery(sql);
-
+            return parsiResultSet(rs);
-            HashMap<Integer, Asiakas> asiakkaat = new HashMap<>(IDt.length);
-            parsiResultSet(rs).forEach(asiakas -> asiakkaat.put(asiakas.getId(), asiakas));
-            ArrayList<Asiakas> järjestetyt = new ArrayList<>(asiakkaat.size());
-            for (int id : IDt) {
-                if (asiakkaat.containsKey(id)) järjestetyt.add(asiakkaat.get(id)); //Huomioidaan, onko asiakas poistettu
-            }
-            return järjestetyt;
         }
         catch (SQLException e) {
             SiirtoVirhe alert = new SiirtoVirhe(e);
@@ -158,38 +145,21 @@ public class Asiakas implements TietokantaOlio {
 
         try {
             while (rs.next()) {
-                int i = 1;
                 Asiakas uusi = new Asiakas(
-                        rs.getInt(i++),
+                        rs.getInt("id"),
-                        rs.getString(i++),
+                        rs.getString("nimi"),
-                        rs.getString(i++),
+                        rs.getString("osoite"),
-                        rs.getInt(i++),
+                        rs.getInt("maanantaiKpl"),
-                        rs.getInt(i++),
+                        rs.getInt("tiistaiKpl"),
-                        rs.getInt(i++),
+                        rs.getInt("keskiviikkoKpl"),
-                        rs.getInt(i++),
+                        rs.getInt("torstaiKpl"),
-                        rs.getInt(i++),
+                        rs.getInt("perjantaiKpl"),
-                        rs.getInt(i++),
+                        rs.getInt("salaatit"),
-                        rs.getInt(i++),
+                        rs.getInt("pääruoat"),
-                        rs.getInt(i++),
+                        rs.getInt("jälkiruoat"),
-                        rs.getString(i++),
+                        rs.getString("erityisruokavalio"),
-                        rs.getString(i++),
+                        rs.getString("avainTieto"),
-                        rs.getString(i++),
+                        rs.getString("lisätiedot")
-                        rs.getString(i++),
-                        rs.getString(i++),
-                        rs.getString(i++),
-                        rs.getString(i++),
-                        rs.getInt(i++),
-                        rs.getInt(i++),
-                        rs.getInt(i++),
-                        rs.getInt(i++),
-                        rs.getInt(i++),
-                        rs.getInt(i++),
-                        rs.getInt(i++),
-                        rs.getInt(i++),
-                        rs.getInt(i++),
-                        rs.getInt(i++),
-                        rs.getInt(i++),
-                        rs.getInt(i++)
                 );
                 asiakkaat.add(uusi);
             }
@@ -208,42 +178,20 @@ public class Asiakas implements TietokantaOlio {
     @Override
     public boolean tallennaUusi() {
         try (Connection tietokanta = Tietokanta.haeYhteys()) {
-            PreparedStatement stmt = tietokanta.prepareStatement("INSERT INTO asiakkaat (nimi, osoite, " +
+            PreparedStatement stmt = tietokanta.prepareStatement("INSERT INTO asiakkaat (nimi, osoite, maanantaiKpl, tiistaiKpl, keskiviikkoKpl, torstaiKpl, perjantaiKpl, salaatit, pääruoat, jälkiruoat, erityisruokavalio, avainTieto, lisätiedot) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)", Statement.RETURN_GENERATED_KEYS);
-                    "maanantaiKpl, tiistaiKpl, keskiviikkoKpl, torstaiKpl, perjantaiKpl, lauantaiKpl, sunnuntaiKpl, salaatit, pääruoat, jälkiruoat, " +
+            stmt.setString(1, nimi);
-                    "dinner_maanantaiKpl, dinner_tiistaiKpl, dinner_keskiviikkoKpl, dinner_torstaiKpl, dinner_perjantaiKpl, dinner_lauantaiKpl, dinner_sunnuntaiKpl, dinner_salaatit, dinner_pääruoat, dinner_jälkiruoat, " +
+            stmt.setString(2, osoite);
-                    "erityisruokavalio, avainTieto, lisätiedot, laskutusosoite, yhteyshenkilönNimi, yhteyshenkilönPuhelinnumero, yhteyshenkilönSähköposti) " +
+            stmt.setInt(3, maanantaiKpl);
-                    "VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)", Statement.RETURN_GENERATED_KEYS);
+            stmt.setInt(4, tiistaiKpl);
-            int i = 1;
+            stmt.setInt(5, keskiviikkoKpl);
-            stmt.setString(i++, nimi);
+            stmt.setInt(6, torstaiKpl);
-            stmt.setString(i++, osoite);
+            stmt.setInt(7, perjantaiKpl);
-            stmt.setInt(i++, maanantaiKpl);
+            stmt.setInt(8, salaatit);
-            stmt.setInt(i++, tiistaiKpl);
+            stmt.setInt(9, pääruoat);
-            stmt.setInt(i++, keskiviikkoKpl);
+            stmt.setInt(10, jälkiruoat);
-            stmt.setInt(i++, torstaiKpl);
+            stmt.setString(11, erityisruokavalio);
-            stmt.setInt(i++, perjantaiKpl);
+            stmt.setString(12, avainTieto);
-            stmt.setInt(i++, lauantaiKpl);
+            stmt.setString(13, lisätiedot);
-            stmt.setInt(i++, sunnuntaiKpl);
-            stmt.setInt(i++, salaatit);
-            stmt.setInt(i++, pääruoat);
-            stmt.setInt(i++, jälkiruoat);
-            stmt.setInt(i++, dinnerMaanantaiKpl);
-            stmt.setInt(i++, dinnerTiistaiKpl);
-            stmt.setInt(i++, dinnerKeskiviikkoKpl);
-            stmt.setInt(i++, dinnerTorstaiKpl);
-            stmt.setInt(i++, dinnerPerjantaiKpl);
-            stmt.setInt(i++, dinnerLauantaiKpl);
-            stmt.setInt(i++, dinnerSunnuntaiKpl);
-            stmt.setInt(i++, dinnerSalaatit);
-            stmt.setInt(i++, dinnerPääruoat);
-            stmt.setInt(i++, dinnerJälkiruoat);
-            stmt.setString(i++, erityisruokavalio);
-            stmt.setString(i++, avainTieto);
-            stmt.setString(i++, lisätiedot);
-            stmt.setString(i++, laskutusOsoite);
-            stmt.setString(i++, yhteyshenkilönNimi);
-            stmt.setString(i++, yhteyshenkilönPuhelinnumero);
-            stmt.setString(i++, yhteyshenkilönSähköposti);
-
             int muokatutRivit = stmt.executeUpdate();
 
             if (muokatutRivit == 0) {
@@ -271,42 +219,21 @@ public class Asiakas implements TietokantaOlio {
     @Override
     public boolean päivitäTietokantaan() {
         try (Connection tietokanta = Tietokanta.haeYhteys()) {
-            PreparedStatement stmt = tietokanta.prepareStatement("UPDATE asiakkaat set nimi = ?, osoite = ?, " +
+            PreparedStatement stmt = tietokanta.prepareStatement("UPDATE asiakkaat set nimi = ?, osoite = ?, maanantaiKpl = ?, tiistaiKpl = ?, keskiviikkoKpl = ?, torstaiKpl = ?, perjantaiKpl = ?, salaatit = ?, pääruoat = ?, jälkiruoat = ?, erityisruokavalio = ?, avainTieto = ?, lisätiedot = ? WHERE id = ?");
-                    "maanantaiKpl = ?, tiistaiKpl = ?, keskiviikkoKpl = ?, torstaiKpl = ?, perjantaiKpl = ?, lauantaiKpl = ?, sunnuntaiKpl = ?, salaatit = ?, pääruoat = ?, jälkiruoat = ?, " +
+            stmt.setString(1, nimi);
-                    "dinner_maanantaiKpl = ?, dinner_tiistaiKpl = ?, dinner_keskiviikkoKpl = ?, dinner_torstaiKpl = ?, dinner_perjantaiKpl = ?, dinner_lauantaiKpl = ?, dinner_sunnuntaiKpl = ?, dinner_salaatit = ?, dinner_pääruoat = ?, dinner_jälkiruoat = ?, " +
+            stmt.setString(2, osoite);
-                    "erityisruokavalio = ?, avainTieto = ?, lisätiedot = ?, laskutusosoite = ?, yhteyshenkilönNimi = ?, yhteyshenkilönPuhelinnumero = ?, yhteyshenkilönSähköposti = ? WHERE id = ?");
+            stmt.setInt(3, maanantaiKpl);
-            int i = 1;
+            stmt.setInt(4, tiistaiKpl);
-            stmt.setString(i++, nimi);
+            stmt.setInt(5, keskiviikkoKpl);
-            stmt.setString(i++, osoite);
+            stmt.setInt(6, torstaiKpl);
-            stmt.setInt(i++, maanantaiKpl);
+            stmt.setInt(7, perjantaiKpl);
-            stmt.setInt(i++, tiistaiKpl);
+            stmt.setInt(8, salaatit);
-            stmt.setInt(i++, keskiviikkoKpl);
+            stmt.setInt(9, pääruoat);
-            stmt.setInt(i++, torstaiKpl);
+            stmt.setInt(10, jälkiruoat);
-            stmt.setInt(i++, perjantaiKpl);
+            stmt.setString(11, erityisruokavalio);
-            stmt.setInt(i++, lauantaiKpl);
+            stmt.setString(12, avainTieto);
-            stmt.setInt(i++, sunnuntaiKpl);
+            stmt.setString(13, lisätiedot);
-            stmt.setInt(i++, salaatit);
+            stmt.setInt(14, id);
-            stmt.setInt(i++, pääruoat);
-            stmt.setInt(i++, jälkiruoat);
-            stmt.setInt(i++, dinnerMaanantaiKpl);
-            stmt.setInt(i++, dinnerTiistaiKpl);
-            stmt.setInt(i++, dinnerKeskiviikkoKpl);
-            stmt.setInt(i++, dinnerTorstaiKpl);
-            stmt.setInt(i++, dinnerPerjantaiKpl);
-            stmt.setInt(i++, dinnerLauantaiKpl);
-            stmt.setInt(i++, dinnerSunnuntaiKpl);
-            stmt.setInt(i++, dinnerSalaatit);
-            stmt.setInt(i++, dinnerPääruoat);
-            stmt.setInt(i++, dinnerJälkiruoat);
-            stmt.setString(i++, erityisruokavalio);
-            stmt.setString(i++, avainTieto);
-            stmt.setString(i++, lisätiedot);
-            stmt.setString(i++, laskutusOsoite);
-            stmt.setString(i++, yhteyshenkilönNimi);
-            stmt.setString(i++, yhteyshenkilönPuhelinnumero);
-            stmt.setString(i++, yhteyshenkilönSähköposti);
-
-            stmt.setInt(i++, id);
             stmt.executeUpdate();
             return true;
         } catch (SQLException e) {
@@ -350,52 +277,6 @@ public class Asiakas implements TietokantaOlio {
         return this.getNimi();
     }
 
-    public int getKuljetuksenSalaatit(LocalDate päivä, Kuljetus.Tyyppi tyyppi) {
-        return switch (tyyppi) {
-            case Kuljetus.Tyyppi.Lounas -> getLounasKuljetusKerroin(päivä) * getSalaatit();
-            case Kuljetus.Tyyppi.Päivällinen -> getPäivällisKuljetusKerroin(päivä) * getDinnerSalaatit();
-        };
-    }
-
-    public int getKuljetuksenPääruoat(LocalDate päivä, Kuljetus.Tyyppi tyyppi) {
-        return switch (tyyppi) {
-            case Kuljetus.Tyyppi.Lounas -> getLounasKuljetusKerroin(päivä) * getPääruoat();
-            case Kuljetus.Tyyppi.Päivällinen -> getPäivällisKuljetusKerroin(päivä) * getDinnerPääruoat();
-        };
-    }
-
-    public int getKuljetuksenJälkiruoat(LocalDate päivä, Kuljetus.Tyyppi tyyppi) {
-        return switch (tyyppi) {
-            case Kuljetus.Tyyppi.Lounas -> getLounasKuljetusKerroin(päivä) * getJälkiruoat();
-            case Kuljetus.Tyyppi.Päivällinen -> getPäivällisKuljetusKerroin(päivä) * getDinnerJälkiruoat();
-        };
-    }
-
-    private int getLounasKuljetusKerroin(LocalDate päivä) {
-        return switch (päivä.getDayOfWeek()) {
-            case DayOfWeek.MONDAY -> getMaanantaiKpl();
-            case DayOfWeek.TUESDAY -> getTiistaiKpl();
-            case DayOfWeek.WEDNESDAY -> getKeskiviikkoKpl();
-            case DayOfWeek.THURSDAY -> getTorstaiKpl();
-            case DayOfWeek.FRIDAY -> getPerjantaiKpl();
-            case DayOfWeek.SATURDAY -> getLauantaiKpl();
-            case DayOfWeek.SUNDAY -> getSunnuntaiKpl();
-        };
-    }
-
-    private int getPäivällisKuljetusKerroin(LocalDate päivä) {
-        return switch (päivä.getDayOfWeek()) {
-            case DayOfWeek.MONDAY -> getDinnerMaanantaiKpl();
-            case DayOfWeek.TUESDAY -> getDinnerTiistaiKpl();
-            case DayOfWeek.WEDNESDAY -> getDinnerKeskiviikkoKpl();
-            case DayOfWeek.THURSDAY -> getDinnerTorstaiKpl();
-            case DayOfWeek.FRIDAY -> getDinnerPerjantaiKpl();
-            case DayOfWeek.SATURDAY -> getDinnerLauantaiKpl();
-            case DayOfWeek.SUNDAY -> getDinnerSunnuntaiKpl();
-        };
-    }
-
-
     public int getId() {
         return id;
     }
@@ -460,22 +341,6 @@ public class Asiakas implements TietokantaOlio {
         this.perjantaiKpl = perjantaiKpl;
     }
 
-    public int getLauantaiKpl() {
-        return lauantaiKpl;
-    }
-
-    public void setLauantaiKpl(int lauantaiKpl) {
-        this.lauantaiKpl = lauantaiKpl;
-    }
-
-    public int getSunnuntaiKpl() {
-        return sunnuntaiKpl;
-    }
-
-    public void setSunnuntaiKpl(int sunnuntaiKpl) {
-        this.sunnuntaiKpl = sunnuntaiKpl;
-    }
-
     public int getSalaatit() {
         return salaatit;
     }
@@ -523,116 +388,4 @@ public class Asiakas implements TietokantaOlio {
     public void setLisätiedot(String lisätiedot) {
         this.lisätiedot = lisätiedot;
     }
-
-    public String getLaskutusOsoite() {
-        return laskutusOsoite;
-    }
-
-    public void setLaskutusOsoite(String laskutusOsoite) {
-        this.laskutusOsoite = laskutusOsoite;
-    }
-
-    public String getYhteyshenkilönNimi() {
-        return yhteyshenkilönNimi;
-    }
-
-    public void setYhteyshenkilönNimi(String yhteyshenkilönNimi) {
-        this.yhteyshenkilönNimi = yhteyshenkilönNimi;
-    }
-
-    public String getYhteyshenkilönPuhelinnumero() {
-        return yhteyshenkilönPuhelinnumero;
-    }
-
-    public void setYhteyshenkilönPuhelinnumero(String yhteyshenkilönPuhelinnumero) {
-        this.yhteyshenkilönPuhelinnumero = yhteyshenkilönPuhelinnumero;
-    }
-
-    public String getYhteyshenkilönSähköposti() {
-        return yhteyshenkilönSähköposti;
-    }
-
-    public void setYhteyshenkilönSähköposti(String yhteyshenkilönSähköposti) {
-        this.yhteyshenkilönSähköposti = yhteyshenkilönSähköposti;
-    }
-
-    public int getDinnerMaanantaiKpl() {
-        return dinnerMaanantaiKpl;
-    }
-
-    public void setDinnerMaanantaiKpl(int dinnerMaanantaiKpl) {
-        this.dinnerMaanantaiKpl = dinnerMaanantaiKpl;
-    }
-
-    public int getDinnerTiistaiKpl() {
-        return dinnerTiistaiKpl;
-    }
-
-    public void setDinnerTiistaiKpl(int dinnerTiistaiKpl) {
-        this.dinnerTiistaiKpl = dinnerTiistaiKpl;
-    }
-
-    public int getDinnerKeskiviikkoKpl() {
-        return dinnerKeskiviikkoKpl;
-    }
-
-    public void setDinnerKeskiviikkoKpl(int dinnerKeskiviikkoKpl) {
-        this.dinnerKeskiviikkoKpl = dinnerKeskiviikkoKpl;
-    }
-
-    public int getDinnerTorstaiKpl() {
-        return dinnerTorstaiKpl;
-    }
-
-    public void setDinnerTorstaiKpl(int dinnerTorstaiKpl) {
-        this.dinnerTorstaiKpl = dinnerTorstaiKpl;
-    }
-
-    public int getDinnerPerjantaiKpl() {
-        return dinnerPerjantaiKpl;
-    }
-
-    public void setDinnerPerjantaiKpl(int dinnerPerjantaiKpl) {
-        this.dinnerPerjantaiKpl = dinnerPerjantaiKpl;
-    }
-
-    public int getDinnerLauantaiKpl() {
-        return dinnerLauantaiKpl;
-    }
-
-    public void setDinnerLauantaiKpl(int dinnerLauantaiKpl) {
-        this.dinnerLauantaiKpl = dinnerLauantaiKpl;
-    }
-
-    public int getDinnerSunnuntaiKpl() {
-        return dinnerSunnuntaiKpl;
-    }
-
-    public void setDinnerSunnuntaiKpl(int dinnerSunnuntaiKpl) {
-        this.dinnerSunnuntaiKpl = dinnerSunnuntaiKpl;
-    }
-
-    public int getDinnerSalaatit() {
-        return dinnerSalaatit;
-    }
-
-    public void setDinnerSalaatit(int dinnerSalaatit) {
-        this.dinnerSalaatit = dinnerSalaatit;
-    }
-
-    public int getDinnerPääruoat() {
-        return dinnerPääruoat;
-    }
-
-    public void setDinnerPääruoat(int dinnerPääruoat) {
-        this.dinnerPääruoat = dinnerPääruoat;
-    }
-
-    public int getDinnerJälkiruoat() {
-        return dinnerJälkiruoat;
-    }
-
-    public void setDinnerJälkiruoat(int dinnerJälkiruoat) {
-        this.dinnerJälkiruoat = dinnerJälkiruoat;
-    }
 }

src/main/java/fi/lpam/dataluokat/Kuljetus.java

@@ -8,17 +8,11 @@ import java.util.ArrayList;
 
 @SuppressWarnings("DuplicatedCode")
 public class Kuljetus implements TietokantaOlio {
-    public enum Tyyppi {
-        Lounas,
-        Päivällinen;
-    }
-
     private int id = -1;
     private int asiakasID;
     private String nimi;
     private LocalDate kuljetusPäivämäärä;
     private int kuljetusKierros;
-    private Tyyppi tyyppi;
     private int salaatit, pääruoat, jälkiruoat;
     private String osoite, lisätieto, avainTieto, ruokarajoite;
 
@@ -28,7 +22,6 @@ public class Kuljetus implements TietokantaOlio {
         this.nimi = vanha.nimi;
         this.kuljetusPäivämäärä = vanha.kuljetusPäivämäärä;
         this.kuljetusKierros = vanha.kuljetusKierros;
-        this.tyyppi = vanha.tyyppi;
         this.salaatit = vanha.salaatit;
         this.pääruoat = vanha.pääruoat;
         this.jälkiruoat = vanha.jälkiruoat;
@@ -38,40 +31,34 @@ public class Kuljetus implements TietokantaOlio {
         this.ruokarajoite = vanha.ruokarajoite;
     }
 
-    public Kuljetus(Asiakas asiakas, LocalDate pvm, int kuljetusKierros, Tyyppi tyyppi) {
+    public Kuljetus(Asiakas asiakas, LocalDate pvm, int kuljetusKierros) {
         this.asiakasID = asiakas.getId();
         this.nimi = asiakas.getNimi();
         this.kuljetusPäivämäärä = pvm;
         this.kuljetusKierros = kuljetusKierros;
-        this.tyyppi = tyyppi;
+        int kuljetusKerroin = switch (kuljetusPäivämäärä.getDayOfWeek()) {
-        this.salaatit = asiakas.getKuljetuksenSalaatit(pvm, tyyppi);
+            case MONDAY -> asiakas.getMaanantaiKpl();
-        this.pääruoat = asiakas.getKuljetuksenPääruoat(pvm, tyyppi);
+            case TUESDAY -> asiakas.getTiistaiKpl();
-        this.jälkiruoat = asiakas.getKuljetuksenJälkiruoat(pvm, tyyppi);
+            case WEDNESDAY -> asiakas.getKeskiviikkoKpl();
+            case THURSDAY -> asiakas.getTorstaiKpl();
+            case FRIDAY -> asiakas.getPerjantaiKpl();
+            default -> 0;
+        };
+        this.salaatit = kuljetusKerroin * asiakas.getSalaatit();
+        this.pääruoat = kuljetusKerroin * asiakas.getPääruoat();
+        this.jälkiruoat = kuljetusKerroin * asiakas.getJälkiruoat();
         this.osoite = asiakas.getOsoite();
         this.avainTieto = asiakas.getAvainTieto();
         this.lisätieto = asiakas.getLisätiedot();
         this.ruokarajoite = asiakas.getErityisruokavalio();
     }
 
-    private Kuljetus(int id,
+    private Kuljetus(int id, int asiakasID, String nimi, LocalDate kuljetusPäivämäärä, int kuljetusKierros, int salaatit, int pääruoat, int jälkiruoat, String osoite, String lisätieto, String avainTieto, String ruokarajoite) {
-                     int asiakasID,
-                     String nimi,
-                     LocalDate kuljetusPäivämäärä,
-                     int kuljetusKierros,
-                     int salaatit,
-                     int pääruoat,
-                     int jälkiruoat,
-                     String osoite,
-                     String lisätieto,
-                     String avainTieto,
-                     String ruokarajoite,
-                     Tyyppi tyyppi) {
         this.id = id;
         this.asiakasID = asiakasID;
         this.nimi = nimi;
         this.kuljetusPäivämäärä = kuljetusPäivämäärä;
         this.kuljetusKierros = kuljetusKierros;
-        this.tyyppi = tyyppi;
         this.salaatit = salaatit;
         this.pääruoat = pääruoat;
         this.jälkiruoat = jälkiruoat;
@@ -81,18 +68,82 @@ public class Kuljetus implements TietokantaOlio {
         this.ruokarajoite = ruokarajoite;
     }
 
+    public static ArrayList<Kuljetus> haeKaikki() {
+        try (Connection tietokanta = Tietokanta.haeYhteys()) {
+            Statement stmt = tietokanta.createStatement();
+            ResultSet rs = stmt.executeQuery("SELECT * FROM kuljetukset;");
+            return parsiResultSet(rs);
+        } catch (SQLException e) {
+            SiirtoVirhe virhe = new SiirtoVirhe(e);
+            virhe.showAndWait();
+            return null;
+        }
+    }
+
+    /**
+     * Hakee yhden päivän kuljetukset riippumatta kierroksesta
+     * @param päivä haettava päivä
+     * @return päivän kuljetukset
+     */
+    public static ArrayList<Kuljetus> haePäivänKuljetukset(LocalDate päivä) {
+        try (Connection tietokanta = Tietokanta.haeYhteys()) {
+            PreparedStatement stmt = tietokanta.prepareStatement("SELECT * FROM kuljetukset WHERE kuljetusPäivämäärä = ?;");
+            stmt.setDate(1, Date.valueOf(päivä));
+            ResultSet rs = stmt.executeQuery();
+            return parsiResultSet(rs);
+        } catch (SQLException e) {
+            SiirtoVirhe virhe = new SiirtoVirhe(e);
+            virhe.showAndWait();
+            return null;
+        }
+    }
+
     /**
      * Hakee tietyn kierroksen kuljetukset
      * @param päivä kierroksen päivä
      * @param kuljetusKierros kierroksen numero
      * @return valitun päivän valitun kierroksen kuljetukset
      */
-    public static ArrayList<Kuljetus> haeKierroksenKuljetukset(LocalDate päivä, int kuljetusKierros, Tyyppi tyyppi) {
+    public static ArrayList<Kuljetus> haePäivänKuljetuksetKierroksella(LocalDate päivä, int kuljetusKierros) {
         try (Connection tietokanta = Tietokanta.haeYhteys()) {
-            PreparedStatement stmt = tietokanta.prepareStatement("SELECT * FROM kuljetukset WHERE kuljetusKierros = ? and kuljetusPäivämäärä = ? and tyyppi = ?");
+            PreparedStatement stmt = tietokanta.prepareStatement("SELECT * FROM kuljetukset WHERE kuljetusKierros = ? and kuljetusPäivämäärä = ?");
             stmt.setInt(1, kuljetusKierros);
             stmt.setDate(2, Date.valueOf(päivä));
-            stmt.setString(3, tyyppi.toString());
+            ResultSet rs = stmt.executeQuery();
+            return parsiResultSet(rs);
+        } catch (SQLException e) {
+            SiirtoVirhe virhe = new SiirtoVirhe(e);
+            virhe.showAndWait();
+            return null;
+        }
+    }
+    /**
+     * Hakee kuljetukset aikajakson perusteella
+     *
+     * @param alku  jakson alku (sisältyen)
+     * @param loppu jakson loppu (sisältyen)
+     * @return Jakson kuljetukset
+     */
+    public static ArrayList<Kuljetus> haeAikaJakso(LocalDate alku, LocalDate loppu) {
+        try (Connection tietokanta = Tietokanta.haeYhteys()) {
+            PreparedStatement stmt = tietokanta.prepareStatement("SELECT * FROM kuljetukset WHERE kuljetusPäivämäärä BETWEEN ? AND ?");
+            stmt.setDate(1, Date.valueOf(alku));
+            stmt.setDate(2, Date.valueOf(loppu));
+            ResultSet rs = stmt.executeQuery();
+            return parsiResultSet(rs);
+        } catch (SQLException e) {
+            SiirtoVirhe virhe = new SiirtoVirhe(e);
+            virhe.showAndWait();
+            return null;
+        }
+    }
+
+    public static ArrayList<Kuljetus> haeAikaJaksollaJaKierroksella(LocalDate alku, LocalDate loppu, int kuljetusKierros) {
+        try (Connection tietokanta = Tietokanta.haeYhteys()) {
+            PreparedStatement stmt = tietokanta.prepareStatement("SELECT * FROM kuljetukset WHERE kuljetusKierros = ? and kuljetusPäivämäärä BETWEEN ? AND ?");
+            stmt.setInt(1, kuljetusKierros);
+            stmt.setDate(2, Date.valueOf(alku));
+            stmt.setDate(3, Date.valueOf(loppu));
             ResultSet rs = stmt.executeQuery();
             return parsiResultSet(rs);
         } catch (SQLException e) {
@@ -105,19 +156,18 @@ public class Kuljetus implements TietokantaOlio {
     @Override
     public boolean tallennaUusi() {
          try (Connection tietokanta = Tietokanta.haeYhteys()) {
-            PreparedStatement stmt = tietokanta.prepareStatement("INSERT INTO kuljetukset (asiakasID, nimi, kuljetusPäivämäärä, kuljetusKierros, tyyppi, salaatit, pääruoat, jälkiruoat, ruokarajoite, osoite, avainTieto, lisätieto) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)", Statement.RETURN_GENERATED_KEYS);
+            PreparedStatement stmt = tietokanta.prepareStatement("INSERT INTO kuljetukset (asiakasID, nimi, kuljetusPäivämäärä, kuljetusKierros, salaatit, pääruoat, jälkiruoat, ruokarajoite, osoite, avainTieto, lisätieto) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)", Statement.RETURN_GENERATED_KEYS);
             stmt.setInt(1, asiakasID);
             stmt.setString(2, nimi);
             stmt.setDate(3, Date.valueOf(kuljetusPäivämäärä));
             stmt.setInt(4, kuljetusKierros);
-            stmt.setString(5, tyyppi.toString());
+            stmt.setInt(5, salaatit);
-            stmt.setInt(6, salaatit);
+            stmt.setInt(6, pääruoat);
-            stmt.setInt(7, pääruoat);
+            stmt.setInt(7, jälkiruoat);
-            stmt.setInt(8, jälkiruoat);
+            stmt.setString(8, ruokarajoite);
-            stmt.setString(9, ruokarajoite);
+            stmt.setString(9, osoite);
-            stmt.setString(10, osoite);
+            stmt.setString(10, avainTieto);
-            stmt.setString(11, avainTieto);
+            stmt.setString(11, lisätieto);
-            stmt.setString(12, lisätieto);
             int muokatutRivit = stmt.executeUpdate();
 
             if (muokatutRivit == 0) {
@@ -141,20 +191,19 @@ public class Kuljetus implements TietokantaOlio {
     @Override
     public boolean päivitäTietokantaan() {
         try (Connection tietokanta = Tietokanta.haeYhteys()) {
-            PreparedStatement stmt = tietokanta.prepareStatement("UPDATE kuljetukset set asiakasID = ?, nimi = ?, kuljetusPäivämäärä = ?, kuljetusKierros = ?, tyyppi = ?, salaatit = ?, pääruoat = ?, jälkiruoat = ?, ruokarajoite = ?, osoite = ?, avainTieto = ?, lisätieto = ? WHERE id = ?");
+            PreparedStatement stmt = tietokanta.prepareStatement("UPDATE kuljetukset set asiakasID = ?, nimi = ?, kuljetusPäivämäärä = ?, kuljetusKierros = ?, salaatit = ?, pääruoat = ?, jälkiruoat = ?, ruokarajoite = ?, osoite = ?, avainTieto = ?, lisätieto = ? WHERE id = ?");
             stmt.setInt(1, asiakasID);
             stmt.setString(2, nimi);
             stmt.setDate(3, Date.valueOf(kuljetusPäivämäärä));
             stmt.setInt(4, kuljetusKierros);
-            stmt.setString(5, tyyppi.toString());
+            stmt.setInt(5, salaatit);
-            stmt.setInt(6, salaatit);
+            stmt.setInt(6, pääruoat);
-            stmt.setInt(7, pääruoat);
+            stmt.setInt(7, jälkiruoat);
-            stmt.setInt(8, jälkiruoat);
+            stmt.setString(8, ruokarajoite);
-            stmt.setString(9, ruokarajoite);
+            stmt.setString(9, osoite);
-            stmt.setString(10, osoite);
+            stmt.setString(10, avainTieto);
-            stmt.setString(11, avainTieto);
+            stmt.setString(11, lisätieto);
-            stmt.setString(12, lisätieto);
+            stmt.setInt(12, id);
-            stmt.setInt(13, id);
             stmt.executeUpdate();
             return true;
         } catch (SQLException e) {
@@ -205,13 +254,8 @@ public class Kuljetus implements TietokantaOlio {
                         rs.getString("osoite"),
                         rs.getString("lisätieto"),
                         rs.getString("avainTieto"),
-                        rs.getString("ruokarajoite"),
+                        rs.getString("ruokarajoite")
-                        Tyyppi.valueOf(rs.getString("tyyppi").strip())
                 );
-                System.out.println(rs.getString("tyyppi").equals(Tyyppi.Lounas.toString()));
-                System.out.println(Tyyppi.Lounas);
-                System.out.println(uusi.getTyyppi());
-                System.out.println(Tyyppi.valueOf(rs.getString("tyyppi")));
                 kuljetukset.add(uusi);
             }
         } catch (SQLException e) {
@@ -229,7 +273,6 @@ public class Kuljetus implements TietokantaOlio {
                 ", nimi='" + nimi + '\'' +
                 ", kuljetusPäivämäärä=" + kuljetusPäivämäärä +
                 ", kuljetusKierros=" + kuljetusKierros +
-                ", tyyppi =" + tyyppi +
                 ", salaatit=" + salaatit +
                 ", pääruoat=" + pääruoat +
                 ", jälkiruoat=" + jälkiruoat +
@@ -248,16 +291,23 @@ public class Kuljetus implements TietokantaOlio {
     }
 
     public void palautaMäärät(Asiakas asiakas) {
-        setSalaatit(asiakas.getKuljetuksenSalaatit(kuljetusPäivämäärä, tyyppi));
+        int kuljetusKerroin = switch (kuljetusPäivämäärä.getDayOfWeek()) {
-        setPääruoat(asiakas.getKuljetuksenPääruoat(kuljetusPäivämäärä, tyyppi));
+            case MONDAY -> asiakas.getMaanantaiKpl();
-        setJälkiruoat(asiakas.getKuljetuksenJälkiruoat(kuljetusPäivämäärä, tyyppi));
+            case TUESDAY -> asiakas.getTiistaiKpl();
+            case WEDNESDAY -> asiakas.getKeskiviikkoKpl();
+            case THURSDAY -> asiakas.getTorstaiKpl();
+            case FRIDAY -> asiakas.getPerjantaiKpl();
+            default -> 0;
+        };
+        setSalaatit(asiakas.getSalaatit() * kuljetusKerroin);
+        setPääruoat(asiakas.getPääruoat() * kuljetusKerroin);
+        setJälkiruoat(asiakas.getJälkiruoat() * kuljetusKerroin);
         setLisätieto(asiakas.getLisätiedot());
     }
 
     public boolean kuljetetaan() {
         return salaatit > 0 || pääruoat > 0 || jälkiruoat > 0 || !lisätieto.isBlank();
     }
-
     public int getId() {
         return id;
     }
@@ -353,12 +403,4 @@ public class Kuljetus implements TietokantaOlio {
     public void setRuokarajoite(String ruokarajoite) {
         this.ruokarajoite = ruokarajoite;
     }
-
-    public Tyyppi getTyyppi() {
-        return tyyppi;
-    }
-
-    public void setTyyppi(Tyyppi tyyppi) {
-        this.tyyppi = tyyppi;
-    }
 }

src/main/java/fi/lpam/dataluokat/RaporttiRivi.java

@@ -6,40 +6,21 @@ import java.sql.*;
 import java.time.LocalDate;
 import java.util.ArrayList;
 
-@SuppressWarnings({"ClassCanBeRecord", "DuplicatedCode"})
+@SuppressWarnings("ClassCanBeRecord")
 public class RaporttiRivi {
     private final String nimi;
-    private final LocalDate pvm;
+    private final LocalDate alkuPvm, loppuPvm;
-    private final Kuljetus.Tyyppi tyyppi;
     private final int salaatit, pääruoat, jälkiruoat;
-    private final String erva;
     private final String lisätiedot;
-    private final String yhteyshenkilönNimi, laskutusOsoite, yhteyshenkilönPuhelinnumero, yhteyshenkilönSähköposti;
 
-    private RaporttiRivi(String nimi,
+    public RaporttiRivi(String nimi, LocalDate alkuPvm, LocalDate loppuPvm, int salaatit, int pääruoat, int jälkiruoat, String lisätiedot) {
-                        LocalDate pvm,
-                        int salaatit,
-                        int pääruoat,
-                        int jälkiruoat,
-                        String lisätiedot,
-                        String yhteyshenkilönNimi,
-                        String laskutusOsoite,
-                        String yhteyshenkilönPuhelinnumero,
-                        String yhteyshenkilönSähköposti,
-                        String erva,
-                        Kuljetus.Tyyppi tyyppi) {
         this.nimi = nimi;
-        this.pvm = pvm;
+        this.alkuPvm = alkuPvm;
-        this.tyyppi = tyyppi;
+        this.loppuPvm = loppuPvm;
         this.salaatit = salaatit;
         this.pääruoat = pääruoat;
         this.jälkiruoat = jälkiruoat;
         this.lisätiedot = lisätiedot;
-        this.erva = erva;
-        this.yhteyshenkilönNimi = yhteyshenkilönNimi;
-        this.laskutusOsoite = laskutusOsoite;
-        this.yhteyshenkilönPuhelinnumero = yhteyshenkilönPuhelinnumero;
-        this.yhteyshenkilönSähköposti = yhteyshenkilönSähköposti;
     }
 
     public static ArrayList<RaporttiRivi> haeRaportti(LocalDate alkuPvm, LocalDate loppuPvm, boolean tarkka) {
@@ -48,90 +29,40 @@ public class RaporttiRivi {
             ResultSet rs;
             if (tarkka) {
                 //Jokainen kuljetus saa oman rivin
-                PreparedStatement stmt = tietokanta.prepareStatement("""
+                PreparedStatement stmt = tietokanta.prepareStatement("select nimi, kuljetusPäivämäärä, salaatit, pääruoat, jälkiruoat, lisätieto from kuljetukset where kuljetusPäivämäärä between ? and ? order by nimi");
-                                            select kuljetukset.asiakasID,
-                                                   kuljetukset.nimi,
-                                                   kuljetukset.kuljetusPäivämäärä,
-                                                   kuljetukset.salaatit,
-                                                   kuljetukset.pääruoat,
-                                                   kuljetukset.jälkiruoat,
-                                                   kuljetukset.lisätieto,
-                                                   asiakkaat.yhteyshenkilönNimi,
-                                                   asiakkaat.laskutusosoite,
-                                                   asiakkaat.yhteyshenkilönPuhelinnumero,
-                                                   asiakkaat.yhteyshenkilönSähköposti,
-                                                   asiakkaat.erityisruokavalio,
-                                                   kuljetukset.tyyppi
-                                            from kuljetukset
-                                            left join asiakkaat on kuljetukset.asiakasID=asiakkaat.id
-                                            where kuljetusPäivämäärä between ? and ?
-                                            order by kuljetukset.nimi, kuljetukset.kuljetusPäivämäärä, kuljetukset.tyyppi
-                                            """);
                 stmt.setDate(1, Date.valueOf(alkuPvm));
                 stmt.setDate(2, Date.valueOf(loppuPvm));
                 rs = stmt.executeQuery();
 
                 while (rs.next()) {
-                    int i = 2;
                     RaporttiRivi uusi = new RaporttiRivi(
-                            rs.getString(i++),
+                            rs.getString(1),
-                            rs.getDate(i++).toLocalDate(),
+                            rs.getDate(2).toLocalDate(),
-                            rs.getInt(i++),
+                            rs.getDate(2).toLocalDate(),
-                            rs.getInt(i++),
+                            rs.getInt(3),
-                            rs.getInt(i++),
+                            rs.getInt(4),
-                            rs.getString(i++),
+                            rs.getInt(5),
-                            rs.getString(i++),
+                            rs.getString(6)
-                            rs.getString(i++),
-                            rs.getString(i++),
-                            rs.getString(i++),
-                            rs.getString(i++),
-                            Kuljetus.Tyyppi.valueOf(rs.getString(i++))
                     );
                     raportti.add(uusi);
                 }
 
             } else {
                 //Kuljetukset summataan yhteen riviin per vastaanottaja
-                PreparedStatement stmt = tietokanta.prepareStatement("""
+                PreparedStatement stmt = tietokanta.prepareStatement("select nimi, kuljetusPäivämäärä, sum(salaatit), sum(pääruoat), sum(jälkiruoat), group_concat(lisätieto, '\n') from kuljetukset where kuljetusPäivämäärä between ? and ? group by nimi order by nimi;");
-                                            select kuljetukset.asiakasID,
+                stmt.setDate(1, Date.valueOf(alkuPvm));
-                                                   kuljetukset.nimi,
+                stmt.setDate(2, Date.valueOf(loppuPvm));
-                                                   kuljetukset.kuljetusPäivämäärä,
-                                                   sum(kuljetukset.salaatit),
-                                                   sum(kuljetukset.pääruoat),
-                                                   sum(kuljetukset.jälkiruoat),
-                                                   group_concat(kuljetukset.lisätieto, ?),
-                                                   asiakkaat.yhteyshenkilönNimi,
-                                                   asiakkaat.laskutusosoite,
-                                                   asiakkaat.yhteyshenkilönPuhelinnumero,
-                                                   asiakkaat.yhteyshenkilönSähköposti,
-                                                   asiakkaat.erityisruokavalio,
-                                                   kuljetukset.tyyppi
-                                            from kuljetukset
-                                            left join asiakkaat on kuljetukset.asiakasID=asiakkaat.id
-                                            where kuljetusPäivämäärä between ? and ?
-                                            group by kuljetukset.nimi, kuljetukset.tyyppi
-                                            order by kuljetukset.nimi, kuljetukset.kuljetusPäivämäärä, kuljetukset.tyyppi
-                                            """);
-                stmt.setString(1, "\n");
-                stmt.setDate(2, Date.valueOf(alkuPvm));
-                stmt.setDate(3, Date.valueOf(loppuPvm));
                 rs = stmt.executeQuery();
 
                 while (rs.next()) {
-                    int i = 2;
                     RaporttiRivi uusi = new RaporttiRivi(
-                            rs.getString(i++),
+                            rs.getString(1),
-                            rs.getDate(i++).toLocalDate(),
+                            alkuPvm,
-                            rs.getInt(i++),
+                            loppuPvm,
-                            rs.getInt(i++),
+                            rs.getInt(3),
-                            rs.getInt(i++),
+                            rs.getInt(4),
-                            rs.getString(i++),
+                            rs.getInt(5),
-                            rs.getString(i++),
+                            rs.getString(6)
-                            rs.getString(i++),
-                            rs.getString(i++),
-                            rs.getString(i++),
-                            rs.getString(i++),
-                            Kuljetus.Tyyppi.valueOf(rs.getString(i++))
                     );
                     raportti.add(uusi);
                 }
@@ -145,25 +76,19 @@ public class RaporttiRivi {
     }
     @Override
     public String toString() {
-
+        return this.nimi + ":\nSal: " + this.salaatit + ", Pr: " + this.pääruoat + ", Jr: " + this.jälkiruoat + "\nLisätiedot:\n" + this.lisätiedot.strip();
-        String laskutustiedot = "\n" + yhteyshenkilönNimi +
-                                "\n" + laskutusOsoite +
-                                "\n" + yhteyshenkilönPuhelinnumero +
-                                "\n" + yhteyshenkilönSähköposti;
-
-        return nimi + " " + erva +
-                "\n" + tyyppi + " Sal: " + salaatit + ", Pr: " + pääruoat + ", Jr: " + jälkiruoat +
-                "\nLisätiedot:\n" +
-                getLisätiedot() +
-                laskutustiedot.replace("\nnull", "").strip();
     }
 
     public String getNimi() {
         return nimi;
     }
 
-    public LocalDate getPvm() {
+    public LocalDate getAlkuPvm() {
-        return pvm;
+        return alkuPvm;
+    }
+
+    public LocalDate getLoppuPvm() {
+        return loppuPvm;
     }
 
     public int getSalaatit() {
@@ -179,31 +104,6 @@ public class RaporttiRivi {
     }
 
     public String getLisätiedot() {
-        if (lisätiedot == null) return null;
+        return lisätiedot;
-        return lisätiedot.strip().trim().stripLeading().stripTrailing();
-    }
-
-    public String getYhteyshenkilönNimi() {
-        return yhteyshenkilönNimi;
-    }
-
-    public String getLaskutusOsoite() {
-        return laskutusOsoite;
-    }
-
-    public String getYhteyshenkilönPuhelinnumero() {
-        return yhteyshenkilönPuhelinnumero;
-    }
-
-    public String getYhteyshenkilönSähköposti() {
-        return yhteyshenkilönSähköposti;
-    }
-
-    public String getErva() {
-        return erva;
-    }
-
-    public Kuljetus.Tyyppi getTyyppi() {
-        return tyyppi;
     }
 }

src/main/java/fi/lpam/dataluokat/Tietokanta.java

@@ -1,62 +1,60 @@
 package fi.lpam.dataluokat;
 
 import fi.lpam.gui.virheDialogit.SiirtoVirhe;
+import fi.lpam.gui.virheDialogit.YhteysVirhe;
 
 import fi.lpam.Main;
+import javafx.scene.control.Alert;
 import org.apache.commons.lang3.SystemUtils;
 
 import java.io.*;
-import java.nio.file.Path;
 import java.sql.*;
 import java.util.Objects;
 
 public class Tietokanta {
     private static String osoite;
-    private static File tietokantatiedosto = Path.of(SystemUtils.getUserHome().getAbsolutePath(), "OneDrive", "Kuljetusruokasovellus", "tietokanta.db").toFile();
+    private static String dbPath = SystemUtils.getUserHome().getAbsolutePath() + "\\OneDrive\\Kuljetusruokasovellus\\tietokanta.db";
 
-    public Tietokanta(boolean dev) {
+    public Tietokanta(boolean dev) throws IOException {
-        if (dev) tietokantatiedosto = new File("testikanta.db");
+        if (dev) dbPath = "testikanta.db";
+        File file = new File(dbPath);
         try {
-            boolean _ = tietokantatiedosto.getParentFile().mkdirs();
+            //noinspection ResultOfMethodCallIgnored
-            boolean _ = tietokantatiedosto.createNewFile();
+            file.getParentFile().mkdirs();
         }
         catch (Exception _) {}
+        //noinspection ResultOfMethodCallIgnored
+        file.createNewFile();
 
-        osoite = "jdbc:sqlite:" + tietokantatiedosto.getAbsolutePath();
+        osoite = "jdbc:sqlite:" + file.getAbsolutePath();
 
-        String sql = "";
+
-        try (BufferedReader br = new BufferedReader(
+        try (Connection conn = haeYhteys()) {
-                new InputStreamReader(
+            InputStream is = Objects.requireNonNull(Main.class.getResourceAsStream("tietokanta.sql"));
-                        Objects.requireNonNull(Main.class.getClassLoader()
+            BufferedReader br = new BufferedReader(new InputStreamReader(is));
-                                .getResourceAsStream("tietokanta.sql"))))) {
+            String sql = br.readAllAsString();
-            sql = br.readAllAsString();
-        } catch (IOException e) {
-            throw new RuntimeException(e);
-        }
 
             for (String query : sql.split(";")) {
-            try (Connection conn = haeYhteys()) {
                 conn.createStatement().execute(query);
-            } catch (SQLException e) {
-                if (!e.getMessage().contains("duplicate column name")) {
-                    handleException(e);
-                }
-            }
-        }
             }
 
-    public static Connection haeYhteys() {
-        Connection connection = null;
-        try {
-            connection = DriverManager.getConnection(osoite);
-        } catch (SQLException e) {
-            handleException(e);
         }
-        return connection;
+        catch (Exception e) {
-    }
-
-    private static void handleException(Exception e) {
             SiirtoVirhe virhe = new SiirtoVirhe(e);
             virhe.showAndWait();
         }
+
+    }
+
+    public static Connection haeYhteys() {
+
+        Connection connection = null;
+        try {
+            connection = DriverManager.getConnection(osoite);
+        } catch (Exception e) {
+            Alert virhe = new YhteysVirhe(e);
+            virhe.showAndWait();
+        }
+        return connection;
+    }
 }

src/main/java/fi/lpam/gui/Asetukset.java

@@ -1,124 +0,0 @@
-package fi.lpam.gui;
-
-import fi.lpam.Main;
-import fi.lpam.gui.elementit.FontLabel;
-import fi.lpam.gui.elementit.TabPohja;
-import javafx.collections.FXCollections;
-import javafx.collections.ObservableList;
-import javafx.geometry.Pos;
-import javafx.scene.control.*;
-import javafx.scene.layout.GridPane;
-import javafx.scene.layout.HBox;
-import javafx.scene.text.Font;
-
-import java.io.IOException;
-import java.util.Properties;
-
-
-public class Asetukset extends TabPohja {
-    private static final Properties prop = Main.properties;
-    private static final ObservableList<String> järjestelmäFontit = FXCollections.observableArrayList(Font.getFamilies());
-
-
-    public Asetukset() {
-        super("Asetukset");
-
-        HBox yläpalkki = new HBox();
-        yläpalkki.setSpacing(5);
-        yläpalkki.setAlignment(Pos.CENTER_RIGHT);
-        Button btnNollaa = new Button("Palauta oletukset");
-        btnNollaa.setFont(BUTTON_FONT);
-        btnNollaa.setOnAction(_ -> palautaOletukset());
-        yläpalkki.getChildren().addAll(
-                new FontLabel(TEKSTI_FONT, "Muutokset astuvat voimaan sovelluksen käynnistyttyä uudelleen"),
-                btnNollaa);
-        root.setTop(yläpalkki);
-
-        root.setCenter(getFonttiAsetukset());
-
-    }
-
-    private Spinner<Integer> tekstiFonttiKoko, buttonFonttiKoko, tulostusFonttiKoko, insets;
-    private ComboBox<String> tekstiFontti, buttonFontti;
-
-    /**
-     * Luo paneelin fonttien asetusten säätämiseen
-     * @return paneeli
-     */
-    private GridPane getFonttiAsetukset() {
-        GridPane root = new GridPane();
-        root.setAlignment(Pos.CENTER);
-        root.setHgap(SPACING *2);
-        root.setVgap(SPACING *2);
-        root.setBorder(BORDER);
-        root.setPadding(INSETS);
-        int row = 0;
-
-        Label otsikko = new Label("Fonttien asetukset");
-        otsikko.setFont(TEKSTI_FONT);
-        root.addRow(row++, otsikko);
-
-        int minFonttiKoko = 8;
-        int maxFonttiKoko = 18;
-        int stepFontti = 1;
-        tekstiFonttiKoko = new Spinner<>(minFonttiKoko, maxFonttiKoko, Integer.parseInt(prop.getProperty("tekstiFonttiKoko")), stepFontti);
-        tekstiFonttiKoko.valueProperty().addListener((_) -> prop.setProperty("tekstiFonttiKoko", String.valueOf(tekstiFonttiKoko.getValue())));
-        Label tekstinKoko = new Label("Tekstin koko");
-        tekstinKoko.setFont(TEKSTI_FONT);
-        root.addRow(row++, tekstinKoko, tekstiFonttiKoko);
-
-        tekstiFontti = new ComboBox<>();
-        tekstiFontti.setItems(järjestelmäFontit);
-        tekstiFontti.getSelectionModel().select(prop.getProperty("tekstiFontti"));
-        tekstiFontti.valueProperty().addListener((_) -> prop.setProperty("tekstiFontti", tekstiFontti.getValue()));
-        Label tekstinFontti = new Label("Tekstin fontti");
-        tekstinFontti.setFont(TEKSTI_FONT);
-        root.addRow(row++, tekstinFontti, tekstiFontti);
-
-        buttonFonttiKoko = new Spinner<>(minFonttiKoko, maxFonttiKoko, Integer.parseInt(prop.getProperty("buttonFonttiKoko")), stepFontti);
-        buttonFonttiKoko.valueProperty().addListener((_) -> prop.setProperty("buttonFonttiKoko", String.valueOf(buttonFonttiKoko.getValue())));
-        Label nappiTekstinKoko = new Label("Painikkeiden koko");
-        nappiTekstinKoko.setFont(TEKSTI_FONT);
-        root.addRow(row++, nappiTekstinKoko, buttonFonttiKoko);
-
-        buttonFontti = new ComboBox<>();
-        buttonFontti.setItems(järjestelmäFontit);
-        buttonFontti.getSelectionModel().select(prop.getProperty("buttonFontti"));
-        buttonFontti.valueProperty().addListener((_) -> prop.setProperty("buttonFontti", buttonFontti.getValue()));
-        Label napinFontti = new Label("Painikkeiden fontti");
-        napinFontti.setFont(TEKSTI_FONT);
-        root.addRow(row++, napinFontti, buttonFontti);
-
-        tulostusFonttiKoko = new Spinner<>(minFonttiKoko, maxFonttiKoko, Integer.parseInt(prop.getProperty("tulostusFonttiKoko")), stepFontti);
-        tulostusFonttiKoko.valueProperty().addListener((_) -> prop.setProperty("tulostusFonttiKoko", String.valueOf(tulostusFonttiKoko.getValue())));
-        Label tulostusFontinKoko = new Label("Tulostusfontin koko");
-        tulostusFontinKoko.setFont(TEKSTI_FONT);
-        root.addRow(row++, tulostusFontinKoko, tulostusFonttiKoko);
-
-        insets = new Spinner<>(0, 10, Integer.parseInt(prop.getProperty("insets")), 1);
-        insets.valueProperty().addListener((_) -> prop.setProperty("insets", String.valueOf(insets.getValue())));
-        Label insetsi = new Label("Yleinen fyllinki");
-        insetsi.setFont(TEKSTI_FONT);
-        root.addRow(row++, insetsi, insets);
-
-        return root;
-    }
-
-    private void palautaOletukset() {
-        try {
-            Properties defaults = new Properties();
-            defaults.load(Main.class.getClassLoader().getResourceAsStream("defaults.properties"));
-
-            tekstiFonttiKoko.getValueFactory().setValue(Integer.valueOf(defaults.getProperty("tekstiFonttiKoko")));
-            tekstiFontti.getSelectionModel().select(defaults.getProperty("tekstiFontti"));
-            buttonFonttiKoko.getValueFactory().setValue(Integer.valueOf(defaults.getProperty("buttonFonttiKoko")));
-            buttonFontti.getSelectionModel().select(defaults.getProperty("buttonFontti"));
-            tulostusFonttiKoko.getValueFactory().setValue(Integer.valueOf(defaults.getProperty("tulostusFonttiKoko")));
-            insets.getValueFactory().setValue(Integer.valueOf(defaults.getProperty("insets")));
-        }
-        catch (IOException e) {
-            //noinspection CallToPrintStackTrace
-            e.printStackTrace();
-        }
-    }
-}

src/main/java/fi/lpam/gui/AsiakasIkkuna.java

@@ -6,14 +6,10 @@ import fi.lpam.gui.elementit.IntegerSpinner;
 import fi.lpam.gui.elementit.TabPohja;
 import fi.lpam.Main;
 import javafx.geometry.Insets;
-import javafx.geometry.Pos;
-import javafx.geometry.Side;
 import javafx.scene.Scene;
 import javafx.scene.control.*;
 import javafx.scene.image.Image;
 import javafx.scene.layout.GridPane;
-import javafx.scene.layout.HBox;
-import javafx.scene.layout.VBox;
 import javafx.stage.Stage;
 
 import java.util.Objects;
@@ -21,9 +17,8 @@ import java.util.Objects;
 
 public class AsiakasIkkuna extends Stage {
     final Asiakas käsiteltäväAsiakas;
-    final TextField tfNimi, tfOsoite, tfRuokarajoitteet, tfAvainTieto, tfLisätiedot, tfYhteyshenkilönNimi, tfLaskutusOsoite, tfYhteyshenkilönPuhelinnumero, tfYhteyshenkilönSähköposti;
+    final TextField tfNimi, tfOsoite, tfRuokarajoitteet, tfAvainTieto, tfLisätiedot;
-    final IntegerSpinner tfMaanantai, tfTiistai, tfKeskiviikko, tfTorstai, tfPerjantai, tfLauantai, tfSunnuntai, tfSalaatit, tfPääruoat, tfJälkiruoat;
+    final IntegerSpinner tfMaanantai, tfTiistai, tfKeskiviikko, tfTorstai, tfPerjantai, tfSalaatit, tfPääruoat, tfJälkiruoat;
-    final IntegerSpinner tfDinnerMaanantai, tfDinnerTiistai, tfDinnerKeskiviikko, tfDinnerTorstai, tfDinnerPerjantai, tfDinnerLauantai, tfDinnerSunnuntai, tfDinnerSalaatit, tfDinnerPääruoat, tfDinnerJälkiruoat;
 
     /**
      * Luo ikkunan jossa muokataan saatua asiakasta
@@ -31,186 +26,95 @@ public class AsiakasIkkuna extends Stage {
      */
     public AsiakasIkkuna(Asiakas käsiteltäväAsiakas) {
         this.käsiteltäväAsiakas = käsiteltäväAsiakas;
-        if (käsiteltäväAsiakas.getNimi() != null) {
+        this.setTitle("Asiakasikkuna");
-            this.setTitle(käsiteltäväAsiakas.getNimi());
-        }
-        else {
-            this.setTitle("Uusi asiakas");
-        }
 
-        TabPane tabPane = new TabPane();
+        GridPane root = new GridPane();
-        tabPane.setSide(Side.LEFT);
+        root.setPadding(new Insets(15));
-        int korkeus = 150;
+        root.setHgap(10);
-        tabPane.setTabMinWidth(korkeus);
+        root.setVgap(10);
-        tabPane.setTabMinWidth(korkeus);
+        root.setOnKeyPressed(e -> {
-        tabPane.setTabMinHeight(35);
-        tabPane.setTabMaxHeight(35);
-        tabPane.setOnKeyPressed(e -> {
             switch (e.getCode()) {
                 case ENTER: tallenna(); break;
                 case ESCAPE: close(); break;
                 default: break;
             }});
 
-        Tab tabLounastiedot, tabPäivällistiedot, tabAsiakastiedot;
+        root.addRow(root.getRowCount(), new Label("Vahvistetut kentät pakollisia"));
-        tabLounastiedot = new TabPohja("Kuljetustiedot");
-        tabPäivällistiedot = new TabPohja("Päivällistiedot");
-        tabAsiakastiedot = new TabPohja("Asiakastiedot");
-        tabPane.getTabs().addAll(tabLounastiedot, tabPäivällistiedot, tabAsiakastiedot);
-
-        GridPane lounasTiedot = new GridPane();
-        lounasTiedot.setPadding(new Insets(15));
-        lounasTiedot.setHgap(10);
-        lounasTiedot.setVgap(10);
-        tabLounastiedot.setContent(lounasTiedot);
-
-        lounasTiedot.addRow(lounasTiedot.getRowCount(), new Label("Lounaat per kuljetus"));
-
-        Label salaatit = new Label("Salaatit");
-        tfSalaatit = new IntegerSpinner();
-        lounasTiedot.addRow(lounasTiedot.getRowCount(), salaatit, tfSalaatit);
-
-        Label pääruoat = new Label("Pääruoat");
-        tfPääruoat = new IntegerSpinner();
-        lounasTiedot.addRow(lounasTiedot.getRowCount(), pääruoat, tfPääruoat);
-
-        Label jälkiruoat = new Label("Jälkiruoat");
-        tfJälkiruoat = new IntegerSpinner();
-        lounasTiedot.addRow(lounasTiedot.getRowCount(), jälkiruoat, tfJälkiruoat);
-
-        lounasTiedot.addRow(lounasTiedot.getRowCount(), new Label("Kuljetukset per päivä"));
-
-        Label maanantai = new Label("Maanantai");
-        tfMaanantai = new IntegerSpinner();
-        lounasTiedot.addRow(lounasTiedot.getRowCount(), maanantai, tfMaanantai);
-
-        Label tiistai = new Label("Tiistai");
-        tfTiistai = new IntegerSpinner();
-        lounasTiedot.addRow(lounasTiedot.getRowCount(), tiistai, tfTiistai);
-
-        Label keskiviikko = new Label("Keskiviikko");
-        tfKeskiviikko = new IntegerSpinner();
-        lounasTiedot.addRow(lounasTiedot.getRowCount(), keskiviikko, tfKeskiviikko);
-
-        Label torstai = new Label("Torstai");
-        tfTorstai = new IntegerSpinner();
-        lounasTiedot.addRow(lounasTiedot.getRowCount(), torstai, tfTorstai);
-
-        Label perjantai = new Label("Perjantai");
-        tfPerjantai = new IntegerSpinner();
-        lounasTiedot.addRow(lounasTiedot.getRowCount(), perjantai, tfPerjantai);
-
-        tfLauantai = new IntegerSpinner();
-        lounasTiedot.addRow(lounasTiedot.getRowCount(), new Label("Lauantai"), tfLauantai);
-
-        tfSunnuntai = new IntegerSpinner();
-        lounasTiedot.addRow(lounasTiedot.getRowCount(), new Label("Sunnuntai"), tfSunnuntai);
-
-        Label ruokarajoitteet = new Label("Ruokarajoitteet");
-        tfRuokarajoitteet = new TextField();
-        lounasTiedot.addRow(lounasTiedot.getRowCount(), ruokarajoitteet, tfRuokarajoitteet);
-
-        Label avainTieto = new Label("Avaintieto");
-        tfAvainTieto = new TextField();
-        lounasTiedot.addRow(lounasTiedot.getRowCount(), avainTieto, tfAvainTieto);
-
-        Label lisätiedot = new Label("Kuljetuksen lisätiedot");
-        tfLisätiedot = new TextField();
-        tfLisätiedot.setPromptText("Näkyy raporteilla");
-        lounasTiedot.addRow(lounasTiedot.getRowCount(), lisätiedot, tfLisätiedot);
-
-
-        GridPane päivällisTiedot = new GridPane();
-        päivällisTiedot.setPadding(new Insets(15));
-        päivällisTiedot.setHgap(10);
-        päivällisTiedot.setVgap(10);
-        tabPäivällistiedot.setContent(päivällisTiedot);
-
-        päivällisTiedot.addRow(päivällisTiedot.getRowCount(), new Label("Päivälliset per kuljetus"));
-
-        tfDinnerSalaatit = new IntegerSpinner();
-        päivällisTiedot.addRow(päivällisTiedot.getRowCount(), new Label("Salaatit"), tfDinnerSalaatit);
-
-        tfDinnerPääruoat = new IntegerSpinner();
-        päivällisTiedot.addRow(päivällisTiedot.getRowCount(), new Label("Pääruoat"), tfDinnerPääruoat);
-
-        tfDinnerJälkiruoat = new IntegerSpinner();
-        päivällisTiedot.addRow(päivällisTiedot.getRowCount(), new Label("Jälkiruoat"), tfDinnerJälkiruoat);
-
-        päivällisTiedot.addRow(päivällisTiedot.getRowCount(), new Label("Kuljetukset per päivä"));
-
-        tfDinnerMaanantai = new IntegerSpinner();
-        päivällisTiedot.addRow(päivällisTiedot.getRowCount(), new Label("Maanantai"), tfDinnerMaanantai);
-
-        tfDinnerTiistai = new IntegerSpinner();
-        päivällisTiedot.addRow(päivällisTiedot.getRowCount(), new Label("Tiistai"), tfDinnerTiistai);
-
-        tfDinnerKeskiviikko = new IntegerSpinner();
-        päivällisTiedot.addRow(päivällisTiedot.getRowCount(), new Label("Keskiviikko"), tfDinnerKeskiviikko);
-
-        tfDinnerTorstai = new IntegerSpinner();
-        päivällisTiedot.addRow(päivällisTiedot.getRowCount(), new Label("Torstai"), tfDinnerTorstai);
-
-        tfDinnerPerjantai = new IntegerSpinner();
-        päivällisTiedot.addRow(päivällisTiedot.getRowCount(), new Label("Perjantai"), tfDinnerPerjantai);
-
-        tfDinnerLauantai = new IntegerSpinner();
-        päivällisTiedot.addRow(päivällisTiedot.getRowCount(), new Label("Lauantai"), tfDinnerLauantai);
-
-        tfDinnerSunnuntai = new IntegerSpinner();
-        päivällisTiedot.addRow(päivällisTiedot.getRowCount(), new Label("Sunnuntai"), tfDinnerSunnuntai);
-
-        GridPane asiakasTiedot = new GridPane();
-        asiakasTiedot.setPadding(new Insets(15));
-        asiakasTiedot.setHgap(10);
-        asiakasTiedot.setVgap(10);
-        tabAsiakastiedot.setContent(asiakasTiedot);
 
         Label nimi = new Label("Asiakkaan etu- ja sukunimi");
         nimi.setStyle("-fx-font-weight: bold;");
         tfNimi = new TextField();
         tfNimi.setPromptText("Pakollinen tieto");
-        asiakasTiedot.addRow(asiakasTiedot.getRowCount(), nimi, tfNimi);
+        root.addRow(root.getRowCount(), nimi, tfNimi);
 
         Label osoite = new Label("Osoite");
         osoite.setStyle("-fx-font-weight: bold;");
         tfOsoite = new TextField();
         tfOsoite.setPromptText("Pakollinen tieto");
-        asiakasTiedot.addRow(asiakasTiedot.getRowCount(), osoite, tfOsoite);
+        root.addRow(root.getRowCount(), osoite, tfOsoite);
 
-        Label yhteyshenkilönNimi = new Label("Yhteyshenkilön etu- ja sukunimi");
+        root.addRow(root.getRowCount(), new Label("Määrät per kuljetus"));
-        tfYhteyshenkilönNimi = new TextField();
-        asiakasTiedot.addRow(asiakasTiedot.getRowCount(), yhteyshenkilönNimi, tfYhteyshenkilönNimi);
 
-        Label laskutusOsoite = new Label("Laskutusosoite");
+        Label salaatit = new Label("Salaatit");
-        tfLaskutusOsoite = new TextField();
+        tfSalaatit = new IntegerSpinner();
-        asiakasTiedot.addRow(asiakasTiedot.getRowCount(), laskutusOsoite, tfLaskutusOsoite);
+        root.addRow(root.getRowCount(), salaatit, tfSalaatit);
 
-        Label yhteyshenkilönPuhelinnumero = new Label("Puhelinnumero");
+        Label pääruoat = new Label("Pääruoat");
-        tfYhteyshenkilönPuhelinnumero = new TextField();
+        tfPääruoat = new IntegerSpinner();
-        asiakasTiedot.addRow(asiakasTiedot.getRowCount(), yhteyshenkilönPuhelinnumero, tfYhteyshenkilönPuhelinnumero);
+        root.addRow(root.getRowCount(), pääruoat, tfPääruoat);
 
-        Label yhteyshenkilönSähköposti = new Label("Sähköposti");
+        Label jälkiruoat = new Label("Jälkiruoat");
-        tfYhteyshenkilönSähköposti = new TextField();
+        tfJälkiruoat = new IntegerSpinner();
-        asiakasTiedot.addRow(asiakasTiedot.getRowCount(), yhteyshenkilönSähköposti, tfYhteyshenkilönSähköposti);
+        root.addRow(root.getRowCount(), jälkiruoat, tfJälkiruoat);
+
+        root.addRow(root.getRowCount(), new Label("Kuljetukset per päivä"));
+
+        Label maanantai = new Label("Maanantai");
+        tfMaanantai = new IntegerSpinner();
+        root.addRow(root.getRowCount(), maanantai, tfMaanantai);
+
+        Label tiistai = new Label("Tiistai");
+        tfTiistai = new IntegerSpinner();
+        root.addRow(root.getRowCount(), tiistai, tfTiistai);
+
+        Label keskiviikko = new Label("Keskiviikko");
+        tfKeskiviikko = new IntegerSpinner();
+        root.addRow(root.getRowCount(), keskiviikko, tfKeskiviikko);
+
+        Label torstai = new Label("Torstai");
+        tfTorstai = new IntegerSpinner();
+        root.addRow(root.getRowCount(), torstai, tfTorstai);
+
+        Label perjantai = new Label("Perjantai");
+        tfPerjantai = new IntegerSpinner();
+        root.addRow(root.getRowCount(), perjantai, tfPerjantai);
+
+        Label ruokarajoitteet = new Label("Ruokarajoitteet");
+        tfRuokarajoitteet = new TextField();
+        root.addRow(root.getRowCount(), ruokarajoitteet, tfRuokarajoitteet);
+
+        Label avainTieto = new Label("Avaintieto");
+        tfAvainTieto = new TextField();
+        root.addRow(root.getRowCount(), avainTieto, tfAvainTieto);
+
+        Label lisätiedot = new Label("Kuljetuksen lisätiedot");
+        tfLisätiedot = new TextField();
+        tfLisätiedot.setPromptText("Näkyy raporteilla");
+        root.addRow(root.getRowCount(), lisätiedot, tfLisätiedot);
 
-        HBox napit = new HBox();
         Button peruuta = new Button("Peruuta");
-        peruuta.setFont(TabPohja.BUTTON_FONT);
+        peruuta.setFont(TabPohja.buttonFont);
+        peruuta.setMinWidth(160);
         peruuta.setOnAction(_ -> this.close());
         Button tallenna = new Button("Tallenna");
-        tallenna.setFont(TabPohja.BUTTON_FONT);
+        tallenna.setFont(TabPohja.buttonFont);
+        tallenna.setMinWidth(160);
         tallenna.setOnAction(_ -> tallenna());
-        napit.getChildren().addAll(peruuta, tallenna);
+        root.addRow(root.getRowCount(), peruuta, tallenna);
-        napit.setAlignment(Pos.CENTER_RIGHT);
-        napit.setSpacing(10);
-        napit.setPadding(TabPohja.INSETS);
 
 
-        VBox root = new VBox();
-        root.getChildren().addAll(tabPane, napit);
         this.setScene(new Scene(root));
-        this.getIcons().add(new Image(Objects.requireNonNull(Main.class.getClassLoader().getResourceAsStream("kuljetusruokalaatikko.jpg"))));
+        this.getIcons().add(new Image(Objects.requireNonNull(Main.class.getResourceAsStream("kuljetusruokalaatikko.jpg"))));
         this.setAlwaysOnTop(true);
         this.setResizable(false);
 
@@ -232,28 +136,12 @@ public class AsiakasIkkuna extends Stage {
         tfKeskiviikko.getValueFactory().setValue(käsiteltäväAsiakas.getKeskiviikkoKpl());
         tfTorstai.getValueFactory().setValue(käsiteltäväAsiakas.getTorstaiKpl());
         tfPerjantai.getValueFactory().setValue(käsiteltäväAsiakas.getPerjantaiKpl());
-        tfLauantai.getValueFactory().setValue(käsiteltäväAsiakas.getLauantaiKpl());
-        tfSunnuntai.getValueFactory().setValue(käsiteltäväAsiakas.getSunnuntaiKpl());
         tfSalaatit.getValueFactory().setValue(käsiteltäväAsiakas.getSalaatit());
         tfPääruoat.getValueFactory().setValue(käsiteltäväAsiakas.getPääruoat());
         tfJälkiruoat.getValueFactory().setValue(käsiteltäväAsiakas.getJälkiruoat());
-        tfDinnerMaanantai.getValueFactory().setValue(käsiteltäväAsiakas.getDinnerMaanantaiKpl());
-        tfDinnerTiistai.getValueFactory().setValue(käsiteltäväAsiakas.getDinnerTiistaiKpl());
-        tfDinnerKeskiviikko.getValueFactory().setValue(käsiteltäväAsiakas.getDinnerKeskiviikkoKpl());
-        tfDinnerTorstai.getValueFactory().setValue(käsiteltäväAsiakas.getDinnerTorstaiKpl());
-        tfDinnerPerjantai.getValueFactory().setValue(käsiteltäväAsiakas.getDinnerPerjantaiKpl());
-        tfDinnerLauantai.getValueFactory().setValue(käsiteltäväAsiakas.getDinnerLauantaiKpl());
-        tfDinnerSunnuntai.getValueFactory().setValue(käsiteltäväAsiakas.getDinnerSunnuntaiKpl());
-        tfDinnerSalaatit.getValueFactory().setValue(käsiteltäväAsiakas.getDinnerSalaatit());
-        tfDinnerPääruoat.getValueFactory().setValue(käsiteltäväAsiakas.getDinnerPääruoat());
-        tfDinnerJälkiruoat.getValueFactory().setValue(käsiteltäväAsiakas.getDinnerJälkiruoat());
         tfRuokarajoitteet.setText(käsiteltäväAsiakas.getErityisruokavalio());
         tfAvainTieto.setText(käsiteltäväAsiakas.getAvainTieto());
         tfLisätiedot.setText(käsiteltäväAsiakas.getLisätiedot());
-        tfYhteyshenkilönNimi.setText(käsiteltäväAsiakas.getYhteyshenkilönNimi());
-        tfLaskutusOsoite.setText(käsiteltäväAsiakas.getLaskutusOsoite());
-        tfYhteyshenkilönPuhelinnumero.setText(käsiteltäväAsiakas.getYhteyshenkilönPuhelinnumero());
-        tfYhteyshenkilönSähköposti.setText(käsiteltäväAsiakas.getYhteyshenkilönSähköposti());
     }
 
     private void tallenna() {
@@ -277,28 +165,12 @@ public class AsiakasIkkuna extends Stage {
         käsiteltäväAsiakas.setKeskiviikkoKpl(tfKeskiviikko.getValue());
         käsiteltäväAsiakas.setTorstaiKpl(tfTorstai.getValue());
         käsiteltäväAsiakas.setPerjantaiKpl(tfPerjantai.getValue());
-        käsiteltäväAsiakas.setLauantaiKpl(tfLauantai.getValue());
-        käsiteltäväAsiakas.setSunnuntaiKpl(tfSunnuntai.getValue());
         käsiteltäväAsiakas.setSalaatit(tfSalaatit.getValue());
         käsiteltäväAsiakas.setPääruoat(tfPääruoat.getValue());
         käsiteltäväAsiakas.setJälkiruoat(tfJälkiruoat.getValue());
-        käsiteltäväAsiakas.setDinnerMaanantaiKpl(tfDinnerMaanantai.getValue());
-        käsiteltäväAsiakas.setDinnerTiistaiKpl(tfDinnerTiistai.getValue());
-        käsiteltäväAsiakas.setDinnerKeskiviikkoKpl(tfDinnerKeskiviikko.getValue());
-        käsiteltäväAsiakas.setDinnerTorstaiKpl(tfDinnerTorstai.getValue());
-        käsiteltäväAsiakas.setDinnerPerjantaiKpl(tfDinnerPerjantai.getValue());
-        käsiteltäväAsiakas.setDinnerLauantaiKpl(tfDinnerLauantai.getValue());
-        käsiteltäväAsiakas.setDinnerSunnuntaiKpl(tfDinnerSunnuntai.getValue());
-        käsiteltäväAsiakas.setDinnerSalaatit(tfDinnerSalaatit.getValue());
-        käsiteltäväAsiakas.setDinnerPääruoat(tfDinnerPääruoat.getValue());
-        käsiteltäväAsiakas.setDinnerJälkiruoat(tfDinnerJälkiruoat.getValue());
         käsiteltäväAsiakas.setErityisruokavalio(tfRuokarajoitteet.getText());
         käsiteltäväAsiakas.setAvainTieto(tfAvainTieto.getText());
         käsiteltäväAsiakas.setLisätiedot(tfLisätiedot.getText());
-        käsiteltäväAsiakas.setYhteyshenkilönNimi(tfYhteyshenkilönNimi.getText());
-        käsiteltäväAsiakas.setLaskutusOsoite(tfLaskutusOsoite.getText());
-        käsiteltäväAsiakas.setYhteyshenkilönPuhelinnumero(tfYhteyshenkilönPuhelinnumero.getText());
-        käsiteltäväAsiakas.setYhteyshenkilönSähköposti(tfYhteyshenkilönSähköposti.getText());
 
         boolean läpi = false;
         int maxYritykset = 3;
@@ -316,22 +188,14 @@ public class AsiakasIkkuna extends Stage {
             }
         }
 
-        if (läpi) {
-            int viikonKuljetukset =
-                    käsiteltäväAsiakas.getMaanantaiKpl() +
-                    käsiteltäväAsiakas.getTiistaiKpl() +
-                    käsiteltäväAsiakas.getKeskiviikkoKpl() +
-                    käsiteltäväAsiakas.getTorstaiKpl() +
-                    käsiteltäväAsiakas.getPerjantaiKpl() +
-                    käsiteltäväAsiakas.getLauantaiKpl() +
-                    käsiteltäväAsiakas.getSunnuntaiKpl();
 
+        if (läpi) {
+            int viikonKuljetukset = käsiteltäväAsiakas.getMaanantaiKpl() + käsiteltäväAsiakas.getTiistaiKpl() + käsiteltäväAsiakas.getKeskiviikkoKpl() + käsiteltäväAsiakas.getTorstaiKpl() + käsiteltäväAsiakas.getPerjantaiKpl();
             Alert alert = new Alert(Alert.AlertType.INFORMATION);
             alert.setTitle("Onnistuimme!");
             alert.setHeaderText("Asiakkaan tallentaminen onnistui!");
-            //TODO tähän myös dinneri määrät...
+            alert.setContentText(String.format("%s saa jatkossa yhteensä \n%s salaattia, %s pääruokaa ja %s jälkiruokaa viikossa",
-//            alert.setContentText(String.format("%s saa jatkossa yhteensä \n%s salaattia, %s pääruokaa ja %s jälkiruokaa viikossa",
+                    käsiteltäväAsiakas.getNimi(), käsiteltäväAsiakas.getSalaatit()*viikonKuljetukset, käsiteltäväAsiakas.getPääruoat()*viikonKuljetukset, käsiteltäväAsiakas.getPääruoat()*viikonKuljetukset));
-//                    käsiteltäväAsiakas.getNimi(), käsiteltäväAsiakas.getSalaatit()*viikonKuljetukset, käsiteltäväAsiakas.getPääruoat()*viikonKuljetukset, käsiteltäväAsiakas.getPääruoat()*viikonKuljetukset));
             alert.showAndWait();
             this.close();
         }

src/main/java/fi/lpam/gui/Asiakashallinta.java

@@ -1,23 +1,15 @@
 package fi.lpam.gui;
 
-import fi.lpam.Main;
 import fi.lpam.dataluokat.Asiakas;
-import fi.lpam.gui.elementit.FontLabel;
 import fi.lpam.gui.elementit.MaaraTableColumn;
 import fi.lpam.gui.elementit.TabPohja;
 import javafx.collections.FXCollections;
 import javafx.scene.control.*;
-import javafx.scene.control.Button;
-import javafx.scene.control.Label;
 import javafx.scene.control.cell.PropertyValueFactory;
 import javafx.scene.input.KeyCode;
 import javafx.scene.input.MouseEvent;
 import javafx.scene.layout.*;
 
-import java.awt.*;
-import java.io.IOException;
-import java.net.URI;
-import java.net.URISyntaxException;
 import java.util.Objects;
 import java.util.Optional;
 
@@ -27,9 +19,34 @@ public class Asiakashallinta extends TabPohja {
     final TableView<Asiakas> tvAsiakkaat = new TableView<>();
 
     public Asiakashallinta() {
-        super("Asiakashallinta");
+        BorderPane root = new BorderPane();
-        root.setTop(getYläpalkki());
+        this.getChildren().add(root);
+        root.setPadding(oletusInsets);
 
+        BorderPane yläpalkki = new BorderPane();
+        root.setTop(yläpalkki);
+        Label otsikko = new Label("Asiakashallinta");
+        otsikko.setFont(tekstiFont);
+        yläpalkki.setLeft(otsikko);
+
+        HBox yläpalkinNapit = new HBox();
+        yläpalkinNapit.setSpacing(5);
+        yläpalkki.setRight(yläpalkinNapit);
+
+        Button poistaValittu = new Button("Poista");
+        poistaValittu.setOnAction(_ -> poistaValittu());
+        poistaValittu.setFont(TabPohja.buttonFont);
+        yläpalkinNapit.getChildren().add(poistaValittu);
+
+        Button muokkaaAsiakasta = new Button("Muokkaa");
+        muokkaaAsiakasta.setOnAction(_ -> muokkaaValittua());
+        muokkaaAsiakasta.setFont(TabPohja.buttonFont);
+        yläpalkinNapit.getChildren().add(muokkaaAsiakasta);
+
+        Button uusiAsiakas = new Button("Uusi asiakas");
+        uusiAsiakas.setOnAction(_ -> uusiAsiakas());
+        uusiAsiakas.setFont(TabPohja.buttonFont);
+        yläpalkinNapit.getChildren().add(uusiAsiakas);
 
         tvAsiakkaat.setEditable(false);
         tvAsiakkaat.setPlaceholder(new Label("Ei vielä tietoja"));
@@ -48,107 +65,57 @@ public class Asiakashallinta extends TabPohja {
         });
         root.setCenter(tvAsiakkaat);
 
-        TableColumn<Asiakas, String> tcId = new TableColumn<>("Id");
-        tcId.setCellValueFactory(new PropertyValueFactory<>("id"));
         TableColumn<Asiakas, String> tcNimi = new TableColumn<>("Nimi");
+        tcNimi.setMinWidth(180);
         tcNimi.setCellValueFactory(new PropertyValueFactory<>("nimi"));
         TableColumn<Asiakas, String> tcOsoite = new TableColumn<>("Osoite");
+        tcOsoite.setMinWidth(250);
         tcOsoite.setCellValueFactory(new PropertyValueFactory<>("osoite"));
 
-        int minWidth = 40;
         TableColumn<Asiakas, MaaraTableColumn<Asiakas, Integer>> tcKuljetuspäivät = new TableColumn<>("Kuljetukset per päivä");
-        MaaraTableColumn<Asiakas, Integer> tcMaanantaiKpl = new MaaraTableColumn<>("MA", minWidth);
+        MaaraTableColumn<Asiakas, Integer> tcMaanantaiKpl = new MaaraTableColumn<>("MA", 20);
         tcMaanantaiKpl.setCellValueFactory(new PropertyValueFactory<>("maanantaiKpl"));
-        MaaraTableColumn<Asiakas, Integer> tcTiistaiKpl = new MaaraTableColumn<>("TI", minWidth);
+        MaaraTableColumn<Asiakas, Integer> tcTiistaiKpl = new MaaraTableColumn<>("TI", 20);
         tcTiistaiKpl.setCellValueFactory(new PropertyValueFactory<>("tiistaiKpl"));
-        MaaraTableColumn<Asiakas, Integer> tcKeskiviikkoKpl = new MaaraTableColumn<>("KE", minWidth);
+        MaaraTableColumn<Asiakas, Integer> tcKeskiviikkoKpl = new MaaraTableColumn<>("KE", 20);
         tcKeskiviikkoKpl.setCellValueFactory(new PropertyValueFactory<>("keskiviikkoKpl"));
-        MaaraTableColumn<Asiakas, Integer> tcTorstaiKpl = new MaaraTableColumn<>("TO", minWidth);
+        MaaraTableColumn<Asiakas, Integer> tcTorstaiKpl = new MaaraTableColumn<>("TO", 20);
         tcTorstaiKpl.setCellValueFactory(new PropertyValueFactory<>("torstaiKpl"));
-        MaaraTableColumn<Asiakas, Integer> tcPerjantaiKpl = new MaaraTableColumn<>("PE", minWidth);
+        MaaraTableColumn<Asiakas, Integer> tcPerjantaiKpl = new MaaraTableColumn<>("PE", 20);
         tcPerjantaiKpl.setCellValueFactory(new PropertyValueFactory<>("perjantaiKpl"));
-        MaaraTableColumn<Asiakas, Integer> tcLauantaiKpl = new MaaraTableColumn<>("LA", minWidth);
+        tcKuljetuspäivät.getColumns().addAll(tcMaanantaiKpl, tcTiistaiKpl, tcKeskiviikkoKpl, tcTorstaiKpl, tcPerjantaiKpl);
-        tcLauantaiKpl.setCellValueFactory(new PropertyValueFactory<>("lauantaiKpl"));
-        MaaraTableColumn<Asiakas, Integer> tcSunnuntaiKpl = new MaaraTableColumn<>("SU", minWidth);
-        tcSunnuntaiKpl.setCellValueFactory(new PropertyValueFactory<>("sunnuntaiKpl"));
-        tcKuljetuspäivät.getColumns().addAll(tcMaanantaiKpl, tcTiistaiKpl, tcKeskiviikkoKpl, tcTorstaiKpl, tcPerjantaiKpl, tcLauantaiKpl, tcSunnuntaiKpl);
 
         TableColumn<Asiakas, MaaraTableColumn<Asiakas, Integer>> tcKuljetusmäärät = new TableColumn<>("Määrät per kuljetus");
-        MaaraTableColumn<Asiakas, Integer> tcSalaatit = new MaaraTableColumn<>("Salaatit", minWidth, 150);
+        MaaraTableColumn<Asiakas, Integer> tcSalaatit = new MaaraTableColumn<>("Salaatit", 90);
         tcSalaatit.setCellValueFactory(new PropertyValueFactory<>("salaatit"));
-        MaaraTableColumn<Asiakas, Integer> tcPääruoat = new MaaraTableColumn<>("Pääruoat", minWidth, 150);
+        MaaraTableColumn<Asiakas, Integer> tcPääruoat = new MaaraTableColumn<>("Pääruoat", 90);
         tcPääruoat.setCellValueFactory(new PropertyValueFactory<>("pääruoat"));
-        MaaraTableColumn<Asiakas, Integer> tcJälkiruoat = new MaaraTableColumn<>("Jälkiruoat", minWidth, 150);
+        MaaraTableColumn<Asiakas, Integer> tcJälkiruoat = new MaaraTableColumn<>("Jälkiruoat", 90);
         tcJälkiruoat.setCellValueFactory(new PropertyValueFactory<>("jälkiruoat"));
         tcKuljetusmäärät.getColumns().addAll(tcSalaatit, tcPääruoat, tcJälkiruoat);
 
         TableColumn<Asiakas, String> tcErityisruokavalio = new TableColumn<>("Ruokarajoitteet");
+        tcErityisruokavalio.setMinWidth(150);
         tcErityisruokavalio.setCellValueFactory(new PropertyValueFactory<>("erityisruokavalio"));
         TableColumn<Asiakas, String> tcAvainTieto = new TableColumn<>("Avaintieto");
+        tcAvainTieto.setMinWidth(150);
         tcAvainTieto.setCellValueFactory(new PropertyValueFactory<>("avainTieto"));
         TableColumn<Asiakas, String> tcLisätiedot = new TableColumn<>("Lisätieto kuljetukselle");
+        tcLisätiedot.setMinWidth(250);
         tcLisätiedot.setCellValueFactory(new PropertyValueFactory<>("lisätiedot"));
-        TableColumn<Asiakas, String> tcYhteyshenkilönNimi = new TableColumn<>("Yhteyshenkilö");
-        tcYhteyshenkilönNimi.setCellValueFactory(new PropertyValueFactory<>("yhteyshenkilönNimi"));
-        TableColumn<Asiakas, String> tcLaskutusOsoite = new TableColumn<>("Laskutusosoite");
-        tcLaskutusOsoite.setCellValueFactory(new PropertyValueFactory<>("laskutusOsoite"));
-        TableColumn<Asiakas, String> tcYhteyshenkilönPuhelinnumero = new TableColumn<>("Yhteyshenkilön puhelinnumero");
-        tcYhteyshenkilönPuhelinnumero.setCellValueFactory(new PropertyValueFactory<>("yhteyshenkilönPuhelinnumero"));
-        TableColumn<Asiakas, String> tcYhteyshenkilönSähköposti = new TableColumn<>("Yhteyshenkilön sähköposti");
-        tcYhteyshenkilönSähköposti.setCellValueFactory(new PropertyValueFactory<>("yhteyshenkilönSähköposti"));
-
-        tvAsiakkaat.getColumns().addAll(tcId, tcNimi, tcOsoite, tcKuljetuspäivät, tcKuljetusmäärät, tcErityisruokavalio, tcAvainTieto, tcLisätiedot, tcYhteyshenkilönNimi, tcLaskutusOsoite, tcYhteyshenkilönPuhelinnumero, tcYhteyshenkilönSähköposti);
 
+        tvAsiakkaat.getColumns().addAll(tcNimi, tcOsoite, tcKuljetuspäivät, tcKuljetusmäärät, tcErityisruokavalio, tcAvainTieto, tcLisätiedot);
         for (TableColumn<Asiakas, ?> column : tvAsiakkaat.getColumns()) {
-            column.getColumns().forEach(this::määritäColumn);
+            column.setEditable(false);
-            määritäColumn(column);
+            column.setReorderable(false);
+            column.setResizable(false);
+            column.setStyle("-fx-alignment: CENTER; -fx-font-size: 16px;");
+
         }
 
         tvAsiakkaat.setItems(FXCollections.observableArrayList(Objects.requireNonNull(Asiakas.haeKaikki())));
-        //tvAsiakkaat.setPrefSize(1920, 1080); //Tämä estää oudon bugin? (asiakkaiden ollessa 31 TV sekoaa)
     }
 
-    private BorderPane getYläpalkki() {
-        BorderPane yläpalkki = new BorderPane();
-        Label otsikko = new FontLabel(TEKSTI_FONT, "Asiakashallinta");
-        yläpalkki.setLeft(otsikko);
-
-        HBox yläpalkinNapit = new HBox();
-        yläpalkinNapit.setSpacing(SPACING);
-        yläpalkinNapit.setPadding(INSETS);
-        yläpalkki.setRight(yläpalkinNapit);
-
-        Button avaaVersioloki = new Button("VERSIOLOKI");
-        avaaVersioloki.setFont(BUTTON_FONT);
-        avaaVersioloki.setOnAction(_ -> {
-            try {
-                Desktop.getDesktop().browse(new URI("https://git.lpam.fi/laurimaaninka/Ruokamanageri/src/branch/master/CHANGELOG.md"));
-            } catch (IOException | URISyntaxException e) {
-                System.out.println(e.getMessage());
-            }
-        });
-        yläpalkinNapit.getChildren().add(avaaVersioloki);
-
-        Button poistaValittu = new Button("Poista");
-        poistaValittu.setOnAction(_ -> poistaValittu());
-        poistaValittu.setFont(BUTTON_FONT);
-        yläpalkinNapit.getChildren().add(poistaValittu);
-
-        Button muokkaaAsiakasta = new Button("Muokkaa");
-        muokkaaAsiakasta.setOnAction(_ -> muokkaaValittua());
-        muokkaaAsiakasta.setFont(BUTTON_FONT);
-        yläpalkinNapit.getChildren().add(muokkaaAsiakasta);
-
-        Button uusiAsiakas = new Button("Uusi asiakas");
-        uusiAsiakas.setOnAction(_ -> uusiAsiakas());
-        uusiAsiakas.setFont(BUTTON_FONT);
-        yläpalkinNapit.getChildren().add(uusiAsiakas);
-
-        return yläpalkki;
-    }
-
-
-
     private void uusiAsiakas() {
         Asiakas luotavaAsiakas = new Asiakas();
         AsiakasIkkuna asiakasIkkuna = new AsiakasIkkuna(luotavaAsiakas);
@@ -186,15 +153,4 @@ public class Asiakashallinta extends TabPohja {
             }
         }
     }
-
-    private void määritäColumn(TableColumn<Asiakas, ?> column) {
-        column.setEditable(false);
-        column.setReorderable(false);
-        column.setResizable(true);
-        column.setStyle(
-                "-fx-alignment: CENTER;" +
-                "-fx-font-size: " + Main.properties.getProperty("tekstiFonttiKoko") + "px;" +
-                "-fx-font-family: " + Main.properties.getProperty("tekstiFontti")
-        );
-    }
 }

src/main/java/fi/lpam/gui/Kierroshallinta.java

@@ -4,8 +4,7 @@ import com.dlsc.gemsfx.MultiColumnListView;
 
 import fi.lpam.dataluokat.Asiakas;
 import fi.lpam.dataluokat.Kierros;
-import fi.lpam.gui.elementit.AsiakasMCLVListCell;
+import fi.lpam.gui.elementit.AsiakasListCell;
-import fi.lpam.gui.elementit.FontLabel;
 import fi.lpam.gui.elementit.TabPohja;
 
 import javafx.collections.FXCollections;
@@ -23,13 +22,15 @@ import java.util.Objects;
 public class Kierroshallinta extends TabPohja {
     private final MultiColumnListView<Asiakas> mclv = new MultiColumnListView<>();
     private final ArrayList<Kierros> kierrokset = Kierros.haeKaikki();
-    private ArrayList<Asiakas> kaikkiAsiakkaat = Asiakas.haeKaikki();
+    private final ArrayList<Asiakas> kaikkiAsiakkaat = Asiakas.haeKaikki();
 
     public Kierroshallinta() {
-        super("Kierroshallinta");
+        BorderPane root = new BorderPane();
+        this.getChildren().add(root);
+        root.setPadding(oletusInsets);
         root.setCenter(mclv);
 
-        mclv.setCellFactory(_ -> new AsiakasMCLVListCell(mclv));
+        mclv.setCellFactory(_ -> new AsiakasListCell(mclv));
         mclv.setPlaceholderFrom(new Asiakas(""));
         mclv.setPlaceholderTo(new Asiakas(""));
         rakennaSarakkeet();
@@ -37,7 +38,8 @@ public class Kierroshallinta extends TabPohja {
         BorderPane yläpalkki = new BorderPane();
         root.setTop(yläpalkki);
 
-        Label infoTeksti = new FontLabel(TEKSTI_FONT, "Siirrä asiakkaat haluamallesi kierrokselle. Kierroksia voit lisätä painamalla lisää uusi kierros painiketta.\nKierrosnumerot päivittyvät, sekä tyhjät poistuvat kierroksia tallentaessa.");
+        Label infoTeksti = new Label("Siirrä asiakkaat haluamallesi kierrokselle. Kierroksia voit lisätä painamalla lisää uusi kierros painiketta.\nKierrosnumerot päivittyvät, sekä tyhjät poistuvat kierroksia tallentaessa.");
+        infoTeksti.setFont(tekstiFont);
         yläpalkki.setLeft(infoTeksti);
 
         HBox yläpalkinNapit = new HBox();
@@ -45,15 +47,12 @@ public class Kierroshallinta extends TabPohja {
         yläpalkki.setRight(yläpalkinNapit);
 
         Button lisääKierros = new Button("Lisää uusi kierros");
-        lisääKierros.setOnAction(_ -> {
+        lisääKierros.setOnAction(_ -> lisaaKierros());
-            int uudenKierroksenNumero = mclv.getColumns().size();
+        lisääKierros.setFont(TabPohja.buttonFont);
-            lisaaKierros(new Kierros(uudenKierroksenNumero, FXCollections.observableArrayList(new ArrayList<>())));
-        });
-        lisääKierros.setFont(BUTTON_FONT);
         yläpalkinNapit.getChildren().add(lisääKierros);
 
         Button tallenna = new Button("Tallenna kierrokset");
-        tallenna.setFont(BUTTON_FONT);
+        tallenna.setFont(TabPohja.buttonFont);
         tallenna.setOnAction(_ ->tallennaKierrokset());
         yläpalkinNapit.getChildren().add(tallenna);
 
@@ -64,16 +63,18 @@ public class Kierroshallinta extends TabPohja {
 
     private void rakennaSarakkeet() {
         mclv.getColumns().clear();
-        kaikkiAsiakkaat = Asiakas.haeKaikki();
         ObservableList<Asiakas> eiKierroksella = FXCollections.observableArrayList(new ArrayList<>());
         HashSet<Integer> kierroksellaOlevatAsiakasIDt = new HashSet<>();
 
         if (!kierrokset.isEmpty()) {
-            for (Kierros kierros : kierrokset) {
+            for (Kierros k : kierrokset) {
-                lisaaKierros(kierros);
+                MultiColumnListView.ListViewColumn<Asiakas> sarake = new MultiColumnListView.ListViewColumn<>();
-                for (Asiakas asiakas : kierros.getAsiakkaat()) {
+                sarake.setHeader(new Label("Kierros " + k.getKierrosNumero()));
+                sarake.setItems(k.getAsiakkaat());
+                for (Asiakas asiakas : k.getAsiakkaat()) {
                     kierroksellaOlevatAsiakasIDt.add(asiakas.getId());
                 }
+                mclv.getColumns().add(sarake);
             }
         }
 
@@ -83,18 +84,19 @@ public class Kierroshallinta extends TabPohja {
             }
         }
         MultiColumnListView.ListViewColumn<Asiakas> colEiKierroksella = new MultiColumnListView.ListViewColumn<>();
-        Label labelEiKierroksella = new FontLabel(TEKSTI_FONT, "Ei vielä kierroksella");
+        colEiKierroksella.setHeader(new Label("Ei vielä kierroksella"));
-        colEiKierroksella.setHeader(labelEiKierroksella);
         colEiKierroksella.setItems(FXCollections.observableArrayList(eiKierroksella));
         mclv.getColumns().add(colEiKierroksella);
     }
 
-    private void lisaaKierros(Kierros kierros) {
+    private void lisaaKierros() {
-        if (!kierrokset.contains(kierros)) kierrokset.add(kierros);
+        int uudenKierroksenNumero = mclv.getColumns().size();
+        Kierros lisättäväKierros = new Kierros(uudenKierroksenNumero, FXCollections.observableArrayList(new ArrayList<>()));
+        kierrokset.add(lisättäväKierros);
         MultiColumnListView.ListViewColumn<Asiakas> uusiSarake = new MultiColumnListView.ListViewColumn<>();
-        uusiSarake.setHeader(new FontLabel(TEKSTI_FONT, "Kierros " + kierros.getKierrosNumero()));
+        uusiSarake.setHeader(new Label("Kierros " + uudenKierroksenNumero));
-        uusiSarake.setItems(kierros.getAsiakkaat());
+        uusiSarake.setItems(lisättäväKierros.getAsiakkaat());
-        mclv.getColumns().add(kierros.getKierrosNumero()-1, uusiSarake);
+        mclv.getColumns().add(uudenKierroksenNumero-1, uusiSarake);
     }
 
     private void tallennaKierrokset() {
@@ -102,7 +104,6 @@ public class Kierroshallinta extends TabPohja {
         int index = 0;
         for (int i = 0; i < kierrokset.size();) {
             Kierros k = kierrokset.get(i);
-            k.setAsiakkaat(FXCollections.observableArrayList(new ArrayList<>()));
             k.setAsiakkaat(mclv.getColumns().get(index).getItems());
             index++;
 
@@ -125,9 +126,4 @@ public class Kierroshallinta extends TabPohja {
         info.setContentText(null);
         info.showAndWait();
     }
-
-    @Override
-    public void päivitä() {
-        rakennaSarakkeet();
-    }
 }

src/main/java/fi/lpam/gui/KuljetusListat.java

@@ -1,12 +1,9 @@
 package fi.lpam.gui;
 
-import fi.lpam.Main;
 import fi.lpam.dataluokat.Asiakas;
 import fi.lpam.dataluokat.Kierros;
 import fi.lpam.dataluokat.Kuljetus;
-import fi.lpam.gui.elementit.FontLabel;
+import fi.lpam.gui.elementit.KuljetaTableCell;
-import fi.lpam.gui.tableCell.IntegerFieldTableCellFactory;
-import fi.lpam.gui.tableCell.KuljetaTableCell;
 import fi.lpam.tulostajat.KuljetusListaTulostaja;
 import fi.lpam.gui.elementit.MaaraTableColumn;
 import fi.lpam.gui.elementit.TabPohja;
@@ -16,13 +13,17 @@ import javafx.collections.FXCollections;
 import javafx.collections.ObservableList;
 import javafx.geometry.Pos;
 import javafx.scene.control.*;
+import javafx.scene.control.cell.ChoiceBoxTableCell;
 import javafx.scene.control.cell.PropertyValueFactory;
 import javafx.scene.control.cell.TextFieldTableCell;
 import javafx.scene.layout.BorderPane;
 import javafx.scene.layout.HBox;
 
 import java.time.LocalDate;
-import java.util.*;
+import java.util.ArrayList;
+import java.util.HashMap;
+import java.util.HashSet;
+import java.util.Objects;
 
 
 @SuppressWarnings("unchecked")
@@ -30,14 +31,11 @@ public class KuljetusListat extends TabPohja {
     private final TableView<Kuljetus> tableView = new TableView<>();
     private final DatePicker datePicker = new DatePicker(LocalDate.now());
     private final ComboBox<Integer> kierrosValinta = new ComboBox<>(FXCollections.observableArrayList(Kierros.haeKierrosNumerot()));
-    private final ComboBox<Kuljetus.Tyyppi> tyyppiValinta = new ComboBox<>(FXCollections.observableArrayList(Kuljetus.Tyyppi.values()));
     public static Label tallennusStatus;
     private final Button btnTulosta;
     private final HashMap<Integer, Asiakas> asiakkaat = new HashMap<>();
 
     public KuljetusListat() {
-        super("Kuljetuslistat");
-
         ObservableList<Integer> määräVaihtoehdot = FXCollections.observableArrayList();
         for (int i = 0; i <= 10; i++) {
             määräVaihtoehdot.add(i);}
@@ -46,47 +44,51 @@ public class KuljetusListat extends TabPohja {
             asiakkaat.put(asiakas.getId(), asiakas);
         }
 
+        BorderPane root = new BorderPane();
+        root.setPadding(oletusInsets);
+        this.getChildren().add(root);
+
         BorderPane yläpalkki =  new BorderPane();
-        Label infoTeksti = new FontLabel(TEKSTI_FONT, "Kuljetuslistojen luonti");
+        Label infoTeksti = new Label("Kuljetuslistojen luonti");
+        infoTeksti.setFont(tekstiFont);
         yläpalkki.setLeft(infoTeksti);
 
         HBox kierroksenValintaLaatikko = new HBox();
-        kierroksenValintaLaatikko.setSpacing(SPACING);
+        kierroksenValintaLaatikko.setSpacing(5);
+        kierroksenValintaLaatikko.setSpacing(5);
         kierroksenValintaLaatikko.setAlignment(Pos.CENTER);
         Button valitse = new Button("Valitse");
-        valitse.setFont(BUTTON_FONT);
+        valitse.setFont(buttonFont);
         valitse.setOnAction(_ -> päivitäListanäkymä());
-        tallennusStatus = new FontLabel(TEKSTI_FONT, "Tallennettu: Kyllä");
+        tallennusStatus = new Label("Tallennettu: Kyllä");
-        tallennusStatus.setFont(TEKSTI_FONT);
+        tallennusStatus.setFont(tekstiFont);
         datePicker.valueProperty().addListener(_ -> päivitäListanäkymä());
         kierrosValinta.valueProperty().addListener(_ -> päivitäListanäkymä());
-        tyyppiValinta.getSelectionModel().select(0);
-        tyyppiValinta.valueProperty().addListener(_ -> päivitäListanäkymä());
         kierroksenValintaLaatikko.getChildren().addAll(
-                new FontLabel(TEKSTI_FONT, "Päivämäärä:"),
+                new Label("Valitse päivämäärä:"),
                 datePicker,
-                new FontLabel(TEKSTI_FONT, "Kierros:"),
+                new Label("Valitse kierros:"),
-                kierrosValinta, tyyppiValinta, tallennusStatus);
+                kierrosValinta, tallennusStatus);
         yläpalkki.setCenter(kierroksenValintaLaatikko);
 
         HBox napit = new HBox();
-        napit.setSpacing(SPACING);
+        napit.setSpacing(5);
         Button poista = new Button("Poista kuljetusrivi");
-        poista.setOnAction(_ -> poistaRivi());
+        poista.setOnAction(_ ->poistaRivi());
-        poista.setFont(BUTTON_FONT);
+        poista.setFont(buttonFont);
         Button tallenna = new Button("Tallenna");
-        tallenna.setFont(BUTTON_FONT);
+        tallenna.setFont(buttonFont);
         tallenna.setOnAction(_ -> tallennaKierros());
         btnTulosta = new Button("Tulosta lista");
         btnTulosta.setDisable(true);
-        btnTulosta.setFont(BUTTON_FONT);
+        btnTulosta.setFont(buttonFont);
-        btnTulosta.setOnAction(_ -> tulostaLista());
+        btnTulosta.setOnAction(_ ->tulostaLista());
         napit.getChildren().addAll(poista, tallenna, btnTulosta);
         yläpalkki.setRight(napit);
         root.setTop(yläpalkki);
 
         tableView.setEditable(true);
-        tableView.setPlaceholder(new FontLabel(BUTTON_FONT, "Valitse kierros aloittaaksesi"));
+        tableView.setPlaceholder(new Label("Valitse kierros aloittaaksesi"));
         root.setCenter(tableView);
 
         TableColumn<Kuljetus, Boolean> tcNappi = new TableColumn<>("Kuljeta");
@@ -95,8 +97,10 @@ public class KuljetusListat extends TabPohja {
         tcNappi.setCellValueFactory(features -> new SimpleBooleanProperty(features.getValue() != null));
 
         TableColumn<Kuljetus, String> tcNimi =  new TableColumn<>("Nimi");
-        tcNimi.setMinWidth(200);
+        tcNimi.setMinWidth(150);
+        tcNimi.setCellFactory(TextFieldTableCell.forTableColumn());
         tcNimi.setCellValueFactory(new PropertyValueFactory<>("nimi"));
+        tcNimi.setOnEditCommit((TableColumn.CellEditEvent<Kuljetus, String> event) -> {event.getTableView().getItems().get(event.getTablePosition().getRow()).setNimi(event.getNewValue()); tallennusStatus.setText("Tallennettu: Ei");});
 
         TableColumn<Kuljetus, String> tcOsoite =  new TableColumn<>("Osoite");
         tcOsoite.setMinWidth(250);
@@ -105,17 +109,17 @@ public class KuljetusListat extends TabPohja {
         tcOsoite.setOnEditCommit((TableColumn.CellEditEvent<Kuljetus, String> event) -> {event.getTableView().getItems().get(event.getTablePosition().getRow()).setOsoite(event.getNewValue()); tallennusStatus.setText("Tallennettu: Ei");});
 
         TableColumn<Kuljetus, Integer> tcSalaatit = new MaaraTableColumn<>("Salaatit", 100);
-        tcSalaatit.setCellFactory(new IntegerFieldTableCellFactory<>());
+        tcSalaatit.setCellFactory(ChoiceBoxTableCell.forTableColumn(määräVaihtoehdot));
         tcSalaatit.setCellValueFactory(new PropertyValueFactory<>("salaatit"));
         tcSalaatit.setOnEditCommit((TableColumn.CellEditEvent<Kuljetus, Integer> event) -> {event.getTableView().getItems().get(event.getTablePosition().getRow()).setSalaatit(event.getNewValue()); tallennusStatus.setText("Tallennettu: Ei");});
 
         TableColumn<Kuljetus, Integer> tcPääruoat = new MaaraTableColumn<>("Pääruoat", 100);
-        tcPääruoat.setCellFactory(new IntegerFieldTableCellFactory<>());
+        tcPääruoat.setCellFactory(ChoiceBoxTableCell.forTableColumn(määräVaihtoehdot));
         tcPääruoat.setCellValueFactory(new PropertyValueFactory<>("pääruoat"));
         tcPääruoat.setOnEditCommit((TableColumn.CellEditEvent<Kuljetus, Integer> event) -> {event.getTableView().getItems().get(event.getTablePosition().getRow()).setPääruoat(event.getNewValue()); tallennusStatus.setText("Tallennettu: Ei");});
 
         TableColumn<Kuljetus, Integer> tcJälkiruoat = new MaaraTableColumn<>("Jälkiruoat", 100);
-        tcJälkiruoat.setCellFactory(new IntegerFieldTableCellFactory<>());
+        tcJälkiruoat.setCellFactory(ChoiceBoxTableCell.forTableColumn(määräVaihtoehdot));
         tcJälkiruoat.setCellValueFactory(new PropertyValueFactory<>("jälkiruoat"));
         tcJälkiruoat.setOnEditCommit((TableColumn.CellEditEvent<Kuljetus, Integer> event) -> {event.getTableView().getItems().get(event.getTablePosition().getRow()).setJälkiruoat(event.getNewValue()); tallennusStatus.setText("Tallennettu: Ei");});
 
@@ -143,26 +147,23 @@ public class KuljetusListat extends TabPohja {
             sarake.setSortable(false);
             sarake.setEditable(true);
             sarake.setResizable(true);
-            sarake.setStyle(
+            sarake.setStyle("-fx-alignment: CENTER; -fx-font-size: 16px;");
-                    "-fx-alignment: CENTER;" +
-                    "-fx-font-size: " + Main.properties.getProperty("tekstiFonttiKoko") + "px;" +
-                    "-fx-font-family: " + Main.properties.getProperty("tekstiFontti"));
         }
         tcNimi.setEditable(false);
+        tcNimi.setResizable(false);
     }
 
     private void päivitäListanäkymä() {
         LocalDate valittuPäivä = datePicker.getValue();
         int valittuKierros = kierrosValinta.getValue();
-        Kuljetus.Tyyppi kuljetusTyyppi = tyyppiValinta.getValue();
 
-        ArrayList<Kuljetus> luetutKuljetuksetKierrokselle = Kuljetus.haeKierroksenKuljetukset(valittuPäivä, valittuKierros, kuljetusTyyppi);
+        ArrayList<Kuljetus> luetutKuljetuksetKierrokselle = Kuljetus.haePäivänKuljetuksetKierroksella(valittuPäivä, valittuKierros);
         ObservableList<Asiakas> kierroksenAsiakkaat = Objects.requireNonNull(Kierros.haeKierros(valittuKierros)).getAsiakkaat();
         ArrayList<Kuljetus> näytettävätKuljetukset = new ArrayList<>();
         HashSet<Integer> näytettävätAsiakasIDt = new HashSet<>();
         //Luodaan kierroksen kuljetusdata kierrosjärjestyksen mukaan
         for (Asiakas asiakas : kierroksenAsiakkaat) {
-            näytettävätKuljetukset.add(new Kuljetus(asiakas, valittuPäivä, valittuKierros, kuljetusTyyppi));
+            näytettävätKuljetukset.add(new Kuljetus(asiakas, valittuPäivä, valittuKierros));
             näytettävätAsiakasIDt.add(asiakas.getId());
         }
         //Yhdistetään tietokannan data järjestys säilyttäen
@@ -216,11 +217,4 @@ public class KuljetusListat extends TabPohja {
     private void tulostaLista() {
         new KuljetusListaTulostaja().luoKuljetusLista(tableView.getItems(), kierrosValinta.getValue());
     }
-
-    @Override
-    public void päivitä() {
-        if (kierrosValinta.getValue() == null) return;
-        kierrosValinta.setItems(FXCollections.observableArrayList(Kierros.haeKierrosNumerot()));
-        päivitäListanäkymä();
-    }
 }

src/main/java/fi/lpam/gui/KuljetusRaportit.java

@@ -1,11 +1,8 @@
 package fi.lpam.gui;
 
-import fi.lpam.Main;
-import fi.lpam.dataluokat.Kuljetus;
 import fi.lpam.dataluokat.RaporttiRivi;
-import fi.lpam.gui.elementit.FontLabel;
 import fi.lpam.tulostajat.KuljetusRaporttiTulostaja;
-import fi.lpam.gui.tableCell.PaivamaaraTableCell;
+import fi.lpam.gui.elementit.PaivamaaraTableCell;
 import fi.lpam.gui.elementit.TabPohja;
 import javafx.collections.FXCollections;
 import javafx.geometry.Pos;
@@ -37,48 +34,43 @@ public class KuljetusRaportit extends TabPohja {
     }
 
     public KuljetusRaportit() {
-        super("Raportit");
+        BorderPane root = new BorderPane();
+        root.setPadding(oletusInsets);
+        this.getChildren().add(root);
 
         BorderPane yläpalkki =  new BorderPane();
         root.setTop(yläpalkki);
 
-        Label infoTeksti = new FontLabel(TEKSTI_FONT, "Kuljetusraportit. \nTarkka-valinnalla koostetun raportin \ntulostaminen paperille ei suositeltua.");
+        Label infoTeksti = new Label("Kuljetusraportit. \nTarkka-valinnalla koostetun raportin tulostaminen paperille ei suositeltua.");
+        infoTeksti.setFont(tekstiFont);
         yläpalkki.setLeft(infoTeksti);
 
         HBox yläpalkinNapit = new HBox();
         yläpalkinNapit.setAlignment(Pos.CENTER);
-        yläpalkinNapit.setSpacing(SPACING);
+        yläpalkinNapit.setSpacing(5);
         yläpalkki.setRight(yläpalkinNapit);
 
         tarkka = new CheckBox("Tarkka");
-        tarkka.setFont(TEKSTI_FONT);
         alkuPvm = new DatePicker(LocalDate.now().minusDays(LocalDate.now().getDayOfMonth() - 1));
         loppuPvm = new DatePicker(LocalDate.now());
         Button haeKuljetukset = new Button("Hae");
-        haeKuljetukset.setFont(BUTTON_FONT);
+        haeKuljetukset.setFont(buttonFont);
         haeKuljetukset.setOnAction(_ ->haeRaportti());
         Button tulostaRaportti = new Button("Tulosta raportti");
-        tulostaRaportti.setFont(BUTTON_FONT);
+        tulostaRaportti.setFont(buttonFont);
         tulostaRaportti.setOnAction(_ ->tulostaRaportti());
-        yläpalkinNapit.getChildren().addAll(tarkka,
+        yläpalkinNapit.getChildren().addAll(tarkka, new Label("Hae kuljetukset välillä:"), alkuPvm, new Label("-"), loppuPvm, haeKuljetukset, tulostaRaportti);
-                //new FontLabel(tekstiFont,"Hae kuljetukset välillä:"),
-                alkuPvm, new FontLabel(TEKSTI_FONT,"-"), loppuPvm,
-                haeKuljetukset, tulostaRaportti);
 
         root.setCenter(tableView);
 
         TableColumn<RaporttiRivi, String> tcNimi =  new TableColumn<>("Nimi");
-        tcNimi.setMinWidth(200);
+        tcNimi.setMinWidth(150);
         tcNimi.setCellValueFactory(new PropertyValueFactory<>("nimi"));
 
         TableColumn<RaporttiRivi, LocalDate> tcPvm =  new TableColumn<>("Päivämäärä");
         tcPvm.setMinWidth(150);
         tcPvm.setCellFactory(_ ->new PaivamaaraTableCell());
-        tcPvm.setCellValueFactory(new PropertyValueFactory<>("pvm"));
+        tcPvm.setCellValueFactory(new PropertyValueFactory<>("loppuPvm"));
-
-        TableColumn<RaporttiRivi, Kuljetus.Tyyppi> tcTyyppi = new TableColumn<>("Tyyppi");
-        tcTyyppi.setMinWidth(150);
-        tcTyyppi.setCellValueFactory(new PropertyValueFactory<>("tyyppi"));
 
         TableColumn<RaporttiRivi, Integer> tcSalaatit =  new TableColumn<>("Salaatit");
         tcSalaatit.setMinWidth(100);
@@ -92,42 +84,17 @@ public class KuljetusRaportit extends TabPohja {
         tcJälkiruoat.setMinWidth(100);
         tcJälkiruoat.setCellValueFactory(new PropertyValueFactory<>("jälkiruoat"));
 
-        TableColumn<RaporttiRivi, String> tcErva = new TableColumn<>("Erityisruokavalio");
-        tcErva.setMinWidth(200);
-        tcErva.setCellValueFactory(new PropertyValueFactory<>("erva"));
-
         TableColumn<RaporttiRivi, String> tcLisätiedot =  new TableColumn<>("Lisätiedot");
-        tcLisätiedot.setMinWidth(300);
+        tcLisätiedot.setMinWidth(500);
         tcLisätiedot.setCellValueFactory(new PropertyValueFactory<>("lisätiedot"));
 
-        TableColumn<RaporttiRivi, String> tcYhteyshenkilönNimi =  new TableColumn<>("Yhteyshenkilön nimi");
+        tableView.getColumns().addAll(tcNimi, tcPvm, tcSalaatit, tcPääruoat, tcJälkiruoat, tcLisätiedot);
-        tcYhteyshenkilönNimi.setMinWidth(200);
+        tableView.setPlaceholder(new Label("Hae raportti"));
-        tcYhteyshenkilönNimi.setCellValueFactory(new PropertyValueFactory<>("yhteyshenkilönNimi"));
-
-        TableColumn<RaporttiRivi, String> tcLaskutusOsoite =  new TableColumn<>("Laskutusosoite");
-        tcLaskutusOsoite.setMinWidth(200);
-        tcLaskutusOsoite.setCellValueFactory(new PropertyValueFactory<>("laskutusOsoite"));
-
-        TableColumn<RaporttiRivi, String> tcYhteyshenkilönPuhelinnumero =  new TableColumn<>("Puhelinnumero");
-        tcYhteyshenkilönPuhelinnumero.setMinWidth(200);
-        tcYhteyshenkilönPuhelinnumero.setCellValueFactory(new PropertyValueFactory<>("yhteyshenkilönPuhelinnumero"));
-
-        TableColumn<RaporttiRivi, String> tcYhteyshenkilönSähköposti =  new TableColumn<>("Sähköposti");
-        tcYhteyshenkilönSähköposti.setMinWidth(200);
-        tcYhteyshenkilönSähköposti.setCellValueFactory(new PropertyValueFactory<>("yhteyshenkilönSähköposti"));
-
-        tableView.getColumns().addAll(tcNimi, tcPvm, tcTyyppi, tcSalaatit, tcPääruoat, tcJälkiruoat, tcErva, tcLisätiedot,
-                tcYhteyshenkilönNimi, tcLaskutusOsoite, tcYhteyshenkilönPuhelinnumero, tcYhteyshenkilönSähköposti);
-        tableView.setPlaceholder(new FontLabel(BUTTON_FONT, "Hae raportti"));
-
         for (TableColumn<RaporttiRivi, ?> sarake : tableView.getColumns()) {
             sarake.setSortable(false);
             sarake.setEditable(false);
             sarake.setResizable(true);
-            sarake.setStyle(
+            sarake.setStyle("-fx-alignment: CENTER; -fx-font-size: 16px;");
-                    "-fx-alignment: CENTER;" +
-                    "-fx-font-size: " + Main.properties.getProperty("tekstiFonttiKoko") + "px;" +
-                    "-fx-font-family: " + Main.properties.getProperty("tekstiFontti"));
         }
     }
 }

src/main/java/fi/lpam/gui/elementit/AsiakasListCell.java

@@ -12,13 +12,15 @@ import javafx.scene.layout.StackPane;
 import javafx.scene.layout.VBox;
 
 
-public class AsiakasMCLVListCell extends MultiColumnListView.ColumnListCell<Asiakas> {
+public class AsiakasListCell extends MultiColumnListView.ColumnListCell<Asiakas> {
     private final StackPane wrapper;
 
-    public AsiakasMCLVListCell(MultiColumnListView<Asiakas> multiColumnListView) {
+    public AsiakasListCell(MultiColumnListView multiColumnListView) {
+        //noinspection unchecked
         super(multiColumnListView);
-        this.setMinHeight(TabPohja.TEKSTI_FONT.getSize()*4);
+        this.setHeight(150);
-        this.setMaxHeight(TabPohja.TEKSTI_FONT.getSize()*4);
+        this.setPrefHeight(50);
+
 
         VBox content = new VBox();
         content.visibleProperty().bind(placeholder.not().and(emptyProperty().not()));
@@ -29,11 +31,9 @@ public class AsiakasMCLVListCell extends MultiColumnListView.ColumnListCell<Asia
         contentPlaceholder.managedProperty().bind(placeholder);
 
         Label label = new Label();
-        label.setFont(TabPohja.TEKSTI_FONT);
         label.textProperty().bind(textProperty());
 
         wrapper = new StackPane(content, contentPlaceholder, label);
-        wrapper.setAlignment(Pos.CENTER_LEFT);
         setGraphic(wrapper);
         setContentDisplay(ContentDisplay.GRAPHIC_ONLY);
     }
@@ -60,7 +60,7 @@ public class AsiakasMCLVListCell extends MultiColumnListView.ColumnListCell<Asia
                 placeholder.set(true);
                 setText("");
             } else {
-                setText(asiakas.getNimi() + "\n" + asiakas.getOsoite());
+                setText(asiakas.getNimi() + ": " + asiakas.getOsoite());
             }
         } else {
             setText("");

src/main/java/fi/lpam/gui/elementit/FontLabel.java

@@ -1,11 +0,0 @@
-package fi.lpam.gui.elementit;
-
-import javafx.scene.control.Label;
-import javafx.scene.text.Font;
-
-public class FontLabel extends Label {
-    public FontLabel(Font font, String text) {
-        super(text);
-        this.setFont(font);
-    }
-}

src/main/java/fi/lpam/gui/elementit/IntegerSpinner.java

@@ -4,8 +4,21 @@ import javafx.scene.control.Spinner;
 import javafx.scene.control.SpinnerValueFactory;
 
 public class IntegerSpinner extends Spinner<Integer> {
+    /**
+     * Min 0, Max 50
+     */
     public IntegerSpinner() {
-        super(new SpinnerValueFactory.IntegerSpinnerValueFactory(0, 500));
+        super(new SpinnerValueFactory.IntegerSpinnerValueFactory(0, 50));
+        this.setEditable(true);
+    }
+
+    /**
+     *
+     * @param min _
+     * @param max _
+     */
+    public IntegerSpinner(int min, int max) {
+        super(new SpinnerValueFactory.IntegerSpinnerValueFactory(min, max));
         this.setEditable(true);
     }
 }

src/main/java/fi/lpam/gui/elementit/KuljetaTableCell.java

@@ -1,9 +1,8 @@
-package fi.lpam.gui.tableCell;
+package fi.lpam.gui.elementit;
 
 import fi.lpam.dataluokat.Asiakas;
 import fi.lpam.dataluokat.Kuljetus;
 import fi.lpam.gui.KuljetusListat;
-import fi.lpam.gui.elementit.TabPohja;
 import javafx.geometry.Insets;
 import javafx.scene.control.Button;
 import javafx.scene.control.ContentDisplay;
@@ -17,8 +16,7 @@ public class KuljetaTableCell extends TableCell<Kuljetus, Boolean> {
 
     public KuljetaTableCell(HashMap<Integer, Asiakas> asiakkaat) {
         paddedNappi.setPadding(new Insets(3));
-        Button nappi = new Button("Kuljeta?");
+        Button nappi = new Button("Muuta");
-        nappi.setFont(TabPohja.TEKSTI_FONT);
         paddedNappi.getChildren().add(nappi);
         nappi.setOnMouseClicked(_ -> {
             getTableView().getSelectionModel().select(getIndex());

src/main/java/fi/lpam/gui/elementit/MaaraTableColumn.java

@@ -3,17 +3,10 @@ package fi.lpam.gui.elementit;
 import javafx.scene.control.TableColumn;
 
 public class MaaraTableColumn<Asiakas, Integer> extends TableColumn<Asiakas, Integer> {
-    public MaaraTableColumn(String columnLabel, int minWidth, int maxWidth) {
+    public MaaraTableColumn(String columnLabel, int minWidth) {
         super(columnLabel);
-        this.setResizable(true);
+        this.setResizable(false);
         this.setReorderable(false);
         this.setMinWidth(minWidth);
-        this.setMaxWidth(maxWidth);
-        this.setStyle("-fx-alignment: CENTER; -fx-font-size: 16px;");
-    }
-
-    public MaaraTableColumn(String columnLabel, int width) {
-        this(columnLabel, width, width);
-        this.setResizable(false);
     }
 }

src/main/java/fi/lpam/gui/elementit/MuutosIlmoitus.java

@@ -1,53 +0,0 @@
-package fi.lpam.gui.elementit;
-
-import fi.lpam.Main;
-import javafx.geometry.Pos;
-import javafx.scene.Scene;
-import javafx.scene.control.Button;
-import javafx.scene.layout.*;
-import javafx.stage.Stage;
-import one.jpro.platform.mdfx.MarkdownView;
-
-import java.io.IOException;
-import java.io.InputStreamReader;
-import java.util.Objects;
-
-public class MuutosIlmoitus extends Stage {
-    public MuutosIlmoitus() {
-        super();
-        VBox root = new VBox();
-        root.setBorder(TabPohja.BORDER);
-        root.setPadding(TabPohja.INSETS);
-        root.setAlignment(Pos.CENTER);
-        root.setSpacing(10);
-
-        String md;
-        try (InputStreamReader isr = new InputStreamReader(Objects.requireNonNull(Main.class.getClassLoader().getResourceAsStream("CHANGELOG.md")))) {
-            md = isr.readAllAsString();
-            int toinenMerkintä = md.indexOf("# ", 1);
-            int kolmasMerkintä = md.indexOf("# ", toinenMerkintä + 1);
-            int endIndex = md.indexOf("# ", kolmasMerkintä + 1);
-            md = md.substring(0, endIndex);
-        }
-        catch (IOException e) {
-            e.printStackTrace();
-            md = "# Virhe versiolokin haussa";
-        }
-
-        MarkdownView markdownView = new MarkdownView(md);
-        root.getChildren().add(markdownView);
-
-        Button ok = new Button("OK");
-        ok.setFont(TabPohja.BUTTON_FONT);
-        ok.setOnAction(e -> this.close());
-        root.getChildren().add(ok);
-
-        Scene scene = new Scene(root);
-        this.setScene(scene);
-        this.setTitle("Uusi versio, katso muutokset alta!");
-        this.setResizable(false);
-        this.show();
-
-        Main.properties.setProperty("viimeisinVersio", Main.properties.getProperty("version"));
-    }
-}

src/main/java/fi/lpam/gui/elementit/PaivamaaraTableCell.java

@@ -1,4 +1,4 @@
-package fi.lpam.gui.tableCell;
+package fi.lpam.gui.elementit;
 
 import fi.lpam.dataluokat.RaporttiRivi;
 import javafx.scene.control.TableCell;

src/main/java/fi/lpam/gui/elementit/TabPohja.java

@@ -1,54 +1,25 @@
 package fi.lpam.gui.elementit;
 
-import fi.lpam.Main;
 import javafx.geometry.Insets;
-import javafx.scene.control.Tab;
 import javafx.scene.layout.*;
 import javafx.scene.paint.Color;
 import javafx.scene.text.Font;
 import javafx.scene.text.FontWeight;
 
-import java.util.Properties;
-
 /**
- * Käyttöliittymän välilehden  pohja
+ * Käyttöliittymän  pohja
  */
-public class TabPohja extends Tab {
+public class TabPohja extends StackPane {
-    public static Insets INSETS;
+    public static final Insets oletusInsets = new Insets(5);
-    public static Border BORDER = new Border(new BorderStroke(Color.BLACK, BorderStrokeStyle.SOLID, new CornerRadii(5), BorderWidths.DEFAULT));
+    public static final String oletusStyle = "-fx-background-color: lightgray;";
-    public static String STYLE;
+    public static final Font otsikkoFont = Font.font("Open Sans", FontWeight.BOLD, 18);
-    public static Font BUTTON_FONT;
+    public static final Font buttonFont = Font.font("Open Sans", FontWeight.BOLD, 16);
-    public static Font TEKSTI_FONT;
+    public static final Font tekstiFont = Font.font("Open Sans", FontWeight.NORMAL, 16);
-    public static double SPACING;
+    public static final Border oletusBorder = new Border(new BorderStroke(Color.BLACK, BorderStrokeStyle.SOLID, new CornerRadii(5), BorderWidths.DEFAULT));
 
-    protected BorderPane root = new BorderPane();
+    public TabPohja() {
-
+        this.setStyle(oletusStyle);
-    public TabPohja(String title) {
+        this.setPadding(oletusInsets);
-        super(title);
+        this.setBorder(oletusBorder);
-        Properties prop = Main.properties;
-        INSETS = new Insets(Double.parseDouble(prop.getProperty("insets")));
-        SPACING = Double.parseDouble(prop.getProperty("insets"));
-        STYLE = "-fx-background-color: " + prop.getProperty("taustaVari");
-        BUTTON_FONT = Font.font(prop.getProperty("buttonFontti"), FontWeight.BOLD, Double.parseDouble(prop.getProperty("buttonFonttiKoko")));
-        TEKSTI_FONT = Font.font(prop.getProperty("tekstiFontti"), FontWeight.NORMAL, Double.parseDouble(prop.getProperty("tekstiFonttiKoko")));
-
-        this.setStyle(
-                "-fx-font-size: " + Main.properties.getProperty("buttonFonttiKoko") + "px;" +
-                "-fx-font-weight: bold;" +
-                "-fx-font-family: " + Main.properties.getProperty("buttonFontti") + ";"
-                );
-        this.setClosable(false);
-        this.setOnSelectionChanged(e -> {if (this.isSelected()) päivitä();});
-
-        root.setStyle(STYLE);
-        root.setPadding(INSETS);
-        root.setBorder(BORDER);
-        this.setContent(root);
     }
-
-    /**
-     * Metodi, jota kutsutaan kun tähän välilehteen palataan
-     * Luokan toteuttaja määrittää, mitä tarvitsee päivittää välilehteen palattaessa
-     */
-    public void päivitä() {}
 }

src/main/java/fi/lpam/gui/tableCell/IntegerFieldTableCellFactory.java

@@ -1,30 +0,0 @@
-package fi.lpam.gui.tableCell;
-
-import javafx.scene.control.TableCell;
-import javafx.scene.control.TableColumn;
-import javafx.scene.control.cell.TextFieldTableCell;
-import javafx.scene.input.KeyEvent;
-import javafx.util.Callback;
-import javafx.util.converter.IntegerStringConverter;
-
-public class IntegerFieldTableCellFactory<T> implements Callback<TableColumn<T, Integer>, TableCell<T, Integer>> {
-    @Override
-    public TableCell<T, Integer> call(TableColumn<T, Integer> param ) {
-        return new TextFieldTableCell<>(new IntegerStringConverter()) {
-            private boolean filterSet = false;
-
-            @Override
-            public void startEdit() {
-                super.startEdit();
-                if (!filterSet) {
-                    getGraphic().addEventFilter(KeyEvent.KEY_TYPED, KE -> {
-                        if (!"0123456789".contains(KE.getCharacter())) {
-                            KE.consume();
-                        }
-                    });
-                    filterSet = true;
-                }
-            }
-        };
-    }
-}

src/main/java/fi/lpam/gui/virheDialogit/TietokantaVirhe.java

@@ -12,7 +12,7 @@ abstract class TietokantaVirhe extends Alert {
     public TietokantaVirhe() {
         super(AlertType.ERROR);
         this.setTitle("Tietokanta");
-        ImageView sql = new ImageView(new Image(Objects.requireNonNull(Main.class.getClassLoader().getResourceAsStream("tietokanta.png"))));
+        ImageView sql = new ImageView(new Image(Objects.requireNonNull(Main.class.getResourceAsStream("tietokanta.png"))));
         sql.setPreserveRatio(true);
         sql.setFitHeight(100);
         this.setGraphic(sql);

src/main/java/fi/lpam/gui/virheDialogit/YhteysVirhe.java

@@ -0,0 +1,8 @@
+package fi.lpam.gui.virheDialogit;
+
+public class YhteysVirhe extends TietokantaVirhe{
+    public YhteysVirhe(Exception error){
+        this.setHeaderText("Virhe yhdistäessä tietokantaan");
+        this.setContentText(error.getMessage());
+    }
+}

src/main/java/fi/lpam/tulostajat/KuljetusListaTulostaja.java

@@ -6,6 +6,7 @@ import javafx.collections.ObservableList;
 import java.io.IOException;
 import java.time.format.DateTimeFormatter;
 
+@SuppressWarnings("DuplicatedCode")
 public class KuljetusListaTulostaja extends Tulostaja{
     final float NIMEN_LEVEYS = 200;
     final float OSOITTEEN_LEVEYS = 200;
@@ -44,10 +45,8 @@ public class KuljetusListaTulostaja extends Tulostaja{
                 siirräOsoitinta(NIMEN_LEVEYS);
                 sisältö.showText(kuljetus.getLisätieto());
                 uusiRivi(RIVI_KORKEUS*0.1F);
-                //67 alaviivaa fonttikoolla 13 (noin sivun leveys)
+                sisältö.showText("___________________________________________________________________");
-                sisältö.showText("_".repeat((int) (67*13/FONTTI_KOKO)));
             }
-
             tulosta();
         }
         catch (IOException e) {

src/main/java/fi/lpam/tulostajat/KuljetusRaporttiTulostaja.java

@@ -2,15 +2,23 @@ package fi.lpam.tulostajat;
 
 import fi.lpam.dataluokat.RaporttiRivi;
 
+import java.awt.print.PrinterJob;
 import java.io.IOException;
 import java.time.LocalDate;
-import java.time.format.DateTimeFormatter;
 import java.util.ArrayList;
 
+import fi.lpam.gui.virheDialogit.SiirtoVirhe;
+import org.apache.pdfbox.pdmodel.PDDocument;
+import org.apache.pdfbox.pdmodel.PDPage;
+import org.apache.pdfbox.pdmodel.PDPageContentStream;
+import org.apache.pdfbox.pdmodel.common.PDRectangle;
+import org.apache.pdfbox.printing.PDFPageable;
+
+@SuppressWarnings("DuplicatedCode")
 public class KuljetusRaporttiTulostaja extends Tulostaja {
     public void luoRaportti(ArrayList<RaporttiRivi> raporttiRivit, LocalDate alkuPvm, LocalDate loppuPvm) {
         try {
-            sisältö.showText("Raportti aikaväliltä " + alkuPvm.format(DateTimeFormatter.ofPattern("dd/MM/yyyy")) + " - " + loppuPvm.format(DateTimeFormatter.ofPattern("dd/MM/yyyy")));
+            sisältö.showText("Raportti aikaväliltä " + alkuPvm + " - " + loppuPvm);
             uusiRivi();
             uusiRivi();
 

src/main/java/fi/lpam/tulostajat/Tulostaja.java

@@ -1,6 +1,5 @@
 package fi.lpam.tulostajat;
 
-import fi.lpam.Main;
 import fi.lpam.gui.virheDialogit.SiirtoVirhe;
 import org.apache.pdfbox.pdmodel.PDDocument;
 import org.apache.pdfbox.pdmodel.PDPage;
@@ -16,18 +15,18 @@ import java.awt.print.PrinterJob;
 import java.io.IOException;
 
 public abstract class Tulostaja {
-    final PDRectangle SIVUKOKO = PDRectangle.A4;
+    PDRectangle SIVUKOKO = PDRectangle.A4;
     final PDFont FONTTI = new PDType1Font(Standard14Fonts.FontName.HELVETICA);
-    final float FONTTI_KOKO = Float.parseFloat(Main.properties.getProperty("tulostusFonttiKoko"));
+    final float FONTTI_KOKO = 13;
     final float MARGINAALI = 50;
     final float RIVI_KORKEUS = FONTTI_KOKO * 1.2F;
 
-    final PDDocument dokumentti;
+    PDDocument dokumentti;
     PDPage nykyinenSivu = null;
     PDPageContentStream sisältö = null;
-    final float sivunLeveys;
+    float sivunLeveys;
-    final float startX;
+    float startX;
-    final float startY;
+    float startY;
     float yOffset;
     float xOffset;
 
@@ -79,7 +78,7 @@ public abstract class Tulostaja {
      uusiRivi(RIVI_KORKEUS);
     }
     /**Siirtää osoitinta sivulla. + oikealle, - vasemmalle*/
-    void siirräOsoitinta(@SuppressWarnings("SameParameterValue") float tX) {
+    void siirräOsoitinta(float tX) {
      try {
          sisältö.newLineAtOffset(tX, 0);
          xOffset += tX;

src/main/java/module-info.java

@@ -8,7 +8,6 @@ module fi.lpam.ruokamanageri {
     requires java.desktop;
     requires org.apache.pdfbox;
     requires org.apache.commons.lang3;
-    requires one.jpro.platform.mdfx;
 
     opens fi.lpam.dataluokat to javafx.base;
     exports fi.lpam;

src/main/resources/CHANGELOG.md

@@ -1,46 +0,0 @@
-# 3.2.1
-- Asiakkaan erityisruokavalio osaksi raporttia
-
-# 3.2.0
-- Lisätty Asetukset -välilehti
-- Asiakkaan voi poistaa ilman, että se rikkoo kierroksia
-- Ohjelma ei enään kaadu, mikäli on tallentamaton kierros kierroshallinnassa
-- Yhtenäistetty ulkoasua välilehtien välillä
-
-# 3.1.0
-- Muutosloki näkyy käynnistäessä uutta versiota ensimmäistä kertaa
-- Ohjelman taustariippuvuuksia päivitetty
-- Yleistä huoltoa
-
-# 3.0.1
-- Kierroshallinnan muokkaus tukemaan paremmin useampaa kierrosta
-- Kosmeettisia viilauksia muuallekin
-
-# 3.0.0
-- Lisätty lauantai ja sunnuntai mahdollisiksi kuljetuspäiviksi
-- Mahdollisuus kuljettaa (ja muokata listaa) yli 10 ateriaa per asiakas
-- Versioloki näkyy taas
-- Pieniä viilauksia asiakashallinnan näkymään
-
-# 2.0.2
-- Korjattu virhe kierroksen asiakkaiden hakemisessa, joka aiheutti vääriä järjestyksiä kierroksella
-
-# 2.0.1
-- Vaihdettu versioloki avautumaan paikallisena tiedostona
-
-# 2.0.0
-### Palaaminen aiempaan versioon ei tuettu
-- Asiakkaalle lisätty tietoihin laskutusosoite ja yhteyshenkilön yhteystiedot
-- Tiedot nousevat kuljetusraportille laskutusta varten
-- Lisätty ohjelmaan linkki muutoslokiin
-
-# 1.1.1
-- Integroitu versionumerointi projektiin
-
-# 1.1.0
-- Kuljetuslistaa luodessa kuljetusrivin nollaaminen/palauttaminen oletusmääriin.
-- Kuljetuslistojen ja raporttien tulostamismekanismin läpikäynti
-- Kierroshallinnassa asiakasta raahattaessa placeholderin fiksaus
-
-# 1.0.0
-- Ohjelman kieli vaihdettu Pythonista Javaan, sekä tietokanta pickle-tiedostosta SQLite tietokantaan

src/main/resources/META-INF/MANIFEST.MF

@@ -0,0 +1,2 @@
+Manifest-Version: 1.0
+Main-Class: fi.lpam.Main

src/main/resources/defaults.properties

@@ -1,13 +0,0 @@
-version=${project.version}
-
-buttonFonttiKoko=16
-buttonFontti=Arial
-
-tekstiFonttiKoko=16
-tekstiFontti=Arial
-
-tulostusFonttiKoko=13
-
-taustaVari=lightgray
-
-insets=5

src/main/resources/fi/lpam/tietokanta.sql

@@ -0,0 +1,39 @@
+create table if not exists asiakkaat (
+    id INTEGER PRIMARY KEY,
+    nimi varchar(255) not null,
+    osoite varchar(255) not null,
+    maanantaiKpl int not null,
+    tiistaiKpl int not null,
+    keskiviikkoKpl int not null,
+    torstaiKpl int not null,
+    perjantaiKpl int not null,
+    salaatit int not null,
+    pääruoat int not null,
+    jälkiruoat int not null,
+    erityisruokavalio varchar(255),
+    avainTieto varchar(255),
+    lisätiedot text
+);
+
+create table if not exists kierrokset (
+    id INTEGER PRIMARY KEY,
+    kierrosNumero int not null,
+    asiakkaat text not null
+);
+
+create table if not exists kuljetukset (
+    id INTEGER PRIMARY KEY,
+    asiakasID int not null,
+    nimi varchar(255) not null,
+    kuljetusPäivämäärä DATE not null,
+    kuljetusKierros int not null,
+    salaatit int not null,
+    pääruoat int not null,
+    jälkiruoat int not null,
+    ruokarajoite varchar(255),
+    osoite varchar(255),
+    avainTieto varchar(255),
+    lisätieto varchar(255)
+);
+
+create index if not exists idx_kuljetuspvm on kuljetukset(kuljetusPäivämäärä);

src/main/resources/tietokanta.sql

@@ -1,60 +0,0 @@
-create table if not exists asiakkaat (
-    id INTEGER PRIMARY KEY,
-    nimi varchar(255) not null,
-    osoite varchar(255) not null,
-    maanantaiKpl int not null,
-    tiistaiKpl int not null,
-    keskiviikkoKpl int not null,
-    torstaiKpl int not null,
-    perjantaiKpl int not null,
-    salaatit int not null,
-    pääruoat int not null,
-    jälkiruoat int not null,
-    erityisruokavalio varchar(255),
-    avainTieto varchar(255),
-    lisätiedot text,
-    laskutusosoite varchar(255),
-    yhteyshenkilönNimi varchar(255),
-    yhteyshenkilönPuhelinnumero varchar(255),
-    yhteyshenkilönSähköposti varchar(255)
-);
-
-create table if not exists kierrokset (
-    id INTEGER PRIMARY KEY,
-    kierrosNumero int not null,
-    asiakkaat text not null
-);
-
-create table if not exists kuljetukset (
-    id INTEGER PRIMARY KEY,
-    asiakasID int not null,
-    nimi varchar(255) not null,
-    kuljetusPäivämäärä DATE not null,
-    kuljetusKierros int not null,
-    salaatit int not null,
-    pääruoat int not null,
-    jälkiruoat int not null,
-    ruokarajoite varchar(255),
-    osoite varchar(255),
-    avainTieto varchar(255),
-    lisätieto varchar(255)
-);
-
-create index if not exists idx_kuljetuspvm on kuljetukset(kuljetusPäivämäärä);
-
---2.0.2->3.x.x
-alter table asiakkaat add lauantaiKpl int not null default 0;
-alter table asiakkaat add sunnuntaiKpl int not null default 0;
-
---3.x.x->4.0.0
-alter table kuljetukset add tyyppi varchar(50);
-alter table asiakkaat add dinner_maanantaiKpl int not null default 0;
-alter table asiakkaat add dinner_tiistaiKpl int not null default 0;
-alter table asiakkaat add dinner_keskiviikkoKpl int not null default 0;
-alter table asiakkaat add dinner_torstaiKpl int not null default 0;
-alter table asiakkaat add dinner_perjantaiKpl int not null default 0;
-alter table asiakkaat add dinner_lauantaiKpl int not null default 0;
-alter table asiakkaat add dinner_sunnuntaiKpl int not null default 0;
-alter table asiakkaat add dinner_salaatit int not null default 0;
-alter table asiakkaat add dinner_pääruoat int not null default 0;
-alter table asiakkaat add dinner_jälkiruoat int not null default 0;