Alert question

[intraforex]

On MT4 I had the code for trigger "alert once on the bar current":

static bool TriggerOnlyThisBar;

...............

...............

if (......trigger condition......) && TriggerOnlyThisBar!=Bars
{
Alert(Symbol()," TEST ");
TriggerOnlyThisBar = Bars;
}


on mt5 I have replaced Bars (MT4) with Bars (symbol (), period_current) but does not work.

Someone explains to me where it is error?

Thanks to all.

[Enivid]

What? Does it continue to trigger the alert without the new bar coming?

[intraforex]

Originally Posted by Enivid

What? Does it continue to trigger the alert without the new bar coming?

Exact.The alarm should be triggered only the first time there is the bar that is forming.
Example:
bar 5 minutes
trigger alarm at second minute
stop alarm until 5 minutes even if the signal comes back
...on MT4 working bat not in MT5:I do not understand ....
Can you help ?

[Enivid]

Can you copy the exact code of that part of your indicator? Maybe there is some syntax typo that causes this error.

[intraforex]

Originally Posted by Enivid

Can you copy the exact code of that part of your indicator? Maybe there is some syntax typo that causes this error.

Yes.This is the code:

#property copyright "Copyright © 2009"
#property link      ""
#property version   ""
#property description ""


#property indicator_chart_window
#property indicator_buffers 1
double CCIBUFFER[];

static bool TriggerOnlyThisBar;
//+------------------------------------------------------------------+
//| Custom indicator initialization function                         |
//+------------------------------------------------------------------+
void OnInit()
{
 
//---- indicator buffers mapping  
   SetIndexBuffer(0, CCIBUFFER);
   
}
//+------------------------------------------------------------------+
//| Custom CCI Arrows                                                |
//+------------------------------------------------------------------+
int OnCalculate(const int rates_total,
                const int prev_calculated,
                const datetime &time[],
                const double &open[],
                const double &High[],
                const double &Low[],
                const double &close[],
                const long &tick_volume[],
                const long &volume[],
                const int &spread[])
{
int handle;
handle = iCCI(NULL, PERIOD_M5, 14, PRICE_CLOSE);
int buffer_num=BarsCalculated(handle);
CopyBuffer(handle,0,0,buffer_num,CCIBUFFER);
ArraySetAsSeries(CCIBUFFER,true);  

   {
      
         if((CCIBUFFER[0] > 0)&&
            (CCIBUFFER[1] < 0)&&
            TriggerOnlyThisBar != Bars(Symbol(),PERIOD_M5)) 
    {
      Alert(Symbol()," SIGNAL DOWN ");
       TriggerOnlyThisBar = Bars(Symbol(),PERIOD_M5);
    }     
   }
   
         if ((CCIBUFFER[0] < 0)&&
             (CCIBUFFER[1] > 0)&&
             TriggerOnlyThisBar != Bars(Symbol(),PERIOD_M5))     
    {
      Alert(Symbol(),"SIGNAL UP ");
       TriggerOnlyThisBar = Bars(Symbol(),PERIOD_M5);
        }

   return(rates_total);
}

[Enivid]

You have an extra pair of "{}" in your code for some reason. But that's not the problem. The problem is that you don't take this condition - TriggerOnlyThisBar != Bars(Symbol(),PERIOD_M5) - in the brackets "()". Correct example:

if((CCIBUFFER[0] > 0)&&
            (CCIBUFFER[1] < 0)&&
            (TriggerOnlyThisBar != Bars(Symbol(),PERIOD_M5)))

[intraforex]

Originally Posted by Enivid

You have an extra pair of "{}" in your code for some reason. But that's not the problem. The problem is that you don't take this condition - TriggerOnlyThisBar != Bars(Symbol(),PERIOD_M5) - in the brackets "()". Correct example:

if((CCIBUFFER[0] > 0)&&
            (CCIBUFFER[1] < 0)&&
            (TriggerOnlyThisBar != Bars(Symbol(),PERIOD_M5)))

Enivid : you have tested the code ?
I do not work: the alarm continues to play until the completion of the bar....

[Enivid]

Originally Posted by intraforex

Enivid : you have tested the code ?
I do not work: the alarm continues to play until the completion of the bar....

Do you use it on M5?

EDIT: I've found the real problem . It shouldn't be bool. Make it int or long. And remove static, it's not needed if you declare it as a global variable.

[intraforex]

Originally Posted by Enivid

Do you use it on M5?

EDIT: I've found the real problem . It shouldn't be bool. Make it int or long. And remove static, it's not needed if you declare it as a global variable.

thanks Enivid ... Monday I'll try your changes and then I tell you if it works.

[intraforex]

I have modify the code:

#property indicator_chart_window
#property indicator_buffers 1
double CCIBUFFER[];

bool TriggerOnlyThisBar;
//+------------------------------------------------------------------+
//| Custom indicator initialization function                         |
//+------------------------------------------------------------------+
void OnInit()
{
 
//---- indicator buffers mapping  
   SetIndexBuffer(0, CCIBUFFER);
   
}
//+------------------------------------------------------------------+
//| Custom CCI Arrows                                                |
//+------------------------------------------------------------------+
int OnCalculate(const int rates_total,
                const int prev_calculated,
                const datetime &time[],
                const double &open[],
                const double &High[],
                const double &Low[],
                const double &close[],
                const long &tick_volume[],
                const long &volume[],
                const int &spread[])
{
int handle;
handle = iCCI(NULL, PERIOD_M3, 14, PRICE_CLOSE);
int buffer_num=BarsCalculated(handle);
CopyBuffer(handle,0,0,buffer_num,CCIBUFFER);
ArraySetAsSeries(CCIBUFFER,true);  

  if((CCIBUFFER[0] > 0)&&
     (CCIBUFFER[1] < 0)&&
     (TriggerOnlyThisBar != Bars(Symbol(),PERIOD_M3)));
      {      
      Alert(Symbol()," UP ");
       TriggerOnlyThisBar = Bars(Symbol(),PERIOD_M3);
      }     
   
   if((CCIBUFFER[0] < 0)&&
      (CCIBUFFER[1] > 0)&&
      (TriggerOnlyThisBar != Bars(Symbol(),PERIOD_M3))) 
      {
      Alert(Symbol()," DOWN ");
       TriggerOnlyThisBar = Bars(Symbol(),PERIOD_M3);
      }
   return(rates_total);
}

....but still sounds alarm continuous incessant on the all bar of trigger