Double CCI with line and histogram for MT5

[intraforex]

Originally Posted by Enivid

Not exactly, but it looks similar. Do you have the code?

Yes,thi is the code only histogram.
1)Add the code (CCI_HISTO.mql5), to chart
2)Insert in indicator (CCI_HIsto) separate windows one CCI indicator standard at 14 period.
3)Insert in indicator (CCI_HIsto) separate windows one CCI indicator standard at 7 period.

I think you see that my programming is not perfect but it works
I would want to improve the CCI inserting other instructions: you can help?
Thanks

[Enivid]

I can try to help .

[intraforex]

Originally Posted by Enivid

I can try to help .

I would like Insert inside of the indicator the value of CCI like the example.
I am interested above all the procedure for viewing... for the values we ca n't.
Thanks for help.

Example :
http://clip2net.com/clip/m19204/1262...-clip-25kb.png

[intraforex]

[Enivid]

Originally Posted by intraforex

I would like Insert inside of the indicator the value of CCI like the example.
I am interested above all the procedure for viewing... for the values we ca n't.
Thanks for help.

Example :
http://clip2net.com/clip/m19204/1262...-clip-25kb.png

I am not sure that you can insert text labels into the separate indicator window.

[fxwalb]

Originally Posted by Enivid

I am not sure that you can insert text labels into the separate indicator window.

ObjectCreate(0, "myobject_name", OBJ_LABEL, 1, 0, 0);

This expression should create the object in the indicator window, but this feature is not working (yet).

[intraforex]

How can I assign to a variable the price of the close of a candle or bar ?
thanks.

[Enivid]

Originally Posted by intraforex

How can I assign to a variable the price of the close of a candle or bar ?
thanks.

In indicator's event handler OnCalculate() you have an access to array Close[], so you can just write:

double a = Close[0];

to assign the close value of the latest candle (the first if it's not set as tiemseries) to variable "a".

[intraforex]

Someone helps me with this simple indicator?
Arrow Green when MA1>MA2
Arrow Red when MA1<MA2

#property copyright "2009, MetaQuotes Software Corp."
#property link      "http://www.mql5.com"
#property version   "1.00"
#property indicator_separate_window

#property indicator_buffers 2
#property indicator_plots  2

#property indicator_label1  "ARROW1"
#property indicator_type1   DRAW_ARROW
#property indicator_style1  STYLE_SOLID
#property indicator_color1  Green
#property indicator_width1  1

#property indicator_label2  "ARROW2"
#property indicator_type2   DRAW_ARROW
#property indicator_style2  STYLE_SOLID
#property indicator_color2  Red
#property indicator_width2  1
//--- input parameters
 
double  MA1;
double  MA2;
double Arrow1[];
double Arrow2[];

//+------------------------------------------------------------------+
//| Custom indicator initialization function                         |
//+------------------------------------------------------------------+
int OnInit()
  {
//--- indicator buffers mapping
 SetIndexBuffer(0,Arrow1,INDICATOR_DATA);
 SetIndexBuffer(1,Arrow2,INDICATOR_DATA);
 //--- indicator name
IndicatorSetString(INDICATOR_SHORTNAME,"TEST");

//---- OnInit done
MA1=iMA(Symbol(),0,34,0,MODE_EMA,PRICE_CLOSE);
MA2=iMA(Symbol(),0,10,0,MODE_EMA,PRICE_CLOSE);
//---
   return(0);
  }
//+------------------------------------------------------------------+
//| Custom indicator iteration function                              |
//+------------------------------------------------------------------+
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 i ;
  for(i=1;i<=rates_total;i--);
  
 if (MA2>MA1);
     {Arrow1[i]; }
 
 if (MA2<MA1);
    {Arrow2[i]; }  
 
//--- return value of prev_calculated for next call
   return(rates_total);
  }
//+------------------------------------------------------------------+

Where is error ?
thanks to all

[Enivid]

1. With iMA() function you get only indicator handle, not the buffer. You should copy the buffer to get the actual vales:

  double MA1Buffer[];
   CopyBuffer(MA1, 0, 0, rates_total, MA1Buffer);
  double MA2Buffer[];
   CopyBuffer(MA2, 0, 0, rates_total, MA2Buffer);

Do this inside your OnCalculate() handler.

2. Then you need compare the buffer values rather than handles when determining the arrow. You also need to give your arrow buffers the actual values rather than just stating them:

 if (MA2Buffer[i]>MA1MABuffer[i]);
     {Arrow1[i] = MA2Buffer[i]; }
 
 if (MA2Buffer[i]<MA1Buffer[i]);
    {Arrow2[i] = MA2Buffer[i]; }

3. When you have this working you may want to modify this code to draw arrows only on MA cross, not on all bars, but I don't know if that's relevant for you.