EA opening only BuY orders/ SELL NOT

[Stanislav Ivanov]

Hi, I`m posting a piece of code from my EA - it has a problem - its not opening any SELL orders.

if(!AreThereOrders() && order4_allow==true)
{    
if( fisher1>0 && cci0<-70 && stoch0>50 && rsi0>45)
{
         ticket=OrderSend(Symbol(),OP_BUY,lotsize_4,Ask,slippage,NormalizeDouble(Bid-(stoploss_order4*point),Digits),NormalizeDouble(Bid+(takeprofit_order4*point),Digits),NULL,magic4,0,clrGreen);
         if(ticket>0)
           {           
               Print("Order Opened BUY - 4!!");
           }
         else
            Print("Error occured BUY -4 not sent! Error code :",GetLastError());
            return;
}
else if( fisher1<0 && cci0>70 && stoch0<50 && rsi0<45)
{
 ticket=OrderSend(Symbol(),OP_SELL,lotsize_4,Bid , slippage , NormalizeDouble(Ask+(stoploss_order4*point),Digits ),NormalizeDouble(Ask-(takeprofit_order4*point),Digits),NULL , magic4,0,clrRed);
        if(ticket>0)
           {           
               Print("Order Opened SELL -4!!");
           }
         else
            Print("Error occured SELL -4 not sent! Error CODE:",GetLastError());   
}

}//

[William Roeder]

Stanislav Ivanov: I`m posting a piece of code from my EA - it has a problem - its not opening any SELL orders.

1. If your condition is working, what is your error code? No error code, your condition isn't working. Use the debugger or print out your variables, including _LastError and find out why.
2. Do NOT use NormalizeDouble, EVER. For ANY Reason. It's a kludge, don't use it. It's use is usually wrong
   • Print out your values to the precision you want with DoubleToString - Conversion Functions - MQL4 Reference.
   • SL/TP (stops) need to be normalized to tick size (not Point.) (On 5Digit Broker Stops are only allowed to be placed on full pip values. How to find out in mql? - MQL4 and MetaTrader 4 - MQL4 programming forum) and abide by the limits Requirements and Limitations in Making Trades - Appendixes - MQL4 Tutorial and that requires understanding floating point equality Can price != price ? - MQL4 and MetaTrader 4 - MQL4 programming forum
   • Open price for pending orders need to be adjusted. On Currencies, Point == TickSize, so you will get the same answer, but it won't work on Metals. So do it right: Trailing Bar Entry EA - MQL4 and MetaTrader 4 - MQL4 programming forum or Bid/Ask: (No Need) to use NormalizeDouble in OrderSend - MQL4 and MetaTrader 4 - MQL4 programming forum
   • Lot size must also be adjusted to a multiple of LotStep and check against min and max. If that is not a power of 1/10 then NormalizeDouble is wrong. Do it right.

[Stanislav Ivanov]

thank you