Function not finding lowest BuyPosition - why? | Forex Factory

Options

Permalinks
Print Thread

Similar Threads

Why is so difficult to find this function on a trade manager? 0 replies

required to function call function 2 replies

Need help finding the broker with lowest spread on EUR/NZD 6 replies

MT4 Lost Horizontal Line Function - Any Clues Why? 4 replies

stop loss function and edit time function for Breakout EA 1 reply

Function not finding lowest BuyPosition - why?

  • #1
  • First Post: Jan 21, 2014 4:31am Jan 21, 2014 4:31am
Hey folks.

I must have made a very stupid mistake, but I can not figure it out on my own.
I did make myself a function to find the Highest/Lowest Buy/Sell Orders.
Everything works fine, but the value for the "LowestBuy". LowestBuy always returns same value as HighestBuy. WHY?
This is driving me nuts!

Inserted Code
//Global variables needed:
/*
double            HighestBuy = 0;
double            LowestBuy = 0;
double            HighestSell = 0;
double            LowestSell = 0;
*/
void GetOpenOrderPriceForMostRecent ()
   {
   HighestBuy = 0;
   LowestBuy = 0;
   HighestSell = 0;
   LowestSell = 0;
   int               TotalOrders;
   TotalOrders = OrdersTotal();
   for(int iii=TotalOrders-1;iii>=0;iii--)
      {
      OrderSelect(iii, SELECT_BY_POS);
      if(OrderSymbol() != Symbol()) continue;
      if(OrderType() == OP_BUY && OrderOpenPrice() > HighestBuy)
         {
         HighestBuy = OrderOpenPrice();
         }
      if(OrderType() == OP_BUY && OrderOpenPrice() < LowestBuy || OrderType() == OP_BUY && LowestBuy == 0)
         {
         LowestBuy = OrderOpenPrice();
         }
      if (OrderType() == OP_SELL && OrderOpenPrice() > HighestSell)
         {
         HighestSell = OrderOpenPrice();
         }
      if (OrderType() == OP_SELL && OrderOpenPrice() < LowestSell  || OrderType() == OP_SELL && LowestSell == 0)
         {
         LowestSell = OrderOpenPrice();
         }
      }
   }
  • #2
  • Jan 21, 2014 4:59am Jan 21, 2014 4:59am
Change this

if(OrderType() == OP_BUY && OrderOpenPrice() < LowestBuy || OrderType() == OP_BUY && LowestBuy == 0)

to

if(OrderType() == OP_BUY && (OrderOpenPrice() < LowestBuy || OrderType() == OP_BUY && LowestBuy == 0))

and vice versa for sell orders.
  • #3
  • Jan 21, 2014 5:10am Jan 21, 2014 5:10am
Thank you! But that's not working!
Still, your idea seemed to make sense in some ways.

This is doing the trick:
Inserted Code
if(OrderType() == OP_BUY && (OrderOpenPrice() < LowestBuy || LowestBuy == 0))

The question now is: Why was the original code forfing for LowestSell, but not for LowestBuy? It's not making sense, even if the problem is solved...
  • #4
  • Jan 21, 2014 8:16am Jan 21, 2014 8:16am
Sorry, I don't quite understand.
What is and what isn't working?
What does each variable return and what do you want them to return?
  • #5
  • Jan 21, 2014 8:38am Jan 21, 2014 8:38am
Sorry, my bad. Missed a bracket.

Try this, it worked for me when I tested it.

if(OrderType() == OP_BUY && (OrderOpenPrice() < LowestBuy || (OrderType() == OP_BUY && LowestBuy == 0)))

and same for sell orders

if (OrderType() == OP_SELL && (OrderOpenPrice() < LowestSell || (OrderType() == OP_SELL && LowestSell == 0)))
  • #6
  • Jan 23, 2014 10:36am Jan 23, 2014 10:36am
Sorry for not being clear.

My original code did give expected results for LowestSell, but not for LowestBuy.

Your suggestions with the brackets works with both!
I still am wondering, why "LowestBuy" was not working as I've written the code, but "LowestSell" was. I'd like to avoid such problems in the future!

Thank you for your help!
  • #7
  • Jan 24, 2014 2:42am Jan 24, 2014 2:42am
Quoting MikeFT
Sorry for not being clear. My original code did give expected results for LowestSell, but not for LowestBuy. Your suggestions with the brackets works with both! I still am wondering, why "LowestBuy" was not working as I've written the code, but "LowestSell" was. I'd like to avoid such problems in the future! Thank you for your help!
Are you sure that LowestSell was working as expected?
I believe that your problem was related to the precedence rules and to be honest, I always have difficulties with precedence when using && and ||.
http://docs.mql4.com/basis/operations/rules
That is why I will always use brackets, just to be sure that I will get the expected result.
I understand precedence quite well with multiplication, adding and subtracting.
Eg.
Inserted Code
      int number = 2*2 + 2*2+1;
      Alert(number);
will result in 9 because the +1 is calculated last
Inserted Code
      int number = 2*2 + 2*(2+1);
      Alert(number);
will result in 10 because the brackets ensure that (2+1) is given precedence

This suggest to me that in your code
Inserted Code
[color][font] if(OrderType() == OP_BUY && OrderOpenPrice() < LowestBuy || OrderType() == OP_BUY && LowestBuy == 0)[/font][/color]
that && LowestBuy == 0 is left out on its own, so is always part of the if condition. In other words the if condition will only be met if LowestBuy == 0
Best to be safe and get into the habit of always using brackets.
Please Do Not PM Me With Coding Enquiries
  • #8
  • Last Post: Jan 29, 2014 2:21am Jan 29, 2014 2:21am
Thank you.

That makes sense! Will keep this in mind for the future.
0 traders viewing now