|
From a quick look, there's a serious issue with the way EOD time is defined.
Following lines are wrong:
EODMinute=StrToInteger(StringSubstr(EOD,0,2));
EODMinute=StrToInteger(StringSubstr(EODHour,2,2));
They should read:
EODHour =StrToInteger(StringSubstr(EOD,0,2));
EODMinute=StrToInteger(StringSubstr(EOD,2,2));
- EODHour is defined as an integer for hosting the 'hour' portion so it cannot be used as a string StringSubstr(EODHour,2,2) in which we have to search for the hour portion.
- The Minutes and Hour were swapped from the search string (minutes are at the 2nd part of the string (2,2) whereas hour is at the first part (0,2)
There is also another mistake in The Best where the defaults settings use a 13 pip lock on a 3 pip B/E. (Move.To.BreakEven.Lock.pips and Move.To.BreakEven.at.pips parameters).
Assuming we have a long entry at let say 1.9050, it won't be possible to lock 13 pips (meaning SL should move at 1.9063) when our trade reaches +3 (at 1.9053)
Yannis
Last edited by Yannis; 02-28-2008 at 09:19 AM.
|