Anzahl der orders ohne Stopploss und mit Magicnr

[Leopold]

Zitat von Hippo25

Ich habe es mal ausprobiert
Habe aber folgende Meldung

return value of 'OrderSelect' should be checked Offenetrades.mq4

Filtert es richtig oder nicht ?

grus Hippo

Hallo!

Ist nicht zwingend notwendig solltest du aber abfragen und in die if-Bedingung einbauen:

bool Select = OrderSelect(.....)

if (Select == True && OrderSymbol() == Symbol && ...)

Hoffe das ist verständlich.

---Leopold---

[Leopold]

Zitat von Hippo25

Also
Nochmal den ganzen code

PHP-Code:

int offeneorders=0; //Speichert die Anzahl der Orders zur späteren Verwendung
   int ordersohnestop=0; //Speichert die Anzahl der Orders ohne ein Stoploss

   int anzahlorders=OrdersTotal();

   for(zaehler=0;zaehler<anzahlorders;zaehler++) // for schleife
     {
      OrderSelect(zaehler, SELECT_BY_POS, MODE_TRADES); // MODE_TRADES = offene und aufgegebene Orders
       // check offene Positionen, Symbol & MagicNumber
       if (OrderSymbol()== Symbol())
        {
          if (OrderMagicNumber()==MagicNumber) 
            {
               offeneorders++;

               if ( OrderStopLoss() == 0 ) 
               {
                  ordersohnestop++;
               }
  
            }
        } 
     }  
        
  Comment (
              "Offene Trades: ",offeneorders,"\n"
               "Offen ohne SL: ",ordersohnestop,"\n"
              
               
               );   
  }
//+------------------------------------------------------------------+ 


Wie ihr seht - int ordersohnestop = 0; vor der for schleife.
Wo müsste ordersohnestop++ hin ?


gruss
Hippo

Das passt so, traderdoc hat nur die Deklaration gemeint, die stimmt bei dir aber. (int ordersohnestop=0; )
Ich denke er ist auf den Kommentar von MA-EA eingegangen, dort war die Deklaration falsch!

---Leopold---

[Leopold]

Hallo zur Vereinfachung und Übersichtlichkeit würde ich das ganze so schreiben:

for(zaehler=0;zaehler<anzahlorders;zaehler++)
     { 
      bool Select = OrderSelect(zaehler, SELECT_BY_POS, MODE_TRADES);

      if (Select == True && OrderSymbol()== Symbol() && OrderMagicNumber()==MagicNumber) 
        { 
         offeneorders++; 
         if (OrderStopLoss() == 0) ordersohnestop++; 
        }  
     }

---Leopold---

[Leopold]

Zitat von MA-EA

Kenn mich mit den Schleifen nicht so aus.

Aber fängt er nicht sowieso bei 0 an zu zählen?

Eine Schleife an sich zählt gar nicht, diese wird solange ausgeführt bis eine Bedingung Diese abbricht.

Ich hoffe ich hab das richtig ausgedrückt.


---Leopold---

[traderdoc]

Sorry, ja ich meinte die falsche Deklaration von @MA-EA.

Es reicht aber völlig aus, zu schreiben:

if (OrderSelect(...) ) {
...
}

Falls false, dann wird die gesamte if-Abfrage ignoriert, incl. der if-Abfragen innerhalb der o.g. Abfrage.

traderdoc

[Leopold]

Stimmt, aber ich finde am Anfang ist es mit der eigenen deklaration leichter zu verstehn. Zumindest hab ich mir da immer leichter getan.

---Leopold---

[Hippo25]

Ich werde immer die komplette Lösung des Problem hier am Ende anfügen.
Stichworte Filter Selekt select auswahl Order Auftrage Trades Filtern

PHP-Code:

//+------------------------------------------------------------------+
//|                                                 Offenetrades.mq4 |
//|                        Copyright 2018, MetaQuotes Software Corp. |
//|                                             https://www.mql5.com |
//+------------------------------------------------------------------+
#property copyright "Copyright 2018, MetaQuotes Software Corp."
#property link      "https://www.mql5.com"
#property version   "1.00"
#property strict


int MagicNumber;
int zaehler;


void OnTick()
  {
//---
   int offeneorders=0; //Speichert die Anzahl der Orders zur späteren Verwendung
   int ordersohnestop=0; //Speichert die Anzahl der Orders ohne ein Stoploss
   int anzahlorders=OrdersTotal();

   for(zaehler=0;zaehler<anzahlorders;zaehler++)
     { 
      bool Select = OrderSelect(zaehler, SELECT_BY_POS, MODE_TRADES);

      if (Select == True && OrderSymbol()== Symbol() && OrderMagicNumber()==MagicNumber) 
        { 
         offeneorders++; 
         if (OrderStopLoss() == 0) ordersohnestop++; 
        }  
     }
        
  Comment (
              "Offene Trades: ",offeneorders,"\n"
               "Offen ohne SL: ",ordersohnestop,"\n"
              
               
               );   
  }
//+------------------------------------------------------------------+ 


Danke an alle.

gruss
Hippo