Does GridBagLayout affect JLabel images?

Question

Currently trying to add a header image to my JFrame based GUI, I've developed the layout of the project and everything looks good, but every time I run the project the image does not load (no error messages).

My code (partial):

import java.awt.GridBagConstraints;
import java.awt.GridBagLayout;
import java.awt.Insets;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.*;
import javax.swing.event.ChangeEvent;
import javax.swing.event.ChangeListener;
import java.awt.*;


public class DeltaFlightFrame extends JFrame implements ActionListener, ChangeListener{
    //<SNIPPED CODE FOR EASE OF VIEWING>



    //Labels for inputs
    //<SNIPPED CODE FOR EASE OF VIEWING>
    private JLabel deltaLogo;

    //icon
    private Icon logo;

   //<SNIPPED CODE FOR EASE OF VIEWING>


    DeltaFlightFrame() {
        GridBagConstraints layoutConst = null;


        //<SNIPPED CODE FOR EASE OF VIEWING>

        setTitle("Delta Flight Price Estimator");

        //<SNIPPED CODE FOR EASE OF VIEWING>

        //initialize delta logo
        logo = new ImageIcon("../img/logo.png");
        deltaLogo = new JLabel(logo);

        System.out.println("Height" + logo.getIconHeight());
        System.out.println("Width" + logo.getIconWidth());

        //<SNIPPED CODE FOR EASE OF VIEWING>

        // Create frame and add components using GridBagLayout
        setLayout(new GridBagLayout());
        layoutConst = new GridBagConstraints();
        layoutConst.insets = new Insets(10, 10, 1, 1);
        //layoutConst.anchor = GridBagConstraints.LINE_START;
        layoutConst.gridx = 0;
        layoutConst.gridy = 0;
        //layoutConst.gridwidth = 4;
        add(deltaLogo, layoutConst);

        //<SNIPPED CODE FOR EASE OF VIEWING>
    }




    //TODO: this
    public void stateChanged(ChangeEvent event) {
    }

    //TODO, also: this
    public void actionPerformed(ActionEvent event) {

    }


    public static void main(String[] args) {
        try {
            UIManager.setLookAndFeel("com.sun.java.swing.plaf.nimbus.NimbusLookAndFeel");
        } catch(Exception ignored){}
        DeltaFlightFrame myFrame = new DeltaFlightFrame();

        myFrame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        myFrame.pack();
        myFrame.setVisible(true);

    }
}

My full code (I didn't want to put it all, it's a disgusting mess and I'm a first year student so this isn't... nice... hahaha):

import java.awt.GridBagConstraints;
import java.awt.GridBagLayout;
import java.awt.Insets;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.*;
import javax.swing.event.ChangeEvent;
import javax.swing.event.ChangeListener;
import java.awt.*;


public class DeltaFlightFrame extends JFrame implements ActionListener, ChangeListener{
    //Text Fields
    private JTextField desCityField;  // Holds destination city abbreviation,
    private JTextField depCityField;  // Holds departure city abbreviation.
    private JTextField finalOutputField; //Shows final price

    //Drop Down
    //=====TODO

    //Labels for inputs
    private JLabel desCityLabel;
    private JLabel depCityLabel;
    private JLabel tripTypeLabel;
    private JLabel seatTypeLabel;
    private JLabel adultTravelerCountLabel;
    private JLabel childTravelerCountLabel;
    private JLabel finalOutput; //final price
    private JLabel flightTitle;
    private JLabel passengerTitle;
    private JLabel deltaLogo;

    //icon
    private Icon logo;

    //dropdown
    String[] seatClassChoices = { "Basic Economy", "Business+ Comfort", "First Class" };
    private JComboBox seatClassDrop;

    //JSpinners
    private JSpinner adultTravelerCount;
    private JSpinner childTravelerCount;

    //radio button
    private JRadioButton oneWay;
    private JRadioButton roundTrip;

    //Buttons
    private JButton calculateButton;



    /* Constructor creates GUI components and adds GUI components
      using a GridBagLayout. */

    DeltaFlightFrame() {
        GridBagConstraints layoutConst = null;
        SpinnerNumberModel spinnerModelAdult = null;
        SpinnerNumberModel spinnerModelChild = null;


        String desInit = "ATL";
        String depInit = "JFK";
        double priceInit = 150.00;

        int passCountMin = 0;
        int passCountMax = 9;
        int passAdultInit = 1;
        int passChildInit = 0;



        //Set Frame Title
        setTitle("Delta Flight Price Estimator");

        //create labels
        depCityLabel = new JLabel("Departure City: ");
        desCityLabel = new JLabel("Destination City: ");
        tripTypeLabel = new JLabel("Trip Type: ");
        seatTypeLabel = new JLabel("Seat Class: ");
        adultTravelerCountLabel = new JLabel("Travelling Adults: ");
        childTravelerCountLabel = new JLabel("Travelling Children: ");
        finalOutput = new JLabel("Price: ");


        flightTitle = new JLabel("Flight Information");
        flightTitle.setFont(new Font("Sans-Serif", Font.BOLD, 20));

        passengerTitle = new JLabel("Passenger Information");
        passengerTitle.setFont(new Font("Sans-Serif", Font.BOLD, 20));


        //create dropdown
        seatClassDrop = new JComboBox<String>(seatClassChoices);

        //create spinners
        spinnerModelAdult = new SpinnerNumberModel(passAdultInit, passCountMin, passCountMax, 1);
        adultTravelerCount = new JSpinner(spinnerModelAdult);

        spinnerModelChild = new SpinnerNumberModel(passChildInit, passCountMin, passCountMax, 1);
        childTravelerCount = new JSpinner(spinnerModelChild);

        //initialize delta logo
        logo = new ImageIcon("../img/logo.png");
        deltaLogo = new JLabel(logo);
        System.out.println("Height" + logo.getIconHeight());
        System.out.println("Width" + logo.getIconWidth());

        //initialize text fields
        desCityField = new JTextField("JFK");
        desCityField.setEditable(true);
        desCityField.setDocument(new LengthRestrictedDocument(3));
        desCityField.setColumns(3);

        depCityField = new JTextField("ATL");
        depCityField.setEditable(true);
        depCityField.setDocument(new LengthRestrictedDocument(3));
        depCityField.setColumns(3);

        //radio button
        oneWay = new JRadioButton("One Way");
        roundTrip = new JRadioButton("Round Trip");


        //button
        calculateButton = new JButton("Calculate");
        calculateButton.addActionListener(this);

        ButtonGroup tripType = new ButtonGroup();
        tripType.add(oneWay);
        tripType.add(roundTrip);

        // Create frame and add components using GridBagLayout


        setLayout(new GridBagLayout());
        layoutConst = new GridBagConstraints();
        layoutConst.insets = new Insets(10, 10, 1, 1);
        //layoutConst.anchor = GridBagConstraints.LINE_START;
        layoutConst.gridx = 0;
        layoutConst.gridy = 0;
        //layoutConst.gridwidth = 4;
        add(deltaLogo, layoutConst);

        setLayout(new GridBagLayout());
        layoutConst = new GridBagConstraints();
        layoutConst.insets = new Insets(10, 10, 1, 1);
        layoutConst.anchor = GridBagConstraints.LINE_START;
        layoutConst.gridx = 0;
        layoutConst.gridy = 1;
        layoutConst.gridwidth = 4;
        add(flightTitle, layoutConst);

        layoutConst = new GridBagConstraints();
        layoutConst.insets = new Insets(10, 10, 1, 1);
        layoutConst.anchor = GridBagConstraints.LINE_START;
        layoutConst.gridx = 0;
        layoutConst.gridy = 2;
        layoutConst.gridwidth = 1;
        add(desCityLabel, layoutConst);

        layoutConst = new GridBagConstraints();
        layoutConst.insets = new Insets(10, 10, 1, 1);
        layoutConst.anchor = GridBagConstraints.LINE_START;
        layoutConst.gridx = 2;
        layoutConst.gridy = 2;
        layoutConst.gridwidth = 1;
        add(depCityLabel, layoutConst);

        layoutConst = new GridBagConstraints();
        layoutConst.insets = new Insets(10, 10, 1, 10);
        layoutConst.fill = GridBagConstraints.HORIZONTAL;
        layoutConst.gridx = 1;
        layoutConst.gridy = 2;
        layoutConst.gridwidth = 1;

        add(desCityField, layoutConst);

        layoutConst = new GridBagConstraints();
        layoutConst.insets = new Insets(10, 10, 1, 10);
        layoutConst.fill = GridBagConstraints.HORIZONTAL;
        layoutConst.gridx = 3;
        layoutConst.gridy = 2;
        layoutConst.gridwidth = 1;
        add(depCityField, layoutConst);

        layoutConst = new GridBagConstraints();
        layoutConst.insets = new Insets(10, 10, 1, 1);
        layoutConst.fill = GridBagConstraints.HORIZONTAL;
        layoutConst.gridx = 0;
        layoutConst.gridy = 4;
        layoutConst.gridwidth = 1;
        add(roundTrip, layoutConst);

        layoutConst = new GridBagConstraints();
        layoutConst.insets = new Insets(10, 10, 1, 10);
        layoutConst.fill = GridBagConstraints.HORIZONTAL;
        layoutConst.gridx = 2;
        layoutConst.gridy = 4;
        layoutConst.gridwidth = 2;
        add(seatClassDrop, layoutConst);

        layoutConst = new GridBagConstraints();
        layoutConst.insets = new Insets(10, 10, 1, 1);
        layoutConst.fill = GridBagConstraints.HORIZONTAL;
        layoutConst.gridx = 0;
        layoutConst.gridy = 5;
        layoutConst.gridwidth = 1;
        add(oneWay, layoutConst);

        layoutConst = new GridBagConstraints();
        layoutConst.insets = new Insets(50, 10, 1, 1);
        layoutConst.anchor = GridBagConstraints.LINE_START;
        layoutConst.gridx = 0;
        layoutConst.gridy = 6;
        layoutConst.gridwidth = 4;
        add(passengerTitle, layoutConst);

        layoutConst = new GridBagConstraints();
        layoutConst.insets = new Insets(10, 10, 10, 1);
        layoutConst.fill = GridBagConstraints.LINE_START;
        layoutConst.gridx = 0;
        layoutConst.gridy = 7;
        layoutConst.gridwidth = 1;
        add(adultTravelerCountLabel, layoutConst);

        layoutConst = new GridBagConstraints();
        layoutConst.insets = new Insets(10, 10, 10, 10);
        layoutConst.fill = GridBagConstraints.HORIZONTAL;
        layoutConst.gridx = 1;
        layoutConst.gridy = 7;
        layoutConst.gridwidth = 1;
        add(adultTravelerCount, layoutConst);

        layoutConst = new GridBagConstraints();
        layoutConst.insets = new Insets(10, 10, 10, 10);
        layoutConst.fill = GridBagConstraints.LINE_START;
        layoutConst.gridx = 2;
        layoutConst.gridy = 7;
        layoutConst.gridwidth = 1;
        add(childTravelerCountLabel, layoutConst);

        layoutConst = new GridBagConstraints();
        layoutConst.insets = new Insets(10, 10, 10, 10);
        layoutConst.fill = GridBagConstraints.HORIZONTAL;
        layoutConst.gridx = 3;
        layoutConst.gridy = 7;
        layoutConst.gridwidth = 1;
        add(childTravelerCount, layoutConst);
    }




    //TODO: this
    public void stateChanged(ChangeEvent event) {
    }

    //TODO, also: this
    public void actionPerformed(ActionEvent event) {

    }


    public static void main(String[] args) {
        try {
            UIManager.setLookAndFeel("com.sun.java.swing.plaf.nimbus.NimbusLookAndFeel");
        } catch(Exception ignored){}
        DeltaFlightFrame myFrame = new DeltaFlightFrame();

        myFrame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        myFrame.pack();
        myFrame.setVisible(true);

    }
}

I did do some messing around, here:

logo = new ImageIcon("../img/logo.png");
deltaLogo = new JLabel(logo);
System.out.println("Height: " + logo.getIconHeight());
System.out.println("Width: " + logo.getIconWidth());

And the output of that was:

Height: -1 Width: -1

Which I do find odd, if that's a starting point.

SO, my question is: Does my setup with GridBagLayout affect the display of the of the JLabel image?

If this were my program and the image wasn't showing, the first thing I'd do to debug the problem would be to create a small program that does *nothing* except try to show the image of interest as an ImageIcon within a JOptionPane. Have you tried this? Does it work? — Hovercraft Full Of Eels, Dec 10 '19 at 02:03
Also, note that an icon might not have any height or width before the GUI has been rendered. — Hovercraft Full Of Eels, Dec 10 '19 at 02:04
Hey there @HovercraftFullOfEels, thanks for the comment. I'll go ahead and create an isolated program to test the GUI element. — Greekin, Dec 10 '19 at 02:07
Also, even after moving the width and height statement to the end of the program after it renders, still shows them both as -1, which I find odd. Might look into this further. — Greekin, Dec 10 '19 at 02:08
But to answer your direct question -- no the GridBagLayout should not harm the display of your image, especially if the image icon has been placed into the GUI before the `pack()` method is called. — Hovercraft Full Of Eels, Dec 10 '19 at 02:09
Hey @HovercraftFullOfEels, thanks for your help so far. So I changed the image directory to the full directory (C:\Users\Nate\Documents\Development\Projects\lab9\img\logo.png) which works, while (..\img\logo.png) does not work. Am I using my path operators wrong? — Greekin, Dec 10 '19 at 02:15
Don't hard code a path like that. The image should be found in your classpath. Read the section from the Swing tutorial on [How to Use Icons](https://docs.oracle.com/javase/tutorial/uiswing/components/icon.html) for a better example on how to load an image. — camickr, Dec 10 '19 at 02:18
"Am I using my path operators wrong?" -- yes. Best to not use file paths and files as you're doing (indirectly). Instead use class resources to get your image via `ImageIO.read(...)` — Hovercraft Full Of Eels, Dec 10 '19 at 02:24
1) Rather than making excuses, make sure the code is properly & consistently formatted! AKA: Use a logical and consistent form of indenting code lines and blocks. The indentation is intended to make the flow of the code easier to follow! Most IDEs have a keyboard shortcut specifically for formatting code. 2) One way to get image(s) for an example is to hot link to images seen in [this Q&A](http://stackoverflow.com/q/19209650/418556). E.G. [This answer](https://stackoverflow.com/a/10862262/418556) hot links to an image embedded in [this question](https://stackoverflow.com/q/10861852/418556). — Andrew Thompson, Dec 10 '19 at 05:54

score 1 · Answer 1 · answered Dec 10 '19 at 02:26

1

Using the website provided by @camickr, I was able to successfully render the image using their "createImageIcon" method.

answered Dec 10 '19 at 02:26

Greekin

13
3

Does GridBagLayout affect JLabel images?

1 Answers1