come aggiungere il codice dell'indicatore nel codice di expertadvisor? - pagina 4

Nuovo commento
 

come scrivere aperto solo 1 modo?

perché il mio ea apre entrambe le posizioni di direzione, acquisto e vendita.

Ho appena modificare il codice e aggiungere il else; su lì,

è corretto?

ecco il codice.

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

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

finalmente fatto il problema della variabile,

ora è solo

( - definizione di funzione inaspettata

) - parentesi destra sbilanciata...

è l sinistra mettere ( e ) sul mio codice?

Hai questo . . . .

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

. . . dove è la tua chiusura } ?

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

l'errore è

ExistPositions - espressione sull'ambito globale non consentita

e } - parentesi sbilanciate.

 

Mi chiedo solo perché l'errore cambia sempre?

dopo aver sistemato questo,

poi errore lì.

dopo risolto lì,

errore qui.

 
albert_lim83:

l'errore è

ExistPositions - espressione sull'ambito globale non consentita

e } - parentesi sbilanciate.

OK, hai questa funzione dichiarata. . . Ho regolato l'indentazione per rendere più facile per me leggere . . .

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

questo codice . . . questa dichiarazione di funzione . . . deve essere al di fuori di qualsiasi altra funzione, quindi deve essere al di fuori di start, init, deinit, ecc.

 
albert_lim83:

Mi chiedo solo perché l'errore cambia sempre?

dopo aver sistemato questo,

poi errore lì.

dopo risolto lì,

errore qui.

Perché un errore ne nasconde un altro. ...correggetene uno e un altro diventa visibile al compilatore. ...per esempio, potreste avere il numero corretto di parentesi graffe ma potrebbero essere nei posti sbagliati. . . o potreste aver dimenticato una parentesi graffa di chiusura e una aperta . . . quindi i numeri corrispondono ma sono sbagliati.
 
RaptorUK:

Ok, avete dichiarato questa funzione... Ho aggiustato l'indentazione per rendere più facile per me leggere . . .

questo codice . . . questa dichiarazione di funzione . . . deve essere al di fuori di qualsiasi altra funzione, quindi deve essere al di fuori di start, init, deinit, ecc.

ancora lo stesso errore...


ExistPositions - espressione sull'ambito globale non consentita

e } - parentesi sbilanciate.

Dove dovrei cambiare?

 
albert_lim83:


Dove dovrei cambiare?

Non lo so...vi mostro il codice completo.
 
RaptorUK:
Non so . . . mostrarvi il codice completo.

Sono d'accordo! Senza codice completo, non possiamo aiutare molto, quindi non aspettatevi di più.

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

Forse questo dovrebbe essere scritto in questo modo. 

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