[JAVA]: Impossibile chiudere l'interfaccia grafica

[zenzero]

Ciao a tutti,
ho un piccolo problema e sicuramente mi sfugge qualche banalità.
Le tre classi mostrate sotto, generano una semplice GUI con un tasto che quando premuto viene mostrata una label sempre all'interno della GUI stessa. Non ottengo nessun errore ma quando provo a chiudere la GUI questa non nè vuole sapere.

Grazie
ciao

P.S.: chiedo scusa al moderatore ALKA che mi ha chiuso il post precedente per mia negligenza, prometto che ciò non accadrà più.

Di seguito il codice delle tre classi:
-----------------------------------------------------------------------------------
prima classe:
-----------------------------------------------------------------------------------
public class EseGestEveClassiAnonime
{
public static void main(String[] args)
{
ModelloDelegazioneAnonimo model=new ModelloDelegazioneAnonimo();
ChiudGui ch=new ChiudGui();
model.setup();
ch.chiudi();
}
}

-------------------------------------------------------------------------------------
seconda clsse:
-------------------------------------------------------------------------------------
import java.awt.*;
import java.awt.event.*;

public class ModelloDelegazioneAnonimo
{
private Frame fr;
private Button butt;
private Label l;

public ModelloDelegazioneAnonimo()
{
fr=new Frame("Modello delegazione anonimo");
butt=new Button("Premi questo tasto");
l=new Label();
}

public void setup()
{
butt.addActionListener(new ActionListener()
{
private int counter;
public void actionPerformed(ActionEvent e)
{
l.setText(e.getActionCommand()+"-"+(++counter));
}
});

fr.add(butt,BorderLayout.SOUTH);
fr.add(l);
fr.pack();
fr.setVisible(true);
}

public Frame get()
{
return fr;
}
}

--------------------------------------------------------------------------------
terza classe:
--------------------------------------------------------------------------------
import java.awt.*;
import java.awt.event.*;

public class ChiudGui extends ModelloDelegazioneAnonimo
{
private Frame frame;
public ChiudGui()
{
frame=this.get();
}

public void chiudi()
{
frame.addWindowListener(new WindowListener()
{
public void windowClosing (WindowEvent ev)
{
System.exit(0);
}
public void windowClosed (WindowEvent ev) {}
public void windowOpened (WindowEvent ev) {}
public void windowActivated (WindowEvent ev) {}
public void windowDeactivated (WindowEvent ev) {}
public void windowIconified (WindowEvent ev) {}
public void windowDeiconified (WindowEvent ev) {}
});
}
}
---------------------------------------------------------------------------------

[LeleFT]

Perchè tu tenti di chiudere (e si chiude!!) una finestra che non hai mai "aperto" (o meglio, non l'hai mai visualizzata).

Tu hai creato 2 oggetti:
1) ModelloDelegazioneAnonimo (che è una finestra, quella che vedi!!)
2) ChiudGui (che in realtà ha una finestra nulla)

Tu chiami il metodo chiudi() sulla seconda (giustamente... la prima non ce l'ha).

Io ti consiglio di cambiare completamente metodo di programmazione: è inutile creare tante classi per niente, puoi fare tutto con una sola. Un'unica classe che estende Frame. La seconda (ChiudGui) non serve proprio a nessuno.


Ciao.

[Matteo982]

Originariamente inviato da LeleFT
Io ti consiglio di cambiare completamente metodo di programmazione: è inutile creare tante classi per niente, puoi fare tutto con una sola. Un'unica classe che estende Frame. La seconda (ChiudGui) non serve proprio a nessuno.

Ciao.

Effettivamente con questo modo di programmare appena ti capita di fare qualcosa di un po' più complesso ti ritrovi con 50 classi e la cosa non è proprio il massimo...è uno spreco inutile

Ciao.