Why Are My Buttons Taking Up The Whole JFrame?

Question

My code was working before, but now one JButton takes up the entire JFrame, any help would be appreciated!

I am using a FlowLayout called fl and a class called Ken

I have ten buttons called (oneButton, twoButton, threeButton, etc..)

MY CODE:

import java.awt.ComponentOrientation;
import java.awt.Dimension;
import java.awt.FlowLayout;


import javax.swing.JButton;
import javax.swing.JFrame;



public class Ken {

public static void frame(){
    JFrame frame = new JFrame("Pow");
    frame.setSize(500, 600);
    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    frame.setVisible(true);
    frame.setResizable(false);
    FlowLayout fl = new FlowLayout(0, 50, 40);


    JButton oneButton = new JButton("         1         ");
    oneButton.setPreferredSize(new Dimension(100, 90));
    frame.add(oneButton);
    JButton twoButton = new JButton("         2         ");
    twoButton.setPreferredSize(new Dimension(100, 90));
    frame.add(twoButton);
    JButton threeButton = new JButton("         3         ");
    threeButton.setPreferredSize(new Dimension(100, 90));
    frame.add(threeButton);
    JButton fourButton = new JButton("         4         ");
    fourButton.setPreferredSize(new Dimension(100, 90));
    frame.add(fourButton);
    JButton fiveButton = new JButton("         5         ");
    fiveButton.setPreferredSize(new Dimension(100, 90));
    frame.add(fiveButton);
    JButton sixButton = new JButton("          6         ");
    sixButton.setPreferredSize(new Dimension(100, 90));
    frame.add(sixButton);
    JButton sevenButton = new JButton("         7         ");
    sevenButton.setPreferredSize(new Dimension(100, 90));
    frame.add(sevenButton);
    JButton eightButton = new JButton("         8         ");
    eightButton.setPreferredSize(new Dimension(100, 90));
    frame.add(eightButton);
    JButton nineButton = new JButton("         9         ");
    nineButton.setPreferredSize(new Dimension(100, 90));
    frame.add(nineButton);
    JButton zeroButton = new JButton("                                                                       
                                  0                               ");
    zeroButton.setPreferredSize(new Dimension(400, 90));
    frame.add(zeroButton);
    frame.setComponentOrientation(
            ComponentOrientation.LEFT_TO_RIGHT);
}



public static void main(String[] args){
    frame();
}
}

Anything I can try?

Answer 1

I am using a FlowLayout called fl ...

You are not in fact using FlowLayout as you never call setLayout(...) on a container. Instead your Frame's contentPane is using the default layout for contentPanes, BorderLayout, which will cause the last component added in a default way to fill up the container's BorderLayout.CENTER location.

So one solution is to use your layout:

FlowLayout fl = new FlowLayout(0, 50, 40);    
frame.getContentPane().setLayout(fl);

But having said that, you will probably be better off not setting the preferred sizes of your components but instead using a combination of nested containers each using its own layout in order to achieve a pleasing and complex GUI that will be much easier to maintain and extend.

Alternatively, you could play with more complex layouts such as the GridBagLayout. For example:

import java.awt.Component;
import java.awt.Font;
import java.awt.GridBagConstraints;
import java.awt.GridBagLayout;
import java.awt.Insets;

import javax.swing.*;

public class Ken2 {
   private static final String[][] NUMBERS = {
      {"1", "2", "3"},
      {"4", "5", "6"},
      {"7", "8", "9"},
      {"0"}
   };
   private static final float BUTTON_FONT_PTS = 45f;
   private static final int INSETS = 20;
   private static final Insets BUTTON_INSETS = new Insets(INSETS, INSETS, 
           INSETS, INSETS);
   private static final int IPAD = 20;
   private JPanel mainPanel = new JPanel();

   public Ken2() {
      mainPanel.setLayout(new GridBagLayout());
      for (int row = 0; row < NUMBERS.length; row++) {
         addRowToPanel(row, NUMBERS[row]);
      }
   }

   private void addRowToPanel(int row, String[] numbersRow) {
      int columns = numbersRow.length;
      for (int col = 0; col < numbersRow.length; col++) {
         addNumberButton(row, col, columns, numbersRow[col]);
      }
   }

   private void addNumberButton(int row, int col, int columns, 
           String numberText) {
      JButton button = new JButton(numberText);
      button.setFont(button.getFont().deriveFont(Font.BOLD, BUTTON_FONT_PTS));
      GridBagConstraints gbc = new GridBagConstraints();
      gbc.gridx = col;
      gbc.gridy = row;
      gbc.gridheight = 1;
      gbc.gridwidth = 3 / columns;
      gbc.weightx = 1.0;
      gbc.weighty = 1.0;
      gbc.anchor = GridBagConstraints.CENTER;
      gbc.fill = GridBagConstraints.BOTH;
      gbc.insets = BUTTON_INSETS;
      gbc.ipadx = IPAD;
      gbc.ipady = IPAD;

      mainPanel.add(button, gbc);
   }

   private static void createAndShowGui() {
      Ken2 ken = new Ken2();

      JFrame frame = new JFrame("Ken2");
      frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
      frame.getContentPane().add(ken.getMainPanel());
      frame.pack();
      frame.setLocationByPlatform(true);
      frame.setVisible(true);
   }

   private Component getMainPanel() {
      return mainPanel;
   }

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

Answer 2

Call setLayout(fl) on your void frame() method.