Pinball program problem

[trueblue]

Dear All,
I am writing a sort of a pinball program that will see a ball bounce around a table, and when it hits any of the four sides it changes colour in the sequence BLUE to RED to YELLOW - and then back to BLUE again. For the colour changes I am using a nested IF statement as follows, but the problem I am having is that the ball stays YELLOW for two bounces before it changes back to BLUE.
For the life of me I can't see why this would be. Can anyone please throw any light on the probably obvious tweak that needs to be made?
I have just written out the code in plain English as it almost works - I think it's just a structural issue. Many thanks!

if (the ball touches any of the sides)
{
if (the ball is BLUE)
{
make the ball RED
}

else
if (the ball is RED)
{
make the ball YELLOW
}

else
if (the ball is YELLOW)
{
make the ball BLUE
}
}

[Singing Boyo]

While plain English is great, sometimes things show up in code that can't be seen. If you post the code as well, someone may be able to help you. Certain statements just don't make sense when written in English.

[angryboy]

you're gona have to post the code.

[Singing Boyo]

you're gona have to post the code.

didn't I just say that? :p

[angryboy]

we posted at the same time. big deal

[trueblue]

OK sorry here's the full code

public void ricochet (int xInc, int yInc, int noOfHits)
{
int hitCount;
hitCount = 0;
while (hitCount < noOfHits)
{
this.ball.setXPos(getBallXPos() + xInc);
this.ball.setYPos(getBallYPos() + yInc);
this.delay(30);

if (this.isInCorner())
{
xInc = - xInc;
yInc = - yInc;
hitCount = hitCount + 1;
Toolkit.getDefaultToolkit().beep();
}
else
{
if (this.isTouchingLeftOrRight())
{
xInc = - xInc;
hitCount = hitCount + 1;
Toolkit.getDefaultToolkit().beep();
}
else
{
if (this.isTouchingTopOrBottom())
{
yInc = - yInc;
hitCount = hitCount + 1;
Toolkit.getDefaultToolkit().beep();
}
}
if (this.isInCorner() || this.isTouchingLeftOrRight() || this.isTouchingTopOrBottom())
{
if (this.ball.getColour() == BLUE)
{
this.ball.setColour(RED);
}
else

if (this.ball.getColour() == RED)
{
this.ball.setColour(YELLOW);
}
else

if (this.ball.getColour() == YELLOW)
{
this.ball.setColour(BLUE);
}
}
}
}
}

[emceenugget]

does the behavior recur for all input? your conditional statements don't seem organized too well. if a corner is hit, the color won't change. better fix that one.

[trueblue]

does the behavior recur for all input? your conditional statements don't seem organized too well. if a corner is hit, the color won't change. better fix that one.

yes the ball changes color in the corners no problem but when the ball is yellow it takes two hits against any side or corner to turn back to blue and re-start the sequence.

[trueblue]

does the behavior recur for all input? your conditional statements don't seem organized too well. if a corner is hit, the color won't change. better fix that one.

Hey dude, you are completely right - I had one pair of braces too many enclosing an else statement so the corner hits were being excluded.

I bow to your superior judgement. Many thanks!