How to get Net Floating Pip Count from all Open Active Orders? | Forex Factory

Options

Permalinks
Print Thread

Similar Threads

Active positions calculator with net volume size 9 replies

Close all pending orders and open orders once they reach profit target 3 replies

Separating Count of Max Pending Orders and Max Open Trades 2 replies

script to "close open orders" and "open opposite orders" 3 replies

convert indicator that count bars to have it count ticks. is this possible? 0 replies

How to get Net Floating Pip Count from all Open Active Orders?

  • #1
  • First Post: Nov 14, 2019 9:38pm Nov 14, 2019 9:38pm
Hi, how do you get the net pip count from all the open orders currently in the terminal in mql4?
Take this course: https://shorturl.at/upBey
  • #2
  • Edited 5:25pm Nov 15, 2019 8:12am | Edited 5:25pm
Quoting tiestobob
Hi, how do you get the net pip count from all the open orders currently in the terminal in mql4?
Something like this:
Inserted Code
double sum=0;
for(int o=OrdersTotal()-1;o>=0;o—)
 if(OrderSelect(o,SELECT_BY_POS,MODE_TRADES){
  RefreshRates();
  if(OrderType()==0)sum+=Bid-OrderOpenPrice();
  if(OrderType())sum+=OrderOpenPrice()-Ask;}
sum*=0.1;
Print(“Order Pip Sum=“+(string)sum);
  • #3
  • Nov 15, 2019 5:08pm Nov 15, 2019 5:08pm
Quoting Beerrun
{quote} Something like this: double sum=0; for(int o=OrdersTotal()-1;o>=0;o—) if(OrderSelect(o,SELECT_BY_POS,MODE_TRADES){ RefreshRates(); if(OrderType==0)sum+=Bid-OrderOpenPrice(); if(OrderType)sum+=OrderOpenPrice()-Ask;} sum*=0.1; Print(“Order Pip Sum=“+(string)sum);
why are you multiplying sum by 0.1? and what about your brackets it seems like they're missing for the for loop?

Here's my attempt....

Inserted Code
double calcNetPips() {
 
double sum=0;
 
for(int i=0; i<OrdersTotal; i++)
{
if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES) == 1)
{
RefreshRates();
if(OrderType==0) 
sum+=Bid-OrderOpenPrice();
else 
sum+=OrderOpenPrice()-Ask;
}
}
Return sum*=0.1;
}
Take this course: https://shorturl.at/upBey
  • #4
  • Nov 15, 2019 5:19pm Nov 15, 2019 5:19pm
Quoting tiestobob
{quote} why are you multiplying sum by 0.1? and what about your brackets it seems like they're missing for the for loop? Here's my attempt.... double calcNetPips() { double sum=0; for(int i=0; i<OrdersTotal; i++) { if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES) == 1) { RefreshRates(); if(OrderType==0) sum+=Bid-OrderOpenPrice(); else sum+=OrderOpenPrice()-Ask; } } Return sum*=0.1; }
OrderSelect is a bool so “==1” isn’t needed.
Multiply by 0.1 to convert from points to pips. And the bracket is not needed if only one thing follows it.
Also always count down your orders.
But yes I had some typos doing it on my phone. I edited it just now.
  • #5
  • Nov 16, 2019 11:17pm Nov 16, 2019 11:17pm
Ah I see. Thank you!
Take this course: https://shorturl.at/upBey
  • #6
  • Nov 18, 2019 1:18am Nov 18, 2019 1:18am
Quoting Beerrun
{quote} OrderSelect is a bool so “==1” isn’t needed. Multiply by 0.1 to convert from points to pips. And the bracket is not needed if only one thing follows it. Also always count down your orders. But yes I had some typos doing it on my phone. I edited it just now.
ok this actually didnt work..

it would always display 35.xx for some reason, even though i know that's not the correct floating pip count.
Take this course: https://shorturl.at/upBey
  • #7
  • Last Post: Edited 6:52am Nov 18, 2019 6:41am | Edited 6:52am
Quoting tiestobob
{quote} ok this actually didnt work.. it would always display 35.xx for some reason, even though i know that's not the correct floating pip count.
Did you use your code, particularly “else”? If you use “else” following “if(OrderType()==0)” then it will also include pending orders in the sum.
Inserted Code
if(OrderType()==0)//buy market order
else {}//all other orders market+pending
I’m not sure if that is your problem but if so you need to explicitly state the order type individually.
Inserted Code
if(OrderType()==0) //buy market order
if(OrderType()==1)// sell market order
also if(OrderType())//sell market order
 
// may also use the enum terms:
//OP_BUY=0,OP_SELL=1
if(OrderType()==OP_BUY)
if(OrderType()==OP_SELL)

Also you cannot return an assignment, so:
return sum*=0.1;
will not work; need to do the multiplication first then return the value:
Inserted Code
sum*=0.1;
return sum;
//or
return sum*0.1;
0 traders viewing now