How to stop trading when Profit is made?

I am struggling to get an EA to stop trading once it makes profit for the month.
So basically I need the EA to stop trading for the month that it makes the first profit order in, then when the next month arrives it continues to make trades.

Here is roughly what I have used:

for (int i = 0; i<=OrdersTotal();i++)
{
OrderSelect(i-1,SELECT_BY_POSITION,MODE_TRADES);
if (type <= OP_SELL blah blah..)
{
if order is a buy order
{
if (OrderProfit() > 0) // check that order made a profit
{
if (TimeMonth(OrderCloseTime()) == TimeMonth(TimeCurrent()))
// check to see if the profit order closed in
the current month
{
buymadeprofit = true; // as it did, it is true.
}
}
}
}
}

if (buymadeprofit == false)
{
// Trade criteria goes here because order profit was not made
in the same month as present month
}


I have tried using static bool, plain bool, custom function, tried using static int instead of bool, and normal int.
All that happens is it will make a profitable trade and then it will never trade again. It is like once buymadeprofit is trigerred it stays triggered and doesnt re-set to 0 or false, even if I place that code outside of the function.

If anyone has any ideas or knows what im doing wrong it wud be very appreciated. Thanks.

How about using TimeMonth() ?

int CurrentMonth;

if(OrderProfit() > 0) {
CurrentMonth = TimeMonth(TimeCurrent);
}

if(CurrentMonth != TimeMonth(TimeCurrent) {

//Enter trading functions here

}

unfortunatly that does the same thing as what I had before. But it is an interesting way of doing it. Thanks for reply.
Maybe I am not returning correctly? It is like it is ignoring it all together?
I have placed this part of ur suggested code into the OrderSelect function I have:
int CurrentMonth;

if(OrderProfit() > 0) {
CurrentMonth = TimeMonth(TimeCurrent);
}


and then when it exits this orderselect function I go to my trade function, and wrap my trade decision as you suggest like so :

if(CurrentMonth != TimeMonth(TimeCurrent) {

//Enter trading functions here

}

Is it somthing to do with I am not using it in the same function? Because I also have (in order select function) code to close an order if it has been opened in a different month, then close the trade, which works perfectly. This code is in the same function so do I need to place my trade decision inside the orderselect function to get this to work?

Well if it's interfering with other functions, I can't do much unless I see the code and perhaps try and fit this part in. But I think you understand the concept.

cool sure. Here is where Im using the code u suggested, aswell as where I have tried the way I done it. I think the problem is somthing to do with its returning of the bool profitmade. When it makes the first profitable trade it stops trading totally.

bool profitmade = false;
int currentmonth;
int i;

for(i=1; i<=OrdersHistoryTotal(); i++)
{
OrderSelect(i,SELECT_BY_POS,MODE_HISTORY);
if(OrderType()<=OP_SELL && OrderSymbol()==Symbol())
{
datetime OCT=OrderCloseTime();
int orderprofit = OrderProfit();
if (OCT > 0)
{
if(OrderType()==OP_BUY) // long position is opened
{
if (orderprofit>0)
{
currentmonth = TimeMonth(TimeCurrent());
}
if (currentmonth != TimeMonth(TimeCurrent()))
{
profitmade = false;
}
else
{
profitmade = true;
return(0);
}
}

else //Go to Short Trade
{
if (orderprofit>0)
{
currentmonth = TimeMonth(TimeCurrent());
}
if(currentmonth != TimeMonth(TimeCurrent()))
{
profitmade = false;
}
else
{
profitmade = true;
return(0);
}
}
}
}
}


if (profitmade == false)
{
if (total < 1)
{

// Trade criteria ....


I am a bit puzzled over the placement of return() etc, I have tried to use return(profitmade) too when I make the bool static. Im really stuck with this one.

finally I have sorted it. I have removed the return(0); from that for loop, also added this to it:

if(TimeMonth(OrderCloseTime()) == Month())
{
profitmade = true;
}
else
{
profitmade = false;
}

So now it is working very well. Thanks for your help ronald.

Ok now I have a new problem. I would like to incorperate martingale strategy to my ea. Would I be correct in using something like this:

double Lot = 0.1;

if (OrderProfit() < 0)
{
Lot = Lot * 2;
}
else
{
Lot = 0.1;
}

With that code, you'd only go to 0.2...