invalid ticket for OrderModify...

[Trading111]

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;

}

[michaelf]

Arbeite doch bitte mal an Deinem Errorhandling und poste davon mal die Ergebnisse. Entweder in Kurzform:

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:

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.

[traderdoc]

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

[Trading111]

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....