¿cómo añadir el código del indicador en el código de expertadvisor? - página 4

Nuevo comentario
 

¿como se escribe abrir solo una direccion?

porque mi ea abre posiciones en ambas direcciones, compra y venta.

solo edito el codigo y agrego el else; ahi,

¿es correcto?

aquí está el código .

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

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

finalmente se hizo el problema de la variable,

ahora es sólo

( - definición de función inesperada

) - paréntesis derecho desequilibrado...

es l izquierda puso ( y ) en mi código??

Tienes esto. . . .

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

. . . donde está su cierre } ?

  
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); }

el error es

ExistPositions - expresión en el ámbito global no permitida

y } - paréntesis desequilibrados.

 

me pregunto por qué el error siempre cambia?

después de arreglar esto,

entonces el error allí.

después de arreglar allí,

error aquí.

 
albert_lim83:

el error es

ExistPositions - expresión en el ámbito global no permitida

y } - paréntesis desequilibrados.

Bien, tienes esta función declarada. . . He ajustado la sangría para facilitar la lectura de

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

este código . . esta declaración de función . . debe estar fuera de cualquier otra función, por lo que debe estar fuera de start, init, deinit, etc.

 
albert_lim83:

me pregunto por qué el error siempre cambia?

después de arreglar esto,

entonces el error allí.

después de arreglar allí,

error aquí.

Porque un error enmascara a otro... arregle uno y otro se hace visible para el compilador... por ejemplo, puede tener el número correcto de llaves pero pueden estar en los lugares equivocados... . . o puede haber omitido una llave de cierre y una llave abierta . . de modo que los números coinciden pero son incorrectos.
 
RaptorUK:

Bien, ya tienes esta función declarada. . . He ajustado la sangría para facilitar la lectura de

este código . . esta declaración de función . . debe estar fuera de cualquier otra función, por lo que debe estar fuera de start, init, deinit, etc.

sigue el mismo error...


ExistPositions - expresión en el ámbito global no permitida

y } - paréntesis desequilibrados.

¿dónde debo cambiar?

 
albert_lim83:


¿Dónde debo cambiar?

No lo sé... muéstrale el código completo.
 
RaptorUK:
No sé ... mostrar el código completo.

De acuerdo. Sin código completo, no podemos ayudar mucho, así que no esperes más.

Combed con 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); }
 

Tal vez esto debería escribirse así... 

 if(!ExistPositions())
     {
      if(diClose0 < diMA1 && (ZigZagLow)) // <<== ???
        {
         OpenBuy();
         return(0);
        }
        else // <<=== ???
        {
        if(diClose2 > diMA3 && (ZigZagHigh)) // <<=== ???
          {
          OpenSell();
          return(0);
          }
        }
     }
12345678910
Nuevo comentario