Why does this class not read and write at same time???

Question

I had another thread where i was trying to combine two class files together
http://forum.java.sun.com/thread.jspa?threadID=5146796
I managed to do it myself but when i run the file it does not work right. If i write a file then try and open the file it says there are no records in the file, but if i close the gui down and open the file everything get read in as normal can anybody tell me why?
import java.io.*;
import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import bank.BankUI;
import bank.*;
public class testing extends JFrame {
   private ObjectOutputStream output;
   private BankUI userInterface;
   private JButton SaveToFile, SaveAs, Exit; //////////savetofile also saves to store need to split up and have 2 buttons
   //private Store store; MIGHT BE SOMETHING TO DO WITH THIS AS I HAD TO COMMENT THIS STORE OUT TO GET IT WORKING AS STORE IS USED BELOW
   private Employee record;
//////////////////////////////////////from read
private ObjectInputStream input;
   private JButton nextButton, openButton, nextRecordButton ;
   private Store store = new Store(100);
     private Employee employeeList[] = new Employee[100];
     private int count = 0, next = 0;
/////////////////////////////////////////////from read end
   // set up GUI
   public testing()
      super( "Employee Data" ); // appears in top of gui
      // create instance of reusable user interface
      userInterface = new BankUI( 9 );  // nine textfields
      getContentPane().add( userInterface, BorderLayout.CENTER );
      // configure button doTask1 for use in this program
      SaveAs = userInterface.getSaveAsButton();
      SaveAs.setText( "Save as.." );
//////////////////from read
openButton = userInterface.getOpenFileButton();
      openButton.setText( "Open File" );
openButton.addActionListener(
         // anonymous inner class to handle openButton event
         new ActionListener() {
            // close file and terminate application
            public void actionPerformed( ActionEvent event )
               openFile();
         } // end anonymous inner class
      ); // end call to addActionListener
      // register window listener for window closing event
      addWindowListener(
         // anonymous inner class to handle windowClosing event
         new WindowAdapter() {
            // close file and terminate application
            public void windowClosing( WindowEvent event )
               if ( input != null )
                         closeFile();
               System.exit( 0 );
         } // end anonymous inner class
      ); // end call to addWindowListener
      // get reference to generic task button doTask2 from BankUI
      nextButton = userInterface.getDoTask2Button();
      nextButton.setText( "Next Record" );
      nextButton.setEnabled( false );
      // register listener to call readRecord when button pressed
      nextButton.addActionListener(
         // anonymous inner class to handle nextRecord event
         new ActionListener() {
            // call readRecord when user clicks nextRecord
            public void actionPerformed( ActionEvent event )
               readRecord();
         } // end anonymous inner class
      ); // end call to addActionListener
//get reference to generic task button do Task3 from BankUI
      // get reference to generic task button doTask3 from BankUI
      nextRecordButton = userInterface.getDoTask3Button();
      nextRecordButton.setText( "Get Next Record" );
      nextRecordButton.setEnabled( false );
      // register listener to call readRecord when button pressed
      nextRecordButton.addActionListener(
         // anonymous inner class to handle nextRecord event
         new ActionListener() {
            // call readRecord when user clicks nextRecord
            public void actionPerformed( ActionEvent event )
               getNextRecord();
         } // end anonymous inner class
      ); // end call to addActionListener
///////////from read end
      // register listener to call openFile when button pressed
      SaveAs.addActionListener(
         // anonymous inner class to handle openButton event
         new ActionListener() {
            // call openFile when button pressed
            public void actionPerformed( ActionEvent event )
               SaveLocation();
         } // end anonymous inner class
      ); // end call to addActionListener
      // configure button doTask2 for use in this program
      SaveToFile = userInterface.getSaveStoreToFileButton();
      SaveToFile.setText( "Save to store and to file need to split this task up" );
      SaveToFile.setEnabled( false );  // disable button
      // register listener to call addRecord when button pressed
      SaveToFile.addActionListener(
         // anonymous inner class to handle enterButton event
         new ActionListener() {
            // call addRecord when button pressed
            public void actionPerformed( ActionEvent event )
               addRecord(); // NEED TO SPLIT UP SO DONT DO BOTH
         } // end anonymous inner class
      ); // end call to addActionListener
Exit = userInterface.getExitAndSaveButton();
      Exit.setText( "Exit " );
      Exit.setEnabled( false );  // disable button
      // register listener to call addRecord when button pressed
      Exit.addActionListener(
         // anonymous inner class to handle enterButton event
         new ActionListener() {
            // call addRecord when button pressed
            public void actionPerformed( ActionEvent event )
               addRecord(); // adds record to to file
               closeFile(); // closes everything
         } // end anonymous inner class
      ); // end call to addActionListener
      // register window listener to handle window closing event
      addWindowListener(
         // anonymous inner class to handle windowClosing event
         new WindowAdapter() {
            // add current record in GUI to file, then close file
            public void windowClosing( WindowEvent event )
                         if ( output != null )
                            addRecord();
                              closeFile();
         } // end anonymous inner class
      ); // end call to addWindowListener
      setSize( 600, 500 );
      setVisible( true );
     store = new Store(100);
   } // end CreateSequentialFile constructor
   // allow user to specify file name
////////////////from read
  // enable user to select file to open
   private void openFile()
      // display file dialog so user can select file to open
      JFileChooser fileChooser = new JFileChooser();
      fileChooser.setFileSelectionMode( JFileChooser.FILES_ONLY );
      int result = fileChooser.showOpenDialog( this );
      // if user clicked Cancel button on dialog, return
      if ( result == JFileChooser.CANCEL_OPTION )
         return;
      // obtain selected file
      File fileName = fileChooser.getSelectedFile();
      // display error if file name invalid
      if ( fileName == null || fileName.getName().equals( "" ) )
         JOptionPane.showMessageDialog( this, "Invalid File Name",
            "Invalid File Name", JOptionPane.ERROR_MESSAGE );
      else {
         // open file
         try {
            input = new ObjectInputStream(
               new FileInputStream( fileName ) );
            openButton.setEnabled( false );
            nextButton.setEnabled( true );
         // process exceptions opening file
         catch ( IOException ioException ) {
            JOptionPane.showMessageDialog( this, "Error Opening File",
               "Error", JOptionPane.ERROR_MESSAGE );
      } // end else
   } // end method openFile
public void readRecord() // need to merger with next record
      Employee record;
      // input the values from the file
      try {
     record = ( Employee ) input.readObject();
               employeeList[count++]= record;
               store.add(record);/////////ADDS record to Store
          store.displayAll();
          System.out.println("Count is " + store.getCount());
         // create array of Strings to display in GUI
         String values[] = {
                    String.valueOf(record.getName()),
                        String.valueOf(record.getGender()),
                    String.valueOf( record.getDateOfBirth()),
                    String.valueOf( record.getID()),
                         String.valueOf( record.getStartDate()),
                    String.valueOf( record.getSalary()),
                    String.valueOf( record.getAddress()),
                       String.valueOf( record.getNatInsNo()),
                    String.valueOf( record.getPhone())
// i added all those bits above split onto one line to look neater
         // display record contents
         userInterface.setFieldValues( values );
      // display message when end-of-file reached
      catch ( EOFException endOfFileException ) {
         nextButton.setEnabled( false );
      nextRecordButton.setEnabled( true );
         JOptionPane.showMessageDialog( this, "No more records in file",
            "End of File", JOptionPane.ERROR_MESSAGE );
      // display error message if class is not found
      catch ( ClassNotFoundException classNotFoundException ) {
         JOptionPane.showMessageDialog( this, "Unable to create object",
            "Class Not Found", JOptionPane.ERROR_MESSAGE );
      // display error message if cannot read due to problem with file
      catch ( IOException ioException ) {
         JOptionPane.showMessageDialog( this,
            "Error during read from file",
            "Read Error", JOptionPane.ERROR_MESSAGE );
   } // end method readRecord
   private void getNextRecord()
           Employee record = employeeList[next++%count];//cycles throught accounts
      //create aray of string to display in GUI
      String values[] = {String.valueOf(record.getName()),
         String.valueOf(record.getGender()),
          String.valueOf( record.getStartDate() ), String.valueOf( record.getAddress()),
     String.valueOf( record.getNatInsNo()),
     String.valueOf( record.getPhone()),
         String.valueOf( record.getID() ),
           String.valueOf( record.getDateOfBirth() ),
     String.valueOf( record.getSalary() ) };
     //display record contents
     userInterface.setFieldValues(values);
     //display record content
///////////////////////////////////from read end
private void SaveLocation()
      // display file dialog, so user can choose file to open
      JFileChooser fileChooser = new JFileChooser();
      fileChooser.setFileSelectionMode( JFileChooser.FILES_ONLY );
      int result = fileChooser.showSaveDialog( this );
      // if user clicked Cancel button on dialog, return
      if ( result == JFileChooser.CANCEL_OPTION )
         return;
      File fileName = fileChooser.getSelectedFile(); // get selected file
      // display error if invalid
      if ( fileName == null || fileName.getName().equals( "" ) )
         JOptionPane.showMessageDialog( this, "Invalid File Name",
            "Invalid File Name", JOptionPane.ERROR_MESSAGE );
      else {
         // open file
         try {
            output = new ObjectOutputStream(
               new FileOutputStream( fileName ) );
            SaveAs.setEnabled( false );
            SaveToFile.setEnabled( true );
          Exit.setEnabled( true );
         // process exceptions from opening file
         catch ( IOException ioException ) {
            JOptionPane.showMessageDialog( this, "Error Opening File",
               "Error", JOptionPane.ERROR_MESSAGE );
      } // end else
   } // end method openFile
   // close file and terminate application
private void closeFile()
      // close file
      try {
          //you want to cycle through each recordand add them to store here.
                                        int storeSize = store.getCount();
                                        for (int i = 0; i<storeSize;i++)
                                        output.writeObject(store.elementAt(i));
         output.close();
input.close();// from read!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
         System.exit( 0 );
      // process exceptions from closing file
      catch( IOException ioException ) {
         JOptionPane.showMessageDialog( this, "Error closing file",
            "Error", JOptionPane.ERROR_MESSAGE );
         System.exit( 1 );
   } // end method closeFile
   // add record to file
   public void addRecord()
      int employeeNumber = 0;
      String fieldValues[] = userInterface.getFieldValues();
      // if account field value is not empty
      if ( ! fieldValues[ BankUI.IDNUMBER ].equals( "" ) ) {
         // output values to file
         try {
            employeeNumber = Integer.parseInt(
               fieldValues[ BankUI.IDNUMBER ] );
                    String dob = fieldValues[ BankUI.DOB ];
                    String[] dateofBirth = dob.split ("-"); // what used to put between number chnage to /
                    String sDate = fieldValues[ BankUI.START ];
                    String[] startDate = sDate.split ("-");
                    String sex = fieldValues[ BankUI.GENDER ];
                    char gender = (sex.charAt(0)); // check if m or f prob check in employee
if ( employeeNumber >= 0 ) {
              /* create new record =String name, char gender, Date dob, String add, String nin, String phone, String id, Date start, float salary*/
                record  = new Employee(
                fieldValues[ BankUI.NAME ],
                    gender,
                new Date(     Integer.parseInt(dateofBirth[0]),
                          Integer.parseInt(dateofBirth[1]),
                          Integer.parseInt(dateofBirth[2])),
                    fieldValues[ BankUI.ADDRESS ],
                    fieldValues[ BankUI.NATINTNO ],
                    fieldValues[ BankUI.PHONE ],
                    fieldValues[ BankUI.IDNUMBER ],
          new Date(     Integer.parseInt(startDate[0]),
                          Integer.parseInt(startDate[1]),
                          Integer.parseInt(startDate[2])),
          Float.parseFloat( fieldValues[ BankUI.SALARY ] ));
                    if (!store.isFull())
                         store.add(record);
                    else
                    JOptionPane.showMessageDialog( this, "The Store is full you cannot add\n"+
                     "anymore employees. \nPlease Save Current File and Create a New File." );
                         System.out.println("Store full");
                    store.displayAll();
                    System.out.println("Count is " + store.getCount());
                              // output record and flush buffer
                              //output.writeObject( record );
               output.flush();
            else
                JOptionPane.showMessageDialog( this,
                   "Account number must be greater than 0",
                   "Bad account number", JOptionPane.ERROR_MESSAGE );
            // clear textfields
            userInterface.clearFields();
         } // end try
         // process invalid account number or balance format
         catch ( NumberFormatException formatException ) {
            JOptionPane.showMessageDialog( this,
               "Bad ID number, Date or Salary", "Invalid Number Format",
               JOptionPane.ERROR_MESSAGE );
         // process exceptions from file output
         catch ( ArrayIndexOutOfBoundsException ArrayException ) {
             JOptionPane.showMessageDialog( this, "Error with Start Date or Date of Birth\nPlease enter like: 01-01-2001",
                "IO Exception", JOptionPane.ERROR_MESSAGE );
                  // process exceptions from file output
         catch ( IOException ioException ) {
             JOptionPane.showMessageDialog( this, "Error writing to file",
                "IO Exception", JOptionPane.ERROR_MESSAGE );
            closeFile();
      } // end if
   } // end method addRecord
   public static void main( String args[] )
      new testing();
} // end class CreateSequentialFile

Answer 1

Sure you can read and write at the same time. But normally you would be reading from one place and writing to another place.
I rather regret avoiding the OP's earlier post asking how to combine two classes. I looked at the two classes posted and realized the best thing to do was actually to break them into more classes. But I also realized it was going to be a big job explaining why and how, and I just don't have the patience for that sort of thing.
So now we have a Big Ball Of Tar&trade; and I feel partly responsible.