A bad practice is comparing boolean to boolean to get boolean. Now you can see it is also dangerous, when you misplace the parenthesis.
if(OrderDelete(OrderTicket()==false)) if(OrderDelete( 123456 ==false)) if(OrderDelete( 123456 == 0 )) if(OrderDelete( false )) if(OrderDelete( 0 )) // there is no order number zero
This is why I say: You would never write if( (2+2 == 4) == true) would you? if(2+2 == 4) is sufficient. So Don't write if(bool == true), just use if(bool) or if(!bool). Code becomes self documenting when you use meaningful variable names, like bool isLongEnabled. Long_Entry sounds like a trigger price or a ticket number and "if long entry" is an incomplete sentence.
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
Hi there,
I have this code:
There are several pending orders on the chart.
When I run the script I get the below the pending orders don't delete.
Thanks in advance.