problem with [i]

hi
in my EA , I try to print my two variables but it's not working:

**********************************************************

double m_10 = iMA(NULL, 0, 10, 0, MODE_SMA, PRICE_CLOSE, Current + 0);
double m_20 = iMA(NULL, 0, 20, 0, MODE_SMA, PRICE_CLOSE, Current + 0);


for(int i=1; i<500 ; i--)
{
m_10[i] = iMA(NULL, 0, 10, 0, MODE_SMA, PRICE_CLOSE, Current + i);
m_20[i] = iMA(NULL, 0, 20, 0, MODE_SMA, PRICE_CLOSE, Current + i);
Print(m_10[i]+ " " + m_20[i]);
}

**********************************************************

this is the message error :
'[' assignment expected

Help please!!

Thanks in advance.

double m_10[500], m_20[500];

double m_10[0] = iMA(NULL, 0, 10, 0, MODE_SMA, PRICE_CLOSE, Current + 0);
double m_20[0] = iMA(NULL, 0, 20, 0, MODE_SMA, PRICE_CLOSE, Current + 0);


the problem is your declaration. you have to declare them as an array not just a single variable.
if you change your for loop for example i<1000, you have to modify the array size in your declaration.

Hi ,

thank you for your help


I try do this :

double m_10[500] ;
double m_20[500];
m_10 = iMA(NULL, 0, 10, 0, MODE_SMA, PRICE_CLOSE, Current + 0);
m_20 = iMA(NULL, 0, 20, 0, MODE_SMA, PRICE_CLOSE, Current + 0);


for(int i=1; i<500 ; i--)
{
m_10[i] = iMA(NULL, 0, 10, 0, MODE_SMA, PRICE_CLOSE, Current + i);
m_20[i] = iMA(NULL, 0, 20, 0, MODE_SMA, PRICE_CLOSE, Current + i);
// Print(m_10[i]+ " " + m_20[i]);
Alert(m_10[i]+ " " + m_20[i]);
}


But the result of the Alert is 0 and O

the problem is in your for loop. You are starting at 1, since this is less than 500 the loop immediately breaks. Your for loop should be:

for(int i=500;i>=0;i--)


this will result in a left to right analysis beginning 500 bars back and continuing to the current bar.

There is no need for the Current statement, I do not even see where you are assigning it a value.

you are right that the loop is a problem, however it doesn't break immediately, but it will be an infinite loop.
and your loop is still wrong. if you declare an array with 500 elements, the array's indexes will be 0....499.

for(int i=1;i<500;i++)
or
for(int i=499;i>0;i--)

the 0.indexed element is filled up before the loop so you doesn't need to fill up again.

so try to use this:

double m_10[500] ;
double m_20[500];
m_10[0] = iMA(NULL, 0, 10, 0, MODE_SMA, PRICE_CLOSE, Current + 0);
m_20[0] = iMA(NULL, 0, 20, 0, MODE_SMA, PRICE_CLOSE, Current + 0);

for(int i=1; i<500 ; i++)
{
m_10[i] = iMA(NULL, 0, 10, 0, MODE_SMA, PRICE_CLOSE, Current + i);
m_20[i] = iMA(NULL, 0, 20, 0, MODE_SMA, PRICE_CLOSE, Current + i);
// Print(m_10[i]+ " " + m_20[i]);
Alert(m_10[i]+ " " + m_20[i]);
}

Hi

Great thanks for your help

I'll try it this evening


Bye

Hi

http://www.forexfactory.com/images/icons/icon14.gif http://www.forexfactory.com/images/icons/icon14.gif http://www.forexfactory.com/images/icons/icon14.gif http://www.forexfactory.com/images/icons/icon7.gif http://www.forexfactory.com/images/icons/icon7.gif

Thank you !

you're welcome