Problem mit bool-Abfrage nach OrderSelect

[MarkusWilhelm89]

Hallo zusammen,

ich hoffe mir kann jemand helfen und mir sagen, wo mein Fehler liegt.


Beschreibung:

Es handelt sich um einen Grid-EA. Zur leicherten Darstellung gehe ich davon aus nur "long" Positionen zu eröffnen.

Der EA soll immer den OrderOpenPrice() der aktuellen/neuesten Market-Order als Basis für weitere Berechnungen nutzen --> double PREISLEVEL()

Ich definiere mit einer input Variable die Maschenweite des Grids, zB 200 --> double Grid_Abstand


In den hier dargestellten Programmzeilen ermittle ich mithilfe des PREISLEVEL() +/- Grid_Abstand, jeweils die Ober-/Untergrenze der Grid-Ebenen:

double GRID_UPPER_1_HIGH()    {double Grid_Upper_1_High = PREISLEVEL()+(1*Grid_Abstand*Point)+(Grid_Abstand/2)*Point;  return(Grid_Upper_1_High);};      double GRID_UPPER_1_LOW()     {double Grid_Upper_1_Low  = PREISLEVEL()+(1*Grid_Abstand*Point)-(Grid_Abstand/2)*Point;  return(Grid_Upper_1_Low);};


double GRID_CURRENT_HIGH()    {double Grid_Current_High = PREISLEVEL()+(Grid_Abstand/2)*Point;                         return(Grid_Current_High);};      double GRID_CURRENT_LOW()     {double Grid_Current_Low  = PREISLEVEL()-(Grid_Abstand/2)*Point;                         return(Grid_Current_Low);};


double GRID_LOWER_1_HIGH()    {double Grid_Lower_1_High = PREISLEVEL()-(1*Grid_Abstand*Point)+(Grid_Abstand/2)*Point;  return(Grid_Lower_1_High);}      double GRID_LOWER_1_LOW()     {double Grid_Lower_1_Low  = PREISLEVEL()-(1*Grid_Abstand*Point)-(Grid_Abstand/2)*Point;  return(Grid_Lower_1_Low);};

Im Anschluss daran, prüfe ich mit bool'schen Abfrage, ob in den zuvor berechneten Grid-Ebenen die jeweiligen Order-Arten vorliegen.

Hintergrund: Es soll immer im Grid-Bereich darüber eine BUYSTOP und im Grid-Bereich darunter eine BUYLIMIT Order liegen.

bool GRID_UPPER_1_BUYSTOP()   {bool Grid_Upper_1_Buystop    =false; for(int i=OrdersTotal()-1;i>=0;i--) if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES)) if(OrderSymbol()==Symbol()&&OrderMagicNumber()==MagicNumber)   if(OrderType()==OP_BUYSTOP)    if(OrderOpenPrice()>GRID_UPPER_1_LOW() && OrderOpenPrice()<GRID_UPPER_1_HIGH())  Grid_Upper_1_Buystop    = true;  return(Grid_Upper_1_Buystop);};

bool GRID_CURRENT_BUY()       {bool Grid_Current_Buy        =false; for(int i=OrdersTotal()-1;i>=0;i--) if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES)) if(OrderSymbol()==Symbol()&&OrderMagicNumber()==MagicNumber)   if(OrderType()==OP_BUY)        if(OrderOpenPrice()<GRID_CURRENT_HIGH() && OrderOpenPrice()>GRID_CURRENT_LOW())  Grid_Current_Buy        = true;  return(Grid_Current_Buy);};

bool GRID_LOWER_1_BUYLIMIT()  {bool Grid_Lower_1_Buylimit   =false; for(int i=OrdersTotal()-1;i>=0;i--) if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES)) if(OrderSymbol()==Symbol()&&OrderMagicNumber()==MagicNumber)   if(OrderType()==OP_BUYLIMIT)   if(OrderOpenPrice()<GRID_LOWER_1_HIGH() && OrderOpenPrice()>GRID_LOWER_1_LOW())  Grid_Lower_1_Buylimit   = true;  return(Grid_Lower_1_Buylimit);};

Nun zu meinem Problem: In der bool'schen Prüfung hat der EA Probleme, den Code nach der OrderType()-Prüfung, nämlich der Prüfung ob der OrderType() in dem Grid-Bereich liegt, zu verarbeiten. Konkret stört sich der EA an folgendem Schnipsel:

if( OrderOpenPrice()<GRID_LOWER_1_HIGH() && OrderOpenPrice()>GRID_LOWER_1_LOW() )

Ich bin mir nicht sicher wo mein Fehler liegt.

Wenn ich die Variable in drei einzelne Variablen zerlege und nach einander bzw. aufeinander aufbauenf nutze, in dem ich zuerst die TicketNummer prüfe, danach mit der gefundenen TicketNummer den OrderOpenPrice ermittle, danach mit dem OrderOpenPrice die bool'sche Abfrage mache.... dann funktioniert das. Aber so wie oben gezeigt in einem Rutsch nicht. :-(

Hoffe ich habe es verständlich erklärt und hoffe noch mehr, mir kann jemand die augen öffnen was ich falsche mache.


Danke vorab

Gruß

[traderdoc]

Nun ja, es gibt Code, da existieren Fehler, die sieht man sofort und es gibt denselben Code, nur "anders" geschrieben, da exisierenn dieselben Fehler, die man aber nicht sofort sieht.

Zu Letzterem gehört Dein Code.
Ich habe schon wirklich viel Code gesehen, aber solche Bandwürmer bisher nicht.
Der ist leider an Unübersichtlichkeit, kaum noch zu übertreffen.
Nimm das jetzt bitte nicht persönlich, sondern versuche, die Abfragen und Schleifen eingerückt in separate Zeilen zu schreiben.
Das macht Dir das Lesen zukünftig einfacher und uns, die die Fehler finden sollen, auch.

traderdoc

[MarkusWilhelm89]

ich versuche mich zu bessern.
Ist folgende Darstellung leserlicher?

bool GRID_LOWER_1_BUYLIMIT()
   {bool Grid_Lower_1_Buylimit = false;
   
   for(int i=OrdersTotal()-1; i>=0; i--) 
      {
       if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES))
         if(OrderSymbol()==Symbol()&&OrderMagicNumber()==MagicNumber)
            if(OrderType()==OP_BUYLIMIT)
               if(OrderOpenPrice()<GRID_LOWER_1_HIGH() && OrderOpenPrice()>GRID_LOWER_1_LOW())  
                  Grid_Lower_1_Buylimit = true;}
                     return(Grid_Lower_1_Buylimit);};

[traderdoc]

Geht so. Ich würde das so scheiben, dann sieht man genaus was wohin gehört.

bool GRID_LOWER_1_BUYLIMIT() {
   bool Grid_Lower_1_Buylimit = false;
   
   for (int i = OrdersTotal()-1; i >= 0; i--) {
      if (OrderSelect(i, SELECT_BY_POS, MODE_TRADES)) {
         if (OrderSymbol() == Symbol() && OrderMagicNumber() == MagicNumber) {
            if (OrderType() == OP_BUYLIMIT) {
               if (OrderOpenPrice() < GRID_LOWER_1_HIGH() && OrderOpenPrice() > GRID_LOWER_1_LOW()) { 
                  Grid_Lower_1_Buylimit = true;
               }
            }
         }
      }
   }
      
   return (Grid_Lower_1_Buylimit);
}

Ob man nun z.B. bei
for (int i = OrdersTotal()-1; i >= 0; i--) {
die { hinter die Zeile schreibt oder so
for (int i = OrdersTotal()-1; i >= 0; i--)
{
ist dann eher Geschmackssache.

Auch die Freizeichen nach den Operatoren und hinter if und for erleichtern die Leseweise enorm. Auch mal eine Freizeile einfügen, um wie hier, die Schleife hervorzuheben.

traderdoc

[MarkusWilhelm89]

Ok verstanden.
Aber um auf die eigentliche Frage zu meinem Problem zurück zu kommen :-D

Eine Idee wo der Fehler liegen könnte?

[traderdoc]

Den Code würde ich mir erst anschauen, wenn er in
lesbarer Form vorliegt.

traderdoc

[MarkusWilhelm89]

Hi,
ich habe versucht den Code in vernünftiger Form mal dar zu stellen.
Für weitere Hinweise zur Darstellung/Schreibweise bin ich dankbar.
Alles beginnt mit:

1. Schritt: durch eine input Variable wird der Grid_Abstand = zB.: 200 definiert
2. Schritt: Ermittlung vom Preislevel: PREISLEVEL() = OrderOpenPrice() der aktuellen MARKET-Order
3. Schritt: Grid-Bereiche bzw. die Ober-/Untergrenzen der Grid-Bereiche ermitteln

   double GRID_LOWER_1_HIGH()
     {
      double Grid_Lower_1_High = PREISLEVEL() - (1 * Grid_Abstand * Point) + (Grid_Abstand / 2) * Point;
      return(Grid_Lower_1_High);
     }

   double GRID_LOWER_1_LOW()
     {
      double Grid_Lower_1_Low = PREISLEVEL() - (1 * Grid_Abstand * Point) - (Grid_Abstand / 2) * Point;
      return(Grid_Lower_1_Low);
     }

Sobald das gemacht ist, soll die Prüfung erfolgen, ob der jeweilige OrderType() in dem ermittelten Grid-Bereich (also zwischen Ober-/Untergrenze) vorliegt.
In meinem Beispiel der OrderType() = OP_BUYLIMIT, da ich den Grid-Bereich unterhalb der aktuellen Market-Order prüfe.
Dazu anbei zwei unterschiedliche Vorgehensweisen.

// Beispiel der aufgeteilten Prüfung

   double Grid_Lower_1_Buylimit_Price = 0;
   for(int i = OrdersTotal()-1; i >= 0; i--)
     {
      if(OrderSelect(i, SELECT_BY_POS, MODE_TRADES))
        {
         if(OrderSymbol() == Symbol() && OrderMagicNumber() == MagicNumber)
           {
            Grid_Lower_1_Buylimit_Price = OrderOpenPrice();
           }
        }
     };

   bool Grid_Lower_1_Buylimit_BOOL = false;
     {
      if(Grid_Lower_1_Buylimit_Price < GRID_LOWER_1_HIGH() && Grid_Lower_1_Buylimit_Price > GRID_LOWER_1_LOW())
        {
         Grid_Lower_1_Buylimit_BOOL = true;
        }
     }

// Beispiel der zusammenhängenden Prüfung

   bool GRID_LOWER_1_BUYLIMIT()
     {
      bool Grid_Lower_1_Buylimit = false;

      for(int i = OrdersTotal()-1; i >= 0; i--)
        {
         if(OrderSelect(i, SELECT_BY_POS, MODE_TRADES))
           {
            if(OrderSymbol() == Symbol() && OrderMagicNumber() == MagicNumber)
              {
               if(OrderType() == OP_BUYLIMIT)
                 {
                  if(OrderOpenPrice() < GRID_LOWER_1_HIGH() && OrderOpenPrice() > GRID_LOWER_1_LOW())
                    {
                     Grid_Lower_1_Buylimit = true;
                    }
                 }
              }
           }
        }

      return (Grid_Lower_1_Buylimit);
     }

Wie anfangs erwähnt, funktioniert die Abfrage bei der "aufgeteilten Prüfung", aber nicht bei der "zusammenhängenden Prüfung".

[Bobs]

Ich sehe nicht den Code aller Funktionen, aber ich kann davon ausgehen, dass das Problem kann durch die Verwendung einer Funktion innerhalb einer anderen entstehen. Beide Funktionen verwenden OrderSelect(i,SELECT_BY_POS), wodurch sich die ausgewählte Order ändert.
Versuchen OrderSelect(i,SELECT_BY_POS) zweites mal danach anwenden.

[traderdoc]

Im zweiten Code fehlt schon mal die Selektion nach OP_BUYLIMIT. Damit wird die tatsächlich letzte Order benutzt, um die Variable Grid_Lower_1_Buylimit_Price zu füllen, egal, von welchem Ordertyp die letzte Order ist.

Wir brauchen noch den Code der Funktion PREISLEVEL().

Dass Funtionen in Abfragen verwendet werden, stellt überhaupt kein Problem dar. D.h.
if(OrderOpenPrice() < GRID_LOWER_1_HIGH() && OrderOpenPrice() > GRID_LOWER_1_LOW())
sollte auch funktionieren.
Das hier verwendete OrderOpenPrice() muss ja auch nicht identisch sein zum OrderOpenPrice() in der Funktion PREISLEVEL().
Existieren z.B. mehrere BuyLimit-Orders, dann wird bei derjenigen BuyLimitOrder ein true folgen, für die die Abfrage als erste mit true endet. Alle evtl. weiteren BuyLimitOrders blieben dann unberücksichtigt, weil Grid_Lower_1_Buylimit dann true bleibt.

Was ermittelt denn der EA im dritten Codebeispiel?

Das prinzipiell nächste Problem besteht eben darin, dass nicht gesamte relevante Code gesendet wird (z.B. Funktion PREISLEVEL()). Da könnten schon einige Informationen auf der Strecke bleiben.

traderdoc

[MarkusWilhelm89]

Anbei der Code

input int Grid_Abstand = 200;
input int MagicNumber  = 1234;


// GRID_START_TICKET() ist die zu letzt eröffnete Pending-Order bzw. neueste Market-Order
   double PREISLEVEL()
     {
      double PreisLevel = 0;

      if(OrderSelect( GRID_START_TICKET() , SELECT_BY_TICKET , MODE_TRADES ) == false )
        {
         PreisLevel = NormalizeDouble( ( Ask + Bid ) / 2 , Digits );
        };
      if(OrderSelect( GRID_START_TICKET() , SELECT_BY_TICKET , MODE_TRADES ) == true )
        {
         PreisLevel = NormalizeDouble( OrderOpenPrice() , Digits );
        };
      return(PreisLevel);
     };


// GRID-Bereiche (Ober-/Untergrenze) ermitteln
   double GRID_UPPER_1_HIGH()
     {
      double Grid_Upper_1_High = PREISLEVEL() + ( 1 * Grid_Abstand * Point ) + ( Grid_Abstand / 2 ) * Point;
      return( Grid_Upper_1_High );
     };

   double GRID_UPPER_1_LOW()
     {
      double Grid_Upper_1_Low  = PREISLEVEL() + ( 1 * Grid_Abstand * Point )-( Grid_Abstand / 2 ) * Point;
      return( Grid_Upper_1_Low );
     };


   double GRID_CURRENT_HIGH()
     {
      double Grid_Current_High = PREISLEVEL() + ( Grid_Abstand / 2 ) * Point;
      return( Grid_Current_High );
     };
   double GRID_CURRENT_LOW()
     {
      double Grid_Current_Low  = PREISLEVEL() - ( Grid_Abstand / 2 ) * Point;
      return( Grid_Current_Low );
     };


   double GRID_LOWER_1_HIGH()
     {
      double Grid_Lower_1_High = PREISLEVEL() - ( 1 * Grid_Abstand * Point ) + ( Grid_Abstand / 2 ) * Point;
      return( Grid_Lower_1_High );
     };
   double GRID_LOWER_1_LOW()
     {
      double Grid_Lower_1_Low  = PREISLEVEL() - ( 1 * Grid_Abstand * Point ) - ( Grid_Abstand / 2 ) * Point;
      return( Grid_Lower_1_Low );
     };


      // Beispiel der zusammenhängenden Prüfung
      bool GRID_LOWER_1_BUYLIMIT()
        {
         bool Grid_Lower_1_Buylimit = false;

         for(int i = OrdersTotal()-1; i >= 0; i--)
           {
            if(OrderSelect(i, SELECT_BY_POS, MODE_TRADES))
              {
               if(OrderSymbol() == Symbol() && OrderMagicNumber() == MagicNumber)
                 {
                  if(OrderType() == OP_BUYLIMIT)
                    {
                     if(OrderOpenPrice() < GRID_LOWER_1_HIGH() && OrderOpenPrice() > GRID_LOWER_1_LOW())
                       {
                        Grid_Lower_1_Buylimit = true;
                       }
                    }
                 }
              }
           }

         return (Grid_Lower_1_Buylimit);
        };