Pusta formatka

Witam,
Chciałbym zapytać o sprawę tworzenia okienka w javie.
Otóż mam taki konstruktor: GUI(){
ramka = new JFrame();
panel1 = new JPanel();
panel2 = new JPanel();
ramka.setDefaultCloseOperation(ramka.EXIT_ON_CLOSE);
ramka.setResizable(false);
ramka.setSize(400, 250);
ramka.setVisible(true);

} 

Reszta operacji, takich jak dodawanie przycisków itp dzieje się w specjalnej funkcji wywoływanej od razy po wykonaniu się tego konstruktora. Pierwsze pytanie: Czy takie podejście jest dobre? Czy może lepiej całą formatkę tworzyć w konstruktorze?

I druga sprawa... otóż raz na jakieś 4-5 uruchomień programu pojawia się pusta formatka (bez przycisków, pól itp), tak jakby sama ramka z szarym wypełnieniem. Gdy tylko zacznę ją choć odrobinkę rozszerzać to pojawiają się wszystkie elementy i wszystko śmiga jak należy. Co może być tym spowodowane? Wg mnie sprawa może leżeć po stronie tworzenia formatki ale co?...
Z góry dzięki za pomoc ;) pzdr

Cześć.
Opisany przez Ciebie problem występuje zwykle, gdy tworzy się GUI po za wątkiem Swinga.

Innymi słowy, najprościej zrobić coś takiego:

 int main(String args[]){
		SwingUtilities.invokeLater(new Runnable() {
			
			@Override
			public void run() {
                               GUI gui = new GUI();
				
			}
		});
} 

Chociaż osobiście preferuję coś takiego:

	public static void main(String[] args) {
		SwingUtilities.invokeLater(new Runnable() {
			
			@Override
			public void run() {
				MainWindow window = new MainWindow();
				window.setVisible(true);
				
			}
		});

	}
 

Konstruktor mainWindow tylko tworzy w pamięci elementy wyglądu i dodaje je na MainWindow.
Natomiast w konstruktorze nie powinno się robić setVisible.

Jeśli kiedyś zrobisz klasę, która dziedziczy po klasie GUI, to w momencie wywołania konstruktora klasy nadrzędnej od razu wyskoczy ci okienko.
I będziesz miał dziwne błędy.
Czyli w twoim wypadku powinieneś w metodzie main stworzyć obiekt ramki i potem dać mu setVisible = true;

W razie pytań służę pomocą.
Pozdrawiam.

w NetBeans wybierająć Nowy projekt /Java/ Java Class Library , a następnie utworzenie klasy (New/JFrame Form) "test" wygeneruję Ci następujący kod:

public class test extends javax.swing.JFrame {

    /** Creates new form test */
    public test() {
        initComponents();
    }

    /** This method is called from within the constructor to
     * initialize the form.
     * WARNING: Do NOT modify this code. The content of this method is
     * always regenerated by the Form Editor.
     */
    @SuppressWarnings("unchecked")
    // <editor-fold defaultstate="collapsed" desc="Generated Code">
    private void initComponents() {

        setDefaultCloseOperation(javax.swing.WindowConstants.EXIT_ON_CLOSE);

        javax.swing.GroupLayout layout = new javax.swing.GroupLayout(getContentPane());
        getContentPane().setLayout(layout);
        layout.setHorizontalGroup(
            layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGap(0, 400, Short.MAX_VALUE)
        );
        layout.setVerticalGroup(
            layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGap(0, 300, Short.MAX_VALUE)
        );

        pack();
    }// </editor-fold>

    /**
    * @param args the command line arguments
    */
    public static void main(String args[]) {
        java.awt.EventQueue.invokeLater(new Runnable() {
            public void run() {
                new test().setVisible(true);
            }
        });
    }

    // Variables declaration - do not modify
    // End of variables declaration

}

gdzie:
new test().setVisible(true);
oznacza utworzenie obiektu, "test" wywołujący metodę setVisible(true) z parametrem o wartości oznaczającym, iż ramka będzie widoczna.