not sure what im doing wrong with these if else statements

[yrvd86]

the first one works but the second does not when its called upon later

if (major == 1)
{
if (gpa > largestm1)
{
largestm1 = gpa;
highnamem1 = last;

}

else if (major == 2)
{

if (gpa > largestm2)
{
largestm2 = gpa;
highnamem2 = last;
}

}

and the next one wont work correctly at all

if (credits <= 30)
{
credits1 = students;
}
else if (credits >=31 && credits <=60)
{
credits2 = students;
}
else if (credits >=61 && credits <=90)
{
credits3 = students;
}
else if (credits >=90)
{
credits4 = students;
}
}

I appreciate any help you can give

[Fr33dan]

When trying to determine errors with nested code blocks indenting each block in helps you see errors:

if (major == 1)
{ // Indent starting here because this is the start 
  // of a code block.
    if (gpa > largestm1)
    {
    largestm1 = gpa;
    highnamem1 = last;
 
    }
  // Now we know the next statement shouldn't go in the
  // code block because it's not indented
 } // and we can see that we missed the end bracket.
else if (major == 2)
{
 
    if (gpa > largestm2)
    {
    largestm2 = gpa;
    highnamem2 = last;
    }
 
}

if (credits <= 30)
{
    credits1 = students;
}
else if (credits >=31 && credits <=60)
{
    credits2 = students;
}
else if (credits >=61 && credits <=90)
{
    credits3 = students;
}
else if (credits >=90)
{
    credits4 = students;
}
 } // We should end go back a step at the end bracket 
  // but we don't have anywhere to go so this must
  // be an accidental extra (possibly added to stop 
  // missing bracket errors).

[jkmyoung]

What does students refer to?
What do credits1, credits2, credits3, and credits4 refer to?