Controlling The # of Orders | Forex Factory

Options

Permalinks
Print Thread

Similar Threads

Controlling Emotions is Not the Goal of Trading Psychology 5 replies

Best Method of Controlling Multiple MT4 Accounts at Once? 28 replies

Controlling your emotions 30 replies

Controlling scale of chart window? 0 replies

Controlling the Time an EA Works - help needed 4 replies

Controlling The # of Orders

  • #1
  • First Post: Nov 28, 2014 4:18am Nov 28, 2014 4:18am
Hello Everyone,

Would someone be able to tell me how to keep an algorithm from opening unwanted orders? I am opening two orders at the same time, and only want to open another set of two orders after both orders have closed. If I remove the check for Ticket1 and Ticket2, then it continues to open a new set of two orders after only one order closes. If I include the Ticket1 and Ticket2 check below, then it will not open any new order at all. Please assist if you can.

int start()
{
int Ticket1 = 0, Ticket2 = 0;
bool IsTrade = False;
if((!IsTrade) && (Ticket1 == 0) && (Ticket2 == 0))
{
RefreshRates();
Ticket1=OrderSend(Symbol(),OP_SELL,Lot,Bid,3,0,0,"",MagicNumber,0,Blue);
Ticket2=OrderSend(Symbol(),OP_BUY,Lot,Ask,3,0,0,"",MagicNumber,0,Blue);
if (Ticket1 < 1 || Ticket2 < 1)
{
lastError = GetLastError();
Print("Error Sending the Market Order: ",ErrorMessage(lastError));
}
}
return(0);
}
  • #2
  • Nov 28, 2014 5:04am Nov 28, 2014 5:04am
Quoting Deut818
Hello Everyone, Would someone be able to tell me how to keep an algorithm from opening unwanted orders? I am opening two orders at the same time, and only want to open another set of two orders after both orders have closed. If I remove the check for Ticket1 and Ticket2, then it continues to open a new set of two orders after only one order closes. If I include the Ticket1 and Ticket2 check below, then it will not open any new order at all. Please assist if you can. int start() { int Ticket1 = 0, Ticket2 = 0; bool IsTrade = False; if((!IsTrade)...
As far as I can see, your " if " is redundant, it will always be true. So if you are using this code, it will try to open 2 orders every tick!
BUT
Inserted Code
Ticket2=OrderSend(Symbol(),OP_BUY,L ot,Ask,3,0,0,"",MagicNumber,0,Blue) ;
I doubt that it will compile with the space in "L ot"
Please Do Not PM Me With Coding Enquiries
  • #3
  • Edited 6:06am Nov 28, 2014 5:54am | Edited 6:06am
Quoting Deut818
I am opening two orders at the same time, and only want to open another set of two orders after both orders have closed.
1. PUT CODE ALWAYS INBETWEEN code TAGS

Now, if the two oders are the only two orders in the system at any given time, then the problem can be solved in a trivial way:

Inserted Code
int start()
{
  if(OrdersTotal() != 0) return;
  OrderSend(Symbol(), OP_SELL, Lot, Bid, 3, 0, 0, "", MagicNumber, 0, Blue);
  OrderSend(Symbol(), OP_BUY, Lot, Ask, 3, 0, 0, "", MagicNumber, 0, Blue);
}

Which in turn implies that in another case you simply can count the orders (via magic number), and if there are more than zero, then pass.
Return To Disorder
  • #4
  • Nov 28, 2014 6:20am Nov 28, 2014 6:20am
Quoting Gumrai
{quote} As far as I can see, your " if " is redundant, it will always be true. So if you are using this code, it will try to open 2 orders every tick! BUT Ticket2=OrderSend(Symbol(),OP_BUY,L ot,Ask,3,0,0,"",MagicNumber,0,Blue) ; I doubt that it will compile with the space in "L ot"
Gumrai, thank you for your time, and pointing out that the "if" statement is worthless to prevent new orders. The code does compile. I just made a typo error here.
  • #5
  • Last Post: Nov 28, 2014 6:22am Nov 28, 2014 6:22am
Quoting rockit
{quote} 1. PUT CODE ALWAYS INBETWEEN code TAGS Now, if the two oders are the only two orders in the system at any given time, then the problem can be solved in a trivial way: int start() { if(OrdersTotal() != 0) return; OrderSend(Symbol(), OP_SELL, Lot, Bid, 3, 0, 0, "", MagicNumber, 0, Blue); OrderSend(Symbol(), OP_BUY, Lot, Ask, 3, 0, 0, "", MagicNumber, 0, Blue); } Which in turn implies that in another case you simply can count the orders (via magic number), and if there are more than zero, then pass.
Rockit, thank you also for your time and showing me an easy way to prevent new trades. I think that will solve the issue.

Thanks again to both of you.
0 traders viewing now