Obscure errors, troubleshooting, anyone?

Okay. look at my program and see if you can figure out these problems...
1- pointer CN[4] (not 3,2,or1) is changed to the value of WE when WFK calls Event()
2- int CEXPGoal[4] is assigned a value of 127 but it is immediately changed to 10 even before you call status()
3- int CMHP[4] is changed from 30 to 1 when sleeping at the inn...

I expect to solve these problems in the next couple of weeks, but any help would be appreciated. also any suggestions/comments you have would be welcome and, indeed, valued.
As of right now, i'm guessing that there is some kind of memory overlay in the variable declaration section (not definition)...
i've just recently figured out how to do pointers/arrays so perhaps i've made some mistake...


Aaron/Golbez/Kainazzo/Valvalis/Ruben/Milon/Zemus/Zeromus
oh, btw i use Borland version whatever...

>>pointer CN[4] (not 3,2,or1)
Thats because you access array element via 0-n, not 1-n. So in your case its, CN[0], CN[1], CN[2], CN[3] only. CN[4] is not in this variables memory scope, you are overwriting someone elses data, and they will happily return the favour.

well i changed all variables to correct the 0-4 problem.
However, i only got more of the same errors. now it changes almost every variable; most change to 10 or 1 but one changes to 420463 or something...
i tested it by printing the values to the screen, even right after i assigned the values, and they were already messed up.
i dont know what is happening, but i wish it would stop.
I will include the updated version of Game.cpp...
thanks in advance for any help i get...
Aaron

>>well i changed all variables to correct the 0-4 problem
Not quite, you didn't ;)

Look:

Code:

---

int CCON[3];
...
CCON[0] = 6;
...
CCON[1] = 10;
...
CCON[2] = 6;
...
CCON[3] =5;

---

You allocated CCON as an array of ints, size 3. Then you filled CCON[0] - CCON[3] with numbers. But that's 4 elements, not 3. So the last one went beyond the end of the array.

You should declare your array with the total number of elements required:
>>int CCON[4];
and then fill the elements, using the notation 0 to N-1, in this case CCON[0] - CCON[3].

This might not be all of your problems, but you need to get this right first.

Quote:

---

I expect to solve these problems in the next couple of weeks,

---

Based on Hammer's corrections and your response, I think it fair to warn you that it is potentially unlikely you will, by yourself, ever solve these problems. You basic problem seems to stem from the fact that you don't even grasp the syntax of the language. Therefore, I would recommend you learn the syntax and language grammar necessary before delving into the deeper mysteries of writing logic.

"If you programmed code using the full extent of your cunning skills, ingenuity, and to the best of your ability, then by definition, you are not smart enough to debug it."