JFileChooser is throwing null exeption error if no file or image is selected or attached.

[Mezta Dodger]

Hi fellas,

I am new to Java and I'm making a simple inventory system which is connected to MySql Phpmyadmin. In this application, I have created a form form called employee_details and inside the employee details form I created a button to open a file chooser for me to be able to select image file. But what I want is to make this feature as optional which means I can save all the entries into mysql database whether I attached an image or not.

Error:Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException

Below are codes for my add or save button.

    private void jButtonAddEmpActionPerformed(java.awt.event.ActionEvent evt) {                                              
       
        int InsertToDB = JOptionPane.showConfirmDialog(null, "Do you want to save this entry?", "SAVE",JOptionPane.YES_NO_OPTION);
        
        if (InsertToDB == 0)
        {   
            
            if(CheckInputs() == true)
            {
                String Gender = null;
            
                if(JRB_Male.isSelected())
                {
                    Gender = "Male";
                }
                else if(JRB_Female.isSelected())
                {
                    Gender = "Female";
                }
                
               
                try {
                    pst = con.prepareStatement("INSERT INTO employee_details (First_Name, Last_Name, Gender, Position_ID, Department_ID, "
                            + "Office_Location_ID, Employee_Status, Office_Number, Mobile_Number, Email_Address, Remarks, Image) "
                            + "VALUES (?,?,?,?,?,?,?,?,?,?,?,?)");
                    
                    pst.setString(1, jTextFieldFname.getText());
                    pst.setString(2, jTextFieldLname.getText());
                    pst.setString(3, Gender);
                    pst.setInt(4, ((PositionWrapper)Emp_Positions.getSelectedItem()).GetId());          // Stroring the FK ID to employee_details table intead of the String reference to the Position string.
                    pst.setInt(5, ((DepartmentWrapper)JCB_Department.getSelectedItem()).getDeptid());
                    pst.setInt(6, ((OfficeLocationWrapper)JCB_Location.getSelectedItem()).getLocId());
                    String EmpStatus = JCB_EmpStat.getSelectedItem().toString();
                    pst.setString(7, EmpStatus);
                    pst.setString(8, jTextFieldOfficeNumber.getText());
                    pst.setString(9, jTextFieldMobNumber.getText());
                    pst.setString(10, jTextFieldEmailAdd.getText());
                    pst.setString(11, jTextAreaRemarks.getText());   
                                           
                    InputStream InsertImg = new FileInputStream(new File(ImagePath));                    
                    pst.setBlob(12, InsertImg);
                                       
                    pst.executeUpdate();
                    JOptionPane.showMessageDialog(null, "Succesfully Saved");
                    
                } catch (SQLException ex) {
                    Logger.getLogger(Employee_Details.class.getName()).log(Level.SEVERE, null, ex);
                } catch (FileNotFoundException ex) {
                    Logger.getLogger(Employee_Details.class.getName()).log(Level.SEVERE, null, ex);
                }
            }
            
            else{
            JOptionPane.showMessageDialog(null, "Check the mandatory fields");
            }
        }
    }   

This is the method for my ImageIcon.

String ImagePath  = null;
    public ImageIcon ResizeImage(String ImagePath, byte[] pictures){
        ImageIcon Photo;
        
        if(ImagePath != null)
        {
            Photo = new ImageIcon(ImagePath);
        }
        else
        {
            Photo = new ImageIcon(pictures);
        }
        Image pic = Photo.getImage();
        Image img = pic.getScaledInstance(jLabelPhoto.getWidth(), jLabelPhoto.getHeight(), Image.SCALE_SMOOTH);
        ImageIcon Img = new ImageIcon(img);
        return Img;
    }

This is the code for my add photo button.

    private void jButtonAddPhotoActionPerformed(java.awt.event.ActionEvent evt) {                                                
        
        JFileChooser SelectImage = new JFileChooser();
        SelectImage.setCurrentDirectory(new File(System.getProperty("user.home")));
        
        FileNameExtensionFilter PicFormat = new FileNameExtensionFilter("*.image", "jpg","png");
        SelectImage.addChoosableFileFilter(PicFormat);
        int result = SelectImage.showSaveDialog(null);
        
        if(result == JFileChooser.APPROVE_OPTION)
        {
            File SelectedImage = SelectImage.getSelectedFile();
            String path = SelectedImage.getAbsolutePath();
            jLabelPhoto.setIcon(ResizeImage(path, null));
            ImagePath = path;
        }
        else 
        {
            JOptionPane.showMessageDialog(null, "No file is selected");
            

        }
    }

Error Stack Trace.

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
	at java.io.File.<init>(File.java:277)
	at iap_it_inventory_system.Employee_Details.jButtonAddEmpActionPerformed(Employee_Details.java:664)
	at iap_it_inventory_system.Employee_Details.access$100(Employee_Details.java:36)
	at iap_it_inventory_system.Employee_Details$3.actionPerformed(Employee_Details.java:373)
	at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:2022)
	at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2348)
	at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:402)
	at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:259)
	at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(BasicButtonListener.java:252)
	at java.awt.Component.processMouseEvent(Component.java:6533)
	at javax.swing.JComponent.processMouseEvent(JComponent.java:3324)
	at java.awt.Component.processEvent(Component.java:6298)
	at java.awt.Container.processEvent(Container.java:2236)
	at java.awt.Component.dispatchEventImpl(Component.java:4889)
	at java.awt.Container.dispatchEventImpl(Container.java:2294)
	at java.awt.Component.dispatchEvent(Component.java:4711)
	at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4888)
	at java.awt.LightweightDispatcher.processMouseEvent(Container.java:4525)
	at java.awt.LightweightDispatcher.dispatchEvent(Container.java:4466)
	at java.awt.Container.dispatchEventImpl(Container.java:2280)
	at java.awt.Window.dispatchEventImpl(Window.java:2746)
	at java.awt.Component.dispatchEvent(Component.java:4711)
	at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:758)
	at java.awt.EventQueue.access$500(EventQueue.java:97)
	at java.awt.EventQueue$3.run(EventQueue.java:709)
	at java.awt.EventQueue$3.run(EventQueue.java:703)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:80)
	at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:90)
	at java.awt.EventQueue$4.run(EventQueue.java:731)
	at java.awt.EventQueue$4.run(EventQueue.java:729)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:80)
	at java.awt.EventQueue.dispatchEvent(EventQueue.java:728)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:201)
	at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:116)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:105)
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:101)
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:93)
	at java.awt.EventDispatchThread.run(EventDispatchThread.java:82)

[Dave Tolls]

Lines 40 and 41 in the first bit of code assume you have a valid image file.
You need to check whether or not one has been supplied.
If it isn't then don't try and create an input stream...I think (though cannot guarantee) that you can just set the Blob to null in that case.

The error should have given you a stack trace, by the way, which would have helped to point us to where in the code you were getting this error.
Alway give the stacktrace, along with the full error message.

[Mezta Dodger]

Is it possible to put a default image that will be inserted to mysql table whenever I save a record without image attached in it. All I want is to make the image field as optional, just like what I have stated above. I the idea but I don't know how to execute it through codes. Can you please guide how to achieve that.

[Mezta Dodger]

@dave Tolls Kindly check my post again I attached the stack trace error.

[Knute Snortum]

Mezta Dodger: instead of editing a prior post, make a new post so that people looking at this thread can follow along.  As it is now, it looks like Dave asked for something that was already there.

[Mezta Dodger]

My apologies, I'm new here that's why I'm not so very familiar with the rules. But here is the stack trace of my code.

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at java.io.File.<init>(File.java:277)
at iap_it_inventory_system.Employee_Details.jButtonAddEmpActionPerformed(Employee_Details.java:664)
at iap_it_inventory_system.Employee_Details.access$100(Employee_Details.java:36)
at iap_it_inventory_system.Employee_Details$3.actionPerformed(Employee_Details.java:373)
at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:2022)
at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2348)
at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:402)
at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:259)
at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(BasicButtonListener.java:252)
at java.awt.Component.processMouseEvent(Component.java:6533)
at javax.swing.JComponent.processMouseEvent(JComponent.java:3324)
at java.awt.Component.processEvent(Component.java:6298)
at java.awt.Container.processEvent(Container.java:2236)
at java.awt.Component.dispatchEventImpl(Component.java:4889)
at java.awt.Container.dispatchEventImpl(Container.java:2294)
at java.awt.Component.dispatchEvent(Component.java:4711)
at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4888)
at java.awt.LightweightDispatcher.processMouseEvent(Container.java:4525)
at java.awt.LightweightDispatcher.dispatchEvent(Container.java:4466)
at java.awt.Container.dispatchEventImpl(Container.java:2280)
at java.awt.Window.dispatchEventImpl(Window.java:2746)
at java.awt.Component.dispatchEvent(Component.java:4711)
at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:758)
at java.awt.EventQueue.access$500(EventQueue.java:97)
at java.awt.EventQueue$3.run(EventQueue.java:709)
at java.awt.EventQueue$3.run(EventQueue.java:703)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:80)
at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:90)
at java.awt.EventQueue$4.run(EventQueue.java:731)
at java.awt.EventQueue$4.run(EventQueue.java:729)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:80)
at java.awt.EventQueue.dispatchEvent(EventQueue.java:728)
at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:201)
at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:116)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:105)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:101)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:93)
at java.awt.EventDispatchThread.run(EventDispatchThread.java:82)
Employee-GUI.jpg

[Dave Tolls]

There's nothing to stop you having a dummy image to use.
No point storing that in the database, just display it if the employee record has no image associated with it.

[Mezta Dodger]

That is exactly what I want to achieve, make a default image that will automatically attached on the display whenever I don't attach any image file on the jlabel. I just don't know how. It's been a week now and I still can't a tutorial or lesson on how to code that.

[Dave Tolls]

Well, currently your ResizeImage method (should be resizeImage, to fit with standard naming practices) accepts both a path and a byte[].
When you want the default image then, if byte[] is null, give that method the path to the default image instead.

Indeed, I would split the resizeImage up.  It's currently doing too much, in my opinion.

First, the resize:

public ImageIcon resizeImage(ImageIcon originalImage) {
    Image pic = originalImage.getImage();
    Image img = pic.getScaledInstance(jLabelPhoto.getWidth(), jLabelPhoto.getHeight(), Image.SCALE_SMOOTH);
    ImageIcon Img = new ImageIcon(img);
    return Img;
}

Then supply two methods to get an image:

public ImageIcon getImage(String path) {
    return resizeImage(new ImageIcon(path));
}

public ImageIcon getImage(byte[] image) {
    if (image == null) {
        return getImage(PATH_TO_DEFAULT);
    }
    return resizeImage(new ImageIcon(image));
}

Depending on where these live, I wouldn't expect them to be public methods, but I don't know your code structure.

ETA: This lot is hand written, so apologies for any compilation issues.

[Mezta Dodger]

Let me create another project to check it, because I'm really confused right now. by the way here is the entire structure of my code. https://pastebin.com/Uagnj9a9

[Dave Tolls]

If it's too much to post here, it's too much for me to go through I'm afraid.

[Mezta Dodger]

It's ok Dave thank you so much for your effort appreciated.