ФорумРазделы

Function "_IsTradeAllowed" is not referenced and will be removed from exp-file - и чего теперь делать - страница 2

Новый комментарий
dmmikl86:

спасибо. убрал _IsTradeAllowed, и заработало. спасибо еще раз

вы не поняли тогда логику автора.

Если будете использовать свою интерпретацию работы функции 

TradeIsBusy()

то будете иметь неожиданные результаты...

Эта функция и 

TradeIsNotBusy()

организуют работу семафоров(глобальных переменных).

А проверку возможности работы следует всё же осуществлять _IsTradeAllowed, которую вы убрали из инклюдника...

;)

//=================================================================
   int TradeAllow = _IsTradeAllowed();
   if(TradeAllow < 0) 
      { 
        return(-1); 
      }
   if(TradeAllow == 0)
   {
   RefreshRates();
   if(TradeIsBusy() < 0) 
      { 
      return(-1); 
      }
   // определение пересичения на бай
   if(Close[1]>jaw && Close[1]>teeth && Close[1]>lips && flag==-1)
      {
       flag=1;
       if(HA_open<HA_close)
         {
          OrderSend(Symbol(), OP_BUY, Lots, NormalizeDouble(Ask, Digits), 3, Low[1]-offset*Point, Ask+TP*Point, "a_BHA_vs_Alligatror buy", magic, 0, Blue);
         }
      }
   //----
   // определение пересичения на 
   if(Close[1]<jaw && Close[1]<teeth && Close[1]<lips && flag==1)
      {
       flag=-1;
       if(HA_open>HA_close)
         {
          OrderSend(Symbol(), OP_SELL, Lots, NormalizeDouble(Bid, Digits), 3, High[1]+(3+offset)*Point, Bid-TP*Point, "a_BHA_vs_Alligatror sell", magic, 0, Red);
         }
      }
   TradeIsNotBusy();
   }
   //=================================================================
вот так правильно будет?
dmmikl86:
вот так правильно будет?

с точки зрения первоначального вопроса и темы топика - да.

о остальном воздержусь...

;)

А не проще было бы спросить о правильности использования библиотеки в комментариях к статье, в которой она была описана? Я на эту тему наткнулся абсолютно случайно, мог просто не заметить...

Если используются TradeIsBusy() и TradeIsNotBusy(), _IsTradeAllowed() вообще не нужна. Помещены они в один файл просто для удобства, неиспользуемый вариант можно удалить.

12
Новый комментарий