[ntski]

Hello.

I'm supposed to be making a simple GUI program for my CSCI class and I pieced together this, yet it's not running correctly; nothing is opening.

Here is my code:

public class GUIassignment extends JFrame
{
    public GUIassignment()
    {
    JPanel p1 = new JPanel();
    
    p1.setLayout(new FlowLayout(FlowLayout.LEFT, 5, 10));
    
    p1.add(new JButton("Button 1"));
    p1.add(new JButton("Button 2"));
    p1.add(new JButton("Button 3"));
    
    JPanel p2 = new JPanel();
    
    p2.setLayout(new FlowLayout(FlowLayout.RIGHT, 5, 10));
    
    p2.add(new JButton("Button 4"));
    p2.add(new JButton("Button 5"));
    p2.add(new JButton("Button 6"));
    
    Container gui = getContentPane();
    gui.add(p1, BorderLayout.NORTH);
    gui.add(p2, BorderLayout.SOUTH);
    }

    public static void main(String[] args) 
    {
        GUIassignment gui = new GUIassignment();
        
    
    }
} 

[Ryano121]

gui.setVisible(true);

[GregBrannon]

Or since you're making your JFrame visible, you simply need:

setVisible( true );

After that, you'll find it all scrunched up in the top left corner of you screen, so expand it to see what you've made.

Edit:

To fix the scrunching, add

pack();

before setting it visible.

This post has been edited by GregBrannon: 03 April 2012 - 02:01 PM

[ntski]

Oddly enough, a window is still not opening when I run the program.

[GregBrannon]

It may be too small or obscured for you to see. Add the pack() statement, and that may make it more obvious. It's there . . . or at least it's visible here.

[Ryano121]

You also will want to add -

setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

To make sure that the program exits when you close the JFrame.

[pbl]

gui in the main() method
not gui in the constructor

How to confuse everybody
Actually in the constructor you don't need to access the ContentPane you can simply write

add(p1, BorderLayout.NORTH);
add(p2, BorderLayout.SOUTH);