Question

I know this is something very simple, but as a complete Java newbie I’m missing it and someone pointing it out would be infinitely helpful. I’ve stared at the screen and moved things around and still nothing.

Screenshot:
https://i.stack.imgur.com/ufCd9.png
This is all that comes up when this is run.

fullGUI.java:

---

import java.awt.BorderLayout;
import javax.swing.JFrame;
import javax.swing.JRadioButton;
import javax.swing.JTextArea;


public class fullGUI extends JFrame

{
    JFrame frame = new JFrame(); //creates frame

    public fullGUI() // constructor
    {
        //setLayout(new BorderLayout());
        //add(new shipGrid(), BorderLayout.NORTH);
        //add(new shipGrid(), BorderLayout.NORTH);
        add(new JRadioButton("Horizontal"), BorderLayout.WEST);
        add(new JRadioButton("Vertical"), BorderLayout.WEST);
        add(new JTextArea("Instructions to player will go here"), BorderLayout.CENTER);
        frame.setSize(400,600);
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.setTitle("Battleship!");
        frame.pack(); //sets appropriate size for frame
        frame.setVisible(true);
    }

}

---

…called by…

test.java

---

public class test 
{

     public static void main(String[] args) 
     {

         new fullGUI();


     }


}

---

Answer 1

1. name classes in Java from a capital letter
2. FullGUI already extends JFrame, so no need to create another JFrame inside it
3. call getContentPane.add() to add to JFrame
4. use SwingUtilities.invokeLater

So overall something like this

import java.awt.BorderLayout;
import javax.swing.JFrame;
import javax.swing.JRadioButton;
import javax.swing.JTextArea;


public class FullGUI extends JFrame

{
    public FullGUI() // constructor
    {
        getContentPane().add(new JRadioButton("Horizontal"), BorderLayout.WEST);
        getContentPane().add(new JRadioButton("Vertical"), BorderLayout.WEST);
        getContentPane().add(new JTextArea("Instructions to player will go here"), BorderLayout.CENTER);
        setSize(400,600);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setTitle("Battleship!");
        pack(); //sets appropriate size for frame
        setVisible(true);
    }

}

public static void main(String[] args) {
    SwingUtilities.invokeLater(new Runnable() {

        @Override
        public void run() {
            new FillGU();
        }
    });
}