Displaying an image in a JFrame

Question

I am currently learning Java, and I am stuck with something at the moment.

I was looking for a way to add an image to my JFrame. I found this on the internet:

ImageIcon image = new ImageIcon("path & name & extension");
JLabel imageLabel = new JLabel(image);

And after implementing it to my own code, it looks like this (this is only the relevant part):

class Game1 extends JFrame
{
    public static Display f = new Display();
    public Game1()
    {
        Game1.f.setSize(1000, 750);
        Game1.f.setResizable(false);
        Game1.f.setVisible(true);
        Game1.f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        Game1.f.setTitle("Online First Person Shooter");

        ImageIcon image = new ImageIcon("C:\\Users\\Meneer\\Pictures\\image.png");
        JLabel imageLabel = new JLabel(image); 
        add(imageLabel);
        }
}

class Display extends JFrame
{
}

When running this code, it doesn't give me any errors, but it also doesn't show the picture. I saw some questions and people having the same problem, but their code was completely different from mine, they used other ways to display the image.

keep `setVisible(true)` after `add(imageLable)`.. – Anirban Nag 'tintinmj' Nov 20 '13 at 14:16 — Anirban Nag 'tintinmj', Nov 20 '13 at 14:16

score 8 · Answer 1 · edited Nov 20 '13 at 17:02

You don't need to use another JFrame instance inside the Game JFrame:
Calling setVisible(flag) from the constructor is not preferable. Rather initialize your JFrame from outside and put your setVisible(true) inside event dispatch thread to maintain Swing's GUI rendering rules using SwingUtilities.invokeLater(Runnable)
Do not give size hint by setSize(Dimension) of the JFrame. Rather use proper layout with your component, call pack() after adding all of your relevant component to the JFrame.
Try using JScrollPane with JLabel for a better user experience with image larger than the label's size can be.

All of the above description is made in the following example:

     class Game1 extends JFrame
    {
       public Game1()
      {
         // setSize(1000, 750);  <---- do not do it
         // setResizable(false); <----- do not do it either, unless any good reason

         setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
         setTitle("Online First Person Shooter");

         ImageIcon image = new ImageIcon("C:\\Users\\Meneer\\Pictures\\image.png");
         JLabel label = new JLabel(image);
         JScrollPane scrollPane = new JScrollPane(label);
         scrollPane.setHorizontalScrollBarPolicy(JScrollPane.HORIZONTAL_SCROLLBAR_AS_NEEDED);
         scrollPane.setVerticalScrollBarPolicy(JScrollPane.VERTICAL_SCROLLBAR_AS_NEEDED);
         add(scrollPane, BorderLayout.CENTER);
         pack();
      }

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

           @Override
           public void run() {
              new Game1().setVisible(true);
           }
        });

      }
    }

AJ. · Accepted Answer · 2013-11-20T14:21:55.297

5

do this after creating Jlabel

imageLabel.setBounds(10, 10, 400, 400);
imageLabel.setVisible(true);

also set the layout to JFrame

Game.f.setLayout(new FlowLayout);

edited Nov 20 '13 at 14:21

answered Nov 20 '13 at 14:15

AJ.

4,526
5
29
41

@alex2410, don't bother. The OP thought this is the best answer ;) – Sage Nov 20 '13 at 14:33
It did alex2410. Thanks for this answer AJ :)! – user2988879 Nov 20 '13 at 14:37

score 0 · Answer 3 · answered Nov 20 '13 at 14:17

You are adding the label to the wrong JFrame. Also, move the setVisible() to the end.

import javax.swing.*;
class Game1 extends JFrame
{
    public static Display f = new Display();
    public Game1()
    {
        // ....
        Game1.f.add(imageLabel);
        Game1.f.setVisible(true);
    }
}

score 0 · Answer 4 · edited May 23 '17 at 12:00

0

Also try to use image from resources, and not from hardcoded path from your PC You can look in here, where sombody asked similar question about images in Jframe: How to add an ImageIcon to a JFrame?

edited May 23 '17 at 12:00

Community

1
1

answered Nov 20 '13 at 14:18

Khobar

486
7
20

score 0 · Answer 5 · answered Nov 20 '13 at 14:25

Your problem in next you add your JLabel to Game1 but you display another Frame(Display f). Change add(imageLabel); to Game1.f.add(imageLabel);.

Recommendations:

1)according to your problem: Game1 extends JFrame seems that Display is also a frame, use only one frame to display content.

2) use pack() method instead of setSize(1000, 750);

3)call setVisible(true); at the end of construction.

4)use LayoutManager to layout components.

Displaying an image in a JFrame

5 Answers5

Linked