Einfach if-Abfrage

[Reality]

Hi,
ich habe ein kleines Fremdwörterbuch programmiert.
Das Problem: Wenn ich nun ein Wort ändern will, zeigt es zwar an, dass die Änderung erfolgreich war, aber das tut es auch, wenn der Eintrag nicht existierte.

Folgendes geht nicht:

Statement stmt=con.createStatement();
int updaten=stmt.executeUpdate(update.toString());
  if(stmt!=null)
  {
  lbUpdate.setText("Update war erfolgreich!");
  }
  else
  {
  lbUpdate.setText("Update fehlgeschlagen");
  }

Es liefert immer, dass der Update erfolgreich war.

Wie kann ich das Problem lösen?

Liebe Grüße
Reality

 

[lhein]

Das Statement-Objekt ist normalerweise nie NULL, weil NULL hiesse nicht initialisiert bzw. instanziiert.
Es gibt aber bei einem Statement Metadaten glaube ich, an denen man erkennt, wieviele rows von dem statement affected sind.

Dein Ansatz scheint mir zumindest nicht korrekt.

LR

 

[Miskral]

Hi reality,

bei deinem Source Code muß das auch so sein.

Da du das Objekt stmt auf null überprüfst. 2 Zeilen vorher erstelltst du aber erst das Objekt somit kann die Variable niemals null sein.

Die richtig if abfrage wäre :

if(updaten != 0)

Gruß Chris

 

[lhein]

Genau,

executeUpdate

public int executeUpdate(String sql)
throws SQLException

Executes the given SQL statement, which may be an INSERT, UPDATE, or DELETE statement or an SQL statement that returns nothing, such as an SQL DDL statement.

Parameters:
sql - an SQL INSERT, UPDATE or DELETE statement or an SQL statement that returns nothing
Returns:
either the row count for INSERT, UPDATE or DELETE statements, or 0 for SQL statements that return nothing
Throws:
SQLException - if a database access error occurs or the given SQL statement produces a ResultSet object

so solltest du die Rows affected rausbekommen.

LR

 

[Java-Kämpfer]

Hallöle,

ich würde keine if Abfrage machen. In der API Steht das da eine SQL Exception geworfen wird.
Mach lieber einen Try Catch Block.

try{
    Statement stmt=con.createStatement(); 
    int updaten=stmt.executeUpdate(update.toString()); 
}
catch  (SQLException e){
     lbUpdate.setText("Update fehlgeschlagen"); 
}

Ich glaube das geht besser

 

[lhein]

Das ist so leider falsch.

Es kann durchaus sein, dass das UPDATE oder DELETE Statement durch eine WHERE Clause so beschränkt wurde, dass es auf keinen Datensatz in der Datenbank zutrifft. In diesem Fall würde es 0 zurückliefern und keine Exception werfen.

Exceptions sind harte Fehler und eignen sich daher für solche Prüfungen eher nicht.

LR

 

[Java-Kämpfer]

wieder schlauer.... :wink:

 

[Reality]

Hi,
danke Miskral für den Lösungsvorschlag und auch an allen anderen! Hätte ich selbst drauf kommen können....
Dass es mit stmt!=null nicht geht, war mir klar. Mir war es wichtig, dass ihr wisst, worauf ich hinaus will.

Danke nochmals!

Liebe Grüße
Reality