C++Talk.NET

rene · Guest

Hi all,
please look into this small piece of code and tell me why my values
weren't stored. Ive searched for hours on an explanation, but found
none.
Ive got an class with (in this example) 2 ints and want to set an read
them
out of another implementation. BUT i always get zeros back.
Ive also tried that with pointer (eg mycur=new curve() Wink

but same
result.

Thanks in advance for any reply
Rene

class definition:

class curve
{
public:
curve()
{
}
int ckeytime;
int ckeyspeed;

void setCurvekeyTime (long t)
{
ckeytime=t;
}

void setCurvekeySpeed (long s)
{
ckeyspeed=s;
}

int CurvekeyTime() {
return ckeytime;
}
int CurvekeySpeed() {
return ckeyspeed;
}
};

implementation other class:
public: curve mycur;

one void:
mycur.setCurvekeyTime(11);
mycur.setCurvekeySpeed(12);

other void:
cout << mycur.CurvekeyTime() << endl;
cout << mycur.CurvekeySpeed() << endl;

results in:
0
0

Ian McCulloch · Guest

rene wrote:

rene · Guest

Hi Ian,
no I couldn't.
The whole code is 16000 lines long.
If I write a little example code. Everythis works fine.
Yes, could be a bug in the compiler, but Ivr tried several gccs and
iccs.
Same result

shez · Guest

rene wrote:

rene · Guest

Are you sure you're calling the first "void"?
Yes, if sourrounded the code with debugging stuff.

the FUNCTION (<- you know)
mycur.setCurvekeyTime(11);
is OK.
debugging cout in the curve class tells me, that 11 is assigned to t
and ckeytime.
But calling mycur.CurvekeyTime() holds the value 0.
So something went wrong with the return value.
If I changed the code to:
int CurvekeyTime() {
//return ckeytime;
return 123;
}
I get the 123.

BTW. Please forgive the typo with long and int. There are all long in
my code.
I have acted desperately and set them all to int.

Tnx Rene

Ian McCulloch · Guest

rene wrote:

teejbee@hotmail.com · Guest

Usually these sorts of problems occur when the code where you set the
value and the code where you read the value actually do not talk to the
same instance of the class. Make *sure* you are dealing with the same
instance of curve class and as posted earlier, you are calling the
setCurvekeyTime() before CurvekeyTime() call.

If you have access for modifying the code, try to print (or by any
other means) check the value of "this" pointer in both the
setCurvekeyTime() and CurvekeyTime() functions.

Hope this helps,
TJ

rene · Guest

Hi TJ,

tried this. Won't help.
Got a question. Do I need that constructor to initialize the integers
(ckeytime, ckeyspeed)?
Will the mem space reserved by initialize the class curve(normal or as
pointer; curve mycur or. mycur= new curve.)

If also tried the set these values via the constructor.
curve::curve(int a, int b){ckeytime=a;ckeyspeed=b;};
mycur=new curve(12,13);

But It don't work either.

TJ · Guest