Trying to get the latest open trade info.

[Tan Chee Ho]

void counttrades() {
//----
   int      total=OrdersTotal();
            
   if (total==0) openedtrades=0;
   else {
      for (int i=0; i<total; i++) {
         if (!OrderSelect(i,SELECT_BY_POS)) continue;
         if (OrderSymbol()==Symbol()) {
            openedtrades++;
            lastticket=OrderTicket();
            lastopenlots=OrderLots();
            lastopentype=OrderType();
            lastopenprice=OrderOpenPrice();
            total_average_price+=(OrderOpenPrice()*OrderLots());
            total_lots+=OrderLots();
            breakeven_price=(total_average_price/total_lots);
         }
      }
   }
}

I am trying to get the latest trade info, and also the breakeven price.... but the return all show 0.0.

hope that can get help from you, so i can learn from my mistake.

[William Roeder]

1. No need for the if.

      openedtrades=0;
         for (int i=OrdersTotal() - 1; i>=0; --i) {

2. Get in the habit of always counting down.

   In the presence of multiple orders (one EA multiple charts, multiple EAs, manual trading,) while you are waiting for the current operation (closing, deleting, modifying) to complete, any number of other operations on other orders could have concurrently happened and changed the position indexing:

   1. For non-FIFO (US brokers,) (or the EA only opens one order per symbol,) you can simply count down in a position loop, and you won't miss orders. Get in the habit of always counting down.
                Loops and Closing or Deleting Orders - MQL4 and MetaTrader 4 - MQL4 programming forum
      For FIFO (US brokers,) and you (potentially) process multiple orders per symbol, you must count up and on a successful operation, reprocess all positions (set index to -1 before continuing.)
   2. and check OrderSelect in case earlier positions were deleted.
                What are Function return values ? How do I use them ? - MQL4 and MetaTrader 4 - MQL4 programming forum
                Common Errors in MQL4 Programs and How to Avoid Them - MQL4 Articles
   3. and if you (potentially) process multiple orders, must call RefreshRates() after server calls if you want to use the Predefined Variables (Bid/Ask) or OrderClosePrice() instead, on the next order/server call.

3. Make Total_Average_Price a local variable. It's an intermediary, it's neither an average nor a price. Average price is your break even.
4. No Magic number filtering on your OrderSelect loop means your code is incompatible with every EA (including itself on other charts and manual trading.)
             Symbol Doesn't equal Ordersymbol when another currency is added to another seperate chart . - MQL4 and MetaTrader 4 - MQL4 programming forum

5. Your problem is elsewhere.
   

[Konstantin Nikitin]

void counttrades() {
//----
   openedtrades=0;
   lastticket = 0;
   for (int i=0; i<OrdersTotal(); i++) {
      if (!OrderSelect(i,SELECT_BY_POS)) continue;
      if (OrderSymbol()!=Symbol()) continue;
      if( OrderType()!=ORDER_TYPE_BUY && OrderType()!=ORDER_TYPE_SELL ) continue;

      openedtrades++;
      total_average_price+=(OrderOpenPrice()*OrderLots());
      total_lots+=OrderLots();
      breakeven_price=(total_average_price/total_lots);

      if(lastticket > OrderTicket()) continue;
      lastticket=OrderTicket();
      lastopenlots=OrderLots();
      lastopentype=OrderType();
      lastopenprice=OrderOpenPrice();
   }
}

Or

void counttrades() {
//----
   openedtrades=0;
   datetime timeOrderOpen = 0;
   for (int i=0; i<OrdersTotal(); i++) {
      if (!OrderSelect(i,SELECT_BY_POS)) continue;
      if (OrderSymbol()!=Symbol()) continue;
      if( OrderType()!=ORDER_TYPE_BUY && OrderType()!=ORDER_TYPE_SELL ) continue;

      openedtrades++;
      total_average_price+=(OrderOpenPrice()*OrderLots());
      total_lots+=OrderLots();
      breakeven_price=(total_average_price/total_lots);

      if(timeOrderOpen > OrderOpenTime()) continue;
      timeOrderOpen = OrderOpenTime();

      lastticket=OrderTicket();
      lastopenlots=OrderLots();
      lastopentype=OrderType();
      lastopenprice=OrderOpenPrice();
   }
}

[Tan Chee Ho]

Konstantin Nikitin:

Or

Thank you very much, brother! appreciate for your suggestion. will certainly try them. Thanks once again.

[Tan Chee Ho]

whroeder1:

1. No need for the if.
2. Get in the habit of always counting down.
3. Make Total_Average_Price a local variable. It's an intermediary, it's neither an average nor a price. Average price is your break even.
4. No Magic number filtering on your OrderSelect loop means your code is incompatible with every EA (including itself on other charts and manual trading.)
             Symbol Doesn't equal Ordersymbol when another currency is added to another seperate chart . - MQL4 and MetaTrader 4 - MQL4 programming forum

5. Your problem is elsewhere.
   

Appreciate your sharing. Thank you for teaching, and suggestion. 

[Tan Chee Ho]

I am actually searching and trying to write an EA to help me to calculate 

1. Breakeven Point + few pips profit.

2. Modify my trades to the price above. 

These trades are manually entered, that's why when at the loop, there is no filter for Magicnumber. 

After few tries, is not success yet, and will try again and again. Thanks guys. 

if you ever come across anything similar, hope you don't mind sharing the link or articles where i can study deeper. 

[Mohamad Zulhairi Baba]

sdahao36:

I am actually searching and trying to write an EA to help me to calculate 

1. Breakeven Point + few pips profit.

2. Modify my trades to the price above. 

These trades are manually entered, that's why when at the loop, there is no filter for Magicnumber. 

After few tries, is not success yet, and will try again and again. Thanks guys. 

if you ever come across anything similar, hope you don't mind sharing the link or articles where i can study deeper. 

Here is some ideas.
Just loop the OrdersTotal() and get OpenPrice & OrderLot of each order, plus Sum of OrderLot.

1. a) if total order = 1, breakeven price = open price.
   b) if total order > 1, breakeven price = ((orderlot1*openPrice1+orderlot2*openPrice2+....)*pipvalue) / totalOrderLot. 
2. Modify order, based on order direction. (buy/sell), above / below.

good luck.

ps: @Konstantin Nikitin did provide the code for breakeven_price, perhaps breakeven_price=((total_average_price*pipsvalue)/total_lots)

[Tan Chee Ho]

Mohamad Zulhairi Baba:

Here is some ideas.
Just loop the OrdersTotal() and get OpenPrice & OrderLot of each order, plus Sum of OrderLot.

1. a) if total order = 1, breakeven price = open price.
   b) if total order > 1, breakeven price = ((orderlot1*openPrice1+orderlot2*openPrice2+....)*pipvalue) / totalOrderLot. 
2. Modify order, based on order direction. (buy/sell), above / below.

good luck.

ps: @Konstantin Nikitin did provide the code for breakeven_price, perhaps breakeven_price=((total_average_price*pipsvalue)/total_lots)

Dear Mohamad,

I want to thank you for pointing out a direction where i can follow. Hope you don't mind asking you a question. 

I understand most of your given advice accept this "pipvalue". Do you have little more info about pipvalue i can refer to?

[Mohamad Zulhairi Baba]

sdahao36:

Dear Mohamad,

I want to thank you for pointing out a direction where i can follow. Hope you don't mind asking you a question. 

I understand most of your given advice accept this "pipvalue". Do you have little more info about pipvalue i can refer to?

if your deposit currency is in USD, and you trade USD based pair, example EURUSD, GBPUSD... pipsvalue = 1.
else if you trade other pair with different USD based currency, you need to accurately calculate every pip( point) movement. example, EURJPY, USDJPY etc...

double pipsvalue = SymbolInfoDouble(_Symbol,SYMBOL_TRADE_TICK_VALUE);

https://docs.mql4.com/constants/environment_state/marketinfoconstants

[William Roeder]

sdahao36: Do you have little more info about pipvalue

1. Pip:

   Using Point means code breaks on 4 digit brokers, exotics (e.g. USDZAR where spread is over 500 points,) and metals. Compute what a PIP is and use it, not points.
             How to manage JPY pairs with parameters? - MQL4 and MetaTrader 4 - MQL4 programming forum
             Slippage defined in index points - Currency Pairs - Expert Advisors and Automated Trading - MQL5 programming forum

2. Delta: Value per price change per lot:

   Risk depends on your initial stop loss, lot size, and the value of the pair.

   1. You place the stop where it needs to be - where the reason for the trade is no longer valid. E.g. trading a support bounce the stop goes below the support.
   2. Account Balance * percent/100 = RISK = OrderLots * (|OrderOpenPrice - OrderStopLoss| * DeltaPerLot + CommissionPerLot) (Note OOP-OSL includes the SPREAD, and DeltaPerLot is usually around $10/pip but it takes account of the exchange rates of the pair vs. your account currency.)
   3. Do NOT use TickValue by itself - DeltaPerLot and verify that MODE_TICKVALUE is returning a value in your deposit currency, as promised by the documentation, or whether it is returning a value in the instrument's base currency.
                MODE_TICKVALUE is not reliable on non-fx instruments with many brokers.
   4. You must normalize lots properly and check against min and max.
   5. You must also check FreeMargin to avoid stop out

   Most pairs are worth about $10 per PIP. A $5 risk with a (very small) 5 PIP SL is $5/$10/5=0.1 Lots maximum.