Swing neues Fenster nicht sichtbar

[Wirago]

Ich habe einfache GUI mit einigen Buttons, ComboBoxen, Labels etc, einer der Buttons soll ein neues Fenster aufscheinen lassen. Allerdings tut sich beim click einfach nichts.
Ist wahrscheinlich ein einfaches Problem nur komm ich leider nicht drauf.

ActionListener des Buttons, der das Fenster aufrufen soll:

    	btnAddItem.addActionListener(new ActionListener()
		{
			
			@Override
			public void actionPerformed(ActionEvent arg0)
			{
				
				javax.swing.SwingUtilities.invokeLater(new Runnable()
				{
					@Override
					public void run()
					{
						new newItem().setVisible(true);
					}
				});
			}
		});

die Klasse newItem ist einfach nur ein im NetBeans zusammengeklicktes Fenster und hat hat eigentlich nur die InitComponents-Methode.

public class newItem extends javax.swing.JPanel {

	private static final long serialVersionUID = 1L;
	public newItem() {
        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.
     */
   
    // <editor-fold defaultstate="collapsed" desc="Generated Code">
    private void initComponents() {

        jLabel1 = new javax.swing.JLabel();
        fieldFinish = new javax.swing.JTextField();
        fieldName = new javax.swing.JTextField();
        jLabel4 = new javax.swing.JLabel();
        jLabel3 = new javax.swing.JLabel();
        jLabel2 = new javax.swing.JLabel();
        fieldColor = new javax.swing.JTextField();
        fieldBrand = new javax.swing.JTextField();
        listTypes = new javax.swing.JComboBox<String>();
        jToggleButton1 = new javax.swing.JToggleButton();
        jScrollPane1 = new javax.swing.JScrollPane();
        fieldComment = new javax.swing.JTextArea();

        jLabel1.setText("Name: ");

        fieldFinish.addActionListener(new java.awt.event.ActionListener() {
            public void actionPerformed(java.awt.event.ActionEvent evt) {
                fieldFinishActionPerformed(evt);
            }
        });

        jLabel4.setText("Finish: ");

        jLabel3.setText("Color: ");

        jLabel2.setText("Brand: ");

        listTypes.setModel(new javax.swing.DefaultComboBoxModel<String>(new String[] { "Item 1", "Item 2", "Item 3", "Item 4" }));

        jToggleButton1.setText("OK");

        fieldComment.setColumns(20);
        fieldComment.setRows(5);
        fieldComment.setText("Comment");
        jScrollPane1.setViewportView(fieldComment);

        javax.swing.GroupLayout layout = new javax.swing.GroupLayout(this);
        this.setLayout(layout);
        layout.setHorizontalGroup(
            layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGroup(layout.createSequentialGroup()
                .addContainerGap()
                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING, false)
                    .addGroup(layout.createSequentialGroup()
                        .addComponent(jLabel1)
                        .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.UNRELATED)
                        .addComponent(fieldName))
                    .addGroup(layout.createSequentialGroup()
                        .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
                            .addComponent(jLabel2)
                            .addComponent(jLabel3)
                            .addComponent(jLabel4))
                        .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.UNRELATED)
                        .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
                            .addComponent(fieldColor)
                            .addComponent(fieldBrand)
                            .addGroup(layout.createSequentialGroup()
                                .addComponent(fieldFinish, javax.swing.GroupLayout.PREFERRED_SIZE, 121, javax.swing.GroupLayout.PREFERRED_SIZE)
                                .addGap(0, 0, Short.MAX_VALUE))))
                    .addComponent(jScrollPane1, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE)
                    .addGroup(layout.createSequentialGroup()
                        .addComponent(listTypes, javax.swing.GroupLayout.PREFERRED_SIZE, 84, javax.swing.GroupLayout.PREFERRED_SIZE)
                        .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.UNRELATED)
                        .addComponent(jToggleButton1, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, Short.MAX_VALUE)))
                .addContainerGap(javax.swing.GroupLayout.DEFAULT_SIZE, Short.MAX_VALUE))
        );
        layout.setVerticalGroup(
            layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGroup(layout.createSequentialGroup()
                .addContainerGap()
                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.BASELINE)
                    .addComponent(jLabel1)
                    .addComponent(fieldName, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE))
                .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.RELATED)
                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.BASELINE)
                    .addComponent(jLabel2)
                    .addComponent(fieldBrand, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE))
                .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.RELATED)
                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.BASELINE)
                    .addComponent(jLabel3)
                    .addComponent(fieldColor, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE))
                .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.RELATED)
                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.BASELINE)
                    .addComponent(jLabel4)
                    .addComponent(fieldFinish, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE))
                .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.UNRELATED)
                .addComponent(jScrollPane1, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE)
                .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.UNRELATED)
                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.BASELINE)
                    .addComponent(listTypes, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE)
                    .addComponent(jToggleButton1))
                .addContainerGap(javax.swing.GroupLayout.DEFAULT_SIZE, Short.MAX_VALUE))
        );
        
    }// </editor-fold>

    private void fieldFinishActionPerformed(java.awt.event.ActionEvent evt) {
        // TODO add your handling code here:
    }

    // Variables declaration - do not modify
    private javax.swing.JTextField fieldBrand;
    private javax.swing.JTextField fieldColor;
    private javax.swing.JTextArea fieldComment;
    private javax.swing.JTextField fieldFinish;
    private javax.swing.JTextField fieldName;
    private javax.swing.JLabel jLabel1;
    private javax.swing.JLabel jLabel2;
    private javax.swing.JLabel jLabel3;
    private javax.swing.JLabel jLabel4;
    private javax.swing.JScrollPane jScrollPane1;
    private javax.swing.JToggleButton jToggleButton1;
    private javax.swing.JComboBox<String> listTypes;
    // End of variables declaration
}

 

[Volvagia]

Gott, der Code sieht zum :wuerg: aus.
Versuch mal die Klasse mit dem mieserablen Namen von einer Subklasse von Window und nicht von einen Panel erben zu lassen.

In einem Listener brauchst du den Code nicht in die Queue invoken, alle Listener werden standartgemäß per EDT abgearbeitet.

 

[Wirago]

oh, das mim listener wusste ich nicht.
Für den tollen Code kann ich nix... der ist generiert

Das mim Window haut nicht ganz so hin, da maleurt mir der Konstruktor.

Vllt. liegts daran, dass das "Haupt"-GUI ein JFrame ist? Kanns da irgendwelche Besonderheiten geben?

 

[Volvagia]

Ne, Subklasse von Window. JFrame, JDialog oder JWindow bei Swing. Hast ja gesagt das er generiert ist, interessanter finde ich es wie du den jemals ändern willst.

 

[Wirago]

bei JDialog/JWindow und JFrame kommt es zur Runtime zu folgendem Fehler:
java.lang.IllegalArgumentException: GroupLayout can only be used with one Container at a time

Das zu Ändern ist natürlich absoluter Wahnsinn. Deshalb steht auch vor der initComponents das Kommentar "WARNING: Do NOT modify this code. The content of this method is always regenerated by the Form Editor."
Im Prinzip geht NetBeans davon aus, dass man den Code weder ansieht noch anfasst, sondern im netBeans-Editor ändert.

 

[Volvagia]

Ich kenn mich (Gott sei dank) damit nicht aus, aber blind geraten musst du dem GroupLayout bei der Erzeugung das ContentPane des Fensters übergeben.

 

[Wirago]

jop, hast recht. habs auch selber grad herausgefunden...

hab einfach

javax.swing.GroupLayout layout = new javax.swing.GroupLayout(this);
this.setLayout(layout);

durch

javax.swing.GroupLayout layout = new javax.swing.GroupLayout(getContentPane());
getContentPane().setLayout(layout);

ersetzt... klappt jetzt einwandfrei.

danke trotzdem vielmals


PS:
ich mochte die GUI-Programmierung sowieso nie ^^