AOS - problém s kódem

Tak je zveřejni a můžem to probrat

Já také obchoduji několik strategií, které mají dobré výsledky až poslední 3 roky. Nevidím v tom problém. Na druhou stranu robustní stratege by měla fungovat o několik let nazpět, nebo alespoň neprodělávat. Také záleží, jestli testuješ základní myšlenku strategie nebo již zahrnutou optimalizaci parametrů, které mohou fungovat dobře na určité fázi trhu.

mam EA ale funguje ve starší verzi a ted mi hlásí chybu 

if(D == 0 && h == 00 && m < 5)

  { ticket = OrderSend(Symbol(),OP_BUY,0.01,Ask,0,Ask-SL,Ask+PT,Text, Magic_number,0,Blue); }

'if' - expressions are not allowed on a global scope

děkuji za pomoc Beri

Pouzij globalni promennou. To se definuje na zacatku kodu, tam nahore jak jsou extern promenny. Ale bez toho extern. ;-) Proste mimo telo void OnTick()...

Dobrý den,

chtěl bych Vás poprosit zda by mi někdo poradil jak z indikátoru FPK-MACD udělat aos aby mi sám vstoupil do obchodu (nejlepší signály jsou když jsem mimo pc ) pokud indikátor přejde přes 0  a to i bez SL a TP (ty si pak zadávám ručně dle fibo)  na 1H - 4H.  

Indikator - http://leteckaposta.cz/225279714

Zkoušel jsem to programovat dle ,,kuchařky,, na internetu ale nenašel jsem kde se dá vložit indikator.

Děkuji

Pořád se mi to nechce nějak načítat. Asi budu muset zkoušet zatím něco lehčího, nebo myslíte si že je to někdo do by mi to mohl udělat, abych podle toho mohl zkoušet další?

Děkuji za odpověď

To je jednoduchý, zkus si ten kód rozepsat, aby byl pro tebe čitelnější...

double POCET_SELL()

{

   for(int poc = OrdersTotal()-1; poc>=0; poc--)

   {

      if (OrderSelect(poc, SELECT_BY_POS, MODE_TRADES)&& OrderType()==OP_SELL)

      {

         return(pocet_lotu+OrderLots());

      }

   }

}

Když se na to podíváš, tak kód vběhne do smyčky, kde spočítá první obchod a return smyčku ukončí dřív, než smyčka proběhne podruhé.

Pro inspiraci přikládám funkci, jak bych to asi řešil já. Vzhledem k názvu POCET_SELL předpokládám, že chceš sčítat všechny short příkazy a long příkazy pak zlvášť. Tohle teda sečte jen SELL, SELL STOP a SELL LIMIT příkazy.

int POCET_SELL()

{

   double soucetLotu = 0;

   for( int pos = 0; pos < OrdersTotal(); pos++ )

   {

      if( OrderSelect( pos, SELECT_BY_POS ) == false )

         continue;

      if( Symbol() != OrderSymbol() )

         continue;

      if( OrderType() == OP_BUY || OrderType() == OP_BUYSTOP || OrderType() == OP_BUYLIMIT )

         continue;

      soucetLotu += OrderLots();

   }

   return( soucetLotu );

}

double POCET_SELL()

{

   double soucetLotu = 0;

   for( int pos = 0; pos < OrdersTotal(); pos++ )

   {

      if( OrderSelect( pos, SELECT_BY_POS ) == false )

         continue;

      if( Symbol() != OrderSymbol() )

         continue;

      if( OrderType() == OP_SELL )   

         soucetLotu += OrderLots();

   }

   return( soucetLotu );

}

No to je bomba, funguje to, hurááá.. Díky, díky, díky moc.

Tak asi úplně nechápu, co vlastně chceš, aby to dělalo, ale řekl bych (a teď se prosím neuraž) že škrábeš, jako čuně . Ono je to čitě o zvyku, jak kdo píše, ale tvůj kód si musím přepsat do srozumitelného textu, abych se v něm vůbec vyznal . Neber to prosím jako urážku, jen jako doporučení, že se dá ještě něco zlepšít.

No a abych nebyl za hulváta, kterej jenom uráží ostatní členy fóra, tak tady přikládám ten kód přepsaný:

   if(1.32000 < Bid && OPEN_TRADE_WAIT( OP_SELLSTOP ) == false )      

      ticket_sell = OrderSend( Symbol(), OP_SELLSTOP, VYSE_LOTU, Bid - 150 * Point, 0, 0, 0, NULL, 0 );  

   if(1.32000 > Ask && OPEN_TRADE_WAIT( OP_BUYSTOP ) == false )      

      ticket_buy = OrderSend( Symbol(), OP_BUYSTOP, VYSE_LOTU, Ask + 150 * Point,0 ,0 ,0 , NULL, 0 );

   for( int tr = 0; tr < OrdersTotal(); tr++ )         

   {

      if( OrderSelect( tr, SELECT_BY_POS, MODE_TRADES ) && OrderType() == OP_BUYSTOP && ( Ask - ( 150 * 0.00001 ) < OrderOpenPrice() ) )

         OrderModify( OrderTicket(), OrderOpenPrice() - 1 * Point, OrderStopLoss(), OrderTakeProfit(), 0, CLR_NONE );

   }  

   for( int tra = 0; tra < OrdersTotal(); tra++ )         

   {

      if( OrderSelect( tra ,SELECT_BY_POS, MODE_TRADES) && OrderType() == OP_SELLSTOP && ( Bid + ( 150 * 0.00001 ) > OrderOpenPrice() ) )

         OrderModify( OrderTicket(), OrderOpenPrice() + 1 * Point, OrderStopLoss(), OrderTakeProfit() , 0, CLR_NONE );

   }

Nevím tedy co dělá OPEN_TRADE_WAIT(), ale předpokládám, že vrací TRUE, pokud je otevřený obchod BUYSTOP, NEBO SELLSTOP. Dál nechápu, proč máš smyčku pro BUYSTOP a SELLSTOP zvlášť. Je to sice pitomost, která nemá vliv na funkci živočicháře, ale můžeš si do kódu zavléct zbytečnou chybu. A co se týče té podmínky ve smyčce, tak to je nejspíš kámen úrazu, který tě trápí.

if( OrderSelect( tr, SELECT_BY_POS, MODE_TRADES ) && OrderType() == OP_BUYSTOP && ( Ask - ( 150 * 0.00001 ) < OrderOpenPrice() ) )

Nejspíš měl básník na mysli něco jako Ask < ( OrderOpenPrice() - 150 * 0.00001 ). Pominu-li zbytek podmínky, kde je nejspíš napsáno opravdu co jsi zamýšlel, tak tady máš napsáno "pokud cena - 0.00150 je menší než otevírací cena příkazu, modifikuj příkaz...", ale chtěl jsi napsat něco jako "pokud je cena o 0.00150 menší, než otevírací cena, tak modifikuj příkaz..." a to je podstatný rozdíl, protože tvoje podmínka je platná i když je cena o 0.00149 výš, než otevírací cena. Takže asi bych to zkusil nějak takto:

   if(1.32000 < Bid && OPEN_TRADE_WAIT( OP_SELLSTOP ) == false )      

      ticket_sell = OrderSend( Symbol(), OP_SELLSTOP, VYSE_LOTU, Bid - 150 * Point, 0, 0, 0, NULL, 0 );  

   if(1.32000 > Ask && OPEN_TRADE_WAIT( OP_BUYSTOP ) == false )      

      ticket_buy = OrderSend( Symbol(), OP_BUYSTOP, VYSE_LOTU, Ask + 150 * Point,0 ,0 ,0 , NULL, 0 );

   for( int tr = 0; tr < OrdersTotal(); tr++ )         

   {

      if( OrderSelect( tr, SELECT_BY_POS, MODE_TRADES ) && OrderType() == OP_BUYSTOP && ( Ask < ( OrderOpenPrice() - ( 150 * 0.00001 ) ) ) )

         OrderModify( OrderTicket(), OrderOpenPrice() - 1 * Point, OrderStopLoss(), OrderTakeProfit(), 0, CLR_NONE );

      if( OrderSelect( tr ,SELECT_BY_POS, MODE_TRADES) && OrderType() == OP_SELLSTOP && ( Bid > ( OrderOpenPrice() + ( 150 * 0.00001 ) ) ) )

         OrderModify( OrderTicket(), OrderOpenPrice() + 1 * Point, OrderStopLoss(), OrderTakeProfit() , 0, CLR_NONE );

   }

No, tak snad jsem tě tedy dobře pochopil a dokonce ti i poradil a ne jen zkritizoval .

Zdravím. Chtěl bych pro AOS udělat SL z indikátoru ZigZag. Nedokážu vypotit kód, který by úspěšně vyvolal poslední hodnotu ZigZagu. (Myslím hodnotu na posledním swingu).

Aktuálně se překresluje. Mně jde o swing hodnotu, která už je vytvořená.

int start() { 

	   double upZZ = 0, dwZZ = 0;

	   

	   for(int i=0; i<Bars; i++){

	      upZZ=iCustom(Symbol(), Period(), "ZigZag", 12, 5, 3, 1, i);

	      if (upZZ != 0) break;

	   }

	   for(int j=0; j<Bars; j++){

	      dwZZ=iCustom(Symbol(), Period(), "ZigZag", 12, 5, 3, 2, j);

	      if (dwZZ != 0) break;

	   }

	   Alert("Up ZZ = " + DoubleToStr(upZZ, Digits) + "  Down ZZ = " + DoubleToStr(dwZZ, Digits));   

	   return(0);

	}

Díky. Pomohl jsi mi. 

1)máš zapnuté strategie? (tj. smajlík vpravo)

2) pokud nejde start() máš chybu v Init sekci popř. deklaraci externích proměnných

krokuj kod a urči, jaký řádek dělá problém

3) podívej se případně do log souboru

double ask = SymbolInfoDouble( Symbol(), SYMBOL_ASK );

Každopádně pokud se nemůžeš spolehnout na otevřený graf, tak nesmíš použít ani Symbol() , ale vypsat název symbolu třeba "EURUSD", nebo "GBPUSD".

Dobry den, mam takovy problem chtel bych aby se po uzavreni vsech pozic a pokynu tato AOS opakovala. Problem je v tom ze se me porad nastavuji dalsi a dalsi pokyny. Timto bych se vas chtel zeptat jestli nemate nekdo s tim zkusenost kde delam chybu :( . Sice se to snazim naucit ale bez znalosti jazyka a i toho Aj to jde dost tezko dekuji za pomoc





int start()

{
int ticket,expiration;
double point;
//--
point=MarketInfo(Symbol(),MODE_POINT);
expiration=CurTime()+PERIOD_D1*60;
//--

while(true)
{


ticket=OrderSend(Symbol(),OP_SELL,0.10,Bid-0*point,0,Bid+20000*point,Bid-20000*point,"BUY 1K H4",16384,expiration,Black);




ticket=OrderSend(Symbol(),OP_BUY,0.10,Bid+0*point,0,Bid-20000*point,Bid+20000*point,"BUY 1K H4",16384,expiration,Black);
ticket=OrderSend(Symbol(),OP_BUYSTOP,0.10,Bid+100*point,0,Bid-20000*point,Bid+20000*point,"BUY 1K H4",16384,expiration,Black);
ticket=OrderSend(Symbol(),OP_BUYSTOP,0.10,Bid+200*point,0,Bid-20000*point,Bid+20000*point,"BUY 1K H4",16384,expiration,Black);
ticket=OrderSend(Symbol(),OP_BUYSTOP,0.10,Bid+300*point,0,Bid-20000*point,Bid+20000*point,"BUY 1K H4",16384,expiration,Black);
ticket=OrderSend(Symbol(),OP_BUYSTOP,0.10,Bid+400*point,0,Bid-20000*point,Bid+20000*point,"BUY 1K H4",16384,expiration,Black);
ticket=OrderSend(Symbol(),OP_BUYSTOP,0.10,Bid+500*point,0,Bid-20000*point,Bid+20000*point,"BUY 1K H4",16384,expiration,Black);




/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////




ticket=OrderSend(Symbol(),OP_SELLSTOP,0.10,Bid-100*point,0,Bid+20000*point,Bid-20000*point,"BUY 1K H4",16384,expiration,Black);
ticket=OrderSend(Symbol(),OP_SELLSTOP,0.10,Bid-200*point,0,Bid+20000*point,Bid-20000*point,"BUY 1K H4",16384,expiration,Black);
ticket=OrderSend(Symbol(),OP_SELLSTOP,0.10,Bid-300*point,0,Bid+20000*point,Bid-20000*point,"BUY 1K H4",16384,expiration,Black);
ticket=OrderSend(Symbol(),OP_SELLSTOP,0.10,Bid-400*point,0,Bid+20000*point,Bid-20000*point,"BUY 1K H4",16384,expiration,Black);
ticket=OrderSend(Symbol(),OP_SELLSTOP,0.10,Bid-500*point,0,Bid+20000*point,Bid-20000*point,"BUY 1K H4",16384,expiration,Black);


if(ticket<=0) Print("Error = ",GetLastError());
else { Print("ticket =1",ticket); break; }
//-- 10 seconds wait
Sleep(10000);
}
////////////////////////////////////////////////////////////////////////////////////////////////////////


for(q=0;q<OrdersTotal();q++)
{if(OrderSelect(q, SELECT_BY_POS, MODE_TRADES) && OrderMagicNumber()==16384)
{return(0);}}




/////////////////////////////////////////////////////////////////////////////////////////////////////////////

//--
return(0);
}