图像不会显示在JPanel上

Question

Just started out using Java and am a beginner. I tried to create a photo viewer which can search a directory for an image and open the image but my program won't display the image.

When i run the program, it opens up and shows a menubar which i use to search my directories, but even if i select an image it won't display. TIA.

import java.awt.*;
import java.awt.event.*;
import java.awt.image.BufferedImage;
import java.io.*;
import javax.swing.*;
import javax.swing.event.*;
import javax.swing.filechooser.FileNameExtensionFilter;
import javax.imageio.ImageIO;

public class ICS
{
private JPanel gui;
private JFileChooser fileChooser;
FilenameFilter fileNameFilter;
private JMenuBar menuBar;
DefaultListModel model; 
public ICS() {
    gui = new JPanel(new GridLayout());

    final JLabel imageView = new JLabel();
    gui.add(imageView);

    fileChooser = new JFileChooser();
    String[] imageTypes = ImageIO.getReaderFileSuffixes();

    menuBar = new JMenuBar();
    JMenu menu = new JMenu("GET PHOTO HERE");
    menuBar.add(menu);
    JMenuItem browse = new JMenuItem("browse");
    menu.add(browse);

    browse.addActionListener(new ActionListener(){
            public void actionPerformed(ActionEvent ae) {
                int result = fileChooser.showOpenDialog(gui);
                if (result==JFileChooser.APPROVE_OPTION) {
                    File eg = fileChooser.getSelectedFile();
                }
            }
        }); 
}

public void loadImages(File directory) throws IOException {
    File[] imageFiles = directory.listFiles(fileNameFilter);
    BufferedImage[] images = new BufferedImage[imageFiles.length];
}

public Container getGui() {
    return gui;
}

public JMenuBar getMenuBar() {
    return menuBar;
}

public static void main(String[] args) {
    SwingUtilities.invokeLater(new Runnable(){
            public void run() {
                ICS imageList = new ICS();

                JFrame f = new JFrame("Image Browser");
                f.setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);
                f.add(imageList.getGui());
                f.setJMenuBar(imageList.getMenuBar());
                f.setLocationByPlatform(true);
                f.pack();
                f.setSize(800,600);
                f.setVisible(true);
            }
        });
}

}

Answer 1

You're not doing anything with your selected file. Given that you have an empty JLabel in your JPanel , you could simply set the Icon for that component:

imageView.setIcon(new ImageIcon(eg.getPath()));

Answer 2

It's not doing anything with the image because you've never told it to. In your action listener, you've created a file chooser and gotten the selected file, but you never do anything with it. You just define it as a local variable inside the action listener, which is then immediately destroyed when your listener exits.

What you should be doing is, inside of your action listener, make a function call that actually displays your image after you retrieve the file the user selected.

Also, ICS is a terrible name for a class. You should be descriptive of your class names for your own reference and sanity when your program gets larger and you're trying to remember what everything did.