Metatrader Forum | Forex Expert-Advisor | Broker & Forex Tools

Metatrader Forum | Forex Expert-Advisor | Broker & Forex Tools (http://www.expert-advisor.com/forum/index.php)

- Programmierung MQL4 (http://www.expert-advisor.com/forum/forumdisplay.php?f=220)

- - History Select (http://www.expert-advisor.com/forum/showthread.php?t=6493)

Hallo,

ich versuche aus der History, Order mit bestimmenten Comment zu selektieren.
Es gibt genau 2 Positionen, welche im Profit geschlossen wurden, aber es werden keine Ergebnisse angezeigt.
Es soll das Ergebnis (+/-) von Profit, Loss, Swap, Commission ausgegeben werden

Habe es mit allen Varianten durch probiert.
Jemand eine Idee die mich weiterbringt, besten Dank

Mimo

HTML-Code:

  double PairHistoryProfitLossCalc(){

   double bapairprofitloss=0,babuyprofit=0,basellprofit=0;

   for(cnt=0;cnt<OrdersTotal();cnt++){

    if(OrderSelect(cnt,SELECT_BY_POS,MODE_HISTORY))

     if(OrderSymbol()==Symbol() && OrderMagicNumber()==111 && StringFind(OrderComment(),"111",0))

       if(OrderType()==OP_BUY )babuyprofit=babuyprofit+OrderProfit()+OrderSwap()+OrderCommission();

       if(OrderType()==OP_SELL)basellprofit=basellprofit+OrderProfit()+OrderSwap()+OrderCommission();

        bapairprofitloss=babuyprofit+basellprofit;

   }

   return(bapairprofitloss);

 }

Nun, dann schreibe mal statt:

for(cnt=0;cnt<OrdersTotal();cnt++){

die Zeile:

for (cnt = 0 ; cnt < OrdersHistoryTotal(); cnt++) {

traderdoc

Du hast vollkommen recht! Danke! Zuviel Arbei, zuwenig Schlaf...

Aber die Summe die nun rauskommt entspricht nicht der Summe in der History für das Symbol + Magic,
erschwerend kommt hinzu wenn ich statt 111 z.B. 777 (ist nirgendwo im Einsatz) kommt die selbe Summe raus, eigentlich müßte es "0", denn es gibt weder als Magic noch im Comment die "777".

Mimo

Zitat:

Zitat von MIMO (Beitrag 42793)

Aber die Summe die nun rauskommt entspricht nicht der Summe in der History für das Symbol + Magic,
erschwerend kommt hinzu wenn ich statt 111 z.B. 777 (ist nirgendwo im Einsatz) kommt die selbe Summe raus, eigentlich müßte es "0", denn es gibt weder als Magic noch im Comment die "777".

So auf die Schnelle fehlt mir ne Klammer um die beiden if OrderType()==xxx.
AVT

Hallo,
ich bin an dem selben Problem. Gibt es eine Lösung zu dem Ansatz hier?

Code:

 

void Historie()

{

     

   double bapairprofitloss=0,babuyprofit=0,basellprofit=0;

   int cnt;

   for(cnt=0;cnt<OrdersHistoryTotal();cnt++)

   {

    if(OrderSelect(cnt,SELECT_BY_POS,MODE_HISTORY))

     if(OrderSymbol()==Symbol() && OrderMagicNumber()==Magicnummer)

       if(OrderType()==OP_BUY )babuyprofit=babuyprofit+OrderProfit()+OrderSwap()+OrderCommission();

       if(OrderType()==OP_SELL)basellprofit=basellprofit+OrderProfit()+OrderSwap()+OrderCommission();

        bapairprofitloss=babuyprofit+basellprofit;

Print("Gewinn fuer Magicnr",Magicnummer,": ",bapairprofitloss);

   } 

 }

Was soll denn gelöst werden?

traderdoc

Fehlen da nicht ein paar { und } ?

https://docs.mql4.com/trading/orderselect

Zitat:

Zitat von Max2018 (Beitrag 42864)

Hallo,
ich bin an dem selben Problem. Gibt es eine Lösung zu dem Ansatz hier?

Ein TestScript, wir haben hier keine MagicNummer, darum habe ich Deine Zeile geändert

Code:

// ===================================================================

void OnStart()

{

  Historie();

}

// ===================================================================

void Historie()

{

     

   double bapairprofitloss=0,babuyprofit=0,basellprofit=0;

   int cnt;

   for(cnt=0;cnt<OrdersHistoryTotal();cnt++)

   {

    if(OrderSelect(cnt,SELECT_BY_POS,MODE_HISTORY))

     //if(OrderSymbol()==Symbol() && OrderMagicNumber()==Magicnummer)

     if(OrderSymbol()==Symbol() )                                                // FürAlleOrdersDisesSymbols

     { 

       if(OrderType()==OP_BUY ) 

          babuyprofit=babuyprofit+OrderProfit()+OrderSwap()+OrderCommission();   //AlleBuysSummieren

       if(OrderType()==OP_SELL) 

          basellprofit=basellprofit+OrderProfit()+OrderSwap()+OrderCommission(); //AlleSellsSummieren

     }

   } 

   bapairprofitloss=babuyprofit+basellprofit;                                    //SummenAddieren

   Print("Gewinn gesamt: ",DoubleToStr(bapairprofitloss,2)," €");                //UndMelden

}

Du hast 2 Operationen auf dem OrderSymbol durchzuführen: die Buys und die Sells zu summieren, also müssen die in Klammern gefaßt werden. AVT

Danke - werde ich gleich mal testen. Wenn Du nix mehr hörst hat´s geklappt

:D
:D