Derby/JavaDB Unfindbarer Fehler im Datenbank-Code

[DaCrazyJavaExpert]

ja, es sollte eigentlich nichts mehr static sein (Ausser main(...)

Ok ist nix mehr static.

Und dein CookieListener ruft database.setAnzahlCookies(..) und setBlaBlala(..) auf und anschliessend database.save().

Ich weiß nur nicht so genau was jetzt in die save() Methode rein soll. Du meinst ja, das gleiche was ich in registerConnectionInListener() habe. Also wozu dann eine neue Methode?

 

[JStein52]

da lässt du Database. weg

Aber statt dessen this.

 

[JStein52]

Du meinst ja, das gleiche was ich in registerConnectionInListener() habe.

Ob es die gleiche ist kann ich gerade nicht sagen, auf jeden Fall war der Name doof. Und sie schreibt die Variablen aus der eigenen Klasse Databes in die Datenbank und nichts von einem GUI-Objekt. Der Rest mag gleich sein, kann sein

 

[JStein52]

//in der Methode initialVariables(), welche im GUI-Konstruktor aufgerufen wird.
private void initialVariables() {
this.cookiesProClick = Database.getCookies_Pro_Klick();
this.anzahlCookies = Database.getAnzahl_Cookies();
}

Die hier schmeisst du ersatzlos weg. In der Klasse GUI gibt es keine Attribute cookiesProClick und anzahlCookies mehr

 

[DaCrazyJavaExpert]

Aber statt dessen this.

Hab ich schon längst . Also die Methode registerConnectionInListener ist sozusagen unnötig, die brauch ich gar nicht?

 

[JStein52]

Also die Methode registerConnectionInListener ist sozusagen unnötig

Die wird zur neuen save()-Methode in der Klasse Database

 

[DaCrazyJavaExpert]

Also die nimmt die Variablen aus der GUI:

    public void save() {
        try(Connection connection = createConnection()) {
            PreparedStatement updateValues = connection.prepareStatement(UPDATE_VALUES);
            updateValues.setInt(1, guiVarObject.getCookiesProClick()); // cookiesProClick ist eine Varaible aus der GUI Klasse
            updateValues.setLong(2, guiVarObject.getAnzahlCookies()); // anzahlCookies ist eine Varible aus der GUI-Klasse
            updateValues.executeUpdate();
        } catch (ClassNotFoundException e) {
            e.printStackTrace();
        } catch (SQLException e) {
            e.printStackTrace();
        }
    }

 

[JStein52]

Also die nimmt die Variablen aus der GUI

Genau das soll sie nicht tun, sondern sie schreibt ihre Variablen (du hast doch jetzt in Database anzahlCookies und cookiesProClick ?!!) in die Datenbank

 

[JStein52]

Deine Klasse Database sollte jetzt die o.g. beiden Attribute haben, dann einen Konstruktor der die Tabelle neu anlegt und/oder per Select die Daten liest. Dann sollte sie getter und setter für die beiden Attribute haben und eine save()-Methode. Zugriff auf irgendwas von GUI sollte da nicht mehr drin sein.

 

[tommysenf]

Es gibt auch die Möglichkeit Parameter zu nutzen, was den ganzen Kladderadatsch meiner Meinung nach wesentlich vereinfachen würde...

 

[DaCrazyJavaExpert]

Ich könnte das noch machen, ergibbt allerdings keinen Sinn, da...

try(Connection connection = createConnection()) {
        PreparedStatement updateValues = connection.prepareStatement(UPDATE_VALUES);
        updateValues.setInt(1, cookies_Pro_Klick);
        updateValues.setLong(2, anzahl_Cookies);
        updateValues.executeUpdate();

Die beiden Variablen die Daten aujs der Datenbank enthalten und gar nicht aktualisiert werden.

    PreparedStatement queryInstructions = connection.prepareStatement("SELECT * FROM COOKIE_VALUES");
            ResultSet resultSet = queryInstructions.executeQuery();
            while (resultSet.next()) {
                id = resultSet.getString("ID");
                cookies_Pro_Klick = resultSet.getInt("COOKIES_PRO_KLICK"); // Daten holen
                anzahl_Cookies = resultSet.getLong("ANZAHL_COOKIES"); // Daten holen
                guiVarObject.setCookiesProClick(cookies_Pro_Klick); // Daten in GUI schreiben
                guiVarObject.setAnzahlCookies(anzahl_Cookies); // Daten in GUI schreiben
            }

 

[JStein52]

In deiner GUI - Klasse sollten jetzt die beiden Attribute anzahlCookies und coopiesProClick nicht (!) mehr drin sein, statt dessen ein Attribut database. Statt dessen hat sie einen Konstruktor der zunächst ein database anlegt, dann per get sich von database die Attribute holt, dann die GUI aufbaut und einen Listener der die Textfelder ausliest, irgendwas damit macht (inkrementiert), dann per set in database speichert und save zum speichern in der Datenbank aufruft.

 

[JStein52]

und gar nicht aktualisiert werden.

doch werden sie. Das macht ja der Listener.

 

[DaCrazyJavaExpert]

Und außerdem hab ich grade ein Error, ein StackOverflowError, welcher auf das hier:

public class CookieClickerGUI extends JFrame {
   
    Database database = new Database(); //HIer

Und dass hier:

public class Database {

    private  CookieClickerGUI guiVarObject = new CookieClickerGUI();

verweist.

 

[JStein52]

            while (resultSet.next()) {
                id = resultSet.getString("ID");
                cookies_Pro_Klick = resultSet.getInt("COOKIES_PRO_KLICK"); // Daten holen
                anzahl_Cookies = resultSet.getLong("ANZAHL_COOKIES"); // Daten holen
                guiVarObject.setCookiesProClick(cookies_Pro_Klick); // Daten in GUI schreiben
                guiVarObject.setAnzahlCookies(anzahl_Cookies); // Daten in GUI schreiben
            }

das gesamte guiVarObject fliegt aus der Database raus

 

[JStein52]

das gesamte guiVarObject fliegt aus der Database raus

sag ich doch

 

[DaCrazyJavaExpert]

das gesamte guiVarObject fliegt aus der Database raus

Dann kann ich die Variablen der Gui klasse ja gar nicht mehr nutzen

 

[JStein52]

Die sollst du ja auch gar nicht nutzen ! Warum oder wofür willst du sie denn nutzen ??

 

[JStein52]

Dann kann ich die Variablen der Gui

Die GUI-Klasse hat keine Variablen mehr. Ein paar posts vorher habe ich dir geschrieben was die GUI haben sollte

 

[DaCrazyJavaExpert]

Die sollst du ja auch gar nicht nutzen ! Warum oder wofür willst du sie denn nutzen ??

Das sind doch die, die Aufgezählt werde, und in der Datenbank gespeichert/aktualisiert werden sollen

 

[JStein52]

Aber die stehen jetzt in der Klasse Database und du hast getter und setter um sie dort zu lesen und zu ändern. Also brauchst du sie nicht in der GUI

 

[DaCrazyJavaExpert]

Aso. Und die save() Methode muss in den Listener? Also die sieht momentan so aus:

    public void save() {
        try(Connection connection = createConnection()) {
        PreparedStatement updateValues = connection.prepareStatement(UPDATE_VALUES);
        updateValues.setInt(1, cookies_Pro_Klick);
        updateValues.setLong(2, anzahl_Cookies);
        updateValues.executeUpdate();
        } catch (ClassNotFoundException e) {
            e.printStackTrace();
        } catch (SQLException e) {
            e.printStackTrace();
        }
    }

Ich hoffe das hab ich jetzt richtig .

Und hier der Momentane Konstruktor der Klasse Database:

    public Database() {
        this.createTablePlusSelect();
        this.save();
    }

+ hier die Methode createTablePlusSelect() :

    public  void createTablePlusSelect() { // bei Programmstart werden alle Daten in die GUI eingetragen
        try (Connection connection = createConnection()) {
            DatabaseMetaData metaData = connection.getMetaData();
            ResultSet tables = metaData.getTables(null, "APP", "COOKIE_VALUES", null);
            if (!tables.next()) { // Nur erstellen wenn Datenbank noch nicht vorhanden ist
                Statement instructions = connection.createStatement();
                instructions.executeUpdate(CREATE_DATA_TABLE);
                PreparedStatement insertStatement = connection.prepareStatement(ADD_INSTRUCTIONS);
                insertStatement.setInt(1, 1); // aus Programm (GUI) holen, in Datenbank setzten
                insertStatement.setLong(2, anzahl_Cookies);
                insertStatement.executeUpdate();
                instructions.close();
            }

            PreparedStatement queryInstructions = connection.prepareStatement("SELECT * FROM COOKIE_VALUES");
            ResultSet resultSet = queryInstructions.executeQuery();
            while (resultSet.next()) {
                id = resultSet.getString("ID");
                cookies_Pro_Klick = resultSet.getInt("COOKIES_PRO_KLICK"); // Daten holen
                anzahl_Cookies = resultSet.getLong("ANZAHL_COOKIES"); // Daten holen
            }
        } catch (ClassNotFoundException e) {
            System.err.println("Database driver not found!");
        } catch (SQLException e) {
            System.err.println("SQL error");
            e.printStackTrace();
        }
    }

Soweit so gut.
Und wo muss ich denn jetzt die Inztanz von Database hinplazieren? In den Konstruktor der GUI-Klasse?

Und dadurch wird sogar immer anzahlCookies + 1 addiert. Allerdings wird cookiesProClick nicht als 1 sondern 0 angezeigt, ich hoffe verständlich was ich meine. Wie ändere ich das?

 

[JStein52]

die save() Methode muss in den Listener

Die save()-Methode wird im Listener aufgerufen, nachdem du mit database.setAnzahlCookies und database.set... aufgreufen hast.
Im Konstruktor von Database brauchst du natürlich nicht gleich nach dem Laden wieder save aufzurufen, das bringt nichts. Und eine Instanz von Database legst du im Konstruktor deiner GUI an. Das hattest du doch schon mal ...

 

[DaCrazyJavaExpert]

Oha EDIT: Es funktioniert wieder alles, bis auf das Updaten der Daten, dass heißt ich bin am gleichen Punkt wie zuvor, nur mit einem anderen "Code-Layout" Das freut mich tatsächlich. Also den Fehler muss ich noch finden

 

[mrBrown]

Die save()-Methode wird im Listener aufgerufen, nachdem du mit database.setAnzahlCookies und database.set... aufgreufen hast.

Besser wäre es, der save-methode beide Werte zu übergeben, statt Setter zu nutzen

 

[DaCrazyJavaExpert]

Die save()-Methode wird im Listener aufgerufen, nachdem du mit database.setAnzahlCookies und database.set... aufgreufen hast.
Im Konstruktor von Database brauchst du natürlich nicht gleich nach dem Laden wieder save aufzurufen, das bringt nichts. Und eine Instanz von Database legst du im Konstruktor deiner GUI an. Das hattest du doch schon mal ...

Aha, also so?

    @Override
            public void mousePressed(MouseEvent e) {
                anzahlCookies += database.getCookies_Pro_Klick();
                lblAnzahlCookies.setText("Cookies: " + anzahlCookies);
                lblCookie.setVisible(false);
                lblCookieClicked.setVisible(true);
                database.setAnzahl_Cookies(anzahlCookies);
                database.setCookies_Pro_Klick(database.getCookies_Pro_Klick());
                database.save(); /hier save()
            }

Ich doch angelegt!?:

    public CookieClickerGUI() {
        Database database = new Database();
        this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        this.setLocationByPlatform(true);
        this.setSize(FRAME_WIGHT, FRAME_HIGH);
        this.initialComponents();
        this.addComponetsToPanelOverlay();
        this.registerCookieListener();
    }

 

[JStein52]

database.setCookies_Pro_Klick(database.getCookies_Pro_Klick()); das musst du natürlich nicht machen wenn sich dieser Wert nicht ändert.
Database database ... das darf nicht lokal im Konstruktor deklariert werden sondern muss hoch in die Klasse. Das new bleibt im Konstruktor.

Edit: sonst sieht es jetzt langsam richtig aus ...

 

[DaCrazyJavaExpert]

JA,

database.setCookies_Pro_Klick(database.getCookies_Pro_Klick()); das musst du natürlich nicht machen wenn sich dieser Wert nicht ändert.

später wird es ein Shopsystem geben(jedenfalls so mein Plan). Wenn man was "einkäuft" werden die cookiesProClicks geändert. Und ja ich weiß, man sollte eigentlich mit der Datenspeicherung erst beginnen, wenn man den Rest des Programms hat. Werd ich bei den folgenden Prgrammen auch so einhalten .

Database database ... das darf nicht lokal im Konstruktor deklariert werden sondern muss hoch in die Klasse. Das new bleibt im Konstruktor.

Aso.

 

[JStein52]

anzahlCookies += database.getCookies_Pro_Klick();
das ist noch falsch. anzahlCookies sollte er gar nicht kennen. Und du wolltest das doch um 1 inkrementieren oder ?

int anzahlCookies = database.getCookies_Pro_Klick() + 1;

 

[DaCrazyJavaExpert]

So isses richtig oder? :

Database database;

    public CookieClickerGUI() {
        database = new Database(); // Hier
        this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        this.setLocationByPlatform(true);
        this.setSize(FRAME_WIGHT, FRAME_HIGH);
        this.initialComponents();
        this.addComponetsToPanelOverlay();
        this.registerCookieListener();
    }

Ist es eigentlich relevant, wo ich new Database() im Konstruktor aufrufe?

 

[DaCrazyJavaExpert]

anzahlCookies += database.getCookies_Pro_Klick();
das ist noch falsch. anzahlCookies sollte er gar nicht kennen. Und du wolltest das doch um 1 inkrementieren oder ?

int anzahlCookies = database.getCookies_Pro_Klick() + 1;

Ähm anzahlCookies kennt er daher:

    // Variables
    private int cookiesProClick; //Das hier ist mittlerweile useless geworden
    private long anzahlCookies;

Und 1 ist es schon, da...:

private final  String ADD_INSTRUCTIONS = "INSERT INTO COOKIE_VALUES VALUES (DEFAULT, ?, ?)"; // Datensätzte
private final  String UPDATE_VALUES = "UPDATE COOKIE_VALUES SET COOKIES_PRO_KLICK= ?, ANZAHL_COOKIES= ?";

if (!tables.next()) { // Nur erstellen wenn Datenbank noch nicht vorhanden ist
                Statement instructions = connection.createStatement();
                instructions.executeUpdate(CREATE_DATA_TABLE);
                PreparedStatement insertStatement = connection.prepareStatement(ADD_INSTRUCTIONS);
                insertStatement.setInt(1, 1); // hier wirds auf 1 gesetzt. Und zwar nur beim ersten Programmstart, danach wird immer der wer genommen, der hinter cookies_Pro_Klick steckt.
                insertStatement.setLong(2, anzahl_Cookies);
                insertStatement.executeUpdate();

 

[DaCrazyJavaExpert]

Ich glaube, ich muss nur noch anzahl_Cookies aus der Klasse Database anzahlCookies aus der GUI Klasse zuweisen.
EDIT: Oh das tu ich ja schon hiermit:

database.setAnzahl_Cookies(anzahlCookies);
Ok, irgenwo liegt immernoch ein Problem, denn die Daten werden nicht abgerufen.

 

[JStein52]

// Variables
private int cookiesProClick; //Das hier ist mittlerweile useless geworden
private long anzahlCookies;

Warum ist das noch da ? Das sollte weg

denn die Daten werden nicht abgerufen.

Was meinst du damit

 

[DaCrazyJavaExpert]

Oh mein Gott. Endlich! Ich hab die Lösung aller Lösungen. Es funktioniert. Danke für die ganze Hilfe (Funfact: ... die ganze Hilfe, die sich über 9 Tabs zog). Ich bin euch so dankbar!

 

[JStein52]

Ist es eigentlich relevant, wo ich new Database() im Konstruktor aufrufe?

Bevor du es verwendest.

 

[JStein52]

ok.

 

[DaCrazyJavaExpert]

Bevor du es verwendest.

Ok hab das Problem sowieso gelöst.

 

[DaCrazyJavaExpert]

Ok danke nochmal vielmals! Und wie versprochen, war das hier der vorerst letzte Thread zum Thema Datenbanken meinerseits. Ok ich könnte mich 10 Stunden lang bei euch bedanken, deshalb jetzt einfach: Ciao bis zum nächsten mal .

PS: Danke!