gdzie jest błąd

NiceFox

nie kompiluje się może ktoś znajdzie błąd

#property copyright "Copyright © 2010, MetaQuotes Software Corp."
#property link      "http://www.metaquotes.net"

//+------------------------------------------------------------------+
//| expert initialization function                                   |
//+------------------------------------------------------------------+

//+------------------------------------------------------------------+
//| expert deinitialization function                                 |
//+------------------------------------------------------------------+

int start()
 
int nieMaOtwartych();
{
int pos=0;
for (int i=0; i< OrdersTotal();i++)
{
if OrderSelect(i,SELECT_BY_POS,MODE_TRADES)==false) break;
if (OrderSymbol()==Symbol())
   if(OrderType()==OP_BUY) pos++;
   }
   
   return (pos);
   }

gieroj · Nieprzeczytany post autor: **gieroj** » 07 gru 2010, 16:46

Nie sprawdzałem ale obstawiam

Kod: Zaznacz cały

 int nieMaOtwartych();

usuń ;

green7 · Nieprzeczytany post autor: **green7** » 07 gru 2010, 16:51

NiceFox pisze:if OrderSelect(i,SELECT_BY_POS,MODE_TRADES)==false) break;

i dodaj ( po if ...

LowcaG · Nieprzeczytany post autor: **LowcaG** » 07 gru 2010, 22:16

kod jest w ogole bez sensu.
1) Funkcja start jest zle zdefiniowana - brak otwarcia i zamkniecia nawiasow klamrowych.
2) pesudo definicja funkcji int nieMaOtwartych(); (nie powinno byc zadnego srednika)

3) Jak juz zostalo wspomniane, brak nawiasu otwierajacego w ifie

4) Z racji, ze nie ma ciala funkcji start , nie ma tez co sie wykonywac
(deifnicje funkcji maja to do siebie, ze aby je uzyc trzeba je wywolac)

NiceFox

nie mogę znaleźć błędu, gdzieś w starcie, pomóżcie')' - wrong parameters count C:\Program Files\XTB-Trader 4 PLN\experts\Piramida5.mq4 (52, 73)
.

Kod: Zaznacz cały

int countOrdersByRef(int oMagic, int& all, int& buy, int& sell,
                                           int& buystop, int& sellstop,
                                           int& buylimit, int& selllimit) {

   all=0;
   buy=0;
   sell=0;
   buystop=0;
   sellstop=0;
   buylimit=0;
   selllimit=0;

   for(int i=0;i<OrdersTotal();i++) {
    if(OrderSelect(i,SELECT_BY_POS)) {
     if(OrderMagicNumber()==oMagic) {
      if(OrderSymbol()==Symbol()) {
       all++;
       if(OrderType()==OP_BUY) buy++;
       if(OrderType()==OP_SELL) sell++;
       if(OrderType()==OP_BUYSTOP) buystop++;
       if(OrderType()==OP_SELLSTOP) sellstop++;
       if(OrderType()==OP_BUYLIMIT) buylimit++;
       if(OrderType()==OP_SELLLIMIT) selllimit++;
      }
     }
    }
   }

}
int start() 
{

int cAll;
int cBuy;
int cSell;
int cBuyStop;
int cSellStop;
int cBuyLimit;
int cSellLimit;

countOrdersByRef(cAll,cBuy,cSell,cBuyStop,cSellStop,cBuyLimit,cSellLimit);

// dalej odwołujemy się do zadeklarowanych zmiennych
if(cBuy>0) 
            Alert(1); 

}

gieroj · Nieprzeczytany post autor: **gieroj** » 08 gru 2010, 14:08

Funkcja którą napisałeś nic nie zwraca liczy sobie ile jest pozycji ale nie zwraca nam żadnego wyniku.

U mnie funkcja zliczająca ile jest otwartych pozycji danego typu wygląda tak :

Kod: Zaznacz cały

int checkorders(int typ)//sprawdza ile jest otwartych pozycji danego typu 0-buy 1-sell 2-buylimit ...
{
int trades=0;

   for (int i=OrdersTotal()-1; i>=0; i--)
   {                                               
      if (OrderSelect(i,SELECT_BY_POS,MODE_TRADES)==true)
      {
         if (OrderSymbol()==Symbol()&& OrderMagicNumber() == magic){
           switch(typ){
            case 0: { if (OrderType()==OP_BUY)trades++; break;}
            case 1: { if (OrderType()==OP_SELL)trades++; break;}
            case 2: { if (OrderType()==OP_BUYLIMIT)trades++; break;}
            case 3: { if (OrderType()==OP_SELLLIMIT)trades++; break;}
            case 4: { if (OrderType()==OP_BUYSTOP)trades++; break;}
            case 5: { if (OrderType()==OP_SELLSTOP)trades++; break;}
           } 
         }  
      }   
   }
 return(trades);  
}

Teraz w funkcji start dajesz odwołanie:

Kod: Zaznacz cały

int start()
{
if (checkorders(0)>0) 
  Print("Masz otwarta pozycje BUY ");

return(0);
}

magic zadeklaruj sobie na początku EA jako zmienna globalną jest wygodniej

Kod: Zaznacz cały

extern int magic=7766;

LowcaG · Nieprzeczytany post autor: **LowcaG** » 08 gru 2010, 14:13

gieroj pisze:Funkcja którą napisałeś nic nie zwraca liczy sobie ile jest pozycji ale nie zwraca nam żadnego wyniku.

Zwraca zwraca...

Blad jest w tym, że nie podales jako parametr oMagic, masz 7 parametrow a powinno byc 8

NiceFox

czy można zapisać dla megica i bez megica w jednym module?

Kod: Zaznacz cały

//============================= spr 1====================================
int checkorders(int typ)//sprawdza ile jest otwartych pozycji danego typu 0-buy 1-sell 2-buylimit ...
{
int trades=0;

   for (int i=OrdersTotal()-1; i>=0; i--)
   {                                               
      if (OrderSelect(i,SELECT_BY_POS,MODE_TRADES)==true)
      {
        if (OrderSymbol()==Symbol()))

         {
           switch(typ){
            case 0: { if (OrderType()==OP_BUY)trades++; break;}
            case 1: { if (OrderType()==OP_SELL)trades++; break;}
            case 2: { if (OrderType()==OP_BUYLIMIT)trades++; break;}
            case 3: { if (OrderType()==OP_SELLLIMIT)trades++; break;}
            case 4: { if (OrderType()==OP_BUYSTOP)trades++; break;}
            case 5: { if (OrderType()==OP_SELLSTOP)trades++; break;}
           }
           
         } 
      }   
   }
 return(trades); 
}
//============================= spr 2 ====================================
int checkorders_1(int typ)//sprawdza ile jest otwartych pozycji danego typu 0-buy 1-sell 2-buylimit ...
{
int trades_1=0;

   for (int j=OrdersTotal()-1; j>=0; j--)
   {                                               
      if (OrderSelect(j,SELECT_BY_POS,MODE_TRADES)==true)
      {
       
 if (OrderSymbol()==Symbol())&& OrderMagicNumber() == magic)
         {
           switch(typ){
            case 6: { if (OrderType()==OP_BUY)trades_1++; break;}
            case 7: { if (OrderType()==OP_SELL)trades_1++; break;}
            case 8: { if (OrderType()==OP_BUYLIMIT)trades_1++; break;}
            case 9: { if (OrderType()==OP_SELLLIMIT)trades_1++; break;}
            case 10: { if (OrderType()==OP_BUYSTOP)trades_1++; break;}
            case 11: { if (OrderType()==OP_SELLSTOP)trades_1++; break;}
           }
           
         } 
      }   
   }
 return(trades_1); 
}

green7 · Nieprzeczytany post autor: **green7** » 31 gru 2010, 13:35

Ło rany .... Czy obie te funkcje pisała ta sama osoba ? Bo jak tak to zupełnie nie kumam podejścia. ....
Tak czy siak odpowiadając na pytanie: da się. Dodaj do parametrów funkcji parametr magic, i jeśli równy będzie on przykładowo -1 to nie sprawdzasz go po orderselect. Wtedy możesz wywołując funkcje z odpowiednimi parametrami zliczac ordery dla wybranego magicno, lub wszystkie.
Pozatym po cholerę tak kombinujesz z parametrem typ ? Przecież to pozbawione zupełnie sensu. Cały ten switch możesz wyrzucić i zastąpić linią: if (OrderType() == typ) trades++
zastanów się co wtedy będziesz podawał przy wywołaniu jako parametr typ : czyż nie zrobi się wtedy prosto jasno i przejrzyście?

NiceFox

Kod jest gieroja nie mój. Nie jestem pewien czy coś takiego miałeś na myśli?

Kod: Zaznacz cały

int checkorders(int typ)//sprawdza ile jest otwartych pozycji danego typu 0-buy 1-sell 2-buylimit ...
{
int trades=0;

   for (int i=OrdersTotal()-1; i>=0; i--)
   {                                               
      if (OrderSelect(i,SELECT_BY_POS,MODE_TRADES)==true)
      {
        if (OrderSymbol()==Symbol()&& OrderMagicNumber() == magic_2)

         {
           switch(typ){
            case 0: { if (OrderType()==OP_BUY)trades++; break;}
            case 1: { if (OrderType()==OP_SELL)trades++; break;}
            case 2: { if (OrderType()==OP_BUYLIMIT)trades++; break;}
            case 3: { if (OrderType()==OP_SELLLIMIT)trades++; break;}
            
           }
           if (OrderSymbol()==Symbol()&& OrderMagicNumber() == magic_1)
           {
           switch(typ){
            case 4: { if (OrderType()==OP_BUY)trades++; break;}
            case 5: { if (OrderType()==OP_SELL)trades++; break;}
            case 6: { if (OrderType()==OP_BUYLIMIT)trades++; break;}
            case 7: { if (OrderType()==OP_SELLLIMIT)trades++; break;} 
           
           }
           
           }
           
           
         } 
      }   
   }
 return(trades); 
}