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

как добавить код индикатора в код expertadvisor? - страница 4

Новый комментарий
[Удален]

как написать открывать только в одну сторону?

потому что моя ea открывает позиции в обоих направлениях, на покупку и на продажу.

Я просто редактирую код и добавляю else;,

правильно ли это?

вот код.

      if ((diClose0<diMA1))&&(ZigZagLow){
         OpenBuy();
         return(0);
      }
      else;

      if ((diClose2>diMA3))&&(ZigZagHigh){
         OpenSell();
         return(0);
      }
albert_lim83:

наконец-то решил проблему с переменными,

теперь только

( - неожиданное определение функции

) - несбалансированная правая скобка...

это левая скобка ( и ) в моем коде?

У вас есть это...

double zag, zig; b=0; while(a<2) {

... где ваша закрывающая } ?

[Удален] 
if (!ExistPositions()){

      if ((diClose0<diMA1))&&(ZigZagLow){
         OpenBuy();
         return(0);
      }
      else;

      if ((diClose2>diMA3))&&(ZigZagHigh){
         OpenSell();
         return(0);
      }
   }
   
   return (0);
}

bool ExistPositions() {
for (int i=100; i<OrdersTotal(); i++) {
if (OrderSelect(i, SELECT_BY_POS, MODE_TRADES)) {
if (OrderSymbol()==Symbol()) {
return(True);
}
} 
} 
return(false);
}
void OpenBuy() { 
   double ldLot, ldStop, ldTake; 
   string lsComm; 
   ldLot = GetSizeLot(); 
   ldStop = GetStopLossBuy(); 
   ldTake = GetTakeProfitBuy(); 
   lsComm = GetCommentForOrder(); 
   OrderSend(Symbol
(),OP_BUY,ldLot,Ask,Slippage,ldStop,ldTake,lsComm,0,0,clOpenBuy); 
 
} 
void OpenSell() { 
   double ldLot, ldStop, ldTake; 
   string lsComm; 
   ldLot = GetSizeLot(); 
   ldStop = GetStopLossSell(); 
   ldTake = GetTakeProfitSell(); 
   lsComm = GetCommentForOrder(); 
   OrderSend(Symbol
(),OP_SELL,ldLot,Bid,Slippage,ldStop,ldTake,lsComm,0,0,clOpenSell); 

} 
string GetCommentForOrder() { return(Name_Expert); } 

double GetSizeLot() { return(Lots); } 
double GetTakeProfitBuy() { return(Ask+lTakeProfit*Point); } 
double GetTakeProfitSell() { return(Bid-sTakeProfit*Point); }
double GetStopLossBuy() { return(Bid-lStopLoss*Point); }
double GetStopLossSell() { return(Ask+sStopLoss*Point); }

ошибка

ExistPositions - выражение в глобальной области видимости недопустимо

и } - несбалансированные круглые скобки.

[Удален]

просто интересно, почему ошибка всегда меняется?

после того, как я исправил это,

то ошибка там.

после исправления там,

ошибка здесь.

albert_lim83:

ошибка

ExistPositions - выражение в глобальной области видимости недопустимо

и } - несбалансированные круглые скобки.

Хорошо, у вас объявлена эта функция... Я изменил отступы, чтобы мне было легче читать ...

bool ExistPositions() 
   {
   for (int i=100; i<OrdersTotal(); i++) 
      {
      if (OrderSelect(i, SELECT_BY_POS, MODE_TRADES)) 
         {
         if (OrderSymbol()==Symbol()) 
            {
            return(True);
            }
         } 
      } 
   return(false);
   }

этот код . ... это объявление функции ... должно быть вне любой другой функции, поэтому оно должно быть вне start, init, deinit и т.д.

albert_lim83:

просто интересно, почему ошибка всегда меняется?

после того, как я исправил это,

то ошибка там.

после исправления там,

ошибка здесь.

Потому что одна ошибка маскирует другую... Исправьте одну, и другая станет видимой для компилятора... Например, у вас может быть правильное количество скобок, но они могут быть в неправильных местах. ... или вы пропустили закрывающую скобку и открывающую скобку... так что числа совпадают, но неверны.
[Удален]
RaptorUK:

Хорошо, у вас объявлена эта функция... Я изменил отступы, чтобы мне было легче читать ...

этот код . ... это объявление функции ... должно быть вне любой другой функции, поэтому оно должно быть вне start, init, deinit и т.д.

все та же ошибка...


ExistPositions - выражение в глобальной области видимости недопустимо

и } - несбалансированные круглые скобки.

Что нужно изменить?

albert_lim83:


Что нужно изменить?

Я не знаю... покажите полный код.
RaptorUK:
Я не знаю ... покажите полный код.

Согласитесь! Без полного кода мы не можем помочь, так что не ждите большего.

Комбинированный с MetaEditor 5.

   if(!ExistPositions())
     {

      if((diClose0<diMA1)) && (ZigZagLow)  // <<== ???
        {
         OpenBuy();
         return(0);
        }
      else;   // <<=== ???

      if((diClose2>diMA3)) && (ZigZagHigh) // <<=== ???
        {
         OpenSell();
         return(0);
        }
     }

   return(0);
  }
//+------------------------------------------------------------------+
//|                                                                  |
//+------------------------------------------------------------------+
bool ExistPositions() 
  {
   for(int i=100; i<OrdersTotal(); i++) // <<=== strange calculation 
     {
      if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES)) 
        {
         if(OrderSymbol()==Symbol()) 
           {
            return(True);
           }
        }
     }
   return(false);
  }
//+------------------------------------------------------------------+
//|                                                                  |
//+------------------------------------------------------------------+
void OpenBuy() 
  {
   double ldLot,ldStop,ldTake;
   string lsComm;
   ldLot=GetSizeLot();
   ldStop = GetStopLossBuy();
   ldTake = GetTakeProfitBuy();
   lsComm = GetCommentForOrder();
   OrderSend(Symbol
             (),OP_BUY,ldLot,Ask,Slippage,ldStop,ldTake,lsComm,0,0,clOpenBuy);

  }
//+------------------------------------------------------------------+
//|                                                                  |
//+------------------------------------------------------------------+
void OpenSell() 
  {
   double ldLot,ldStop,ldTake;
   string lsComm;
   ldLot=GetSizeLot();
   ldStop = GetStopLossSell();
   ldTake = GetTakeProfitSell();
   lsComm = GetCommentForOrder();
   OrderSend(Symbol
             (),OP_SELL,ldLot,Bid,Slippage,ldStop,ldTake,lsComm,0,0,clOpenSell);

  }
string GetCommentForOrder() { return(Name_Expert); }

double GetSizeLot() { return(Lots); }
double GetTakeProfitBuy() { return(Ask+lTakeProfit*Point); }
double GetTakeProfitSell() { return(Bid-sTakeProfit*Point); }
double GetStopLossBuy() { return(Bid-lStopLoss*Point); }
double GetStopLossSell() { return(Ask+sStopLoss*Point); }

Возможно, это должно быть написано так ??? 

 if(!ExistPositions())
     {
      if(diClose0 < diMA1 && (ZigZagLow)) // <<== ???
        {
         OpenBuy();
         return(0);
        }
        else // <<=== ???
        {
        if(diClose2 > diMA3 && (ZigZagHigh)) // <<=== ???
          {
          OpenSell();
          return(0);
          }
        }
     }
12345678910
Новый комментарий