JOptionPane sinnvoll benutzen

[schina02]

Heyho, hab da mal eine bestimmt ganz einfache Frage an euch. hab folgendes vor:

Wenn der LÖSCHEN-Button gedrückt wird, soll noch einmal sicher gestellt werden, ob der User das wirklich wollte, also ein Fenster mit Ja und Nein. Er soll logischerweise nur bei Ja wirklich löschen.

mein quellcode sieht so aus:

            Object[] options = { "Ja", "Nein" };
            int eingabe =
                JOptionPane.showOptionDialog(null, "Sind Sie sicher, dass dieser Datensatz gelöscht werden soll?",
                                             "Sicher?", JOptionPane.DEFAULT_OPTION, JOptionPane.INFORMATION_MESSAGE,
                                             null, options, options[0]);
            if ((eingabe == 0)) {
                JOptionPane.showMessageDialog(null, "Okay, ein Datensatz wurde gelöscht.");
            }
            else // ???????????

mein problem: er löscht, egal was gedrückt wird: bei Ja, bei Nein und sogar beim Kreuz oben rechts.

was muss ich in die else reinschreiben?

 

[Camino]

mein problem: er löscht, egal was gedrückt wird: bei Ja, bei Nein und sogar beim Kreuz oben rechts.

was muss ich in die else reinschreiben?

Dann hast du dein tatsächliches Löschen wohl an der falschen Stelle. In dem Code kann ich das leider nirgendwo entdecken. Wo wird denn gelöscht? Du musst schon den kompletten und tatsächlichen Code hier posten, bei dem man das dann auch sehen und erkennen kann.

Bei else musst du das reinschreiben, was du machen willst, wenn die Bedingung bei if nicht zutrifft.

 

[schina02]

sorry, wieder mein fehler:

    private void btnDelete_actionPerformed(ActionEvent e) {

        try {
            String delete = "DELETE FROM sortiment WHERE id = ? ";
            PreparedStatement ps = con.prepareStatement(delete);

            ps.setInt(1, Integer.parseInt(txtID.getText()));

            int n = ps.executeUpdate();
            Object[] options = { "Ja", "Nein" };
            int eingabe =
                JOptionPane.showOptionDialog(null, "Sind Sie sicher, dass dieser Datensatz gelöscht werden soll?",
                                             "Sicher?", JOptionPane.DEFAULT_OPTION, JOptionPane.INFORMATION_MESSAGE,
                                             null, options, options[0]);
            
            if ((eingabe == 0)) {
                JOptionPane.showMessageDialog(null, "Okay, ein Datensatz wurde gelöscht.");
            }
            
            else // ????????????


        } catch (Exception ex) {
            ex.printStackTrace();
            JOptionPane.showMessageDialog(null, "Oh nein, das Löschen hat nicht funktioniert.");
        }


    }

was kann ich in die else schreiben, damit er das löschen abbricht?

 

[HimBromBeere]

was muss ich in die else reinschreiben?

Die Frage ist weniger, was DORT reingehört als vielmehr, ob dieser Codeteil überhaupt erreicht wird. Ich sehe aber nichts, was dafür spräche, dass der else-Zwieg nicht erriecht wird, das musst du schon selbst beuirteilen (kommt die "Okay"-Nachricht immer oder nur bei "ja"?).

[EDIT]Schonmal dasangeguckt? Da steht doch genau, was z.B. für den von Kreuz gedrückt von der Funktion zurückgegeben wird.[/EDIT]

 

[Camino]

was kann ich in die else schreiben, damit er das löschen abbricht?

Du hast eine falsche Reihenfolge drin. Du löscht zuerst, und danach fragst du ab, ob gelöscht werden soll. Du musst das tatsächliche Löschen natürlich in den if-Block verschieben, also nur wenn die if-Bedingung zutrifft, wird auch gelöscht. Wenn nicht, dann passiert halt nichts (und du brauchst das else garnicht), oder es soll was anderes passieren, das schreibst du dann bei else rein.

 

[schina02]

Jo, der Codeteil wird erreicht und das genauso wie ich es will. Ich drück den Button btnDelete und zack kommt mein JOptionPane.

Die Okay-Nachricht kommt nur bei "Ja"

Bei "Nein" und beim "Kreuz drücken" soll er das Löschen abbrechen

 

[Camino]

Jo, der Codeteil wird erreicht und das genauso wie ich es will. Ich drück den Button btnDelete und zack kommt mein JOptionPane.

Die Okay-Nachricht kommt nur bei "Ja"

Bei "Nein" und beim "Kreuz drücken" soll er das Löschen abbrechen

Das ist zu spät. Wenn das OptionPane kommt, ist doch schon gelöscht, weil du das Löschen vor der Anzeige des OptionPane eingebaut hast. Das kannst du dann garnicht mehr abbrechen.

 

[schina02]

Du hast eine falsche Reihenfolge drin. Du löscht zuerst, und danach fragst du ab, ob gelöscht werden soll.

das war der hinweis, den ich brauchte. sorry, richtig richtig blöd alles nur menschen zum glück!!!

fettes dankeschön, so funktioniert es:

    private void btnDelete_actionPerformed(ActionEvent e) {

        try {
            Object[] options = { "Ja", "Nein" };
            int eingabe =
                JOptionPane.showOptionDialog(null, "Sind Sie sicher, dass dieser Datensatz gelöscht werden soll?",
                                             "Sicher?", JOptionPane.DEFAULT_OPTION, JOptionPane.INFORMATION_MESSAGE,
                                             null, options, options[0]);

            if ((eingabe == 0)) {


                String delete = "DELETE FROM sortiment WHERE id = ? ";
                PreparedStatement ps = con.prepareStatement(delete);

                ps.setInt(1, Integer.parseInt(txtID.getText()));

                int n = ps.executeUpdate();
            }
            
        } catch (Exception ex) {
            ex.printStackTrace();
            JOptionPane.showMessageDialog(null, "Oh nein, das Löschen hat nicht funktioniert.");
        }

    }