Close all orders with magic number at certain profit

I have an incomplete script which will close all orders with a certain magic number at a certain profit target. I can not get the script to work and seems to be missing some basic formatting/function calls. Can anyone take a look, I'm sure it just needs a few things added (int init() etc etc).

Thanks


...
extern bool AutoClosePositions = true;
...
double MaxTargetProfit;
MaxTargetProfit = TotalOrders()*10; //This is up to your profit target or formula!
...
...
if(AutoClosePositions && TotalOrders()!= 0) {
if(Profit()>=MaxTargetProfit){
int total = OrdersTotal();
for(int i=total-1;i>=0;i--){
OrderSelect(i, SELECT_BY_POS);
int type = OrderType();
bool result = false;

if(OrderMagicNumber()==MAGIC){
switch(type){

case OP_BUY : result = OrderClose(OrderTicket(),OrderLots(),MarketInfo(OrderSymbol(),MODE_BID),5,Red);
break;

case OP_SELL : result = OrderClose(OrderTicket(),OrderLots(),MarketInfo(OrderSymbol(),MODE_ASK),5, Blue);
} //switch

if(result == false){
Print("Order ", OrderTicket(), " failed to close. Error:", GetLastError() );
}//if(result == false)
}//if(OrderMagicNumber()==MAGIC){
}//for(int i= ...
}//if(AccountProfit()>=MaxTargetProfit)
}//if(AutoClose...

int TotalOrders() {
int cnt=0;
for (int i=0; i<OrdersTotal(); i++) {
if (OrderSelect(i, SELECT_BY_POS, MODE_TRADES)) {
if (OrderMagicNumber()==MAGIC && (OrderType()==OP_BUY || OrderType()==OP_SELL)) {
cnt++;
}
}
}
return(cnt);
}

double Profit(){
double Prof=0;
for (int i=0; i<OrdersTotal(); i++) {
if (OrderSelect(i, SELECT_BY_POS, MODE_TRADES)) {
if (OrderMagicNumber()==MAGIC) {
Prof = Prof + OrderProfit();
}
}
}
return(Prof);
}

based on this section of code it will close based on just magic number:

if(OrderMagicNumber()==MAGIC){
switch(type){

case OP_BUY : result = OrderClose(OrderTicket(),OrderLots( ),MarketInfo(OrderSymbol(),MODE_BID ),5,Red);
break;

case OP_SELL : result = OrderClose(OrderTicket(),OrderLots( ),MarketInfo(OrderSymbol(),MODE_ASK ),5, Blue);
} //switch


take all of that switch statement out, it is not needed. when you close an order you only need ordercloseprice(). so it would be OrderClose(OrderTicket(),OrderLots(),OrderClosePrice(),5,Blue);

so are you looking for something to close all orders with the correct magic number if the total profit reaches x amount? Do you need to close any pending orders as well??

Yes, just looking to close all orders with the correct magic number if the total profit reaches x amount for just the orders with a certain magic number. So it would total the orders with a certain magic number and close just those if the total profit target is met.

Isn't the code I posted missing something? When I put that exactly in a new blank script file it complies with a million errors.

Actually I guess this should be an EA and not a script???

Try this EA.........

Thanks for posting....this looks good.

can u modify my ea to close all entry pos when account equity/account balance=1.003 . figure can be adjust. email me at [email protected] .

Thank you for profit ea, can you explain how it works? I mean I have one EA on different currencies and I want when let's say when is +100pips/-100pips profit/lose to close orders and no more trades for that day.

extern int StartLockPips=200;
extern int MagicNumber=1803;
extern bool CloseAtPipsProfitReached=false;
extern int ScalpProfitPips=30;
extern bool AllowTradeClosure=true;

Thanks.

> HERE < is Solution for same Requirement.

Hope that helps