C++ Help!

[PrincessJasmine]

Hiya, I'm trying to get some code to work...I am tryna get a shape to roll from left to right. So far i have written this code and my shape does go from left to right but does not roll. It is part of a GUI when i press the button it moves 2 to the right then stops. I find myslef having to press the button continously to make it roll. But what i want is to press the button once and make the shape roll from left to right and stop at position 300. Any suggestions on where im going wrong on my code? thanks

//---------------------------------------------------------------------------
void __fastcall TForm1::TimerTimer(TObject *Sender)
{
Timer->Enabled= false;
int TotalShapeAmount;
int ShapeAmount;
TotalShapeAmount = 0;
ShapeAmount = 2;
if(Shape->Left<=300)
{
TotalShapeAmount = Shape->Left+ShapeAmount;
Shape->Left=TotalShapeAmount;

}
}}
//---------------------------------------------------------------------------

void __fastcall TForm1::ButtonClick(TObject *Sender)
{
Timer->Enabled = true;
}
//---------------------------------------------------------------------------

[TheAnimus]

Timer->Enabled= false;

is stopping subsiquent requests to timer, without the button been clicked, I'm guessing you want that after the IF.

[Butcher]

Yeah the problem is you're switching off the timer each time it moves the shape. Try putting the Timer->Enabled= false; as an else clause after the if.

[PrincessJasmine]

Oh right i see, I removed the Timer->Enabled = false; statement and it worked.
In the inpsector in Borland Builder 5, I had enabled it to false; so really i was repeating myself.

Thank you to every one who replied.
Jas

[PrincessJasmine]

ok now ive got another problem... My shape rolls, but ihave to get it move roll gradually to stop slowly.
Using this code it rolls and then rolls FASTER to a stop. but i want it to roll and SLOWLY stop :S
Any suggestion on this code!

{

int TotalShapeAmount;
int ShapeAmount;
TotalShapeAmount = 0;
ShapeAmount = 6;
Button->OnClick;
if(Shape->Left<=550)
{
TotalShapeAmount = Shape->Left+ShapeAmount;
Shape->Left=TotalShapeAmount;
if (Shape->Left>350)
{
TotalShapeAmount = Shape->Left+(ShapeAmount*0.5);
Shape->Left=TotalShapeAmount;
}
}
}

[Butcher]

You're adding to it twice if it's close to the end which makes it accelerate, you only want to add to it once.

[PrincessJasmine]

How do u mean twice? by the two if statements? I cant think of anything else LOL

[Butcher]

TotalShapeAmount = Shape->Left+ShapeAmount;
Shape->Left=TotalShapeAmount;
if (Shape->Left>350)
{
  TotalShapeAmount = Shape->Left+(ShapeAmount*0.5);
  Shape->Left=TotalShapeAmount;
}

This moves it twice if Shape->Left > 350, if you put the first two lines into an else it wouldn't.

[Butcher]

Also you should use ShapeAmount/2 instead of ShapeAmount*0.5 since you're using ints.

[PrincessJasmine]

I'm trying but i seem to be making a worse mess of it :S