Metatrader Forum | Forex Expert-Advisor | Broker & Forex Tools
Zurück   Metatrader Forum | Forex Expert-Advisor | Broker & Forex Tools > Metatrader 4 > Programmierung MQL4

Programmierung MQL4 Hier gehts rund ums Programmieren in MQL4.


Portalsuche


Werbung

Login
Benutzername:
Kennwort:


Statistik
Themen: 3359
Beiträge: 33848
Benutzer: 5.342
Aktive Benutzer: 599
Links: 79
Wir begrüßen unseren neuesten Benutzer: maxi
Mit 483 Benutzern waren die meisten Benutzer gleichzeitig online (21.04.16 um 17:12).
Neue Benutzer:
vor 5 Stunden
- maxi
vor 20 Stunden
- athaclena
vor einem Tag
- Naale
vor einem Tag
- csabi911
vor einem Tag
- Adlerflüge...

Onlineuser
'Wer ist online' anzeigen Benutzer: 3
Gäste: 35
Gesamt: 38
Team: 0
Team:  
Benutzer:  fxnoob, Naale, SentiTrader
Freunde anzeigen

Empfehlungen

Werbung

Like Tree1Likes
  • 1 Post By Trading111
Antwort
 
Themen-Optionen Thema durchsuchen Ansicht
  #1 (permalink)  
Alt 20.10.16
Neues Mitglied
 
Registriert seit: Sep 2016
Beiträge: 11
Trading111 befindet sich auf einem aufstrebenden Ast
Standard invalid ticket for OrderModify...

Hallo, ich habe einen EA programmiert, der auch im Strategiefenster wunderbar funktioniert - so wie ich will......doch wenn ich ihn onlive live schalte will er nicht - da komme ich nicht weiter......

bei der OrderModify passt ihm die Ticketnummer scheinbar nicht....(OrderMod. wird bei mir im Trailingstop angewendet)........ich verstehe das nicht - mit OrderModify(LongOrder,.....) muss ich doch an den offenen Orders rumschrauben können (bzw. OrderModify(ShortOrder,...))

Oft kommt eben dann "invalid ticket for OrderModify function"...
(in mein Code ist das unwichtige klein und die angesprochenen Sachen farbig) - danke schon mal falls einer antwortet...


extern int MA_Periode_lang = 200;
extern int MA_Periode_kurz = 30;
extern double HandelsLots = 0.1;
extern int MagicNummer = 12345;
extern double TrailingStart = 1;
extern double TrailingStep = 1;


// Globale Variable
datetime PeriodenStartZeit;
bool NeuePeriodeBegonnen,LongSignal,ShortSignal;
int LongOrder, ShortOrder;

// für Traidingzeit...
extern bool MON = true;
extern bool TUE = true;
extern bool WED = true;
extern bool THU = true;
extern bool FRI = true;
extern int StartHour = 10;
extern int EndHour = 20;


int CurrentTime;
bool TradingDay[6]= { false, false, false, false, false, false };


int OnInit()
{

PeriodenStartZeit = Time[0];

if(MON) TradingDay[1]=true;
if(TUE) TradingDay[2]=true;
if(WED) TradingDay[3]=true;
if(THU) TradingDay[4]=true;
if(FRI) TradingDay[5]=true;

return(INIT_SUCCEEDED);
}

bool TradingTime()
{
CurrentTime = TimeHour(TimeCurrent());
if (StartHour < EndHour)
{
if ((CurrentTime < StartHour) || (CurrentTime >= EndHour)) return(false);
}
if (StartHour > EndHour)
{
if ((CurrentTime < StartHour) && (CurrentTime >= EndHour)) return(false);
}
return(true);
}


void TrailOrder(double TrailingStart,double TrailingStep) // Funktion für TrailingOrder
{
double Anz_Steps=100; // Anzahl Steps
if(OrderType()==OP_BUY && OrdersTotal()>0)
{

OrderSelect(LongOrder,SELECT_BY_TICKET); // Auswahl der offenen Order
double Kaufpreis = OrderOpenPrice();
for(int n=1;n<=Anz_Steps;n++)
{
if(n==1)
{
// erster SL des Programms...
if (OrderStopLoss()==0 || OrderStopLoss()==OrderOpenPrice()-2)
{
if(Bid>NormalizeDouble(OrderOpenPrice()+(TrailingS tart),Digits))
{
double SL_neu=Kaufpreis+TrailingStart;
SL_neu=NormalizeDouble(SL_neu*Point*10,Digits);
OrderModify(LongOrder,OrderOpenPrice(),SL_neu,OrderTakeProfit(),NULL,Bl ue);
}
}
}
if(n>1)
{
if (Bid>NormalizeDouble(OrderOpenPrice()+TrailingStar t+(TrailingStep*n),Digits))
{
if(OrderStopLoss()<(Kaufpreis+TrailingStart+(Trail ingStep*n)))
{
double SL_neu=Kaufpreis+TrailingStart+(TrailingStep*n);
SL_neu=NormalizeDouble(SL_neu*Point*10,Digits);
OrderModify(LongOrder,OrderOpenPrice(),SL_neu,OrderTakeProfit(),NULL,Bl ue);
}
}

}
}
}
if(OrderType()==OP_SELL && OrdersTotal()>0)
{
OrderSelect(ShortOrder,SELECT_BY_TICKET); // Auswahl der offenen Order
double Kaufpreis = OrderOpenPrice();
for (int n=1;n<=Anz_Steps;n++)
{
if(n==1)
{
// erster SL des Programms...
if (OrderStopLoss()==0 || OrderStopLoss()==OrderOpenPrice()+2)
{
if(Ask<NormalizeDouble(OrderOpenPrice()-TrailingStart,Digits))
{
double SL_neu=Kaufpreis-TrailingStart;
SL_neu=NormalizeDouble(SL_neu*Point*10,Digits);
OrderModify(ShortOrder,OrderOpenPrice(),SL_neu,OrderTakeProfit(),NULL,Bl ue);
}
}
}

if(n>1)
{

if (Ask<NormalizeDouble(OrderOpenPrice()-TrailingStart-(TrailingStep*n),Digits))
{
if(OrderStopLoss()>(Kaufpreis-TrailingStart-(TrailingStep*(n-1))))
{
double SL_neu=Kaufpreis-TrailingStart-(TrailingStep*(n-1));
SL_neu=NormalizeDouble(SL_neu*Point*10,Digits);
OrderModify(ShortOrder,OrderOpenPrice(),SL_neu,OrderTakeProfit(),NULL,Bl ue);
}
}
}
}
}
}

// expert function
void OnTick ()
{
if (TradingDay[DayOfWeek()] == false) return;
if (TradingTime() == false) return;

// Überprüfen ob neuer Periodenbeginn vorliegt
if(PeriodenStartZeit != Time[0])
{
NeuePeriodeBegonnen = true;
PeriodenStartZeit = Time[0];
}

else NeuePeriodeBegonnen = false;

// Marktdaten ermitteln
double MA_lang = iMA(NULL,0,MA_Periode_lang,0,1,0,1);
double MA_lang_Vorperiode = iMA(NULL,0,MA_Periode_lang,0,1,0,2);
double MA_kurz = iMA(NULL,0,MA_Periode_kurz,0,1,0,1);
double MA_kurz_Vorperiode = iMA(NULL,0,MA_Periode_kurz,0,1,0,2);

if (LongSignal == false && ShortSignal == false)
{
// Handelssignale ermitteln

if(NeuePeriodeBegonnen == true)
{
int anzahl_mind_kerzen;
int order_type;
int Nummer_Order = OrdersHistoryTotal();
if(OrderSelect(Nummer_Order,SELECT_BY_TICKET) == true)
{
order_type = OrderType();
}

// EA soll erst anlaufen um
anzahl_mind_kerzen = MA_Periode_lang;
if(iBars(NULL,0) > anzahl_mind_kerzen+1)
{
if (OrdersHistoryTotal() == 0)
{
// Buy Signal
if(MA_lang < MA_kurz && MA_lang_Vorperiode >= MA_kurz_Vorperiode)
{
LongSignal = true;
}
else LongSignal = false;
// Sell Signal
if(MA_lang > MA_kurz && MA_lang_Vorperiode <= MA_kurz_Vorperiode)
{
ShortSignal = true;
}
else ShortSignal =false;
}
if(OrdersHistoryTotal() > 0)
{
// Buy Signal
if(MA_lang < MA_kurz && MA_lang_Vorperiode >= MA_kurz_Vorperiode && OrdersTotal()==0) // && order_type == OP_SELL)
{
LongSignal = true;
}
else LongSignal = false;
}
if(OrdersHistoryTotal() > 0)
{
// Sell Signal
if(MA_lang > MA_kurz && MA_lang_Vorperiode <= MA_kurz_Vorperiode && OrdersTotal()==0) // && order_type == OP_BUY)
{
ShortSignal = true;
}
else ShortSignal =false;
}
}
}
else
{
LongSignal = false;
ShortSignal = false;
}
}

// LongSignale umsetzen
if (LongSignal == true)
{
while (LongOrder <= 0)
{
LongOrder = OrderSend(Symbol(),OP_BUY, HandelsLots,Ask,10,0,0,"MAXing Long",MagicNummer,0,Green);
}
}

if (ShortSignal == true)
{
while (ShortOrder <= 0)
{
ShortOrder = OrderSend(Symbol(),OP_SELL,HandelsLots,Bid,10,0,0, "MAXing Short",MagicNummer,0,Red);
}
}

// Trailingstop...
if (LongSignal == true || ShortSignal == true)
{
if(TrailingStart > 0) TrailOrder(TrailingStart,TrailingStep); // Funtion TrailOrder von oben
}

// Ticketnummern nach Closing zurücksetzen
if (OrderSelect(LongOrder,SELECT_BY_TICKET) == true)
{
if (OrderTicket()>0 && OrderCloseTime()>0)
{
LongOrder = 0;
LongSignal = false;
}
}

if (OrderSelect(ShortOrder,SELECT_BY_TICKET) == true)
{
if (OrderTicket()>0 && OrderCloseTime()>0)
{
ShortOrder = 0;
ShortSignal = false;
}
}
return;

}
Mit Zitat antworten
  #2 (permalink)  
Alt 20.10.16
Mitglied
 
Registriert seit: Nov 2015
Beiträge: 61
michaelf befindet sich auf einem aufstrebenden Ast
Standard

Arbeite doch bitte mal an Deinem Errorhandling und poste davon mal die Ergebnisse. Entweder in Kurzform:
Code:
if( !OrderModify(ticket, price,SL, TP, 0) ){
      Print( "OrderModify(ticket=", ticket, ", price=", DoubleToStr(price,Digits)," SL=", DoubleToStr(SL,Digits), "), TP=",   DoubleToStr(TP,Digits)," ...) failed: Error ", GetLastError(), ", bid=", DoubleToStr(Bid,Digits) );}
oder ausführlicher:
Code:
bool bresult=OrderModify(ticket, price,SL, TP, 0);
if( !bresult ){
      Print( "OrderModify(ticket=", ticket, ", price=", DoubleToStr(price,Digits)," SL=", DoubleToStr(SL,Digits), "), TP=",   DoubleToStr(TP,Digits)," ...) failed: Error ", GetLastError(), ", bid=", DoubleToStr(Bid,Digits) );}
Entsprechend mit allen Funktionen, die Du rund um die Orders verwendest, und immer auf den Datentyp achten, der zurück gegeben wird. Im gezeigten Beispiel ist es bool, bei manchen ist es int. ==> Siehe MQL4-Doku!
Ich weiß, Errorhandling ist lästig, aber es macht Sinn und spart letztlich Zeit bei der Fehlersuche.
Mit Zitat antworten
  #3 (permalink)  
Alt 20.10.16
Elite Mitglied
 
Registriert seit: Apr 2011
Beiträge: 1.518
traderdoc befindet sich auf einem aufstrebenden Ast
Standard

Ich würde entweder in der Funktion
void TrailOrder(double TrailingStart,double TrailingStep) // Funktion für TrailingOrder
mittels OrderSelect() die TicketNumber abfragen, sofern es nur 1 Buy und/oder 1 SellOrder pro Währungspaar und TimeFrame gibt (ansonsten mit MagicNumbers arbeiten).

Oder man übergibt der Funktion
TrailOrder(TrailingStart,TrailingStep, TicketNumber);
die TicketNumber, die ggf. trotzdem vorher nochmals sicher ausgelesen werden sollte.

traderdoc
__________________
Ich erfülle Euch gern Eure EA-, Indikator- und Script-Programmierungswünsche auf Honorarbasis.
Mit Zitat antworten
  #4 (permalink)  
Alt 28.10.16
Neues Mitglied
 
Registriert seit: Sep 2016
Beiträge: 11
Trading111 befindet sich auf einem aufstrebenden Ast
Standard

vielen Dank für die guten Tipps...
@traderdoc: ja das mit dem Hereinnehmen des Parameters Ticketnummer in die Funktion TrailOrder(...) war genau der richtige Ansatz....Danke....
michaelf likes this.
Mit Zitat antworten
Antwort

Lesezeichen

Stichworte
mql4, ordermodify, ordermodify ticketnummer, programmierung, programmierung metatrader, ticketnummer


Themen-Optionen Thema durchsuchen
Thema durchsuchen:

Erweiterte Suche
Ansicht

Forumregeln
Es ist Ihnen nicht erlaubt, neue Themen zu verfassen.
Es ist Ihnen nicht erlaubt, auf Beiträge zu antworten.
Es ist Ihnen nicht erlaubt, Anhänge hochzuladen.
Es ist Ihnen nicht erlaubt, Ihre Beiträge zu bearbeiten.

BB-Code ist an.
Smileys sind an.
[IMG] Code ist an.
HTML-Code ist aus.
Trackbacks are aus
Pingbacks are aus
Refbacks are aus




Alle Zeitangaben in WEZ +1. Es ist jetzt 15:13 Uhr.



-----------------------------------------------------------------------------------------------------------------------------
Powered by vBulletin® Version 3.8.5 (Deutsch)
Copyright ©2000 - 2016, Jelsoft Enterprises Ltd.
Powered by vBCMS® 2.7.0 ©2002 - 2016 vbdesigns.de
SEO by vBSEO 3.6.1
Copyright ©2009 - 2016 by Expert-Advisor.com - Das Metatrader Forum
MetaTrader bzw. MetaTrader 4 und MetaTrader 5 sind eingetragene Marken der MetaQuotes Software Corp.
-----------------------------------------------------------------------------------------------------------------------------