problems with return method involving arrays

[Mulk Abdulhadi]

public class School
{
  public final int MIN_GRADE = 0;
  public final int MAX_GRADE = 100;
  private int [] grades;


  public School( int students )
  {
    grades = new int[students];

    for ( int i = 0; i < grades.length; i++ )
       grades[i] = (int) ( Math.random( ) * ( MAX_GRADE - MIN_GRADE + 1 ) );
  }

// Accessor Method
public int [] getArray( )
{
 int [] temp = new int [grades.length]; 
 for ( int i = 0; i <grades.length; i++)
 
  temp[i]=grades[i];
  return temp; 
 
}
// Mutator method 
public void SetArray(int [] setarray )
{
  grades = new int [setarray.length];
  
  for( int i = 0; i<grades.length; i++)
  {
   grades[i]=setarray[i];
  }
   
}

[b]public int HighestGrade ( )
{  
  int max = 0;
  for( int i = 1; i<grades.length;i++)
  {
   if(grades[i ]> grades[max])
       max= i ; 
  }
  
  return max;

  }[/b]


public class TestSchool{
  public static void main( String [] args)
  { 
    
    int [] Blue = {1,2, 3, 4, 6,7, 8, 9, 10 , 13, 14 , 15};    
    final int NUMBER_OF_STUDENTS = Blue.length;

    // instatiate an object with 20 students
    School g1 = new School( NUMBER_OF_STUDENTS );

    g1.SetArray(Blue);
    
    System.out.print( "The numbers in the array are:     " );
    int [] grades = g1.getArray( );
    for ( int i = 0; i < grades.length; i++ )
       System.out.print( grades[i] + " " );
    System.out.println( );

    
   [b]System.out.println("The highest grade is:  " +g1.HighestGrade( )  ) ;
    [/b]
    
  }
}

The output is :

The numbers in the array are: 1 2 3 4 6 7 8 9 10 13 14 15
The highest grade is: 11



I have no idea what is wrong with the code. Please Help
For some reason it is not giving me the highest number in my array. Did I write the method wrong ? Is my array even passing through HighestGrade ?

[Carey Brown]

Not sure what you intended on doing but you are printing the index into the array and not the value at that index.

[Mulk Abdulhadi]

Carey Brown wrote:Not sure what you intended on doing but you are printing the index into the array and not the value at that index.

I am just trying to print out the highest grade which would be 15 .

[Carey Brown]

Mulk Abdulhadi wrote:

Carey Brown wrote:Not sure what you intended on doing but you are printing the index into the array and not the value at that index.

I am just trying to print out the highest grade which would be 15 .

Yes, and you're printing out the index into the array at which '15' resides.

[Mulk Abdulhadi]

So , how do I make it print out 15 ?

[Carey Brown]

public int HighestGrade ( )
{ 
  int max = 0;
  for( int i = 1; i<grades.length;i++)
  {
   if(grades[i ]> grades[max])
       max= i ;
  }
   
  return max;
 
  }

In this method you already have an example of how to get a grade from the max index (hint: line 6).

[Mulk Abdulhadi]

WOW , okay I got it . Thanks for your help .

I just changed:

public int HighestGrade ( )
{  
  int max = 0;
  for( int i = 1; i<grades.length;i++)
  {
   if(grades[i]> grades[max])
       max= i ; 
  }
  
  return grades[max];