My max function always goes to the last number

[plususerz]

Why does my max function (meant to find the highest number within the array of 10 numbers) ALWAYS state the very last number in the array?

#include <iostream>
#include <ctime>
#include <limits>

using namespace std;

int main( )
{
   int Input;
   int x;
   int arr[10];

   for (x = 0; x < 10; x++)
   {
       cout << "Enter the # of pancakes person #" << x << " ate:" << endl;
       cin >> Input;

       arr[x] = Input; // This creates an array of 10 numbers I input
   }

   int max = 0;

   cout << " " << endl;

   for (x = 0; x < 10; x++)
   {
       if (max < arr[x]); // This is my (defective) max function
       {
           max = arr[x];
       }

   }
cout << " " << endl;

cout << max << endl;

   return 0;
}

I've also tried editing the code to have:

for (x = 0; x < 10; x++)
{
     if (max < arr[x]);
     max = arr[x];
}

but the problem still isn't fixed.

For example, if I say person #1 ate 99 pancakes, person #2 - #8 ate 0 pancakes, and person #9 at 2 pancakes, the last statement would say "max = 2" instead of "max = 99."

[ne555]

       if (max < arr[x]); //<-- look at the semicolon
       {
           max = arr[x];
       }

that could be writted as

if (max < arr[x])
	; //<--
max = arr[x];

As you may note by the indentation the max = arr[x]; statement does not form part of the if.
It is executed always on every iteration, and so, `max' would hold the last value of the array.

Simply remove the semicolon at line 27

[plususerz]

@ne555 DAMN wow.... can't believe I wasted 30min to an hour on that. Thanks so much :D and for everyone else who's replied on my threads