Fehler in While-Schleife?

[icepick1661]

Hallo,

zum Test habe ich folgende While-Schleife separat vom Hauptcode getestet (hier mit einfacheren, lesbareren Variablen...).

Bei einem Startkurs (Ask) von DAX 12.041,8 wird der StopBuy bei 12.072,5 gesetzt. Er sollte aber 1,5% über 11.980,5, also bei 12.160,2 gesetzt werden.

Das Ziel: Hätte ich bei einer Basis von 11.980,5 einen Ask von 12.300, soll 11.980,5 + 1,5% = 12160,2, dann (weil <Ask) bei 12160,2 + 1,5% = 12.342,6 der StopBuy gesetzt werden (weil ja nun >Ask...

Noch ein Hinweis: Prüfuwert "Ask + 30" tut nichts zur Sache, den hab ich nur gesetzt, um eine Spanne zu bekommen...

Danke für eure Hilfe!

Hier der Code:

   bool Ereignis=true;
   int ticketL;
   
   double Price=11980.5;

      if (Ereignis == true)
         {
         if (OrderType() == OP_BUYSTOP)
            {
            bool LongResult=true;
            }
         }
   
      if (LongResult)

         double StopBuyOpenValue = Price;
         double StopBuy2Low;
         double LongValuePerc;
         
            while (Ask +30 > StopBuyOpenValue)
            if (LongResult)
               {
               LongValuePerc = NormalizeDouble(StopBuyOpenValue * NextBuyPercent / 100,Digits); //NextBuyPercent = 1.5
               
               StopBuy2Low = StopBuyOpenValue + LongValuePerc;
               StopBuyOpenValue++;
               }

            bool LStopBuyResult_Restore=false;
            LStopBuyResult_Restore = ticketL= OrderSend(Symbol(),OP_BUYSTOP,LotsTraded,StopBuyOpenValue,Slippage,0,0,"BUY(#" + MagicNumberLong + ")",MagicNumberLong,0,Green);
            LongResult=false;

[next user]

Ich habs grad mal simuliert und bei mir erhalte ich das richtige Ergebnis. Paar Dinge zu deinem Code:

if (Ereignis == true)
{
   if (OrderType() == OP_BUYSTOP)
   {
      bool LongResult=true;
   }
}

Du kannst LongResult nicht innerhalb eines Blocks erstellen, und sie dann auserhalb dieses Blocks weiter nutzen.
LongResult müsstest du am besten oben bei Ereignis erstellen.

if (LongResult)

   double StopBuyOpenValue = Price;
   double StopBuy2Low;
   double LongValuePerc;
   
   while (Ask +30 > StopBuyOpenValue)
   if (LongResult)
   {
      LongValuePerc = NormalizeDouble(StopBuyOpenValue * NextBuyPercent / 100,Digits); //NextBuyPercent = 1.5
      
      StopBuy2Low = StopBuyOpenValue + LongValuePerc;
      StopBuyOpenValue++;
   }
   
   bool LStopBuyResult_Restore=false;
   LStopBuyResult_Restore = ticketL= OrderSend(Symbol(),OP_BUYSTOP,LotsTraded,StopBuyOpenValue,Slippage,0,0,"BUY(#" + MagicNumberLong + ")",MagicNumberLong,0,Green);
   LongResult=false;

• Das obige "if(LongResult)" bezieht sich lediglich auf "double StopBuyOpenValue = Price;", da du keine Klammern gesetzt hast.
• Deine 1.5% befinden sich in der Variablen "StopBuy2Low". "StopBuyOpenValue" inkrementierst du lediglich bei jedem Durchlauf, addierst also immer nur +1.
• Schmeiß die Variable "StopBuy2Low" raus und setze die Zeile so um
  

  StopBuyOpenValue = StopBuyOpenValue + LongValuePerc;

bool LStopBuyResult_Restore=false;
LStopBuyResult_Restore = ticketL= OrderSend(Symbol(),OP_BUYSTOP,0.10,StopBuyOpenValue,3,0,0);

Nicht gut formuliert. Da OrderSend() im Fehlerfall -1 liefert, wäre deine Variable "LStopBuyResult_Restore" trotzdem TRUE.
Um es weiter zu verarbeiten, hättest du einfach abgefragt, ob ticketL == -1. Da brauchst du dann auch keine "LStopBuyResult_Restore" mehr.