Wie kann ein Swing WindowListener veto JFrame schließen

Ich habe ein frame und möchte auffordern, wenn der Benutzer es schließt, das Dokument zu speichern. Aber, wenn Sie Abbrechen, wird der frame nicht in der Nähe.

frame.addWindowListener(new SaveOnCloseWindowListener(fileState));
...
public class SaveOnCloseWindowListener extends WindowAdapter {
    private final FileState fileState;

    public SaveOnCloseWindowListener(FileState fileState) {
        this.fileState = fileState;
    }

    public void windowClosing(WindowEvent e) {
        if (!fileState.onQuit())
            cancelClose();  
    }
}

FileState sieht, ob das Dokument fehlerhaft ist. Wenn es nicht, es tut nichts und liefert true. Wenn es schmutzig ist, es fragt den Benutzer, ob er speichern möchte (JA/NEIN/ABBRECHEN). Wenn der Benutzer bricht an diesem Punkt sollte es zum Abbruch der windowClosing.

Alle Vorschläge, die ich gesehen habe im Netz beinhalten explizit verlassen in die windowClosing-Methode, so überschreiben die Verwendung von JFrame.setDefaultCloseOperation(), und duplizieren Sie den code in JFrame.processWindowEvent().

Habe ich eigentlich eine dirty Lösung, aber würde gerne sehen, ob es irgendwelche Reiniger lieben.

Cheers

InformationsquelleAutor Duncan McGregor | 2010-09-23
  1. 15

    Den richtigen Weg gesetzt JFrame.setDefaultCloseOperation zu DO_NOTHING_ON_CLOSE wenn das Fenster erstellt wird. Und dann einfach aufrufen setVisible(false) oder dispose() wenn Ihre Benutzer akzeptiert die in der Nähe, oder nichts zu tun, wenn die enge nicht akzeptiert.

    Ist der ganze Zweck des JFrame.setDefaultCloseOperation ist nur zu verhindern, dass die Notwendigkeit zur Umsetzung WindowListeners für die meisten einfachen Aktionen. Die Handlungen, die diese Standard-schließen-Operationen sind sehr einfach.

    EDIT:

    Habe ich die Lösung, die ich beschreibe. Dies setzt Voraus, Sie wollen die Rahmen werden komplett gelöscht.

    frame.setDefaultCloseOperation(setDefaultCloseOperation);
frame.addWindowListener(new SaveOnCloseWindowListener(fileState));
...

public class SaveOnCloseWindowListener extends WindowAdapter {
    private final FileState fileState;

    public SaveOnCloseWindowListener(FileState fileState) {
        this.fileState = fileState;
    }

    public void windowClosing(WindowEvent e) {
        if (fileState.onQuit())
            frame.dispose();
    }
}
    • Aber dann meine Zuhörer brauchen würde, zu wissen, was die gewünschte operation war. Ich glaube, ich könnte es passieren das default-close-operation, und duplizieren Sie den code in processWindowEvent, aber es scheint eine Schande.
    • Wenn Sie meinen, die gewünschte operation durch den Benutzer, dann ja. In den listener, den Sie haben würde, um pop-up-dialog. Also, wenn ich verstehe, was Sie wollen, dass Sie richtig: Sie sollten nicht Fragen, was der Nutzer will, bis Sie die close-Ereignis aus.
    • In der Tat - ich habe es bearbeitet die Frage, um dies klarer, hoffe ich.
    • Eigentlich ist meine Antwort noch gilt. Die einzige änderung ist erforderlich, um die von Ihr angegebenen Beispiel ist, dass Sie aufrufen müssen setDefaultCloseOperation auf die Schöpfung zu hindern, etwas zu tun. Und anstelle der Abbruch der schließen Sie explizit schließen Sie das Fenster, wenn Sie denken, es ist notwendig.
    • Lesen Sie Ihre Antwort, die ich tun, wie "Der ganze Zweck von JFrame.setDefaultCloseOperation ist nur zu verhindern, dass die Notwendigkeit zur Umsetzung WindowListeners für die meisten einfachen Aktionen." Das wäre viel einfacher Ausdrücken, wenn JFrame processWindowEvent delegiert, um einen processCloseOperation, die ich nennen könnte.
    • Ich habe akzeptiert diese Antwort zu geben Thirler den Kredit, und damit die Frage nicht offen bleiben. Persönlich bleibe ich mit meinen hässlichen Ausnahme hack, Sie können wählen Sie die 2.

    InformationsquelleAutor Thirler
  2. 1

    Schließen einer Anwendung machen könnte, den Prozess ein wenig einfacher für Sie.

    • Danke, aber ich verstehe, wie das Zeug wirkt (ich programmiere schon im Gange seit dem import com.Sonne.java.swing.*). Ich versuche zu finden, eine elegante Weise, das problem zu lösen.
    • Ich dachte, ich habe eine elegante Lösung. Gelingt es "fordert" und die "schließen-Vorgang" für Sie. Ich glaube, ich verstehe nicht, was Ihr Anliegen ist.
    • Ich muss gestehen, dass ich gar nicht klicken, um Ihren code. Nun, dass ich - ExitAction nicht wirklich verlassen, schließt er sich der aktive Rahmen, die möglicherweise oder möglicherweise nicht die app beenden.
    • Und ich kann nicht entscheiden, ob ich wie CloseListener Manipulation defaultCloseOperation auf die Fliegen. Es fühlt sich für mich falsch, aber ich kann sehen, dass es macht den job. Also danke, ich werde versuchen, ein bisschen weniger ungracious 😉
    InformationsquelleAutor camickr
  3. 0

    Ich denke, dies ist der logische Ausdruck der Thirler beantworten, und Art von dem, was ich versucht habe zu vermeiden.

    frame.setDefaultCloseOperation(JFrame.DO_NOTHING_ON_CLOSE);
frame.addWindowListener(new SaveOnCloseWindowListener(fileState, JFrame.EXIT_ON_CLOSE));

public class SaveOnCloseWindowListener extends WindowAdapter {

    private final int closeOperation;
    private final Document document;

    public SaveOnCloseWindowListener(int closeOperation, Document document) {
        this.closeOperation = closeOperation;
        this.document = document;
    }

    public void windowClosing(WindowEvent e) {
        if (!document.onClose())
            doClose((Window) e.getSource());
    }

    private void doClose(Window w) {
        switch (closeOperation) {
        case JFrame.HIDE_ON_CLOSE:
            w.setVisible(false);
            break;
        case JFrame.DISPOSE_ON_CLOSE:
            w.dispose();
            break;
        case JFrame.DO_NOTHING_ON_CLOSE:
        default:
            break;
        case JFrame.EXIT_ON_CLOSE:
            System.exit(0);
            break;
        }
    }
}
    • Der Grund, warum ich nicht wie das ist, weil es kopiert den code in JFrame, und erfordert die frame-Unterklasse, zu wissen, was die erforderliche operation ist, wenn Sie es schafft das Fenster Zuhörer. Ich nehme an, ich könnte das überschreiben setDefaultCloseOperation weitergeben der operation an die Hörer...
    • Ich passe meine Antwort mit der Lösung, die ich meine. Im Allgemeinen sind Sie derjenige, der macht den JFrame, damit Sie wissen, welche operation zu schließen, die Sie wollen, im Kern deaktivieren Sie die operation zu schließen.
    • Ich denke, das ist, wo wir uns unterscheiden. Ich möchte ein JFrame, das in der Lage ist, ein veto einzulegen schließt, wenn der Benutzer sagt, dass Sie nicht wollen, um es zu schließen, aber wo die default-close-operation ist eine Politik, von der Anwendung, nicht der Rahmen.
    InformationsquelleAutor Duncan McGregor
  4. 0

    Dank der Eingabe von Thirler und camickr. Dies ist meine Lösung, die ich sicher bin, dass einige hassen werden, aber vermeidet das duplizieren von Logik im JFrame, und erlaubt den client-code zu setDefaultCloseOperation wie gewohnt.

    class MyFrame {        
    @Override protected void processWindowEvent(WindowEvent e) {
        try {
            super.processWindowEvent(e);
        } catch (SaveOnCloseWindowListener.VetoException x) {}
    }
}

public class SaveOnCloseWindowListener extends WindowAdapter {

    public static class VetoException extends RuntimeException {
    }

    private final DocumentController documentController;

    public SaveOnCloseWindowListener(DocumentController documentController) {
        this.documentController = documentController;
    }

    public void windowClosing(WindowEvent e) {
        if (!documentController.onClose())
            throw new VetoException();
    }
}
    InformationsquelleAutor Duncan McGregor
Schreibe einen Kommentar