EA not triggering buy or sell trades

[catsoda]

Hi all,

I'm relatively new to coding, but can someone take a look at this code and tell my if there's a reason the trades aren't triggering in back testing? I'm getting no errors back from MetaEditor, and the defined variables are how I'd like them. Just not getting any trades lighting up in MT5 when back testing. Any clues would be greatly appreciated. 

Merci!

// Define input parameters

input double lots = 0.1;
input int fastEMA = 5;
input int mediumEMA = 10;
input int slowEMA = 50;
input int RSIperiod = 14;
input int RSIupper = 83;
input int RSIlower = 17;
input int slippage = 3;
input double stopLoss = 50.0;
input double takeProfit = 150.0;
input int barsAndPrice = 50;

// Define global variables
double lastBuyPrice = 0;
double lastSellPrice = 0;

// Define initialization function
void OnInit()
{
   
}



// Define trading function
void OnTick()
{
     // Get the current price
     double price = SymbolInfoDouble(_Symbol, SYMBOL_BID);

     // Calculate EMAs
     double fastMA = iMA(_Symbol, PERIOD_CURRENT, fastEMA, 0, MODE_EMA, PRICE_CLOSE);
     double mediumMA = iMA(_Symbol, PERIOD_CURRENT, mediumEMA, 0, MODE_EMA, PRICE_CLOSE);
     double slowMA = iMA(_Symbol, PERIOD_CURRENT, slowEMA, 0, MODE_EMA, PRICE_CLOSE);

    // Calculate RSI
    double rsi = iRSI(_Symbol, 0, RSIperiod, PRICE_CLOSE);

    // Check if there's an open position
    bool isBuyPosition = PositionGetInteger(POSITION_TYPE) == POSITION_TYPE_BUY;
    bool isSellPosition = PositionGetInteger(POSITION_TYPE) == POSITION_TYPE_SELL;

    // Check if previous buy/sell signal was within 50 candles and 50 pips
    bool isWithinRange = false;
    if(isBuyPosition)
    {
        if(lastBuyPrice > 0 && (price - lastBuyPrice) <= barsAndPrice)
        {
            isWithinRange = true;
        }
    }
    else if(isSellPosition)
    {
        if(lastSellPrice > 0 && (lastSellPrice - price) <= barsAndPrice)
        {
            isWithinRange = true;
        }
    }

    // Place a buy trade if the EMAs are in the right order and there's no open position and RSI is below the lower level
    if(fastMA > mediumMA && mediumMA > slowMA && !isBuyPosition && !isWithinRange && rsi < RSIupper)
    {
         lastBuyPrice = price;
        MqlTradeRequest request = {};
        MqlTradeResult result;
        request.action = TRADE_ACTION_DEAL;
        request.symbol = _Symbol;
        request.volume = lots;
        request.price = price;
        request.deviation = slippage;
        request.type_filling = ORDER_FILLING_FOK;
        request.type = ORDER_TYPE_BUY;
        request.tp = price + (takeProfit * _Point);
        request.sl = price - (stopLoss * _Point);
        request.comment = "Buy order";
        request.magic = 123456;
        if(OrderSend(request,result))
        {
            Print("Buy order opened with ticket #", result.order);
        }
        else
        {
            Print("Error opening buy order: ", result.retcode);
        }
    }

    // Place a sell trade if the EMAs are in the right order and there's no open position and RSI is above the upper level
      if(fastMA < mediumMA && mediumMA < slowMA && !isSellPosition && !isWithinRange && rsi > RSIlower)
      {
      
       lastSellPrice = price;
       MqlTradeRequest request = {};
       request.action = TRADE_ACTION_DEAL;
       request.symbol = _Symbol;
       request.volume = lots;
       request.price = price;
       request.deviation = slippage;
       request.type_filling = ORDER_FILLING_FOK;
       request.type = ORDER_TYPE_SELL;
       request.tp = price + (takeProfit * _Point);
       request.sl = price - (stopLoss * _Point);
       request.comment = "Sell order";
       request.magic = 1;
       MqlTradeResult result;
         if(OrderSend(request, result))
    {
        Print("Sell order opened with ticket #", result.order);
    }
         else
    {
        Print("Error opening sell order: ", result.retcode);
    }
}
}

[Alain Verleyen]

ChatGPT code ?

[William Roeder]

1. void OnTick()
   {
        // Get the current price
        double price = SymbolInfoDouble(_Symbol, SYMBOL_BID);
   
        // Calculate EMAs
        double fastMA = iMA(_Symbol, PERIOD_CURRENT, fastEMA, 0, MODE_EMA, PRICE_CLOSE);

   Perhaps you should read the manual, especially the examples.
      How To Ask Questions The Smart Way. (2004)
         How To Interpret Answers.
            RTFM and STFW: How To Tell You've Seriously Screwed Up.

   They all (including iCustom) return a handle (an int). You get that in OnInit. In OnTick/OnCalculate (after the indicator has updated its buffers), you use the handle, shift and count to get the data.
             Technical Indicators - Reference on algorithmic/automated trading language for MetaTrader 5
             Timeseries and Indicators Access / CopyBuffer - Reference on algorithmic/automated trading language for MetaTrader 5
             How to start with MQL5 - General - MQL5 programming forum - Page 3 #22 (2020)
             How to start with MQL5 - MetaTrader 5 - General - MQL5 programming forum - Page 7 #61 (2020)
             MQL5 for Newbies: Guide to Using Technical Indicators in Expert Advisors - MQL5 Articles (2010)
             How to call indicators in MQL5 - MQL5 Articles (2010)

2.     bool isBuyPosition = PositionGetInteger(POSITION_TYPE) == POSITION_TYPE_BUY;
       bool isSellPosition = PositionGetInteger(POSITION_TYPE) == POSITION_TYPE_SELL;

   No positions selected, bogus calls
   

3. Stop using ChatGPT.
             Help needed to debug and fix an AI EA - Trading Systems - MQL5 programming forum (2023)

[catsoda]

Alain Verleyen #:
ChatGPT code ?

Yes sir, but it's got me a basic framework of what I need, however don't understand what's missing. Can you help?

[Alain Verleyen]

catsoda #:

Yes sir, but it's got me a basic framework of what I need, however don't understand what's missing. Can you help?

No, your code doesn't make sense. It mixed mql4 and mql5.

[Fernando Carreiro]

catsoda #: Yes sir, but it's got me a basic framework of what I need, however don't understand what's missing. Can you help?

You also have the option to hire a programmer in the Freelance section. Given that you don't know how to code, it may be the better option.

[catsoda]

Alain Verleyen #:
No, your code doesn't make sense. It mixed mql4 and mql5.

I had a feeling that might be the case as I've spent all afternoon fixing errors in MetaEditor. Got rid of all the errors, so thought that may be it, but obvs more to it than that.