I don't trust this code...
if (deleted==false) Print ("Error: ", GetLastError());
when deleting of the selected trade fails
the only thing the code is doing
printing 'Error: ....'
It is not handling the error to delete the trade for cases like ERR_BROKER_BUSY or ERR_TRADE_CONTEXT_BUSY
the code is only trying once to delete the trade and will not give info what trade is failing to delete and there is nothing
when it fails to try again deleting that trade
It simply keep the pending trade that fails with trying once to delete
Also you could add inside the code
if(OrderType < 2)continue;
It looks to me you were only advertising your website adding this code to Code Base
I don't trust this code...
if (deleted==false) Print ("Error: ", GetLastError());
when deleting of the selected trade fails
the only thing the code is doing
printing 'Error: ....'
It is not handling the error to delete the trade for cases like ERR_BROKER_BUSY or ERR_TRADE_CONTEXT_BUSY
the code is only trying once to delete the trade and will not give info what trade is failing to delete and there is nothing
when it fails to try again deleting that trade
It simply keep the pending trade that fails with trying once to delete
Also you could add inside the code
if(OrderType < 2)continue;
It looks to me you were only advertising your website adding this code to Code Base
Thanks for your input, but I don't think it worth it to add error handling to this script. If some orders are not deleted, you can just drag the script to the chart again.
If somebody will ask for that, I'll do it.
Thanks for your input, but I don't think it worth it to add error handling to this script. If some orders are not deleted, you can just drag the script to the chart again.
If somebody will ask for that, I'll do it.
OK, then I'll ask ....
Please add error handling to this script
Because if you wanna advertise your website with its forex lessons and free tools then i think you should provide good tools
Just drag the script to the chart again when a pending trade is not deleted is a solution you don't expect
from someone giving forex lessons and free tools on the website he or she is advertising.
I think bad coding shouldn't belong into CodeBase.
Thanks for your input, but I don't think it worth it to add error handling to this script. If some orders are not deleted, you can just drag the script to the chart again.
If somebody will ask for that, I'll do it.
OK, then I'll ask ....
Please add error handling to this script
Because if you wanna advertise your website with its forex lessons and free tools then i think you should provide good tools
Just drag the script to the chart again when a pending trade is not deleted is a solution you don't expect
from someone giving forex lessons and free tools on the website he or she is advertising.
I think bad coding shouldn't belong into CodeBase.
You ask for it...? You obviously can do it yourself if you want and seems you have too much free time, so I'll not do it for you.
You really mind that much that short reference to my site?
396 downloads so far and nobody had any problem. Code passed proofreading as well. So please stop bothering me and mind your own business.
You ask for it...? You obviously can do it yourself if you want and seems you have too much free time, so I'll not do it for you.
You really mind that much that short reference to my site?
396 downloads so far and nobody had any problem. Code passed proofreading as well. So please stop bothering me and mind your own business.
Maybe you are not willing it to do for me but you could do it for all others that try to learn how to make a good written program
.
Stop bothering you ?? You are the one that post a Code to CodeBase where everyone can give comments to the post you did....
If that is bothering you then stop advertising your website with your Code that can be done better....
and yes i code myself and i can make such kind of code also better working then this code does
396 downloads or a million downloads will not say that your program is used and well tested.... Not everyone will comment his problem with codes downloaded at codebase
About passed proofreading as well ...
That is no proof your code does what you want us to believe that it has to do
....... CLOSE ALL PENDING that you want to have to close
There are a lot bad written codes inside CodeBase that also passed proofreading and some of them totally useless using to trade forex
So accept the fact to get comments to the Code you post at CodeBase....
I still believe you were only advertising your website adding this code to Code Base and everyone can give its comments
I did ...
You ask for it...? You obviously can do it yourself if you want and seems you have too much free time, so I'll not do it for you.
You really mind that much that short reference to my site?
396 downloads so far and nobody had any problem. Code passed proofreading as well. So please stop bothering me and mind your own business.
Maybe you are not willing it to do for me but you could do it for all others that try to learn how to make a good written program
.
Stop bothering you ?? You are the one that post a Code to CodeBase where everyone can give comments to the post you did....
If that is bothering you then stop advertising your website with your Code that can be done better....
and yes i code myself and i can make such kind of code also better working then this code does
396 downloads or a million downloads will not say that your program is used and well tested.... Not everyone will comment his problem with codes downloaded at codebase
About passed proofreading as well ...
That is no proof your code does what you want us to believe that it has to do
....... CLOSE ALL PENDING that you want to have to close
There are a lot bad written codes inside CodeBase that also passed proofreading and some of them totally useless using to trade forex
So accept the fact to get comments to the Code you post at CodeBase....
I still believe you were only advertising your website adding this code to Code Base and everyone can give its comments
I did ...
It's so simple, if you don't like my code, don't use it.
If you want to save the world, create a better one as an alternative. People will love it.
So far it seems you are more passionate about commenting then coding.
Advertising? Come on, what is your problem?
I don't know how to code this stuff, and I am not taking sides here.
But if there is a possibility that this script fails, I agree that it would be nice to have some sort of built-in error checking.
I am thinking of end of trading week, for example, where I might be in a situation where I forgot to close pending orders, and I can just launch a script like this to make sure everything is closed before my broker cuts off any new orders. a sort of last minute panic-button, if you will.
It sounds like it is not difficult to code, from your discussion, so I think it might be nice to have... if that doesn't add a lot of processing overhead to mt4.
thanks.
Thank you abundantly.
It helps me cleaning up my messy stop orders.
- Free trading apps
- Over 8,000 signals for copying
- Economic news for exploring financial markets
You agree to website policy and terms of use
Close Pending Orders.mq4:
Author: George Alexander