Limiting number of trades per candle

Jan-HarmZ63 2021.01.28 10:12

Good day,

I am trying to limit the number of trades per candle to 1. I am using a method that has previously worked for me but I can't figure out why it doesn't work in the function below:

void BuyAndSell()
{
        static int CandleOpenTime;
        if((CandleOpenTime==iTime("XX",PERIOD_M5,0)))
            {
               Print("It is NOT a new candle");
               return;
            }
            else
            
               {  

                   if( // First Buy condition
                        (
                           (MorningStar()==true)
                           &&(CandleOpenTime!=iTime("XX",PERIOD_M5,0))
                           &&(MathAbs(TrendChannelGradient)>=TrendChannelGradientReq)
                           &&(MathAbs(QuickChannelGradient)>=QuickChannelGradientReq)
                           &&(QuickChannelValue1<QuickChannelValue2)
                           &&(TrendChannelValue1<TrendChannelValue2)
                        )
                     )
                  
                       {
                        CloseSELL();
                        if(CanITrade())
                        {
                           int lowV = iLowest("XX",PERIOD_M5,MODE_LOW,3,0);
                           trade.Buy(LotSize,NULL,ask,(iLow("XX",PERIOD_M5,lowV)-(ATR_Array[0]*0.1*ATRMultiplier*pips)),ask+(TakePofitPips*pips),NULL);
                           CandleOpenTime = iTime("XX",PERIOD_M5,0);//If it was a new candle, save the new time in CandleOpenTime
                        }
                        
                       }
                  
                  //+------------------------------------------------------------------+
                  //|                    Sell Conditions                               |
                  //+------------------------------------------------------------------+
                  
                  if(//First Sell condition
                        (
                           (EveningStar()==true)
                           &&(CandleOpenTime!=iTime("XX",PERIOD_M5,0))
                           &&(MathAbs(TrendChannelGradient)>=TrendChannelGradientReq)
                           &&(MathAbs(QuickChannelGradient)>=QuickChannelGradientReq)
                           &&(QuickChannelValue1>QuickChannelValue2)
                           &&(TrendChannelValue1>TrendChannelValue2)
                        )
                     )
                  
                        {
                           CloseBUY();
                           if(CanITrade())
                           {           
                           int highV = iHighest("XX",PERIOD_M5,MODE_HIGH,3,0);
                           trade.Sell(LotSize,NULL,bid,(iHigh("XX",PERIOD_M5,highV)+(ATR_Array[0]*0.1*ATRMultiplier*pips)),bid-(TakePofitPips*pips),NULL);      
                           CandleOpenTime = iTime("XX",PERIOD_M5,0);//If it was a new candle, save the new time in CandleOpenTime
                           }
                        }
                        
               }          

}

I am calling void BuyAndSell () in void OnTick ().

Any constructive help will be appreciated

Flavio Javier Jarabeck 2021.01.28 12:33 #1

Jan-HarmZ63:

Good day,

I am trying to limit the number of trades per candle to 1. I am using a method that has previously worked for me but I can't figure out why it doesn't work in the function below:

I am calling void BuyAndSell () in void OnTick ().

Any constructive help will be appreciated

If you read carefully the documentation of iTime() you will find the reason of the failure of your code...

;)

https://www.mql5.com/en/docs/series/itime

Documentation on MQL5: Timeseries and Indicators Access / iTime

www.mql5.com

iTime - Timeseries and Indicators Access - MQL5 Reference - Reference on algorithmic/automated trading language for MetaTrader 5

Jan-HarmZ63 2021.01.28 12:47 #2

Flavio Jarabeck:

If you read carefully the documentation of iTime() you will find the reason of the failure of your code...

;)

https://www.mql5.com/en/docs/series/itime

Hi Flavio,

Thanks for the reply. Are you referring to the fact that I saved the time to an "int" and not "datetime"?

If not can you please spoon feed me a little? :-)

Joe Varcoe 2021.01.28 16:15 #3

Jan-HarmZ63:

Good day,

I am trying to limit the number of trades per candle to 1. I am using a method that has previously worked for me but I can't figure out why it doesn't work in the function below:

I am calling void BuyAndSell () in void OnTick ().

Any constructive help will be appreciated

Why 'XX' in your iTime? Why are you not including the symbol?

Jan-HarmZ63 2021.01.28 16:28 #4

Joe Varcoe:

Why 'XX' in your iTime? Why are you not including the symbol?

I just removed the Symbol name and added "XX" as a placeholder for this post.

Sorry for the confusion.

William Roeder 2021.01.28 17:00 #5

Jan-HarmZ63: I just removed the Symbol name and added "XX" as a placeholder for this post.

Why not just use _Symbol?

Jan-HarmZ63 2021.01.28 17:19 #6

William Roeder:

Why not just use _Symbol?

Sorry for the confusion.

Any idea what could be wrong with the functions behind CandleOpenTime and trying to to limit trades to 1 per candle?

Marco vd Heijden 2021.01.28 17:38 #7

Did you look up the return value of iTime() yet ?

datetime  iTime(
   string           symbol,          // symbol
   int              timeframe,       // timeframe
   int              shift            // shift
   );

Jan-HarmZ63 2021.01.28 18:46 #8

Re-posting code to clear the confusion created on my part. Sorry for that

void BuyAndSell()
{
        static datetime CandleOpenTime;
         Print(
               "Candle Open Time is: " + CandleOpenTime
             + "\n" + "iTime is: " + " " + iTime(_Symbol,PERIOD_M5,0)
              );

        if((CandleOpenTime==iTime(_Symbol,PERIOD_M5,0)))
            {
               Print("It is NOT a new candle");
               return;
            }
            else
            
               {  

                   if( // First Buy condition
                        (
                           (MorningStar()==true)
                           &&(CandleOpenTime!=iTime(_Symbol,PERIOD_M5,0))
                           &&(MathAbs(TrendChannelGradient)>=TrendChannelGradientReq)
                           &&(MathAbs(QuickChannelGradient)>=QuickChannelGradientReq)
                           &&(QuickChannelValue1<QuickChannelValue2)
                           &&(TrendChannelValue1<TrendChannelValue2)
                        )
                     )
                  
                       {
                        CloseSELL();
                        if(CanITrade())
                        {
                           int lowV = iLowest(_Symbol,PERIOD_M5,MODE_LOW,3,0);
                           trade.Buy(LotSize,NULL,ask,(iLow(_Symbol,PERIOD_M5,lowV)-(ATR_Array[0]*0.1*ATRMultiplier*pips)),ask+(TakePofitPips*pips),NULL);
                           CandleOpenTime = iTime(_Symbol,PERIOD_M5,0);//If it was a new candle, save the new time in CandleOpenTime
                        }
                        
                       }
                  
                  //+------------------------------------------------------------------+
                  //|                    Sell Conditions                               |
                  //+------------------------------------------------------------------+
                  
                  if(//First Sell condition
                        (
                           (EveningStar()==true)
                           &&(CandleOpenTime!=iTime(_Symbol,PERIOD_M5,0))
                           &&(MathAbs(TrendChannelGradient)>=TrendChannelGradientReq)
                           &&(MathAbs(QuickChannelGradient)>=QuickChannelGradientReq)
                           &&(QuickChannelValue1>QuickChannelValue2)
                           &&(TrendChannelValue1>TrendChannelValue2)
                        )
                     )
                  
                        {
                           CloseBUY();
                           if(CanITrade())
                           {           
                           int highV = iHighest(_Symbol,PERIOD_M5,MODE_HIGH,3,0);
                           trade.Sell(LotSize,NULL,bid,(iHigh(_Symbol,PERIOD_M5,highV)+(ATR_Array[0]*0.1*ATRMultiplier*pips)),bid-(TakePofitPips*pips),NULL);      
                           CandleOpenTime = iTime(_Symbol,PERIOD_M5,0);//If it was a new candle, save the new time in CandleOpenTime
                           }
                        }
                        
               }          

}

position objects using time Candle countin' of candles Trailing Stop not working

Flavio Javier Jarabeck 2021.01.28 22:56 #9

Jan-HarmZ63:

Hi Flavio,

Thanks for the reply. Are you referring to the fact that I saved the time to an "int" and not "datetime"?

If not can you please spoon feed me a little? :-)

READ the WHOLE Documentation!

"The function always returns actual data. For this purpose it performs a request to the timeseries for the specified symbol/period during each call. This means that if there is no ready data during the first function call, some time may be taken to prepare the result.

The function does not store previous calls results, and there is no local cache for quick value return."

Probably you are getting Random fails... using your code like this...

Loop to count consecutive iClose iHigh

Jan-HarmZ63 2021.01.29 09:21 #10

Flavio Jarabeck:

READ the WHOLE Documentation!

The function does not store previous calls results, and there is no local cache for quick value return."

Probably you are getting Random fails... using your code like this...

Hi Flavio,

Thanks for this. You are hitting the nail on the head. Probably also why the problem seems to occur more frequent when running a strategy test.

Thank you!!

New comment