Thread: programmimng problem or bug?

NetBeans 8.0.1
This is the 3rd time I've rebuilt this program.
My problem seems to be when I add the Menu's.
When I use RUN, the menu's sometimes show up or sometimes NOT.
When I use DEBUG it works every time.
This is the smallest I can get the program before the problem
starts.
When I add the menu's it fails?

 
 
package myclockpaneltest;
 
import java.awt.Color;
import java.awt.Dimension;
import java.awt.Font;
import javax.swing.*;
 
public class MyclockPanelTest {
 
    public JFrame f = new JFrame();
    public JPanel main = new JPanel();
    public JPanel p1 = new JPanel();
    public JPanel p2 = new JPanel();
    public JMenuBar jmb = new JMenuBar();
    public JMenu jm0, jm1;
    public JMenuItem mi0, mi1;
 
 
    public MyclockPanelTest(){
        f.setVisible(true);
        f.setSize(357,165);
        f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
 
    main.setBackground(Color.BLUE);
    f.add(main);
    main.add(p1);
    main.add(p2);
    p1.setPreferredSize(new Dimension(70,90));
    p1.setBackground(Color.ORANGE);
    p2.setPreferredSize(new Dimension(260,90));
    p2.setBackground(Color.BLACK);
 
    f.setJMenuBar(jmb);
    jm0 = new JMenu("Close");
          mi0 = new JMenuItem("Exit");
          jm0.add(mi0);
    jm1 = new JMenu("Time");
          mi1 = new JMenuItem("24-Hour clock");   
          jm1.add(mi1);
      jmb.add(jm0);
      jmb.add(jm1);     
    }
 
 
 
    public static void main(String[] args) {
        // TODO code application logic here
    new MyclockPanelTest();
    }
}

Post the code that adds the menus and causes the failure.

Swing applications should be run on the EDT.

Use better variable names.

Build the UI, pack() it, and then make it visible.

Originally Posted by GregBrannon

Post the code that adds the menus and causes the failure.

Swing applications should be run on the EDT.

Use better variable names.

Build the UI, pack() it, and then make it visible.

The entire code is posted.
the menus are below setJMenubar

The code I posted is completely runnable.

I don't understand Swing applications should be run on the EDT.

I misunderstood, "This is the smallest I can get the program before the problem starts."

"Build the UI, pack() it, and then make it visible," means to move the following statement to the last line in the constructor:

f.setVisible(true);

Originally Posted by GregBrannon

I misunderstood, "This is the smallest I can get the program before the problem starts."

"Build the UI, pack() it, and then make it visible," means to move the following statement to the last line in the constructor:

f.setVisible(true);

Thanks, it works.
I viewed some my programs done with the IDE.
It all makes sense.
I'm not a student of Java, it's just a hobby. It first 10 tutorials are my reference.

JoeB

You should still google for the EDT or EventDispatchThread.
All Swing applications must be started on the EDT (and NOT your main thread) and every update to any Swing component must also happen on the EDT or Swing might not work.

Originally Posted by Cornix

You should still google for the EDT or EventDispatchThread.
All Swing applications must be started on the EDT (and NOT your main thread) and every update to any Swing component must also happen on the EDT or Swing might not work.

Thanks, Now I know where invokeLater comes from in my IDE in my IDE programs.
This gives me something new to learn.

Joe