Variable doesn't increment in FOR statement

[kepler]

Hi,

I have a code like this:

for(i=0;i<n;i++){

vstep = jd + i * increment

calculate(vstep,....)


}

When I run it, the calculate function gives me the results all equal to the first value - vstep doesn't increase.

Anybody has any idea why?

Kind regards,

Kepler

[jonnin]

could be anything. What does calculate have for side effects, does it mess up vstep? What is increment, if it were zero, would that be the issue? Is jd constant or changed by calculate?

In short, there is no answer without more info ... but the most likely cause is probably a zero increment, assuming you didnt do anything weird.

[kepler]

could be anything. What does calculate have for side effects, does it mess up vstep? What is increment, if it were zero, would that be the issue? Is jd constant or changed by calculate?

In short, there is no answer without more info ... but the most likely cause is probably a zero increment, assuming you didnt do anything weird.

Hi,

I've putted the increment equal to 3 and vstep equal to 0:

vstep = 0;

for(i=0;i<n;i++){

vstep = jd + i * 3;

calculate(vstep,....)


}

The function calculates for the first value ( jd ) and then repeats all the values...

I don't know what's wrong. The routine is simple. The function "calculate" gives different values for different inputs...

This is puzzling me...

Kind regards,

Kepler

[jonnin]

well, if i is 0 to start and increments by 1, the step is 3, and jd is a constant, you should get

jd, jd+3, jd+6, ... jd + 3* i

for vstep. The code is correct to do this ASSUMING that jd is not changed each time.

So my advice is to just do this:

vstep = 0;
jd = 0;
calculate(vstep,....) (to get jd value???)

for(i=0;i<n;i++)
{
vstep = jd + i * 3;
cout << vstep << endl;
}

See what that prints. If vstep is correct, then calculate has an error. If vstep is wrong, what values are expected?

[kepler]

Hi,

I did the test. The first part, went well - gave the correct result.
As for the second part - All repeated....

I don't understant...

Since vstep is a double and i is an integer I've tryed:

vstep = (double) jd + i * 3;

but it didn't work either. I made an output of vstep for n=20, and vstep came out equal to jd 20 times...

Kind regards,

Kepler

[jonnin]

OK other stuff to look at.

Print i each time in the loop when it is going wrong. 'i' may be a global that is being reset somewhere, or something (???) along those lines? You may also have a memory error (clobber) that changes the value of 'i' or something.

All I know to do, since all I have is your black box snippets of code, is to tell you to keep printing every possible variable in the system until you figure out what is going wrong and where it goes wrong.

[kepler]

Hi,

The i prints allright - but I'm making something wrong... I made a test program:

#include <iostream.h>


int main (int argc, char *argv[]) {

double jd, vstep,calc;
int i,n;

jd = 425.65;
n = 20;



          for(i = 0;i < n;i++){

          vstep = jd + i * 3;
          
          calc = 2 * vstep;

          cout << vstep << "\n";
		  
          }


return(0);
}

that outputs vstep correctly:

425.65
428.65
431.65
434.65
437.65
440.65
443.65
446.65
449.65
452.65
455.65
458.65
461.65
464.65
467.65
470.65
473.65
476.65
479.65
482.65

It's really puzzling me...

Kepler

[jonnin]

Something odd is going on in the calculate funciton then, if the test program works and the original does not. That is the only difference between them, really.

If 'i' prints out ok in the original, look at JD or make JD a constant there too.

The problem can only really be 1 of 4 things:

1) vstep is broken somewhere
2) i is broken somewhere
3) jd is broken somewhere
4) calculate function does something really funky.

Also:
5) Your program is correct and you think its wrong, but it is not (it can happen and at this point is a possible issue).

If you look in depth (with print statements) at all 4 of those, something has to bubble out as the issue.

Post the calculate function if you can? I can try to look at that too.

[Ron Weller]

You can eliminate 'i' as a problem by making it local to the for block like this:

for(int i=0;i<n;i++){

vstep = jd + i * 3;

calculate(vstep,....)

}