Method to display not working

[Maureen Charlton]

Hi,

When someone has a moment could they suggest why the following method is only printing out the last entry?

 
    //Method to display the students 
    public void display ( )
    {        
        //Display each Student object in the Storage object
        for (int lp = 0; lp < numberOfStudents; lp++)
        {              
            //Make a temporary variable called 'output', of type Student,
            //this will be used to refer to the Student object retrieved
            //on each pass through the for loop
            Student output = retrieveStudent(lp);
            System.out.print("\n          "+output.getName( )  );
            System.out.print("       "+output.getCourse( )  );          
        }//end for loop
        System.out.println("\n\nEnd of method Display");
    }//end method display
 

Thanking you in advance!

[Paul Sturrock]

Whats in this method:

retrieveStudent(int index)

?
[ November 24, 2004: Message edited by: Paul Sturrock ]

[Maureen Charlton]

Method to retrieve the Student objects from the students array
-needed to print out the student data


public Student retrieveStudent (int index)
{
return students[index];
}//end Method retrieveStudent

This is also in my Storage class.

[Maureen Charlton]

I am currently testing.

In my test file I have the following:

 
public class StorageTest
{
    public static void main(String [ ]args) 
    {
        //Class to test the operation of the Storage class
        int number = 6;                                                                              //Defines the number of Student objects 
 
  //Make an instance of Storage class to store Student object references in
        Storage store = new Storage (number);
        store.storeStudent ("Joe Bloggs", "Java");
        store.storeStudent ("Fred Smyth", "Visual Basic");
        store.storeStudent ("Sally Collins", "History");
        store.storeStudent ("JOHN      DOE", "Java");
        store.storeStudent ("JOE DOHN", "Visual Basic");
        store.storeStudent ("Joe Dohn", "Visual C#");
 
        //Print out each Student object in the Storage object
        store.display( );                  
    }//end main program
}//end public class StorageTest   
 

Currently I am getting the following:

The details entered are as follows:


Student Name: Student Course:
============= ==============+

Joe Dohn Visual C#
Joe Dohn Visual C#
Joe Dohn Visual C#
Joe Dohn Visual C#
Joe Dohn Visual C#
Joe Dohn Visual C#

[Paul Sturrock]

So, it might be this method:

store.storeStudent ("blah", "blah");

which is giving you a problem. What you can say with a degree of certainty is the index you access you students array with is being incremented, so it looks like the contents of this array may well be six entries of the same value. What does you Storage class look like?

[Maureen Charlton]

Paul,

GOT IT!!

I wrote one of my classes four months ago and used static for my variable.

Once I changed these my output was as expected.

Thanks for your help!!