How to calclate total profit

Cromo 2018.09.03 02:49

I want to know total profit of open orders (BUY position + SELL position)

I also want to include swap + comission.

I don't want to use AccountProfit, so I tried to calculate total (BUY+SELL orders).

I coded below but when I check with Print(Total_profit), the value is different from MT4 current profit value.

What is the problem?

   double Total_profit= NULL;int cnt = NULL;
   for(cnt=0;cnt<OrdersTotal();cnt++){
   OrderSelect(cnt,SELECT_BY_POS);
   if(OrderSymbol() == Symbol() && OrderMagicNumber()==magic)
   {if( OrderType() == OP_BUY || OrderType() == OP_SELL )
      {Total_profit= Total_profit+ OrderProfit() + OrderSwap() + OrderCommission();}    
   }
   }

Fernando Morales 2018.09.03 08:04 #1

kajironpu:

I want to know total profit of open orders (BUY position + SELL position)

I also want to include swap + comission.

I don't want to use AccountProfit, so I tried to calculate total (BUY+SELL orders).

I coded below but when I check with Print(Total_profit), the value is different from MT4 current profit value.

What is the problem?

For the account current PnL do not filter by Symbol nor by Magic. Also, compare cnt values with lower or equal than OrdersTotal()

   double Total_profit= NULL;int cnt = NULL;
   for(cnt=0;cnt<=OrdersTotal();cnt++){
   OrderSelect(cnt,SELECT_BY_POS);
   if(OrderSymbol() == Symbol() && OrderMagicNumber()==magic)
   {if( OrderType() == OP_BUY || OrderType() == OP_SELL )
      {Total_profit= Total_profit+ OrderProfit() + OrderSwap() + OrderCommission();}    
   }
   }

For the account realised PnL evaluate today's closed trades

double Total_profit= 0;

for( int cnt = 0; cnt <= OrdersHistoryTotal(); cnt++)
        if ( OrderSelect( cnt, SELECT_BY_POS, MODE_HISTORY ) )
                if( OrderType() < 2 && iTime(NULL, PERIOD_D1,0) <= OrderCloseTime() )
                        Total_profit += OrderProfit() + OrderSwap() + OrderCommission();

Cromo 2018.09.03 08:20 #2

Fernando Morales:

For the account current PnL do not filter by Symbol nor by Magic. Also, compare cnt values with lower or equal than OrdersTotal()

For the account realised PnL evaluate today's closed trades

Hi, Fernando san

Thank you for your help.

If I put different EA in different chart, if I do not filter by MAGIC and Symbol, will it be correct value?

I mean I want to know current total profit which now I have positions where I attached EA.

EA problem :: operator Magic Number for different

hiroshitiu 2018.09.03 08:48 #3

Kajironpu,

Try remove ordersymbol()==Symbol() bool check, as magicnumber is the unique identification code for EA.

For example, you put EA in both usdjpy and eurjpy, the magicnumber of EA should be the same but ordersymbol parameter is different, this" and" condition checking might be the root cause

I will write an SYMBOL() Questions from Beginners MQL4

Cromo 2018.09.03 09:20 #4

hiroshitiu:
Kajironpu,

Try remove ordersymbol()==Symbol() bool check, as magicnumber is the unique identification code for EA.

For example, you put EA in both usdjpy and eurjpy, the magicnumber of EA should be the same but ordersymbol parameter is different, this" and" condition checking might be the root cause

Hi, hiroshitiu

I am confused.

For example,

In the same account, I run 2 EAs (different EA).

EA1: EURUSD magic=111 TF=M1 and order#1(winning) order#2(wining) order#3 (losing) / total profit $10

EA2: EURUSD magic=222 TF=M15 and order#1(losing) order#2(losing) / total profit $-20

I want to get profit for each.

Even tough this condition, I need to remove symbol and magic number filter?

I thought if I don't filter by magic and symbol, total profit would be combined all in my account.

I mean I want to get current profit of EA attached only.

Select By Magic Number Trouble using EA with Firebird EA

hiroshitiu 2018.09.03 13:15 #5

kajironpu:

Hi, hiroshitiu

I am confused.

For example,

In the same account, I run 2 EAs (different EA).

EA1: EURUSD magic=111 TF=M1 and order#1(winning) order#2(wining) order#3 (losing) / total profit $10

EA2: EURUSD magic=222 TF=M15 and order#1(losing) order#2(losing) / total profit $-20

I want to get profit for each.

Even tough this condition, I need to remove symbol and magic number filter?

I thought if I don't filter by magic and symbol, total profit would be combined all in my account.

I mean I want to get current profit of EA attached only.

Kajironpu san,

Let me explain with your example above, and make it more generic.

Assume 7 open orders exist (2 manual released, 5 EA released

Order #1, EURUSD magic =111

Order #2, EURUSD magic =111

Order #3, EURAUD magic =111

Order #4, EURGBP magic =222

Order #5, EURJPY magic =222

Order #6, EURJPY magic =0 (manual)

Order #7, EURCHF magic =0 (manual)

To get the total current profit of EA released order (#1-#5) the solution as below, I cut the codes and brackets which not needed.

double Total_profit= NULL;
   for(int cnt=0;cnt<OrdersTotal();cnt++)
	if (OrderSelect(cnt,SELECT_BY_POS))
   		if(OrderMagicNumber()!=0)
			Total_profit+= OrderProfit() + OrderSwap() + OrderCommission();

As manual orders' magicnumber can be 0 ONLY, if you want to get all EA orders, the solution is to filter out the manual orders with magicnumber=0

Why OrderSymbol()==Symbol() does not work?

Because when the codes run in the market, the return value of "Symbol()" is the name of market that the EA/Script running

if you run the codes at EURUSD, it will not be able to read order #3-#7 because the condition "Symbol()==OrderSymbol()"("EURUSD" not equal to "EURAUD" for #3) cannot fulfill

Loops and Closing or Return data from Closed Ordermodify error - compare

Fernando Morales 2018.09.03 13:45 #6

kajironpu:

Hi, Fernando san

Thank you for your help.

If I put different EA in different chart, if I do not filter by MAGIC and Symbol, will it be correct value?

I mean I want to know current total profit which now I have positions where I attached EA.

If you want to calculate the PnL of each EA, then you must filter my OrderMagicNumber(), never by Symbol. hiroshitiu posted the code to do this for the floating PnL.

For closed trades you have to use my second code example, which it just need the filter for OrderMagicNumber()

Coding help I will program your Manual order and EA

Cromo 2018.09.05 03:40 #7

hiroshitiu:

Kajironpu san,

Let me explain with your example above, and make it more generic.

Assume 7 open orders exist (2 manual released, 5 EA released

Order #1, EURUSD magic =111

Order #2, EURUSD magic =111

Order #3, EURAUD magic =111

Order #4, EURGBP magic =222

Order #5, EURJPY magic =222

Order #6, EURJPY magic =0 (manual)

Order #7, EURCHF magic =0 (manual)

To get the total current profit of EA released order (#1-#5) the solution as below, I cut the codes and brackets which not needed.

As manual orders' magicnumber can be 0 ONLY, if you want to get all EA orders, the solution is to filter out the manual orders with magicnumber=0

Why OrderSymbol()==Symbol() does not work?

Because when the codes run in the market, the return value of "Symbol()" is the name of market that the EA/Script running

if you run the codes at EURUSD, it will not be able to read order #3-#7 because the condition "Symbol()==OrderSymbol()"("EURUSD" not equal to "EURAUD" for #3) cannot fulfill

Hi, hiroshitiu san Sorry for my late reply. Now I am clear that manual order MAGIC is 0 , wow I thought that it is same as EA order....

Thank you so much.

Cromo 2018.09.05 03:44 #8

Fernando Morales:

If you want to calculate the PnL of each EA, then you must filter my OrderMagicNumber(), never by Symbol. hiroshitiu posted the code to do this for the floating PnL.

For closed trades you have to use my second code example, which it just need the filter for OrderMagicNumber()

Hi,Fernando

Thank you for your kind reply.

If I want to calculate the PnL of each EA, then I must filter my OrderMagicNumber(), never by Symbol....

I got it. Your second code is very useful and I will try with it. Gracious.

How to show BTCUSD Any rookie question, so If 1% damage, position

New comment