View Single Post
  #19 (permalink)  
Old 02-28-2008, 09:10 AM
Yannis Yannis is offline
Member
 
Join Date: Sep 2005
Location: Athens
Posts: 93
Yannis is on a distinguished road
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.
Reply With Quote