Event Handling Externe Method schliesst Hauptapplikation

[henri21]

Ich versuche gerade einen externen PDF Viewer zu starten. Dies funktioniert aber sobald ich das Fenster des PDFViewers schliesse, schliesst dieser das ganze Programm.

Der Viewer ist so in die Hauptapplikation eingebunden:

package app.out;

import com.sun.pdfview.PDFViewer;
import java.io.File;
import java.io.IOException;
import app.GUI;
import support.E_loglevel;
import support.log;

/**
 * Interner PDF Viewer
 * @author 
 * @version 1.0
 */
public class PDFView{
 
    /**
     * PDF öffnen
     * @param file Datei
     * @param title Fenstertitel
     */
    public void viewPDF(File file, String title){

       PDFViewer pdfv = new PDFViewer(false);
       
       try {
            pdfv.openFile(file) ;
        } catch (IOException ex) {
            log.new_msg("Internes PDF konnte nicht geöffnet werden!", E_loglevel.error);
        }
       
        pdfv.setTitle(title);
        pdfv.setIconImage(GUI.get_FGUI().getIconImage()); 
        pdfv.getJMenuBar().getMenu(0).getItem(0).setEnabled(false);
        pdfv.setVisible(true);
      
    }
    
}

Ich habe jergendwie keinen Zugriff auf das JFrame des PDFViewers. Kann ich sonst die Methode setDefaultClose() ändern? Oder Ähnlich?

 

[IMartin]

PDFViewer

ist ein

JFrame

. Der besitzt einen

WindowListener

, der in

windowClosing(WindowEvent)

die Methode

doQuit()

aufruft, welche

System.exit(0)

ausführt.

Möglichkeiten unter anderem: Viewer umschreiben, eigenen Viewer schreiben oder den Listener ersetzen.

 

[Noctarius]

Jeder der irgendwo System.exit aufruft (vor allem in wiederverwendbarem Code) gehört erschlagen, so einfach ist das ;-)
Und wenn jemand meint, bei UI Apps müsste man das, der hat Multithreading nicht verstanden.

 

[henri21]

Problem gelöst.
PDFViewer.class decompiliert -> PDFViewer.jad bearbeitet und wieder compiliert.

Original:

public void doQuit()
    {
        doClose();
        dispose();  
        System.exit(0)
    }

Neu:

public void doQuit()
    {
        doClose();
        this.setVisible(false);
        dispose();   
    }

Vielen Dank!

 

[Noctarius]

Vielleicht wäre es besser eine Subclass mit überschriebener doQuit Methode zu erzeugen anstatt zu dekompilieren. Mit deinem Ansatz musst du daran denken den Schritt jedesmal zu machen wenn du die Version aktualisierst.

 

[henri21]

Stimmt! Besser ist:

package support;

import com.sun.pdfview.PDFViewer;

/**
 * Startet internen PDF Viewer
 * @author
 * @version 1.0
 */
public class kkp7PDFViewer extends PDFViewer {

    public kkp7PDFViewer(boolean thumbs) {
        super(thumbs);
    }
    
    // doQuit überschreiben um System.exit(0) zu umgehen
    @Override
    public void doQuit(){
        doClose();
        this.setVisible(false);
        dispose();
    }
    
}

 

[Noctarius]

Argh der Klassenname entspricht aber nicht den Code Conventions