Null Pointer Exception

[Charles Sexton]

What is causing the null point exception? The system says line 221 which is

input = new BufferedReader(new InputStreamReader(connect.getInputStream()));

Here is the method and can post more code if you need.

    public void getStreams() {
        try {
            if (connect == null) {
                System.err.println("getStreams. connect is NULL *****\n");  ///LOOK
                ///LOOK   Somehow 'connect' has been reset to null. 
            }
            input = new BufferedReader(new InputStreamReader(connect.getInputStream()));
            output = new DataOutputStream(connect.getOutputStream());
            output.flush();
        } catch (IOException ex) {
            System.out.println("IOException within getStreams Method!!!!!!!");
        }
    }

[Richard Tookey]

My best guess is that 'connect' is null and though you test for null and print an error message you don't actually abort when 'connect' is null. Since we can't see the rest of your code it is impossible to say why 'connect' is null.

[Joe San]

The source for NullPointerExceptions are when doing a .(dot) operation on a method with an object that is null.

[K. Tsang]

From the code "connect" is null. To avoid this you can either do:

if(connect != null) { // using != null
 input = ....
}

Or

if (connect == null) { System.err.println(...); } // same as your code
else { 
input = ...
}

Now the question is where and when you initialize "connect"?

[Richard Tookey]

K. Tsang wrote:From the code "connect" is null. To avoid this you can either do:

if(connect != null) { // using != null
 input = ....
}

Or

if (connect == null) { System.err.println(...); } // same as your code
else { 
input = ...
}

Now the question is where and when you initialize "connect"?

The chances are that 'connect' being null represents a system error so I would no neither of these and would in fact remove the null check altogether and just let any NPE propagate back to the caller. What I would do is extract the connect.getInputStream() as a separate line then the cause of the NPE would be easier to diagnose.

[Charles Sexton]

Still having issues with the NPE!!!

Here are my two classes, each is in a separate project and the project is a basic client server app where the server reads from two files based on the requests of the client. This application is not finished and still needs work as you can see so please feel free to comment on whatever you please. Thank You!

///LOOK for comments like this, below

package sexton_it351_p1ip_client;



import java.io.DataInputStream;
import java.io.DataOutputStream;
import java.io.IOException;
import java.net.Socket;
import java.net.UnknownHostException;



public class Client extends javax.swing.JPanel {

    private Socket connect;
    private DataInputStream input;
    private DataOutputStream output;

    public Client() throws IOException {
        initComponents();
        customerError.setVisible(false);
        productError.setVisible(false);
    }

    public void runClient() {
        connectToServer();
        getStreams();
        processConnection();
    }

    @SuppressWarnings("unchecked")
    // <editor-fold defaultstate="collapsed" desc="Generated Code">                          
    private void initComponents() {

        customerTextField = new javax.swing.JTextField();
        customerLabel = new javax.swing.JLabel();
        productLabel = new javax.swing.JLabel();
        displayCustomerButton = new javax.swing.JButton();
        displayProductButton = new javax.swing.JButton();
        displayAllButton = new javax.swing.JButton();
        closeButton = new javax.swing.JButton();
        jScrollPane1 = new javax.swing.JScrollPane();
        displayTextArea = new javax.swing.JTextArea();
        titleLabel = new javax.swing.JLabel();
        clearButton = new javax.swing.JButton();
        customerError = new javax.swing.JLabel();
        productError = new javax.swing.JLabel();
        productTextField = new javax.swing.JTextField();

        customerTextField.setFont(new java.awt.Font("Times New Roman", 0, 12)); // NOI18N
        customerTextField.setMinimumSize(new java.awt.Dimension(166, 22));
        customerTextField.setName(""); // NOI18N
        customerTextField.setPreferredSize(new java.awt.Dimension(166, 22));

        customerLabel.setFont(new java.awt.Font("Times New Roman", 0, 14)); // NOI18N
        customerLabel.setText("Customer");

        productLabel.setFont(new java.awt.Font("Times New Roman", 0, 14)); // NOI18N
        productLabel.setText("Product");

        displayCustomerButton.setFont(new java.awt.Font("Times New Roman", 0, 14)); // NOI18N
        displayCustomerButton.setText("Display Customer");
        displayCustomerButton.addActionListener(new java.awt.event.ActionListener() {
            public void actionPerformed(java.awt.event.ActionEvent evt) {
                displayCustomerButtonActionPerformed(evt);
            }
        });

        displayProductButton.setFont(new java.awt.Font("Times New Roman", 0, 14)); // NOI18N
        displayProductButton.setText("Display Product");
        displayProductButton.addActionListener(new java.awt.event.ActionListener() {
            public void actionPerformed(java.awt.event.ActionEvent evt) {
                displayProductButtonActionPerformed(evt);
            }
        });

        displayAllButton.setFont(new java.awt.Font("Times New Roman", 0, 14)); // NOI18N
        displayAllButton.setText("Display All");
        displayAllButton.addActionListener(new java.awt.event.ActionListener() {
            public void actionPerformed(java.awt.event.ActionEvent evt) {
                displayAllButtonActionPerformed(evt);
            }
        });

        closeButton.setFont(new java.awt.Font("Times New Roman", 0, 14)); // NOI18N
        closeButton.setText("Close");
        closeButton.addActionListener(new java.awt.event.ActionListener() {
            public void actionPerformed(java.awt.event.ActionEvent evt) {
                closeButtonActionPerformed(evt);
            }
        });

        displayTextArea.setColumns(20);
        displayTextArea.setFont(new java.awt.Font("Times New Roman", 0, 14)); // NOI18N
        displayTextArea.setRows(5);
        jScrollPane1.setViewportView(displayTextArea);

        titleLabel.setFont(new java.awt.Font("Times New Roman", 0, 18)); // NOI18N
        titleLabel.setText("Music Universal Client/Server Software System");

        clearButton.setFont(new java.awt.Font("Times New Roman", 0, 14)); // NOI18N
        clearButton.setText("Clear");
        clearButton.addActionListener(new java.awt.event.ActionListener() {
            public void actionPerformed(java.awt.event.ActionEvent evt) {
                clearButtonActionPerformed(evt);
            }
        });

        customerError.setFont(new java.awt.Font("Times New Roman", 2, 14)); // NOI18N
        customerError.setForeground(new java.awt.Color(204, 0, 0));
        customerError.setText("Please insert customer name!!!!");

        productError.setFont(new java.awt.Font("Times New Roman", 2, 14)); // NOI18N
        productError.setForeground(new java.awt.Color(204, 0, 0));
        productError.setText("Please insert product name!!!!");

        productTextField.setFont(new java.awt.Font("Times New Roman", 0, 14)); // NOI18N
        productTextField.setMinimumSize(new java.awt.Dimension(166, 23));
        productTextField.setPreferredSize(new java.awt.Dimension(166, 23));

        javax.swing.GroupLayout layout = new javax.swing.GroupLayout(this);
        this.setLayout(layout);
        layout.setHorizontalGroup(
            layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGroup(layout.createSequentialGroup()
                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
                    .addGroup(layout.createSequentialGroup()
                        .addContainerGap()
                        .addComponent(jScrollPane1))
                    .addGroup(layout.createSequentialGroup()
                        .addGap(49, 49, 49)
                        .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
                            .addGroup(layout.createSequentialGroup()
                                .addComponent(displayAllButton)
                                .addGap(52, 52, 52)
                                .addComponent(clearButton)
                                .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.RELATED, javax.swing.GroupLayout.DEFAULT_SIZE, Short.MAX_VALUE)
                                .addComponent(closeButton)
                                .addGap(15, 15, 15))
                            .addGroup(layout.createSequentialGroup()
                                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
                                    .addComponent(customerLabel)
                                    .addComponent(productLabel))
                                .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.UNRELATED)
                                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
                                    .addComponent(customerError)
                                    .addComponent(customerTextField, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE)
                                    .addComponent(productError)
                                    .addComponent(productTextField, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE))
                                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
                                    .addGroup(layout.createSequentialGroup()
                                        .addGap(18, 18, 18)
                                        .addComponent(displayCustomerButton))
                                    .addGroup(layout.createSequentialGroup()
                                        .addGap(18, 18, 18)
                                        .addComponent(displayProductButton)))
                                .addGap(0, 0, Short.MAX_VALUE)))))
                .addContainerGap())
            .addGroup(layout.createSequentialGroup()
                .addGap(41, 41, 41)
                .addComponent(titleLabel)
                .addContainerGap(javax.swing.GroupLayout.DEFAULT_SIZE, Short.MAX_VALUE))
        );
        layout.setVerticalGroup(
            layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGroup(layout.createSequentialGroup()
                .addContainerGap()
                .addComponent(titleLabel)
                .addGap(35, 35, 35)
                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.BASELINE)
                    .addComponent(customerTextField, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE)
                    .addComponent(customerLabel)
                    .addComponent(displayCustomerButton))
                .addGap(12, 12, 12)
                .addComponent(customerError)
                .addGap(18, 18, 18)
                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.BASELINE)
                    .addComponent(productLabel)
                    .addComponent(displayProductButton)
                    .addComponent(productTextField, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE))
                .addGap(8, 8, 8)
                .addComponent(productError)
                .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.UNRELATED)
                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.BASELINE)
                    .addComponent(displayAllButton)
                    .addComponent(closeButton)
                    .addComponent(clearButton))
                .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.RELATED)
                .addComponent(jScrollPane1, javax.swing.GroupLayout.DEFAULT_SIZE, 163, Short.MAX_VALUE)
                .addContainerGap())
        );
    }// </editor-fold>                        

    public void connectToServer() {
        try {
            connect = new Socket("localhost", 9999);
            if (connect == null) {
                System.err.println("\nconnectToServer. connect is NULL *****\n");
            }
        } catch (UnknownHostException ex) {
            System.out.println("System unknown host error!!!!");
        } catch (IOException ex) {
            System.out.println("IOException within connectToServer method!!!!!!");
        }
    }

    public void getStreams() {
        try {
            if (connect == null) {
                System.err.println("getStreams. connect is NULL *****\n");  ///LOOK
                ///LOOK   Somehow 'connect' has been reset to null. 
            }
            input = new DataInputStream(connect.getInputStream());
            output = new DataOutputStream(connect.getOutputStream());
            output.flush();
        } catch (IOException ex) {
            System.out.println("IOException within getStreams Method!!!!!!!");
        }
    }

    public void processConnection() {
        if (true) ///LOOK What are you trying to read from the Server at this point?  And, WHY?
        {
            try {
                String display = input.readLine();
                displayTextArea.setText(display);
            } catch (IOException ex) {
                System.out.println("Error processing the connection from the server!");
            }
        }
    }

    private void displayCustomerButtonActionPerformed(java.awt.event.ActionEvent evt) {                                                      
        try {
            Client displayCustomer = new Client();
            //Initialize boolean variable valid and set equal to true
            boolean valid = true;
            //Declare string variable salesID equal to iDTextField.getText.trim
            String input2 = customerTextField.getText().trim();
            //If salesID length is equal to 0 or greater than 3
            if (input2.length() == 0 || input2.length() < 3) {
                //Set salesRepIDError visibility equal to true
                customerError.setVisible(valid);
                //valid is equal to false
                valid = false;
            } else { //End if statement
                //Initialize else statement
                customerError.setVisible(false);
                String customerData = customerTextField.getText().trim();
                getStreams();
                output.writeChars(customerData);
                output.flush();
            }//End else Statement
        } catch (IOException ex) {
            ex.printStackTrace();
        }
    }                                                     

    private void displayProductButtonActionPerformed(java.awt.event.ActionEvent evt) {                                                     
        try {
            Client displayProduct = new Client();
            //Initialize boolean variable valid and set equal to true
            boolean valid = true;
            //Declare string variable salesID equal to iDTextField.getText.trim
            String input2 = productTextField.getText().trim();
            //If salesID length is equal to 0 or greater than 5
            if (input2.length() == 0 || input2.length() > 3) {
                //Set salesRepIDError visibility equal to true
                productError.setVisible(true);
                //valid is equal to false
                valid = false;
            }//End if statement
            //Initialize else statement
            else {
                productError.setVisible(false);
                String productData = productTextField.getText().trim();
                output.writeBytes(productData + "\n");
                output.flush();
            }//End else Statement
        } catch (IOException ex) {
            ex.printStackTrace();
        }

    }                                                    

    private void displayAllButtonActionPerformed(java.awt.event.ActionEvent evt) {                                                 
        try {
            if (output == null) {
                System.err.println("displayAllButtonActionPerformed. output is NULL *****\n");   ///LOOK
                ///LOOK Somehow 'output' has been reset to null !!!
            }
            output.writeBytes("Display All");
            output.flush();
        } catch (IOException ex) {
            ex.printStackTrace();
        }
    }                                                

    private void closeButtonActionPerformed(java.awt.event.ActionEvent evt) {                                            
        System.exit(0);
    }                                           

    private void clearButtonActionPerformed(java.awt.event.ActionEvent evt) {                                            
        customerTextField.setText("");
        productTextField.setText("");
        displayTextArea.setText("");
    }                                           
    // Variables declaration - do not modify                     
    private javax.swing.JButton clearButton;
    private javax.swing.JButton closeButton;
    private javax.swing.JLabel customerError;
    private javax.swing.JLabel customerLabel;
    private javax.swing.JTextField customerTextField;
    private javax.swing.JButton displayAllButton;
    private javax.swing.JButton displayCustomerButton;
    private javax.swing.JButton displayProductButton;
    private javax.swing.JTextArea displayTextArea;
    private javax.swing.JScrollPane jScrollPane1;
    private javax.swing.JLabel productError;
    private javax.swing.JLabel productLabel;
    private javax.swing.JTextField productTextField;
    private javax.swing.JLabel titleLabel;
    // End of variables declaration                   
}

package sexton_it351_p1ip_server;


import java.io.DataInputStream;
import java.io.DataOutputStream;
import java.io.IOException;
import java.net.ServerSocket;
import java.net.Socket;


/**
 *
 * @author CharlesSexton
 */
public class Server {
    private ServerSocket server;
    private DataOutputStream output;
    private DataInputStream  input;
    private Socket connect;

    public void runServer() {
        try {
            server = new ServerSocket(9999, 25);
            while (true) {
                try {
                    waitConnection();
                    getStreams();
                    processConnection();
                } finally {
                    closeConnection();
                }
            }
        } catch (IOException ex) {
            System.out.println("Couldn't connect to server!!!!!");
        }
    }

    public void waitConnection() {
        try {
            connect = server.accept();
        } catch (IOException ex) {
            System.out.println("IOException within waitConnection method!!!!!!!");
        }
    }

    public void getStreams() {
        try {
            
            input = new DataInputStream(connect.getInputStream());
            output = new DataOutputStream(connect.getOutputStream());
            output.flush();
        } catch (IOException ex) {
            System.out.println("IOException within getStreams method!!!!!!");
        }
    }

    public void processConnection() {
        try {
            String displayAll = "Display All";
            String message = input.readLine();

            if (message == displayAll) {        ///LOOK  Do NOT try to compare strings using ==   use equals()
                System.out.print("It Worked");
            } else {
                System.out.print("This Workd!!!!!!!!!!!!!!!!!!!!!!");
            }
        } catch (IOException ex) {
            System.out.println("IOException within processConnection method!!!!!!!!");
        }
    }

    public void closeConnection() {
        try {
            input.close();
            output.close();
            connect.close();
        } catch (IOException ex) {
            System.out.println("Server couldn't close properly!!!!!!!");
        }
    }
}

[E Armitage]

 if (connect == null) {  
                System.err.println("getStreams. connect is NULL *****\n");  ///LOOK  
                ///LOOK   Somehow 'connect' has been reset to null.   
            }  
            input = new DataInputStream(connect.getInputStream());  
            output = new DataOutputStream(connect.getOutputStream()); 

Is not going to stop the null pointer. You must not do connect.getInputStream() if connect is null.

[Richard Tookey]

I don't have access to your main() methods but I can see no path through the client that actually calls runClient() which is needed to connect to the server so 'connect' is always null.

P.S. Don't throw debug information away by converting exceptions to simple System.err.println("some error message"). Print the stack trace!

[Charles Sexton]

Richard Tookey wrote:I don't have access to your main() methods but I can see no path through the client that actually calls runClient() which is needed to connect to the server so 'connect' is always null.

P.S. Don't throw debug information away by converting exceptions to simple System.err.println("some error message"). Print the stack trace!

Hey Richard, I just did the System.out.println(); to pin point the methods. I will eventually take them out and use ex.printStackTrace();

Also the main method calls runClient(); through an object.

[Charles Sexton]

E Armitage wrote:

 if (connect == null) {  
                System.err.println("getStreams. connect is NULL *****\n");  ///LOOK  
                ///LOOK   Somehow 'connect' has been reset to null.   
            }  
            input = new DataInputStream(connect.getInputStream());  
            output = new DataOutputStream(connect.getOutputStream()); 

Is not going to stop the null pointer. You must not do connect.getInputStream() if connect is null.

Would you suggest system.exit(0); ?

[Richard Tookey]

Charles Sexton wrote:

Richard Tookey wrote:I don't have access to your main() methods but I can see no path through the client that actually calls runClient() which is needed to connect to the server so 'connect' is always null.

P.S. Don't throw debug information away by converting exceptions to simple System.err.println("some error message"). Print the stack trace!

Hey Richard, I just did the System.out.println(); to pin point the methods. I will eventually take them out and use ex.printStackTrace();

You have it back to front! You need the stack trace while you are in your current state i.e. debugging! In this situation you get far more information out of a stack trace than you will ever get from a System.out !

Also the main method calls runClient(); through an object.

Which of course we can't verify that it is done correctly since you have not posted the main methods. The fact is 'connect' is most probably null so you have to verify that you have called runClient() on the correct object before the GUI is constructed. We can't do that since we don't have access to the code that is supposed to do it!

[Charles Sexton]

Richard Tookey wrote:

Charles Sexton wrote:

Richard Tookey wrote:I don't have access to your main() methods but I can see no path through the client that actually calls runClient() which is needed to connect to the server so 'connect' is always null.

P.S. Don't throw debug information away by converting exceptions to simple System.err.println("some error message"). Print the stack trace!

Hey Richard, I just did the System.out.println(); to pin point the methods. I will eventually take them out and use ex.printStackTrace();

You have it back to front! You need the stack trace while you are in your current state i.e. debugging! In this situation you get far more information out of a stack trace than you will ever get from a System.out !

Also the main method calls runClient(); through an object.

Which of course we can't verify that it is done correctly since you have not posted the main methods. The fact is 'connect' is most probably null so you have to verify that you have called runClient() on the correct object before the GUI is constructed. We can't do that since we don't have access to the code that is supposed to do it!

Here are the main classes and the two that I updated with errors.

Thank You

package sexton_it351_p1ip_client;



import java.io.*;
import java.net.Socket;
import java.net.UnknownHostException;
import java.util.logging.Level;
import java.util.logging.Logger;



public class Client extends javax.swing.JPanel {

    private Socket connect;
    private ObjectInputStream input;
    private ObjectOutputStream output;

    public Client() throws IOException {
        initComponents();
        customerError.setVisible(false);
        productError.setVisible(false);
    }

    public void runClient() {
        connectToServer();
        getStreams();
    }

    @SuppressWarnings("unchecked")
    // <editor-fold defaultstate="collapsed" desc="Generated Code">                          
    private void initComponents() {

        customerTextField = new javax.swing.JTextField();
        customerLabel = new javax.swing.JLabel();
        productLabel = new javax.swing.JLabel();
        displayCustomerButton = new javax.swing.JButton();
        displayProductButton = new javax.swing.JButton();
        displayAllButton = new javax.swing.JButton();
        closeButton = new javax.swing.JButton();
        jScrollPane1 = new javax.swing.JScrollPane();
        displayTextArea = new javax.swing.JTextArea();
        titleLabel = new javax.swing.JLabel();
        clearButton = new javax.swing.JButton();
        customerError = new javax.swing.JLabel();
        productError = new javax.swing.JLabel();
        productTextField = new javax.swing.JTextField();

        customerTextField.setFont(new java.awt.Font("Times New Roman", 0, 12)); // NOI18N
        customerTextField.setMinimumSize(new java.awt.Dimension(166, 22));
        customerTextField.setName(""); // NOI18N
        customerTextField.setPreferredSize(new java.awt.Dimension(166, 22));

        customerLabel.setFont(new java.awt.Font("Times New Roman", 0, 14)); // NOI18N
        customerLabel.setText("Customer");

        productLabel.setFont(new java.awt.Font("Times New Roman", 0, 14)); // NOI18N
        productLabel.setText("Product");

        displayCustomerButton.setFont(new java.awt.Font("Times New Roman", 0, 14)); // NOI18N
        displayCustomerButton.setText("Display Customer");
        displayCustomerButton.addActionListener(new java.awt.event.ActionListener() {
            public void actionPerformed(java.awt.event.ActionEvent evt) {
                displayCustomerButtonActionPerformed(evt);
            }
        });

        displayProductButton.setFont(new java.awt.Font("Times New Roman", 0, 14)); // NOI18N
        displayProductButton.setText("Display Product");
        displayProductButton.addActionListener(new java.awt.event.ActionListener() {
            public void actionPerformed(java.awt.event.ActionEvent evt) {
                displayProductButtonActionPerformed(evt);
            }
        });

        displayAllButton.setFont(new java.awt.Font("Times New Roman", 0, 14)); // NOI18N
        displayAllButton.setText("Display All");
        displayAllButton.addActionListener(new java.awt.event.ActionListener() {
            public void actionPerformed(java.awt.event.ActionEvent evt) {
                displayAllButtonActionPerformed(evt);
            }
        });

        closeButton.setFont(new java.awt.Font("Times New Roman", 0, 14)); // NOI18N
        closeButton.setText("Close");
        closeButton.addActionListener(new java.awt.event.ActionListener() {
            public void actionPerformed(java.awt.event.ActionEvent evt) {
                closeButtonActionPerformed(evt);
            }
        });

        displayTextArea.setColumns(20);
        displayTextArea.setFont(new java.awt.Font("Times New Roman", 0, 14)); // NOI18N
        displayTextArea.setRows(5);
        jScrollPane1.setViewportView(displayTextArea);

        titleLabel.setFont(new java.awt.Font("Times New Roman", 0, 18)); // NOI18N
        titleLabel.setText("Music Universal Client/Server Software System");

        clearButton.setFont(new java.awt.Font("Times New Roman", 0, 14)); // NOI18N
        clearButton.setText("Clear");
        clearButton.addActionListener(new java.awt.event.ActionListener() {
            public void actionPerformed(java.awt.event.ActionEvent evt) {
                clearButtonActionPerformed(evt);
            }
        });

        customerError.setFont(new java.awt.Font("Times New Roman", 2, 14)); // NOI18N
        customerError.setForeground(new java.awt.Color(204, 0, 0));
        customerError.setText("Please insert customer name!!!!");

        productError.setFont(new java.awt.Font("Times New Roman", 2, 14)); // NOI18N
        productError.setForeground(new java.awt.Color(204, 0, 0));
        productError.setText("Please insert product name!!!!");

        productTextField.setFont(new java.awt.Font("Times New Roman", 0, 14)); // NOI18N
        productTextField.setMinimumSize(new java.awt.Dimension(166, 23));
        productTextField.setPreferredSize(new java.awt.Dimension(166, 23));

        javax.swing.GroupLayout layout = new javax.swing.GroupLayout(this);
        this.setLayout(layout);
        layout.setHorizontalGroup(
            layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGroup(layout.createSequentialGroup()
                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
                    .addGroup(layout.createSequentialGroup()
                        .addContainerGap()
                        .addComponent(jScrollPane1))
                    .addGroup(layout.createSequentialGroup()
                        .addGap(49, 49, 49)
                        .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
                            .addGroup(layout.createSequentialGroup()
                                .addComponent(displayAllButton)
                                .addGap(52, 52, 52)
                                .addComponent(clearButton)
                                .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.RELATED, javax.swing.GroupLayout.DEFAULT_SIZE, Short.MAX_VALUE)
                                .addComponent(closeButton)
                                .addGap(15, 15, 15))
                            .addGroup(layout.createSequentialGroup()
                                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
                                    .addComponent(customerLabel)
                                    .addComponent(productLabel))
                                .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.UNRELATED)
                                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
                                    .addComponent(customerError)
                                    .addComponent(customerTextField, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE)
                                    .addComponent(productError)
                                    .addComponent(productTextField, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE))
                                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
                                    .addGroup(layout.createSequentialGroup()
                                        .addGap(18, 18, 18)
                                        .addComponent(displayCustomerButton))
                                    .addGroup(layout.createSequentialGroup()
                                        .addGap(18, 18, 18)
                                        .addComponent(displayProductButton)))
                                .addGap(0, 0, Short.MAX_VALUE)))))
                .addContainerGap())
            .addGroup(layout.createSequentialGroup()
                .addGap(41, 41, 41)
                .addComponent(titleLabel)
                .addContainerGap(javax.swing.GroupLayout.DEFAULT_SIZE, Short.MAX_VALUE))
        );
        layout.setVerticalGroup(
            layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGroup(layout.createSequentialGroup()
                .addContainerGap()
                .addComponent(titleLabel)
                .addGap(35, 35, 35)
                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.BASELINE)
                    .addComponent(customerTextField, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE)
                    .addComponent(customerLabel)
                    .addComponent(displayCustomerButton))
                .addGap(12, 12, 12)
                .addComponent(customerError)
                .addGap(18, 18, 18)
                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.BASELINE)
                    .addComponent(productLabel)
                    .addComponent(displayProductButton)
                    .addComponent(productTextField, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE))
                .addGap(8, 8, 8)
                .addComponent(productError)
                .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.UNRELATED)
                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.BASELINE)
                    .addComponent(displayAllButton)
                    .addComponent(closeButton)
                    .addComponent(clearButton))
                .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.RELATED)
                .addComponent(jScrollPane1, javax.swing.GroupLayout.DEFAULT_SIZE, 163, Short.MAX_VALUE)
                .addContainerGap())
        );
    }// </editor-fold>                        

    public void connectToServer() {
        try {
            connect = new Socket("localhost", 9999);
            if (connect == null) {
                System.err.println("\nconnectToServer. connect is NULL *****\n");
                System.exit(0);
            }
        } catch (UnknownHostException ex) {
            ex.printStackTrace();
            
            ex.printStackTrace();
        } catch (IOException ex) {
            ex.printStackTrace();
            
        }
    }

    public void getStreams() {
        try {
            if (connect == null) {
                System.err.println("getStreams. connect is NULL *****\n");  ///LOOK
                ///LOOK   Somehow 'connect' has been reset to null. 
                System.exit(0);
            }
            
            input = new ObjectInputStream(connect.getInputStream());
            output = new ObjectOutputStream(connect.getOutputStream());

        } catch (IOException ex) {
            ex.printStackTrace();
        }
    }

    private void displayCustomerButtonActionPerformed(java.awt.event.ActionEvent evt) {                                                      
        try {
            
            Client displayCustomer = new Client();
            //Initialize boolean variable valid and set equal to true
            boolean valid = true;
            //Declare string variable salesID equal to iDTextField.getText.trim
            String input2 = customerTextField.getText().trim();
            //If salesID length is equal to 0 or greater than 3
            if (input2.length() == 0 || input2.length() < 3) {
                //Set salesRepIDError visibility equal to true
                customerError.setVisible(valid);
                //valid is equal to false
                valid = false;
            } else { //End if statement
                //Initialize else statement
                
                
                customerError.setVisible(false);
                
                
                getStreams();
                output.writeChars(input2);
                output.flush();
            }//End else Statement
        } catch (IOException ex) {
            ex.printStackTrace();
        }
    }                                                     

    private void displayProductButtonActionPerformed(java.awt.event.ActionEvent evt) {                                                     
        try {
            
            Client displayProduct = new Client();
            //Initialize boolean variable valid and set equal to true
            boolean valid = true;
            //Declare string variable salesID equal to iDTextField.getText.trim
            String input2 = productTextField.getText().trim();
            //If salesID length is equal to 0 or greater than 5
            if (input2.length() == 0 || input2.length() > 3) {
                //Set salesRepIDError visibility equal to true
                productError.setVisible(true);
                //valid is equal to false
                valid = false;
            }//End if statement
            //Initialize else statement
            else {
                productError.setVisible(false);
                try {
                    input2 = (String) input.readObject();
                } catch (ClassNotFoundException ex) {
                    ex.printStackTrace();
                }
                String productData = productTextField.getText().trim();
  
                output.writeBytes(productData + "\n");
                output.flush();
            }//End else Statement
        } catch (IOException ex) {
            ex.printStackTrace();
        }

    }                                                    

    private void displayAllButtonActionPerformed(java.awt.event.ActionEvent evt) {                                                 
        try {
            
            if (output == null) {
                System.err.println("displayAllButtonActionPerformed. output is NULL *****\n");   ///LOOK
                ///LOOK Somehow 'output' has been reset to null !!!
                System.exit(0);
            }
            getStreams();
            output.writeBytes("Display All");
            output.flush();
        } catch (IOException ex) {
            ex.printStackTrace();
        }
    }                                                

    private void closeButtonActionPerformed(java.awt.event.ActionEvent evt) {                                            
        
        getStreams();
        try {
            if (output != null) {
                output.close();         // close output stream
            }
            if (input != null) {
                input.close();          // close input stream 1
            }
            if (connect != null) {
                connect.close();         // close socket
            }
        } catch (IOException ex) {
            ex.printStackTrace();
        }
        
        System.exit(0);
    }                                           

    private void clearButtonActionPerformed(java.awt.event.ActionEvent evt) {                                            
        customerTextField.setText("");
        productTextField.setText("");
        displayTextArea.setText("");
    }                                           
    // Variables declaration - do not modify                     
    private javax.swing.JButton clearButton;
    private javax.swing.JButton closeButton;
    private javax.swing.JLabel customerError;
    private javax.swing.JLabel customerLabel;
    private javax.swing.JTextField customerTextField;
    private javax.swing.JButton displayAllButton;
    private javax.swing.JButton displayCustomerButton;
    private javax.swing.JButton displayProductButton;
    private javax.swing.JTextArea displayTextArea;
    private javax.swing.JScrollPane jScrollPane1;
    private javax.swing.JLabel productError;
    private javax.swing.JLabel productLabel;
    private javax.swing.JTextField productTextField;
    private javax.swing.JLabel titleLabel;
    // End of variables declaration                   
}

package sexton_it351_p1ip_client;

import java.io.IOException;
import javax.swing.JFrame;

public class Sexton_IT351_P1IP_Client {

///LOOK If main can throw an uncaught exception, then you lose points because I WILL crash your program
public static void main(String[] args) throws IOException {



JFrame window = new JFrame("Client");
//Add to window new SalesPanel
window.add(new Client());
//Set window default close operation to JFrame exit on close
window.setDefaultCloseOperation(JFrame.DO_NOTHING_ON_CLOSE); ///LOOK WHY DISABLE THE 'Quit' BUTTON ??
//Set window visibility true
window.setVisible(true);
//Set window bounds to 800,800,600,550
window.setBounds(800, 800, 600, 550);
//Set window set location 700, 300
window.setLocation(700, 300);

Client app = new Client();
app.runClient();

}
}


package sexton_it351_p1ip_server;

import java.io.*;
import java.net.ServerSocket;
import java.net.Socket;

/**
*
* @author CharlesSexton
*/
public class Server {

private ServerSocket server;
private ObjectOutputStream output;
private ObjectInputStream input;
private Socket connect;

public void runServer() {
try {
server = new ServerSocket(9999, 25);
while (true) {

waitConnection();
getStreams();
processConnection();

}
} catch (IOException ex) {
ex.printStackTrace();
}
}

public void waitConnection() {
try {
connect = server.accept();

System.out.println(connect.getLocalAddress());
System.out.println(connect.getPort());
} catch (IOException ex) {
ex.printStackTrace();
}
}

public void getStreams() {
try {

input = new ObjectInputStream(connect.getInputStream());
output.flush();
output = new ObjectOutputStream(connect.getOutputStream());
} catch (IOException ex) {
ex.printStackTrace();
}
}

public void processConnection() throws IOException {

getStreams();




do { // process messages sent from server
try { // read message and display it
String input2; // display message
input2 = (String) input.readObject();

System.out.println(input2);
} // end try
catch (ClassNotFoundException ex) {
ex.printStackTrace();
} // end catch
catch (java.net.SocketException ex) {
ex.printStackTrace();
break;
}
}while(input != null);


}

public void closeConnection() {

getStreams();

try {
input.close();
output.close();


connect.close();
} catch (IOException ex) {
ex.printStackTrace();
}
}
}


package sexton_it351_p1ip_server;

import java.io.IOException;

public class Sexton_IT351_P1IP_Server {

public static void main(String[] args) throws IOException {
Server app = new Server();
app.runServer();
}
}



PRINT STACK TRACE!!! When ran and Customer display button is pushed with dddd in the correct JTextField


Server Project/Application

java.net.SocketException: Connection reset
at java.net.SocketInputStream.read(SocketInputStream.java:189)
at java.net.SocketInputStream.read(SocketInputStream.java:121)
at java.io.ObjectInputStream$PeekInputStream.read(ObjectInputStream.java:2308)
at java.io.ObjectInputStream$PeekInputStream.readFully(ObjectInputStream.java:2321)
at java.io.ObjectInputStream$BlockDataInputStream.readShort(ObjectInputStream.java:2792)
at java.io.ObjectInputStream.readStreamHeader(ObjectInputStream.java:799)
at java.io.ObjectInputStream.<init>(ObjectInputStream.java:299)
at sexton_it351_p1ip_server.Server.getStreams(Server.java:49)
at sexton_it351_p1ip_server.Server.runServer(Server.java:26)
at sexton_it351_p1ip_server.Sexton_IT351_P1IP_Server.main(Sexton_IT351_P1IP_Server.java:9)
java.net.SocketException: Connection reset
at java.net.SocketInputStream.read(SocketInputStream.java:134)
at java.net.SocketInputStream.read(SocketInputStream.java:121)
at java.io.ObjectInputStream$PeekInputStream.read(ObjectInputStream.java:2308)
at java.io.ObjectInputStream$PeekInputStream.readFully(ObjectInputStream.java:2321)
at java.io.ObjectInputStream$BlockDataInputStream.readShort(ObjectInputStream.java:2792)
at java.io.ObjectInputStream.readStreamHeader(ObjectInputStream.java:799)
at java.io.ObjectInputStream.<init>(ObjectInputStream.java:299)
at sexton_it351_p1ip_server.Server.getStreams(Server.java:49)
at sexton_it351_p1ip_server.Server.processConnection(Server.java:59)
at sexton_it351_p1ip_server.Server.runServer(Server.java:27)
at sexton_it351_p1ip_server.Sexton_IT351_P1IP_Server.main(Sexton_IT351_P1IP_Server.java:9)
Exception in thread "main" java.lang.NullPointerException
at sexton_it351_p1ip_server.Server.processConnection(Server.java:67)
at sexton_it351_p1ip_server.Server.runServer(Server.java:27)
at sexton_it351_p1ip_server.Sexton_IT351_P1IP_Server.main(Sexton_IT351_P1IP_Server.java:9)
Java Result: 1
BUILD SUCCESSFUL (total time: 7 seconds)



Print Stack Trace for Client Project/Application

getStreams. connect is NULL *****

BUILD SUCCESSFUL (total time: 3 seconds)

[Charles Sexton]

I apologize for the forum rules error.....



Client Application main method repost due to error.....

package sexton_it351_p1ip_client;

import java.io.IOException;
import javax.swing.JFrame;

public class Sexton_IT351_P1IP_Client {

    ///LOOK   If main can throw an uncaught exception, then you lose points because I WILL crash your program
    public static void main(String[] args) throws IOException {
       

        
        JFrame window = new JFrame("Client");
        //Add to window new SalesPanel
        window.add(new Client());
        //Set window default close operation to JFrame exit on close
        window.setDefaultCloseOperation(JFrame.DO_NOTHING_ON_CLOSE);    ///LOOK   WHY DISABLE THE 'Quit' BUTTON ??
        //Set window visibility true
        window.setVisible(true);
        //Set window bounds to 800,800,600,550
        window.setBounds(800, 800, 600, 550);
        //Set window set location 700, 300
        window.setLocation(700, 300);

        Client app = new Client();
        app.runClient();

    }
}

Main Method for Server Application repost due to error.......

package sexton_it351_p1ip_server;

import java.io.IOException;

public class Sexton_IT351_P1IP_Server {

    public static void main(String[] args) throws IOException {
        Server app = new Server();
        app.runServer();
    }
}

Server Class within Server Application repost due to error......

package sexton_it351_p1ip_server;

import java.io.*;
import java.net.ServerSocket;
import java.net.Socket;

/**
 *
 * @author CharlesSexton
 */
public class Server {

    private ServerSocket server;
    private ObjectOutputStream output;
    private ObjectInputStream input;
    private Socket connect;

    public void runServer() {
        try {
            server = new ServerSocket(9999, 25);
            while (true) {
                
                    waitConnection();
                    getStreams();
                    processConnection();
       
            }
        } catch (IOException ex) {
            ex.printStackTrace();
        }
    }

    public void waitConnection() {
        try {
            connect = server.accept();

            System.out.println(connect.getLocalAddress());
            System.out.println(connect.getPort());
        } catch (IOException ex) {
            ex.printStackTrace();
        }
    }

    public void getStreams() {
        try {

            input = new ObjectInputStream(connect.getInputStream());
            output.flush();
            output = new ObjectOutputStream(connect.getOutputStream());
        } catch (IOException ex) {
            ex.printStackTrace();
        }
    }

    public void processConnection() throws IOException {

        getStreams();




        do {            // process messages sent from server
            try {       // read message and display it
                String input2; // display message
                input2 = (String) input.readObject();
                
                System.out.println(input2);
            } // end try
            catch (ClassNotFoundException ex) {
                ex.printStackTrace();
            } // end catch
            catch (java.net.SocketException ex) {
                ex.printStackTrace();
                break;
            }
        }while(input != null);


    }

    public void closeConnection() {

        getStreams();

        try {
            input.close();
            output.close();
            
            
            connect.close();
        } catch (IOException ex) {
            ex.printStackTrace();
        }
    }
}

[Charles Sexton]

Also please take a look at the close button and the customerDisplay button......they are both related to the input/output streams and I haven't even started on displayAll button and productButton......the application will eventually read from txt files.....just trying to establish a firm connect using the close and customer display buttons.....without errors.....but I keep running into lots of errors and having to repeatedly change things.

[Richard Tookey]

For some reason that totally escapes me your client code creates 4 instances of class Client but only applies the runClient() on one of them so 3 of the clients will never have made a connection to the server! I suspect you should stand back and review your design because your client code looks to be creating too many instances of your client and, since it is single threaded, your server code can handle only one client at a time.

Your big-bang approach rarely works. You should start with a much more simple client-server program. The server should be able to handle multiple simultaneous connections and the client should be able to handle multiple request/response actions. Initially there should be no GUI. Until you have the simple system working and you fully understand the concepts you are probably wasting your time trying anything a complex as your current work.

In my view you should really spend a lot of time studying http://docs.oracle.com/javase/tutorial/networking/sockets/clientServer.html and in particular the "Supporting Multiple Clients" section.

[Charles Sexton]

Richard Tookey wrote:For some reason that totally escapes me your client code creates 4 instances of class Client but only applies the runClient() on one of them so 3 of the clients will never have made a connection to the server! I suspect you should stand back and review your design because your client code looks to be creating too many instances of your client and, since it is single threaded, your server code can handle only one client at a time.

Your big-bang approach rarely works. You should start with a much more simple client-server program. The server should be able to handle multiple simultaneous connections and the client should be able to handle multiple request/response actions. Initially there should be no GUI. Until you have the simple system working and you fully understand the concepts you are probably wasting your time trying anything a complex as your current work.

In my view you should really spend a lot of time studying http://docs.oracle.com/javase/tutorial/networking/sockets/clientServer.html and in particular the "Supporting Multiple Clients" section.

This is a past due school assignment in which I am trying to get help in and I assume you are saying start over with something that just writes to the system?

[Richard Tookey]

Charles Sexton wrote:
This is a past due school assignment in which I am trying to get help in

As was said on day one of this thread, 'connect' is null and you need to address that problem and the only way I can see to address that is to address the fundamental design issue. Since you are already in extra time I can't see that I can help further without actually re-writing the code for you which would not really help you.

and I assume you are saying start over with something that just writes to the system?

In effect yes but your time constraints make that sound impossible.

[Charles Sexton]

Richard Tookey wrote:

Charles Sexton wrote:
This is a past due school assignment in which I am trying to get help in

As was said on day one of this thread, 'connect' is null and you need to address that problem and the only way I can see to address that is to address the fundamental design issue. Since you are already in extra time I can't see that I can help further without actually re-writing the code for you which would not really help you.

and I assume you are saying start over with something that just writes to the system?

In effect yes but your time constraints make that sound impossible.

Thank you, sounds like I have no choice.....