urgent help needed please

1. Do not call GetLastError() (or _LastError) unless you have an error. Delete that Print, it is after the loop and is useless.
2. In the presence of multiple orders (one EA multiple charts, multiple EA's, manual trading) you must count down when closing/deleting/modifying in a position loop. Get in the habit of always counting down. Loops and Closing or Deleting Orders - MQL4 forum

3.          if( Type == OP_BUY )
            if(OrderMagicNumber()!= magicbuy )continue;     // Another security

   This rejects orders with another magic only if Type == OP_BUY. Otherwise it ignores the magic. Shouldn't you be testing against OrderType()
   

   ---

4. if(Opn_B==false)continue;

   You would never write if( (2+2 == 4) == true) would you? if(2+2 == 4) is sufficient. So Don't write if(bool == true), just use if(bool) or if(!bool). Code becomes self documenting when you use meaningful variable names, like bool isLongEnabled. Long_Entry sounds like a trigger price or a ticket number and "if long entry" is an incomplete sentence.

   ---

5. Your code

   I would simplify

   for(int i=0; i < OrdersTotal(); i++) // Loop through orders   {    if(OrderSelect(i,SELECT_BY_POS, MODE_TRADES)) // If there is the next one      {         if(OrderSymbol()!=Symb)continue;     // Another security       if( Type == OP_BUY )       if(OrderMagicNumber()!= magicbuy )continue;     // Another security       if(OrderLots() > lastTradeLots && OrderTicket() > Ticket && OrderOpenPrice() > OpenPrice )       if(Opn_B==false)continue;     Total++;

   if(Opn_B && Type == OP_BUY) // Constant inside the loop. for(int i=OrdersTotal() - 1; i >= 0; --i) if(    OrderSelect(i,SELECT_BY_POS) && OrderMagicNumber() == magicbuy && OrderSymbol()      == Symb && OrderLots()        > lastTradeLots && OrderTicket()      > Ticket && OrderOpenPrice()   > OpenPrice ){   Total++;

thanks WHRoeder for your correction well correlated. now......

why i write my ea separately is because of debugging errors and also to be able to use different magic numbers because of my lotstep

1.even though i know there are better ways. i know i will bring out time to joint my ea to one single ea and not handle orders separately.

2. i started call for error when i discovered that my sell loop works but  buying loop on backtest and live test fails with

error 4105 meaning "no selected order".

Now WHRoeder like you corrected can i make the code fit my style like this below..........can it work this way

remember i buy low

// Orders accounting 
     Symb=Symbol();                           
     OpenPrice=0;
     Ticket =0;
      lastTradeLots=0;
     
for(int i=OrdersTotal() - 1; i >= 0; --i) 
if( OrderSelect(i,SELECT_BY_POS)
&& OrderMagicNumber() == magicbuy
&& OrderType()        == OP_BUY                 <===============================
&& OrderSymbol()      == Symb
&& OrderLots()        > lastTradeLots 
&& OrderTicket()      > Ticket 
&& OrderOpenPrice()   > OpenPrice               <===============================        
)
  Total++;
  {  
     }
      { 


//My criteria for opening buy orders
 
  rsi1=iRSI(NULL,0,14,PRICE_OPEN,0);  
  Opn_B=rsi1<=30 && Ask<=OpenPrice;              <===============================




                                                                       or can i do it like this will it work 


// Orders accounting 
     Symb=Symbol();                           
     OpenPrice=0;
     Ticket =0;
      lastTradeLots=0;


if(Opn_B && Type == OP_BUY &&  rsi1<=30 && Ask<=OpenPrice) // Constant inside the loop.
for(int i=OrdersTotal() - 1; i >= 0; --i) if(
   OrderSelect(i,SELECT_BY_POS)
&& OrderMagicNumber() == magicbuy
&& OrderSymbol()      == Symb
&& OrderLots()        > lastTradeLots 
&& OrderTicket()      > Ticket 
&& OrderOpenPrice()   > OpenPrice 
){
  Total++;

1. I have no idea. Your code is unreadable.

   )
     Total++;
     {  
        }
         { 

   If ends ")" but only does Tottal++. Empty "{}" does nothing. Opening "{" starts a block independent of the loop. Unreadable.
2. "no selected order" has been answered - Don't look at LastError after the fact.

That is one good thing they have done here that I have seen in practice in Java (and I presume C variations do as well, as Java is based on it) is the whitespace allowed for programming clarity, instead of the strict spacing rules that some other programming languages have.

1.am sorry actually the braces is doing nothing there,i have remove it.

2. also second question answered work perfectly ok......thanks i appreciate

final step question pls WHRoeder in my

//My criteria for opening buy orders
 
  rsi1=iRSI(NULL,0,14,PRICE_OPEN,0);  
  Opn_B=rsi1<=30 && Ask <= OpenPrice;              <===============================

i don't know trying to get the value of my orderselect which is

OrderOpenPrice()   > OpenPrice 

openprice being a double that get its value from OrderOpenPrice()

and the advantage/value of openprice is  0  which is last position on the current chart.

and the value is gotten from orderselect if selected, so that i can use it on my whole code parts the way i like or seems fit.

because it resets the  variable  to 0 after each close ticket/positions on current chart.

now my question is why did   double "openprice" refuse to work with my code on buying position

 Opn_B=rsi1<=30 && Ask <= OpenPrice; <===============================

instead it work with 

Opn_B=rsi1<=30 && Ask <= OrderOpenPrice(); <===============================

which is variable and does not reset to 0 after each close order/position INSTEAD IT LOOKS for stopped price to continue.

But sell position loop WORKS well with the double "OpenPrice" and resets to 0 for another level in the current chart 

my question is why is it fails on buying levels low and work well on sell levels high. pls assist me with code examples as you normally do .thanks

OpenPrice=0;
:
&& OrderOpenPrice()   > OpenPrice          <== OrderOpenPrice is always greater than zero. Test does nothing.
:
Opn_B=rsi1<=30 && Ask<=OpenPrice;        <== Ask is always greater than zero. Opn_B always false.

OpenPrice=0;
:
&& OrderOpenPrice()   > OpenPrice          <== OrderOpenPrice is always greater than zero. Test does nothing.
:
Opn_B=rsi1<=30 && Ask<=OpenPrice;        <== Ask is always greater than zero. Opn_B always false.

ok....i thought before that

OrderOpenPrice()   > OpenPrice    : for buy high 

while OrderOpenPrice()   < OpenPrice  : for sell low    loops.....

but now i have pick corrections that  OrderOpenPrice() > OpenPrice is 0 for all OrderType().
will i say its constant not changing          thank you very much for the code example





      but WHRoeder why is 


//My criteria for opening buy orders 
 
  rsi1=iRSI(NULL,0,14,PRICE_OPEN,0);  
  Opn_B=rsi1<=30 && Ask < OpenPrice;     <====== do not open any order /FAILS

            WHILE

//My criteria for opening buy orders
 
  rsi1=iRSI(NULL,0,14,PRICE_OPEN,0);  
  Opn_B=rsi1<=30 && Ask < OrderOpenPrice();     <======  opens AN order /WORKS


BUT the right one is suppose to be OpenPrice since it got its value ALREADY from OrderOpenPrice()  
and also reset to 0 at any close levels for another entry point

why is that so or is there any solution to that or something am missing.....because 
OrderOpenPrice()  stores even the last close order/open on current chart and continues when price comeback to it.

so how can i use OpenPrice  instead of OrderOpenPrice()  .......thanks for your support already,

 

updated....processing........Done.

thanks WHRoeder i finally found the answer to this question on you previous answer 
and it was over helpful and quit easy.thanks again for your help God bless you.