MySQL Löschen von Rows mit JDBC

[JDE4D]

public static void remCandidate(String RUBCardID, String serviceID) {
        if (!existsRubID(RUBCardID, serviceID)) {
            try {
                statement = connect.createStatement();
                preparedStatement = connect.prepareStatement("SELECT * FROM dienstberechtigung WHERE Person = ? AND Dienst = ? AND Rolle = ?");
                preparedStatement.setString(1, RUBCardID);
                preparedStatement.setString(2, serviceID);
                preparedStatement.setString(3, "Candidate");

                resultSet = preparedStatement.executeQuery();

                if (resultSet.getConcurrency() == ResultSet.CONCUR_READ_ONLY) {
                    System.out.println("ResultSet non-updatable.");
                } else {
                    System.out.println("ResultSet updatable.");
                }
                while (resultSet.next()) {
                    System.out.println(resultSet.getString("Person"));
                    resultSet.deleteRow();
                }
            } catch (SQLException e) {
                System.out.println(e);
            }
        } else {
        }
    }

Ich weiss nicht wo das problem liegt.. bekomme keinelei Exception zurück... Es tut sich einfach nichts in der DB.

Verbindung steht, ich kann in der Tabelle von einer anderen Methode was reinschreiben aber das löschen scheint nicht zu klappen.

Einer ne idee?

 

[Peter W. Marth]

Hallo,

man sollte zunächst sicherstellen, dass das ResultSet upgedatet werden kann:

A default ResultSet object is not updatable and has a cursor that moves forward only. Thus, you can iterate through it only once and only from the first row to the last row. It is possible to produce ResultSet objects that are scrollable and/or updatable. The following code fragment, in which con is a valid Connection object, illustrates how to make a result set that is scrollable and insensitive to updates by others, and that is updatable. See ResultSet fields for other options.

Statement stmt = con.createStatement(
ResultSet.TYPE_SCROLL_INSENSITIVE,
ResultSet.CONCUR_UPDATABLE);
ResultSet rs = stmt.executeQuery("SELECT a, b FROM TABLE2");
// rs will be scrollable, will not show changes made by others,
// and will be updatable

(Quelle: ResultSet (Java Platform SE 6))

Auch kann es sein, dass einfach ein commit fehlt, falls bei der JDBC-Connection kein Autocommit eingestellt ist.

 

[JDE4D]

Wie du im Code festgestellt hast, uberprüfe ich ja ob es updateable... er sagt mir aber das es nicht ist.. was zutun?

 

[SlaterB]

du überprüfst OB, setzt aber nicht wie vorgeschlagen die Einstellung, DAMIT es Updatable ist,
bemerkst du den Unterschied?

wenn alles nicht geht dann musst du die Ids oder sonst was genaues merken und später ein normales DELETE per SQL durchführen

 

[JDE4D]

So sieht der Code aus:

public static void remCandidate(String RUBCardID, String serviceID) {
            if (existsRubID(RUBCardID, serviceID)) {
                try {
                    statement = connect.createStatement(ResultSet.TYPE_SCROLL_INSENSITIVE, ResultSet.CONCUR_UPDATABLE);
                    preparedStatement = connect.prepareStatement("SELECT * FROM dienstberechtigung WHERE Person = ? AND Dienst = ? AND Rolle = ?");
                    preparedStatement.setString(1, RUBCardID);
                    preparedStatement.setString(2, serviceID);
                    preparedStatement.setString(3, "Candidate");

                    resultSet = preparedStatement.executeQuery();

                    if (resultSet.getConcurrency() == ResultSet.CONCUR_READ_ONLY) {
                        System.out.println("ResultSet non-updatable.");
                    } else {
                        System.out.println("ResultSet updatable.");
                    }
                    while (resultSet.next()) {
                        System.out.println(resultSet.getString("Person"));
                        resultSet.deleteRow();
                    }
                } catch (SQLException e) {
                    System.out.println(e);
                }
            } else {
            }
        }

bekomme jetzt diesen Error :

com.mysql.jdbc.NotUpdatable: Result Set not updatable.This result set must come from a statement that was created with a result set type of ResultSet.CONCUR_UPDATABLE, the query must select only one table, can not use functions and must select all primary keys from that table. See the JDBC 2.1 API Specification, section 5.6 for more details.This result set must come from a statement that was created with a result set type of ResultSet.CONCUR_UPDATABLE, the query must select only one table, can not use functions and must select all primary keys from that table. See the JDBC 2.1 API Specification, section 5.6 for more details.

 

[SlaterB]

und vorher wirklich nichts?
erstaunlich, du hast doch nur die Zeile 5 geändert, und die spielt letztlich keine Rolle,
du verwendest das PrepatedStatement von Zeile 6, also solltest du auf diese Zeile mehr Wert legen

 

[JDE4D]

was muss ich dort ändern? Hab versucht das updatable dort zu setzen aber das mag er nicht. Ich arbeite seit eine Woche mit MySQL und JDBC..

 

[SlaterB]

welchen Code hast du mit welcher Fehlermeldung?
grob in die API geschaut musst du die Parameter nur exakt herübernehmen, aber in die API kannst du auch selber schauen,

und provozierter Kommentar:
eine Woche an die Wand starren hilft nix, lieber 10 Min. strukturiert vorgehen, z.B. jetzt schauen was Connection an Methoden bietet um PreparedStatement zu erzeugen, mit welchen Parametern, welche Werte sind dort für was geeignet usw.

 

[JDE4D]

public static void remCandidate(String RUBCardID, String serviceID) {
            if (!existsRubID(RUBCardID, serviceID)) {
                try {
                    //statement = connect.createStatement(ResultSet.TYPE_SCROLL_INSENSITIVE, ResultSet.CONCUR_UPDATABLE);
                    preparedStatement = connect.prepareStatement("SELECT * FROM dienstberechtigung WHERE Person = ? AND Dienst = ? AND Rolle = ?", ResultSet.TYPE_SCROLL_SENSITIVE, ResultSet.CONCUR_UPDATABLE);
                    preparedStatement.setString(1, RUBCardID);
                    preparedStatement.setString(2, serviceID);
                    preparedStatement.setString(3, "Candidate");

                    resultSet = preparedStatement.executeQuery();

                    if (resultSet.getConcurrency() == ResultSet.CONCUR_READ_ONLY) {
                        System.out.println("ResultSet non-updatable.");
                    } else {
                        System.out.println("ResultSet updatable.");
                    }
                    while (resultSet.next()) {
                        System.out.println(resultSet.getString("Person"));
                        resultSet.deleteRow();
                    }
                } catch (SQLException e) {
                    System.out.println(e);
                }
            } else {
            }
        }

Wie du siehst müsste jetzt das Resultset updatable sein, ist es aber anscheinend nicht:

com.mysql.jdbc.NotUpdatable: Result Set not updatable (referenced table has no primary keys).This result set must come from a statement that was created with a result set type of ResultSet.CONCUR_UPDATABLE, the query must select only one table, can not use functions and must select all primary keys from that table. See the JDBC 2.1 API Specification, section 5.6 for more details.

;(

 

[SlaterB]

die geänderte Fehlermeldung gibt anscheinend auch den genauen Grund: keine Primary Key,

wie in meinem ersten Posting gesagt kann man letztendlich evtl. auch Pech haben, dass der Treiber das grundsätzlich nicht unterstützt,
die Alternative hatte ich da genannt:
> wenn alles nicht geht dann musst du die Ids oder sonst was genaues merken und später ein normales DELETE per SQL durchführen