Любой вопрос новичка, чтоб не захламлять форум. Профи, не проходите мимо. Без вас никуда - 6. - страница 661

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

Доброго времени суток! Подскажите пожалуйста как решить такой момент:

Использую в сове:

Сова работает на каждом тике, иначе основные условия не будут выполнятся. Использую всего 20 баров для iMAOnArray, но на тестере сова очень долго соображает. Задаю и вызываю это в блоке старт.

Как оптимизировать использование этой приблуды, так чтобы и основное условие проверялось на каждом тике и тестер не тормозил. А то у  меня тест совы на 4хчасовом графике за год проходит в течение 2х часов. Я боюсь представить сколько придется оптимизацию проводить)))


Без кода Вам никто помочь не сможет
 

Вот в принципе весь код:

//------- Внешние параметры советника -----------------------------------------+
extern string _P_Expert = "---------- Параметры советника";
extern int      Magic      = 777;                 // Магический номер позиций
extern double   SL=10;                            // Размер лося
extern double   TP=10000;                           // Размер профита 
extern string     ____= "Параметры блока MoneyManagement";
extern bool      MoneyManagement=false;
extern double    Lots          = 0.01;  
extern int       MarginPercent=3;
//------- Параметры трала -----------------------------------------------------+
extern string     ______= "Параметры блока TrallingStop";
extern bool      TralSimple=true;
//------- Характкристики баров ------------------------------------------------+

//------- Подключение внешних модулей -----------------------------------------+
#include <stdlib.mqh>
#include <stderror.mqh>

int start()
{

   total=OrdersTotal();
   Comment( LotsCounting() );
//+=================================================================================+
  // Проверка средств
   if(AccountFreeMargin()<(1000*Lots)){
      Print("We have no money. Free Margin = ", AccountFreeMargin());   
      return(0);  
   }
//===========================  Открытие позиций  ===================================+ 
//==========================================================================================+
//=============================   Center_array      ========================================+
      double Center_array[50];
      int    h,limit=ArraySize(Center_array);
      ArraySetAsSeries(Center_array,true);
      for(h=0; h<limit; h++)
         Center_array[h]=iRSI(NULL,0,14,0,h);
      double MA_1 =iMAOnArray(Center_array,0,14,0,MODE_SMA,1);
      double rs_1 = iRSI(NULL,0,14,0,1);
      
      if (Last_Max1!=Max1){
      if (Close[1]>Close[2] && rs_1<MA_1){
               lots=LotsCounting();
               OrderSend(Symbol(),OP_BUYSTOP,lots,(Max1),0,SL_buy,TP_buy, "Покупаем ",Magic,Expiration,Blue);
               Last_Max1=Max1;
               PlaySound("ok.wav");
               {
               Print ("При открытии произошла ошибка ", GetLastError());
               return;
               }
            }
         }
       }

//==========================================================================================+
      if (delet) DeleteOppositeOrders("",-1,Magic);
//==========================================================================================+
//=======================================TrallingStop=======================================+
      for ( int v = OrdersTotal() - 1; v >= 0; v -- ){       
         if (OrderSelect(v, SELECT_BY_POS, MODE_TRADES)){           
            if (OrderSymbol()==Symbol() && OrderMagicNumber()==Magic){ 
               //+=================================================================================+
               ///+================================Обычный трал ====================================+
               if (TralSimple){
                  //+=================================================================================+
                  if(OrderType() == OP_BUY){
                     if((Bid-OrderOpenPrice()) > (Point*TrailingStop)){
                        if(OrderStopLoss() < Bid-Point*TrailingStop || (OrderStopLoss()==0)){
                           OrderModify(OrderTicket(),OrderOpenPrice(),Bid-Point*TrailingStop,OrderTakeProfit(),0,Blue);  
                        }
                     }
                  }
               }                                   
            }  // Symbol()  
         } // select
      } //total 
        //============================== Конец блока закрытия ==============================+
 return(0);
}
//==========================================================================================+

Если убрать приблуду Center_errey, то тест проходит меньше чем за минуту. Если всеже включить в код это условие то тест проходит часа 2

 
berezhnuy:

Вот в принципе весь код:

Если убрать приблуду Center_errey, то тест проходит меньше чем за минуту. Если всеже включить в код это условие то тест проходит часа 2


А индикатор не пробовали сделать?
 

имеем переменную А, которая может принимать значения 0,1,2.

Будут ли равнозначны эти сравнения?

if(A=0 || A=1){do} и if(A != 2){do}

 
evillive:

имеем переменную А, которая может принимать значения 0,1,2.

Будут ли равнозначны эти сравнения?

if(A=0 || A=1){do} и if(A != 2){do}

Если переменная имеет тип перечисление (enum), то выражения идентичны.
 
evillive:

имеем переменную А, которая может принимать значения 0,1,2.

Будут ли равнозначны эти сравнения?

if(A=0 || A=1){do} и if(A != 2){do}

Нет, потому что ни "A=0", ни "A=1" сравнением не является.

Зато сравнением является "A == 0" и "A == 1".

 
simpleton:

Нет, потому что ни "A=0", ни "A=1" сравнением не является.

Зато сравнением является "A == 0" и "A == 1".

+1! :)
 
berezhnuy:

Вот в принципе весь код:

Если убрать приблуду Center_errey, то тест проходит меньше чем за минуту. Если всеже включить в код это условие то тест проходит часа 2

По условиям пересчеты и проверка актуальны не на каждом тике, а лишь при появлении нового бара.

Поэтому целесообразно применить соответствующую функцию, типа этой:

bool NevBar(){
   static int PrevTime=0;
   if (PrevTime==Time[0]) return(false);
   PrevTime=Time[0];
   return(true);}

и запускать Center_errey лишь в случае появления такового.

Еще можно всякий "мусор" повыкидывать. Но принципиально на скорости это не скажется.

Также у Вас в коде грубейшая ошибка - в случае отсутствия маржи Вы отменяете весь код. А как же трал?... Хотя... возможно, это из-за того, что это лишь часть кода, как я понимаю из комментариев.

 
simpleton:

Нет, потому что ни "A=0", ни "A=1" сравнением не является.

Зато сравнением является "A == 0" и "A == 1".


Зачот )))


Только я не на MQL писал, а формально, вот ведь закавыка (:

 
evillive:

Зачот )))


Только я не на MQL писал, а формально, вот ведь закавыка (:

Если формально, то нет, поскольку может быть ещё  NULL
1...654655656657658659660661662663664665666667668...1178
Новый комментарий
Причина обращения: