about for()
Please use the </> button to insert your above code.
- When you post code please use the CODE button (Alt-S)!
(For large amounts of code, attach it.) Please edit your (original) post.
General rules and best pratices of the Forum. - General - MQL5 programming forum
Messages Editor for(int k=0;k<=OrdersHistoryTotal();k++)
If there are n entries their position is [0 .. n-1]. Drop the equal sign.if(OrderComment()=="onebuystop1" ||OrderComment()=="onesellstop1")
Not a good idea to use comments, brokers can change comments, including complete replacement.if(OrdersHistoryTotal()==0) //////if have not close order { TimeOpenFirstOrder=0; /////// TimeOpenFirstOrder=0 }
This code will never run. If the total is zero, the for loop exits immediately. Initialize your variable when declared.
You are missing trading opportunities:
- Free trading apps
- Over 8,000 signals for copying
- Economic news for exploring financial markets
Registration
Log in
You agree to website policy and terms of use
If you do not have an account, please register
how to fix
int TimeOpenFirstOrder;
for(int k=0;k<=OrdersHistoryTotal();k++)
{
if(OrderSelect(k,SELECT_BY_POS,MODE_HISTORY))
{
if(OrderSymbol()==Symbol() && OrderMagicNumber()==MagicNumber)
{
if(OrderType()==OP_BUY ||OrderType()==OP_SELL )
{
if(OrderComment()=="onebuystop1" ||OrderComment()=="onesellstop1")
{
TimeOpenFirstOrder=TimeDayOfYear(OrderOpenTime()); /////////if have Order ... TimeOpenFirstOrder is ....
}
}
if(OrdersHistoryTotal()==0) //////if have not close order
{
TimeOpenFirstOrder=0; /////// TimeOpenFirstOrder=0
}
Print("TimeOpenFirstOrder=",TimeOpenFirstOrder);
////want value "TimeOpenFirstOrder" to use down
for(int i=0;i<= (TimeDayOfYear( TimeCurrent() ) - TimeOpenFirstOrder );i++)
{
}
}