diff --git a/src/main/java/fi/lpam/dataluokat/Asiakas.java b/src/main/java/fi/lpam/dataluokat/Asiakas.java index 521908d..cd847b9 100644 --- a/src/main/java/fi/lpam/dataluokat/Asiakas.java +++ b/src/main/java/fi/lpam/dataluokat/Asiakas.java @@ -205,8 +205,7 @@ public class Asiakas implements TietokantaOlio { * Tallentaa uuden asiakkaan tietokantaan. Tietokanta luo automaattisesti id:n asiakkaalle. * */ - @Override - public boolean tallennaUusi() { + private boolean tallennaUusi() { try (Connection tietokanta = Tietokanta.haeYhteys()) { PreparedStatement stmt = tietokanta.prepareStatement("INSERT INTO asiakkaat (nimi, osoite, " + "maanantaiKpl, tiistaiKpl, keskiviikkoKpl, torstaiKpl, perjantaiKpl, lauantaiKpl, sunnuntaiKpl, salaatit, pääruoat, jälkiruoat, " + @@ -268,8 +267,7 @@ public class Asiakas implements TietokantaOlio { /** * Päivittää kutsuvan asiakkaan tiedot tietokantaan */ - @Override - public boolean päivitäTietokantaan() { + private boolean päivitäTietokantaan() { try (Connection tietokanta = Tietokanta.haeYhteys()) { PreparedStatement stmt = tietokanta.prepareStatement("UPDATE asiakkaat set nimi = ?, osoite = ?, " + "maanantaiKpl = ?, tiistaiKpl = ?, keskiviikkoKpl = ?, torstaiKpl = ?, perjantaiKpl = ?, lauantaiKpl = ?, sunnuntaiKpl = ?, salaatit = ?, pääruoat = ?, jälkiruoat = ?, " + diff --git a/src/main/java/fi/lpam/dataluokat/Kierros.java b/src/main/java/fi/lpam/dataluokat/Kierros.java index a8a5a65..4b5bb19 100644 --- a/src/main/java/fi/lpam/dataluokat/Kierros.java +++ b/src/main/java/fi/lpam/dataluokat/Kierros.java @@ -86,8 +86,7 @@ public class Kierros implements TietokantaOlio { } - @Override - public boolean tallennaUusi() { + private boolean tallennaUusi() { try (Connection tietokanta = Tietokanta.haeYhteys()) { PreparedStatement stmt = tietokanta.prepareStatement("insert into kierrokset (kierrosNumero, asiakkaat) values (?, ?)", Statement.RETURN_GENERATED_KEYS); stmt.setInt(1, kierrosNumero); @@ -114,8 +113,7 @@ public class Kierros implements TietokantaOlio { return true; } - @Override - public boolean päivitäTietokantaan() { + private boolean päivitäTietokantaan() { try (Connection tietokanta = Tietokanta.haeYhteys()) { PreparedStatement stmt = tietokanta.prepareStatement("update kierrokset set asiakkaat = ?, kierrosnumero = ? where id = ?"); stmt.setString(1, muutaAsiakkaatMerkkijonoksi()); diff --git a/src/main/java/fi/lpam/dataluokat/Kuljetus.java b/src/main/java/fi/lpam/dataluokat/Kuljetus.java index d5ae55c..c5a1d4b 100644 --- a/src/main/java/fi/lpam/dataluokat/Kuljetus.java +++ b/src/main/java/fi/lpam/dataluokat/Kuljetus.java @@ -103,8 +103,7 @@ public class Kuljetus implements TietokantaOlio { } } - @Override - public boolean tallennaUusi() { + private 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); stmt.setInt(1, asiakasID); @@ -139,8 +138,7 @@ public class Kuljetus implements TietokantaOlio { return true; } - @Override - public boolean päivitäTietokantaan() { + private 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 = ?"); stmt.setInt(1, asiakasID); diff --git a/src/main/java/fi/lpam/dataluokat/TietokantaOlio.java b/src/main/java/fi/lpam/dataluokat/TietokantaOlio.java index 6f698ac..7839449 100644 --- a/src/main/java/fi/lpam/dataluokat/TietokantaOlio.java +++ b/src/main/java/fi/lpam/dataluokat/TietokantaOlio.java @@ -1,12 +1,16 @@ package fi.lpam.dataluokat; public interface TietokantaOlio { + /** + * Tallentaa joko uuden, tai päivittää olemassa olevaa riviä + * @return onnistuiko operaatio + */ + boolean tallenna(); - boolean tallennaUusi(); - - boolean päivitäTietokantaan(); - + /** + * Poistaa kutsuvan olion rivin tietokannasta + * @return onnistuiko operaatio + */ boolean poista(); - boolean tallenna(); } \ No newline at end of file diff --git a/src/main/java/fi/lpam/gui/AsiakasIkkuna.java b/src/main/java/fi/lpam/gui/AsiakasIkkuna.java index 8b6e231..3a86cca 100644 --- a/src/main/java/fi/lpam/gui/AsiakasIkkuna.java +++ b/src/main/java/fi/lpam/gui/AsiakasIkkuna.java @@ -7,7 +7,6 @@ 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; @@ -298,12 +297,7 @@ public class AsiakasIkkuna extends Stage { int maxYritykset = 3; int yritykset = 0; while (!läpi) { - if (käsiteltäväAsiakas.getId() > 0) { - läpi = käsiteltäväAsiakas.päivitäTietokantaan(); - } - else { - läpi = käsiteltäväAsiakas.tallennaUusi(); - } + läpi = käsiteltäväAsiakas.tallenna(); yritykset++; if (yritykset > maxYritykset) { break; diff --git a/src/main/java/fi/lpam/gui/KuljetusListat.java b/src/main/java/fi/lpam/gui/KuljetusListat.java index 443a187..2b80d3a 100644 --- a/src/main/java/fi/lpam/gui/KuljetusListat.java +++ b/src/main/java/fi/lpam/gui/KuljetusListat.java @@ -192,11 +192,7 @@ public class KuljetusListat extends TabPohja { private void tallennaKierros() { try { for (Kuljetus kuljetus : tableView.getItems()) { - if (kuljetus.getId() > 0) { - kuljetus.päivitäTietokantaan(); - } else { - kuljetus.tallennaUusi(); - } + kuljetus.tallenna(); } tallennusStatus.setText("Tallennettu: Kyllä"); btnTulosta.setDisable(false);