Little problem

Back to topics list To post a new topic, please log in or register
avatar
5
looper 2015.03.25 13:10
 

hi,

i am trying to  make an alert with laguerre indicator

the alert show a message when laguerre is 1 or 0

the firs part il laguerre indicator, the sencond part il the alert which dosen't work

it shows me only over sold alert  and this alertisn't even correct 

 

tranks 

#property indicator_separate_window
#property  indicator_buffers 1
#property indicator_color1 Red
#property indicator_level1 0.80
#property indicator_level2 0.20
#property indicator_minimum 0
#property indicator_maximum 1

//---- input parameters
extern double gamma=0.7;
extern int CountBars=950;

double L0 = 0;
double L1 = 0;
double L2 = 0;
double L3 = 0;
double L0A = 0;
double L1A = 0;
double L2A = 0;
double L3A = 0;
double LRSI = 0;
double CU = 0;
double CD = 0;

datetime Timee;

double val1[];
//+------------------------------------------------------------------+
//| Custom indicator initialization function                         |
//+------------------------------------------------------------------+
      
int init()
  {
//---- indicators
//----
   SetIndexBuffer(0,val1);
   return(0);
  }
//+------------------------------------------------------------------+
//| Custor indicator deinitialization function                       |
//+------------------------------------------------------------------+
int deinit()
  {
//---- TODO: add your code here
   
//----
   return(0);
  }
//+------------------------------------------------------------------+
//| Custom indicator iteration function                              |
//+------------------------------------------------------------------+
int start()
  {
   if (CountBars>Bars) CountBars=Bars;
   SetIndexDrawBegin(0,Bars-CountBars);
   
   int i;
   int    counted_bars=IndicatorCounted();

   //if(CountBars<=Lookback) return(0);
   //---- initial zero
   //if(counted_bars<1)
   //{
   //   for(i=1;i<=Lookback;i++) val1[CountBars-i]=0.0;
   //}

   i=CountBars-1;
   while(i>=0)
   {
      L0A = L0;
      L1A = L1;
      L2A = L2;
      L3A = L3;
      L0 = (1 - gamma)*Close[i] + gamma*L0A;
      L1 = - gamma *L0 + L0A + gamma *L1A;
      L2 = - gamma *L1 + L1A + gamma *L2A;
      L3 = - gamma *L2 + L2A + gamma *L3A;

      CU = 0;
      CD = 0;
      
      if (L0 >= L1) CU = L0 - L1; else CD = L1 - L0;
      if (L1 >= L2) CU = CU + L1 - L2; else CD = CD + L2 - L1;
      if (L2 >= L3) CU = CU + L2 - L3; else CD = CD + L3 - L2;

      if (CU + CD != 0) LRSI = CU / (CU + CD);
      val1[i] = LRSI;
          i--;
        }

   if(val1==0 && Timee!=Time[0])
{
   Alert("Over Sold "+Symbol()+" TF: "+Period());
}

   if(val1==1 && Timee!=Time[0])
{
   Alert("Over Bought "+Symbol()+" TF: "+Period());
}
   return(0);
 }

 

avatar
12983
WHRoeder 2015.03.25 13:28 #
 
Post code that actually compiles and you might know why.
avatar
5
looper 2015.03.25 14:23 #
 
i don't understand
avatar
735
honest_knave 2015.03.25 18:05 #
 

This is an array:

double val1[];

 Which is bound to a buffer: 

SetIndexBuffer(0,val1);

 This is not how to access an array:

 if(val1==0 && Timee!=Time[0])

or like this:

 if(val1==1 && Timee!=Time[0])

Which is why the code will not even compile in Metaeditor (as WHRoeder points out)

 

Back to topics list  

To add comments, please log in or register