Jedna transakcja dziennie

[m_c]

Witam,

mam problem z kodem dotyczący jednej transakcji na dzień. Chodzi o transakcję w ciągu dnia a nie 24h. Zrobiłem funkcję, według wskazówek z forum mql4. Ale nie działa. Nie wiem czy jest to mniej więcej poprawnie zrobione lub nie powinienem zamienić na sekundy (86400). Dodatkowo mam trailing stop, w funkcji start(), ponieważ założenie było takie, że po sygnale jest wejście w rynek z trailing stopem. Po zrealizowaniu SL nie są już dokonywane żadne transakcje tego dnia. Macie może jakieś sugestie co może być źle ?

Kod: Zaznacz cały

bool TradePlacedToday()
    {
      for (int k=OrdersTotal()-1;k>=0;k--)
      {
         if (OrderSelect(k,SELECT_BY_POS,MODE_HISTORY))
         {      
            datetime opp=OrderCloseTime();
             if (opp>0) return(0);
         }
            else {Ticket=OrderTicket();}     
      }    
    }   

[m_c]

Funkcja jest wywoływana za pomocą bool TPT=TradePlacedToday();
Przyszło mi jeszcze coś takiego do głowy.
1. Może zrobić to tak żeby zamiast Ticket=OrderTicket() było od razu Ticket=OrderSend(....). Jednak wtedy musiałbym to połączyć z warunkami transakcyjnymi.

2. Funkcja jest typu bool czyli zwraca true lub false, ale wynik po else czyli Ticket, OrderTicket jest typu int. może tutaj jest błąd?

[David_Plavko]

Wersja 1 na dzień:
1. Zmienna globalna czyDzisByloZlecenie = false
2. Przy składaniu zlecenia czyDzisByloZlecenie = true
3. Po północy czyDzisByloZlecenie = false

Wersja 1 na 24 godziny
1. Zmienna globalna czyDzisByloZlecenie = false
1a. zmienna globalna czasZlecenia
2. Przy składaniu zlecenia czyDzisByloZlecenie = true
2a. czasZlecenia = czasAktualny
3. jeżeli czasAktualny > czasZlecenia + 24h, to czyDzisByloZlecenie = false

Jak widać nie trzeba kombinować z dodatkowymi funkcjami i pętlami...

[m_c]

I działa Ci to poprawnie?
Rozpatrzmy ver. 1.
pkt. 2 czyli jeśli: if(Ticket>0)czyDzisByloZlecenie=true; // no bo jeśli nie zostało złożone to co wtedy?
po co pkt. 3 jeśli zmienna globalna jest zawsze false ?

[David_Plavko]

Funkcja OrderSend(), zwraca wartość int, jest to OrderTicket() albo -1 gdy wysłanie się nie powiedzie, co za tym idzie nie musisz bawić się w zaznaczanie ostatniego zlecenia ;) Jeśli się nie powiedzie możesz sprawdzić dlaczego funkcją GetLastError();


A więc wracając do mojego pseudokodu:

Na początku parametr czyDzisByloZlecenie jest false (przy inicjowaniu EA).
Czas mija .... nadchodzi sygnał, więc zlecenie idzie w rynek - czyDzisByloZlecenie = true; (jeśli się nie powiodło wysyłanie to sprawdzasz dlaczego a czyDzisByloZlecenie nie zmienia wartości).
Czas mija, nadchodzi kolejny sygnał, sprawdzasz czyDzisByloZlecenie ? Jeśli było to nie wysyłasz zlecenia.
Czas mija, mija północ -> czyDzisByloZlecenie = false;

Specjalnie nie daje Ci gotowego kodu, abyś mógł się trochę przy okazji nauczyć różnych funkcji ;)

[m_c]

Dzięki. Mi z tym false i true coś nie bardzo szło:) Będę musiał spróbować jeszcze raz tak z ciekawości. Na chwilę obecną zrobiłem to trochę inaczej i działa (chyba :) ). Wykorzystałem dwie pętle, pierwsza sprawdza czy były otwarte, druga w historii sprawdza. Obie są w start() a nie w oddzielnej funkcji (miałem błąd przy wywołaniu, tak że będę musiał wymyślić coś co będzie zwracane). Z tego by wynikało że da się to zrobić przynajmniej na 3 rożne sposoby.