I have an EA that works pretty good for me. The only problem is that it only makes one trade at a time and wont open another until the last trade is closed. What would I need to change in the code to let it make multiple trades?
Joined Jun 2008
|
Status: Programming for a better future.
|652 Posts
I think you'll get better response if you share the EA that worked well for you.
anyway, what you can do is to NOT have your EA discard signals while a trade is running. BUT you do have to discard signals generated within the same candle after the initial position is opened. I know you're concerned with it opening multiple positions for every tick within the same candle. All you have to do is... before opening a position, check if one is already opened, then compare the time the trade is opened with the time of the candle, if the time of the trade opening is later than the opening of the candle, discard the signal, else, take it!
on a side note, you should also do the same check on the trade history. this will prevent you opening multiple positions if the trade close out before that candle close.
Joined Jun 2008
|
Status: Programming for a better future.
|652 Posts
oh.. in case you're confused with the candle time, you only get the Opening time of the candle when you use the Time series. so... e.g. if at the current candle, a position is already opened,
Inserted Code
if (Time[0] <= OrderOpenTime()) // don't open a new position if we're still on the same candle
{
return (false);
}
I think you'll get better response if you share the EA that worked well for you.
anyway, what you can do is to NOT have your EA discard signals while a trade is running. BUT you do have to discard signals generated within the same candle after the initial position is opened. I know you're concerned with it opening multiple positions for every tick within the same candle. All you have to do is... before opening a position, check if one is already opened, then compare the time the trade is opened with the time of the candle, if the time of the trade opening is later than the opening of the candle, discard the signal, else, take it!
on a side note, you should also do the same check on the trade history. this will prevent you opening multiple positions if the trade close out before that candle close.
Joined Jun 2008
|
Status: Programming for a better future.
|652 Posts
Here you go. I added in an additional boolean function to do the checking and that substitutes your "IsTrade" variable. test it on demo first just in case.
p.s., just a suggestion, try not to pack all your codes into start() function, break them up into smaller functions instead. its neater and easier to read for old men like me. =)
That looks great Zen! Would it be possible to add a feature "inputs" menu to where you can set the max number of simultaneous trades? Like if I only wanted it to trade a max of 3 trades at any one time?
That looks great Zen! Would it be possible to add a feature "inputs" menu to where you can set the max number of simultaneous trades? Like if I only wanted it to trade a max of 3 trades at any one time?
Thanks!
Ignored
can I get back to you tomorrow? Its 1:37am and I really to sleep...
