![MQL5 - Language of trade strategies built-in the MetaTrader 5 client terminal](https://c.mql5.com/i/registerlandings/logo-2.png)
You are missing trading opportunities:
- Free trading apps
- Over 8,000 signals for copying
- Economic news for exploring financial markets
Registration
Log in
You agree to website policy and terms of use
If you do not have an account, please register
Good morning Mladen,
I don't understand, the indicator seems react with level 0 instead of SMA...what's wrong? It should show the signal of cci crossing the sma, not the 0 level.
I tried on m1 with period1=3 and period2=3Doc
Try out this one. It shows the moving average and CCI in the sub-window too so it can be controlled. The change is in the CopyBuffer() line
Doc Try out this one. It shows the moving average and CCI in the sub-window too so it can be controlled. The change is in the CopyBuffer() line
Ok I got it, the problem is the code
where obvious it's not appropriate for this and give me this problem:![](https://c.mql5.com/forextsd/forum/112/eurusdm1_3.png)
Sould be red signal there, but how to do in an easy method?
Ok I got it, the problem is the code
where obvious it's not appropriate for this and give me this problem:![](https://c.mql5.com/forextsd/forum/112/eurusdm1_3.png)
Sould be red signal there, but how to do in an easy method?Both buffers should be set as series then in order that that comparison works as it should. Here is the corrected version
this is abstract art, and you are the master
thanks
Both buffers should be set as series then in order that that comparison works as it should. Here is the corrected version
Another question about the code:
to add more SMA with different periods, do I need to make a void? So SimpleMAOnBuffer is useless, or am I wrong?
Another question about the code: to add more SMA with different periods, do I need to make a void? So SimpleMAOnBuffer is useless, or am I wrong?
No, you don't need to make a void
But you have to have an extra buffer (for results) for each new MA period you plan to use
No, you don't need to make a void But you have to have an extra buffer (for results) for each new MA period you plan to use
Before wrote my last post, I did try it but without find the problem![](https://c.mql5.com/forextsd/smiles/sad_smile.png)
The problem is here I think:
if(CopyBuffer (handle_cciValue1, 0, 0, 2, cciValue1) < 0)
return(0);
ArraySetAsSeries(cciValue2, true);
if(CopyBuffer (handle_cciValue2, 1, 0, 2, cciValue2) < 0)
return(0);
//-------
SimpleMAOnBuffer(rates_total, prev_calculated, period1+period2, period2, cciValue1, maValue1);
SimpleMAOnBuffer(rates_total, prev_calculated, period2+period3, period3, cciValue2, maValue2);Before wrote my last post, I did try it but without find the problem
The problem is here I think:
if(CopyBuffer (handle_cciValue1, 0, 0, 2, cciValue1) < 0)
return(0);
ArraySetAsSeries(cciValue2, true);
if(CopyBuffer (handle_cciValue2, 1, 0, 2, cciValue2) < 0)
return(0);
//-------
SimpleMAOnBuffer(rates_total, prev_calculated, period1+period2, period2, cciValue1, maValue1);
SimpleMAOnBuffer(rates_total, prev_calculated, period2+period3, period3, cciValue2, maValue2);It should be something like this one
hey mladen can you modify this indicator to show a daily open line aswell? at 00:00 gmt maybe add timeshift option
It should be something like this one
Thank you very much!
just to learn something...why here:
copybuffer for the second has number 0?
From mql5 website:
after the indicator handle there is this: " int buffer_num, // indicator buffer number"
So I'm little confuse, cciValue2 has no. 1 so why we put 0?