MySQL Daten werden aus Datenbank gelesen, obwohl sie dort gar nicht angekommen sind

[dackelfrosch]

Ich schreibe momentan an einem Programm, das Daten in eine Datenbank schreibt und aus dieser herausliest. Auf meinem Rechner funktioniert dies auch einwandfrei (sowohl aus Eclipse gestartet, wie auch die .jar).
Nun habe ich das Programm (als executable jar) aber auch auf die Rechner zweier Kollegen kopiert und ihnen den gleichen DB Server (MySQL 5.5) installiert. Auch die Java Version ist auf allen Rechnern die gleiche (7).

Das Problem: Bei meinen Kollegen werden neu angelegte Daten teilweise nicht in der Datenbank gespeichert. Es werden aber auch keine Fehler geworfen!
Das Kurioseste ist, dass das Programm, solange es geöffnet ist die Daten aber kennt und zwar indem es sie aus der DB ausliest!
Ein Beispiel um mein kurioses Problem zu verstehen. Das Programm bietet die Möglichkeit einen Datensatz anzulegen. An ganz anderer Stelle kann ein PopUp-Menu alle angelegten Datensätze anzeigen. Ich habe es hundertmal überprüft: das PopUp holt sich die Datensätze aus der Datenbank!

So wird das Menü befüllt (vereinfacht):

JPopupMenu opp = new JPopupMenu("Dienste");
try {
   List<Duty> duties = this.connector.findAllDutys();
   Collections.sort(duties);
   for (Duty dutyHelper : duties) {
       JMenuItem item = new JMenuItem(dutyHelper.getName());
       opp.add(item);
   }
}

Die Methode im Datenbank-Connector (vereinfacht):

public List<Duty> findAllDutys() throws SQLException 
	{
		List<Duty> duties = new ArrayList<Duty>();		
		
		Statement stmt = this.con.createStatement();
		ResultSet rs = stmt.executeQuery("SELECT * FROM DUTIES");
		while (rs.next()) {
			Duty duty = new Duty();
			duty.setDutyID(rs.getInt("DUTY_ID"));
			duty.setName(rs.getString("NAME"));
                        duties.add(duty);
		}

		rs.close();
		stmt.close();
		
		
		return duties;

	}

Man kann nun also einen Datensatz anlegen, im Programm hin und her klicken, einen Kaffee trinken gehen und was weiß ich nicht treiben, wenn man das PopUp-Menü öffnet sind die Daten drin.
Aber wenn ich in die Datenbank schaue, sind sie dort nicht! Wenn ich das Programm schließe und neu starte, sind auch dort die Daten nicht mehr zu sehen.

Wie gesagt, das Problem tritt nur bei meinen Kollegen auf und auch nicht immer.

Vll. noch zur Vervollständigung die Methode, die die Daten in die DB speichert:

public void createDuty(Duty duty) throws SQLException 
	{		
		Statement stmt = this.con.createStatement();

		stmt.executeUpdate("INSERT INTO DUTIES (NAME)"
				+ "VALUES ('"
				+ duty.getName()
				+")");

		stmt.close();		
	}

Ich bin dezent verzweifelt. Hat jemand eine Idee?

 

[nillehammer]

Puuhh, plain JDBC ist schon ganz schön her... aber versuch mal, nach Deinem executeUpdate ein commit() auf der Connection aufzurufen (also bei Dir this.con.commit() ). Oder hast du die Connection mit autoCommit=true aufgemacht?

 

[dackelfrosch]

Nee, das ist ein guter Einwand. Das werde ich mal machen und die Sache beobachten.
Wäre schön, wenn's klappt.
Danke!

 

[Camino]

Vll. noch zur Vervollständigung die Methode, die die Daten in die DB speichert:

public void createDuty(Duty duty) throws SQLException 
	{		
		Statement stmt = this.con.createStatement();

		stmt.executeUpdate("INSERT INTO DUTIES (NAME)"
				+ "VALUES ('"
				+ duty.getName()
				+")");

		stmt.close();		
	}

Ich bin dezent verzweifelt. Hat jemand eine Idee?

Hmm, mir ist hier aufgefallen, dass in dem SQL-Statement zwischen

(NAME)

und

VALUES

ein Leerzeichen fehlt. Keine Ahnung, ob das ausschlaggebend für das Problem ist. Ansonsten: wenn etwas aus der Datenbank ausgelesen wird, muss es ja auch drinstehen...

 

[nillehammer]

Wenn die Query syntaktisch falsch wäre, würdest Du eine Exception bekommen. Ich könnte mir in diesem Fall vorstellen, dass die DB das fehlende Leerzeichen akzeptiert, weil durch die schließende Klammer die Trennung eindeutig ist.

Aber, wo wir grad dabei sind, in der Query fehlt das schließende ' (also einfaches Tüddelchen). Das Zusammenbauen von Strings ist mit dem +, wie Du siehst schon bei drei Zeilen ziemlich unübersichtlich. Überlege, ob Du evtl. besser mit String.format oder mit einem StringBuilder arbeitest. String.format finde ich persönlich schöner, der StringBuilder ist aber performanter (wenn Du den Unterschied überhaupt bemerkst). Code könnte dann ungefähr so aussehen:

public class DutyDao {
  // Strings für Zusammenbau der Query mittels StringBuilder
  private static final String CREATE_DUTY_1 = "INSERT INTO DUTIES (NAME) VALUES ('";
  private static final String CREATE_DUTY_2 = "')";

  // String als Format für Zusammenbau der Query mittels String.format.
  private static final String CREATE_DUTY_FORMAT = "INSERT INTO DUTIES (NAME) VALUES ('%s')";

    public void createDutyWithStringBuilder(Duty duty) throws SQLException {  
        final String updateQuery = 
            new StringBuilder(CREATE_DUTY_1).append(duty.getName).append(CREATE_DUTY_2);

        doUpdate(updateQuery);         
    }

    public void createDutyWithStringFormat(Duty duty) throws SQLException {

        final String updateQuery = String.format(CREATE_DUTY_FORMAT, duty.getName);

        doUpdate(updateQuery);         
    }


    private static int doUpdate(final String updateQuery) throws SQLException {
        
        final Statement stmt = this.con.createStatement();
 
        final int updatedRowsCount = stmt.executeUpdate(updateQuery);
 
        stmt.close();

        return updatedRowsCount;
    }     
}

 

[Camino]

Oder mit PreparedStatements arbeiten...

 

[Gast2]

Oder mit PreparedStatements arbeiten...

Würde ich auch empfehlen...

 

[dackelfrosch]

Ja, ich werde definitiv noch auf prepared Statements wechseln. Das Programm ist größer angewachsen, als zunächst gedacht. Deswegen ist noch alles in einzelnen SQL-Strings. Und da ständig irgendwelche Neuerungswünsche eintrudeln ist das Refactoring ziemlich auf der Strecke geblieben...

Vielen Dank für die vielen Anmerkungen wg. dem "fehlerhaften" String. Das ist allerdings durchs Kürzen der Methode fürs Forum entstanden. Eigentlich ist der String viel länger, da mein Duty deutlich mehr Attribute halt, als ID und Namen.

Ich habe nillehammers Vorschlag umgesetzt und auf Auto-commit umgestellt. Bisher haben meine Kollegen aber noch nicht wieder mit dem Programm gearbeitet. Ich werde berichten, sobald es erste Erkenntnisse gibt!

 

[dackelfrosch]

Wollte mich wie versprochen nochmal melden: Es scheint bis jetzt zu funktionieren. Danke!