Come può un utente Swing WindowListener JFrame di chiusura

Question

Ho un frame e voglio chiedere quando l'utente lo chiude per salvare il documento. Ma se annullano, la cornice non dovrebbe chiudersi.

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 controlla se il documento è sporco. Se non lo è, non fa nulla e restituisce il vero. Se è sporco, chiede all'utente se vuole salvare (SÌ/NO/ANNULLA). Se l'utente annulla a questo punto, dovrebbe interrompere la finestra Chiusura.

Tutti i suggerimenti che ho visto in rete implicano esplicitamente l'uscita dal metodo windowClosing, annullando quindi l'uso di JFrame.setDefaultCloseOperation() e la duplicazione del codice in JFrame.processWindowEvent().

In realtà ho una soluzione sporca, ma vorrei vedere se ci sono quelli più puliti.

Acclamazioni

Answer 1

Il modo giusto è impostato JFrame.setDefaultCloseOperation-DO_NOTHING_ON_CLOSE quando viene creata la finestra. E poi chiama semplicemente setVisible(false) o dispose() quando l'utente accetta la chiusura o non fa nulla quando la chiusura non viene accettata.

L'intero scopo di JFrame.setDefaultCloseOperation è solo per evitare la necessità di implementare WindowListeners per le azioni più semplici. Le azioni eseguite da queste operazioni di chiusura predefinite sono molto semplici.

EDIT:

Ho aggiunto la soluzione che sto descrivendo. Questo presuppone che tu voglia che il frame sia completamente cancellato.

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(); 
    } 
} 

Answer 2

Penso che questo sia l'espressione logica di risposta di Thirler, e il tipo di quello che stavo cercando di evitare.

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; 
     } 
    } 
} 

Answer 3

Closing an Application potrebbe rendere il processo un po 'più facile per voi.

Answer 4

Grazie all'input di Thirler e camickr. Questa è la mia soluzione, che sono sicuro che alcuni odieranno, ma eviterà la duplicazione della logica in JFrame e consente al codice client di impostareDefaultCloseOperation come al solito.

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(); 
    } 
}