DislikedThat is expected behavior.
The EA is basically running on an infinite loop. Cycle count is the number of 100 ms intervals that have passed between the last tick and the present.Ignored
DislikedIf ReverseTrades is set to false, the EA tests all trades with a SL setting of whatever SL you set.
If ReverseTrades is set to true, the EA tests all trades with a TP setting of whatever TP you set.Ignored
if(!ReverseTrades) TakeProfit = CurrentHighTP;
if(ReverseTrades) StopLoss = CurrentHighTP;
Which leads me to believe that setting Reverse Trades = True does what one what would think it does, and simply reverses the trading logic rather then sets use of SL and TP levels, which struck me as odd in your description of the setting. Upon further examining the code I believe it would make more sense to create additional settings "Use Hard SL" and "Use Hard TP" and then change the current "SL and TP" descriptions to read "SL Range" and "TP Range"
Also your comment string needs to be changed from:
string CommentString = StringConcatenate("Broker Type: ", BrokerType, "n",
"Last Optimized: ", LastOptimize, "n",
"Bars Used: ", BarCount, "n",
"Total Bars: ", Bars, "n",
"Current Hour: ", CurrentHour, "n",
"Current TP: ", CurrentHightTP, "n",
TO:
string CommentString = StringConcatenate("Broker Type: ", BrokerType, "n",
"Last Optimized: ", LastOptimize, "n",
"Bars Used: ", BarCount, "n",
"Total Bars: ", Bars, "n",
"Current Hour: ", CurrentHour, "n",
"Current TP: ", TakeProfit, "n",
"Current SL: ", StopLoss, "n",
So that it displays both the SL and TP levels, and will correctly display what the actual TP and SL are