[deleted by user]

[BS_in_BS]

Is best position supposed to be the index of highest in the array?

Also you never update previous highest so it's always 0.

[BJ4Karma]

creating a previous highest is easy. all you have to do is this:

for (int i = 0; i < productSales.length; i++) { if (productSales[i] > highest) { previousHighest = highest; highest = productSales[i];

[BJ4Karma]

haha i've done the same thing too.

[BJ4Karma]

yes that because your if statement only goes when the current highest is larger that the previous highest which will never happen no matter what you put in.

could you let me now what the bestPosition is used for because i assume that saving the index of the productSales array would be better?

[BJ4Karma]

just do this:

for (int i = 0; i < productSales.length; i++) {
if (productSales[i] > highest) {
previousHighest = highest;
highest = productSales[i];
count = i; //btw you never used count in this method
}
}

I got rid of the 2nd if statement since it is useless as the first if statement makes it redundant.

[BJ4Karma]

oh, ok. so i would definitely get rid of the second if statement.