AOS - problém s kódem

Zdravim 

pokousel sem se napsat si jednoducheho robota ( krizeni  Ema + filtry )

ale ani za boha se mi to ne a ne podarit :( porad mi to pise "\end_of_program - unbalanced left paranthesis" prochazel sem to x krat , a porad se mi zda, ze vsechny zavorky jsou sparovane a nemuzu taam zadnou chybu najit :( muze te mi prosim poradit kde je chyba ??  - vim, ze mezi klavesnici a zidli :)

extern double Lots = 0.01;

extern int EmaS = 5;

extern int CCI = 14;

extern int BB_Perioda = 12;  

extern int MACD_fast = 3;

extern int MACD_slow = 15;

extern int MACD_signal = 3;

extern int Stoch_K = 3;

extern int Stoch_D = 3;

extern int Stoch_Period = 8;

extern int Stoch_Citlivost = 20;

int start()

{

  int ticket, Sl, Tp;

  double EMA=iMA(NULL,0,EmaS,0,MODE_EMA,PRICE_CLOSE,1);

  double EMAP=iMA(NULL,0,EmaS,0,MODE_EMA,PRICE_CLOSE,2);

  double BBStred=iBands(NULL,0,BB_Perioda,2,0,PRICE_TYPICAL,MODE_MAIN,1);

  double BBStredP=iBands(NULL,0,BB_Perioda,2,0,PRICE_TYPICAL,MODE_MAIN,2);

  double BBHorni = iBands(NULL,0,BB_Perioda,2,0,PRICE_TYPICAL,MODE_UPPER,1);

  double BBSpodni = iBands(NULL,0,BB_Perioda,2,0,PRICE_TYPICAL,MODE_LOWER,1);

  double CCIH = iCCI(NULL,0,CCI,PRICE_TYPICAL,1);

  double MACDH = iMACD(NULL,0,MACD_fast,MACD_slow,MACD_signal,PRICE_TYPICAL,MODE_MAIN,1);

  double Stoch = iStochastic(NULL,0,Stoch_K,Stoch_D,Stoch_Period,MODE_SMA,0,MODE_MAIN,1);

if (EMAP<BBStredP  && EMA>BBStred && CCIH>0 && MACDH>0 && Stoch<Stoch_Citlivost )

   {

     Sl = MathRound((Ask - BBSpodni)*(MathPow(10,(Digits-1)));

     Tp = MathRound((BBHorni - Bid) *(MathPow(10,(Digits-1))*2));

    ticket=OrderSend(Symbol(),OP_BUY,Lots,Ask,3,Bid-Sl*Point,Ask+Tp*Point,"",0,0,Green);

    if(ticket>0)(OrderSelect(ticket,SELECT_BY_TICKET,MODE_TRADES)) Print("Vstup BUY : ",OrderOpenPrice())

       else Print("Chyba Buy : ",GetLastError());          

   return(0);

  }

 if (EMAP > BBStredP  && EMA < BBStred && CCIH < 0 && MACDH < 0 && Stoch < (100-Stoch_Citlivost) ) 

   {

    Sl = MathRound((BBHorni - Bid)*(MathPow(10,(Digits-1)));

    Tp = MathRound((Ask - BBSpodni) *(MathPow(10,(Digits-1))*2));

    ticket=OrderSend(Symbol(),OP_SELL,Lots,Bid,3,Ask+Sl*Point,Bid-Tp*Point,"-",0,0,Red);

    if(ticket>0)(OrderSelect(ticket,SELECT_BY_TICKET,MODE_TRADES)) Print("Vstup SELL : ",OrderOpenPrice()) else Print("Chyba Sell: ",GetLastError());

  }

}

taky sem se divil, ze tam neni na kolik mist zaokrouhlit, ale v helpu je u MathRound psano pouze 1 cislo s tim ze zjevne se to zaokrouhluje na cele cislo - alespon tak sem to z heplu pochopil

já nevím, že něktěří lidé mají tak neskutečný problém s pochopením psaného textu. V helpu od MathRound se jasně píše

"Returns value rounded to the nearest integer of the specified numeric value." takže mi tak nějak uniká, proč by to mělo mít víc parametrů. 

Mike spíš myslel, že ti chybí jedna pravá závorka v tom řádku, co ti vykopíroval... 

ano dekuji moc :) nasel sem to :) asi mi bude chvilku trvat se v tech zavorkach zorientovat uz to alespon funguje:) si spatne , ale da se to testovat :) 

Trosku sem to upravil a vysledek nevypada zrovna nejhur :) -teda az na fakt ze to udela jenom 25 obchodu za pul roku :(

tak se to jeste upravil uz to opravdu otevira obchody se zadanym riskem :)

takhle to vypada ted :) kdyby si to chtel nekdo testnout :)

a risk se zadava v celych cislech ( risk=5 -- risk 5% uctu na 1 obchod ) EMA_CCI

mozna nemas nacteny data :( skus si najet na 15 min. graf a proscrolovat ho nadoraz ( kdyz si zhustis cas tak to nezabere moc casu )

Ten volume limit exceeded neřešte, běžná chyba v datech, se kterou se toho v MT nedá moc udělat. Běžná a neškodná chyba, nepátral jsem po příčinách, ale pravděpodobně nesedí "volume" z Vámi testované TF a volume na barech, ze kterých se daný bar skládá. Na testování to efekt nemá.

Spíše důrazně doporučuji si pohlídat, zda máte poravdu konzistentní data, velmi často se stává, že když stáhnete data z history centra, tak máte bar 23.08.2010 v 15:00 a dalš bar je až (třeba) 14.10.2010 v 11:00. TOHLE dost výrazně ovlivní test, na to pozor.

Alfou i omegou jakéhokoli zpětného testu je mít v pořádku data. Nemáte-li, nic nenatestujete a o nějakých nedej bože optimalizacích nemůže být ani řeč.

Jinak, k tématu tohoto vlákna, ale už jste to tady pravděpodobně vyřešili, problém, že máte moc levých závorek jak těch pravých, takže volání funkce máte vlastně přes celý zbytkový program.

A ohledně MathRound : ano, to zaokrouhluje na celé číslo, pro přidávání nul nebo naopak zkracování desetinných míst slouží funkce s prototypem NormalizeDouble(double number, int precision) . Mohu doporučit tuto funkci pro "urovnání" parmetrů pokynu při použití funkce OrderSend() , kde precision nastavte proměnnou Digits, abyste nemusel zvlášť specifikovat JPY a non-JPY :)

Prihodil by niekto alert pri otvorení každej pozície (poprípade aj s poslaním sms) do tohoto celkom úspešného EA?

http://forexchat.eu/phpBB3/viewtopic.php?f=4&t=23  ja som v programovaní laik, ale mala by tam byť nejaká inštrukcia e-mail alert alebo tak nejako 

na forexchat musí být registrace chlape Předpokládám, že všichni z bývalého chatu ji dostali automaticky(třeba já), ale noví členové tam podle mě přístup nemají a musí si napsat adminovi.. 

jenom sem tam pripsal alert, tak by to nemelo kod nijak ovlidnit :) X_Metr

opet mam mensi problemek s robotem :(

v testru to udela spoustu peknych obchodu, ale v realu to neudela ani jeden :(

i kdyz podle zpetnyho testu by mel udelat nekolik obchodu muze mi nekdo prosim ukazat kde mam chybu :(

Robot

tady je EA a tu nastaveni na EUR/USD :)

kdyby se chtel nekdo mrknout :)

zdar potreboval bych pomoci zde kod EA

//---- Trades limits
extern double    TakeProfit=160;
extern double    TrailingStop=20;
extern double    StopLoss=70;
extern bool      UseStopLoss = false;

//---- EMAs paris
extern int ShortEma = 10;
extern int LongEma = 80;

//---- Crossing options
extern bool immediate_trade = true; //Open trades immediately or wait for cross.
extern bool reversal = true; //Use the originally reversal crossing method or not

//---- Money Management
extern double Lots = 1;
extern bool MM = true; //Use Money Management or not
extern bool AccountIsMicro = false; //Use Micro-Account or not
extern int Risk = 10; //10%

extern bool Show_Settings = true;

//---- Global varaibles
static int TimeFrame = 0;


//+------------------------------------------------------------------+
//| expert initialization function                                   |
//+------------------------------------------------------------------+
int init()
  {
//----
   if(Show_Settings) Print_Details();
   else Comment("");
//----
  
   return(0);
  }
//+------------------------------------------------------------------+
//| expert deinitialization function                                 |
//+------------------------------------------------------------------+
int deinit()
  {
//----
      TimeFrame=Period(); //Prevent counting the cross while the user changing the timeframe
//----
   return(0);
  }
 
bool isNewSumbol(string current_symbol)
  {
   //loop through all the opened order and compare the symbols
   int total  = OrdersTotal();
   for(int cnt = 0 ; cnt < total ; cnt++)
   {
      OrderSelect(cnt, SELECT_BY_POS, MODE_TRADES);
      string selected_symbol = OrderSymbol();
      if (current_symbol == selected_symbol)
      return (False);
    }
    return (True);
}

int Crossed (double line1 , double line2)
   {
      static int last_direction = 0;
      static int current_direction = 0;
     
      if(TimeFrame!=Period())
      {
         TimeFrame=Period();
         return (0);
      }
     
      if(line1>line2)current_direction = 1; //up
      if(line1<line2)current_direction = 2; //down
     
      if(immediate_trade==false)
      {
         if(last_direction == 0) //first use
         {
               last_direction = current_direction;
               return(0);
         }
      }

      if(current_direction != last_direction) //changed
      {
            last_direction = current_direction;
            return (last_direction);
      }
      else
      {
            return (0); //not changed
      }
   }

//--- Bassed on Alex idea! More ideas are coming
double LotSize()
{
     double lotMM = MathCeil(AccountFreeMargin() *  Risk / 1000) / 100;
     
      if(AccountIsMicro==false) //normal account
      {
         if (lotMM < 0.1) lotMM = Lots;
         if (lotMM > 1.0) lotMM = MathCeil(lotMM);
         if  (lotMM > 100) lotMM = 100;
      }
      else //micro account
      {
         if (lotMM < 0.01) lotMM = Lots;
         if (lotMM > 1.0) lotMM = MathCeil(lotMM);
         if  (lotMM > 100) lotMM = 100;
      }
     
      return (lotMM);
}

string BoolToStr ( bool value)
{
   if(value) return ("True");
   else return ("False");
}
void Print_Details()
{
   string sComment = "";
   string sp = "----------------------------------------\n";
   string NL = "\n";

   sComment = sp;
   sComment = sComment + "TakeProfit=" + DoubleToStr(TakeProfit,0) + " | ";
   sComment = sComment + "TrailingStop=" + DoubleToStr(TrailingStop,0) + " | ";
   sComment = sComment + "StopLoss=" + DoubleToStr(StopLoss,0) + " | ";
   sComment = sComment + "UseStopLoss=" + BoolToStr(UseStopLoss) + NL;
   sComment = sComment + sp;
   sComment = sComment + "immediate_trade=" + BoolToStr(immediate_trade) + " | ";
   sComment = sComment + "reversal=" + BoolToStr(reversal) + NL;
   sComment = sComment + sp;
   sComment = sComment + "Lots=" + DoubleToStr(Lots,0) + " | ";
   sComment = sComment + "MM=" + BoolToStr(MM) + " | ";
   sComment = sComment + "Risk=" + DoubleToStr(Risk,0) + "%" + NL;
   sComment = sComment + sp;
 
   Comment(sComment);
}

//+------------------------------------------------------------------+
//| expert start function                                            |
//+------------------------------------------------------------------+
int start()
  {
//----

   int cnt, ticket, total;
   double SEma, LEma;
  
   string comment = "";
   if(reversal==true) comment = "EMA_CROSS_Counter-Trend";
   if(reversal==false) comment = "EMA_CROSS_Trend-Following";

   if(Bars<100)
     {
      Print("bars less than 100");
      return(0); 
     }
   if(TakeProfit<10)
     {
      Print("TakeProfit less than 10");
      return(0);  // check TakeProfit
     }
    
   SEma = iMA(NULL,0,ShortEma,0,MODE_EMA,PRICE_CLOSE,0);
   LEma = iMA(NULL,0,LongEma,0,MODE_EMA,PRICE_CLOSE,0);
  
  
  
   static int isCrossed  = 0;
   isCrossed = Crossed (LEma,SEma);
  
   if(reversal==false)
   {
      if(isCrossed==1) isCrossed=2;
      if(isCrossed==2) isCrossed=1;
   }
  
   if(MM==true) Lots = LotSize(); //Adjust the lot size
 
  
   total  = OrdersTotal();

   if(total < 1 || isNewSumbol(Symbol()))
     {
       if(isCrossed == 1)
         {
           
            if(UseStopLoss)
               ticket=OrderSend(Symbol(),OP_BUY,Lots,Ask,3,Ask-StopLoss*Point,Ask+TakeProfit*Point,comment,MAGICMA,0,Green);
            else
               ticket=OrderSend(Symbol(),OP_BUY,Lots,Ask,3,0,Ask+TakeProfit*Point,comment,MAGICMA,0,Green);
           
            if(ticket>0)
              {
               if(OrderSelect(ticket,SELECT_BY_TICKET,MODE_TRADES)) Print("BUY order opened : ",OrderOpenPrice());
              }
            else Print("Error opening BUY order : ",GetLastError());
            return(0);
         }
         if(isCrossed == 2)
         {
            if(UseStopLoss)
               ticket=OrderSend(Symbol(),OP_SELL,Lots,Bid,3,Bid+StopLoss*Point,Bid-TakeProfit*Point,comment,MAGICMA,0,Red);
            else
               ticket=OrderSend(Symbol(),OP_SELL,Lots,Bid,3,0,Bid-TakeProfit*Point,comment,MAGICMA,0,Red);
           
            if(ticket>0)
              {
               if(OrderSelect(ticket,SELECT_BY_TICKET,MODE_TRADES)) Print("SELL order opened : ",OrderOpenPrice());
              }
            else Print("Error opening SELL order : ",GetLastError());
            return(0);
         }
         return(0);
     }
    
    
   for(cnt=0;cnt<total;cnt++)
     {
      OrderSelect(cnt, SELECT_BY_POS, MODE_TRADES);

      if(OrderType()<=OP_SELL && OrderSymbol()==Symbol())
        {
         if(OrderType()==OP_BUY)   // long position is opened
           {
            // check for trailing stop
            if(TrailingStop>0) 
              {                
               if(Bid-OrderOpenPrice()>Point*TrailingStop)
                 {
                  if(OrderStopLoss()<Bid-Point*TrailingStop)
                    {
                     OrderModify(OrderTicket(),OrderOpenPrice(),Bid-Point*TrailingStop,OrderTakeProfit(),0,Green);
                     return(0);
                    }
                 }
              }
           }
         else // go to short position
           {
            // check for trailing stop
            if(TrailingStop>0) 
              {                
               if((OrderOpenPrice()-Ask)>(Point*TrailingStop))
                 {
                  if((OrderStopLoss()>(Ask+Point*TrailingStop)) || (OrderStopLoss()==0))
                    {
                     OrderModify(OrderTicket(),OrderOpenPrice(),Ask+Point*TrailingStop,OrderTakeProfit(),0,Red);
                     return(0);
                    }
                 }
              }
           }
        }
     }
   return(0);
  }
//+------------------------------------------------------------------+

jde mi o to ze se mi ta ea zda dobra ,ale ma par chyb a na forum kde jsem ho stahnul je uplne mrtvo a ja zrovna programovat neumim tudiz vas zadam o pomoc

.jde o to ze pri nastaveny counterTrend pri falce nefunguje a otvyra pouze buy signaly coz je velka skoda

.dalsi vec je ze kdyz si nastavite sema1 lema13 tp51 trailing 117 sl 120 tak v urcityych bodech na grafu tj treba skok 100pips mozna vic to v testeru hodi 10 plusovej obchodu v jedny minute kdyz by tam vubec nemely byt nevim cim to ale strasne to znehodnocuje vysledek testu .

mohl by prosimvas nekdo z vas dobrych dusi na to kouknout dekuju