class members accessors called unexpectedly

[steveevets]

Hi

I have written a class with a private member & public member with accessors
thus

public class Wierd
{
private int _MyInt = 0;
private bool _MyIntGot = false;
public int MyInt
{
get
{
_MyIntGot = true;
return _MyInt;
}
set
{
_MyInt = value;
_MyIntGot = true;
}
}
public Wierd()
{

}
}

then on a windows form i have

private void button1_Click(object sender, System.EventArgs e)
{
Wierd _MyWierd = new Wierd();
string g = "g";
g = "hh"; // do something
g = "kk"; // do something
int i = _MyWierd.MyInt;
}

In the debugger I can see that _MyWierd._MyIntGot becomes true before the line
int i = _MyWierd.MyInt;

The get accessor for _MyWierd.MyInt seems to be being called, though no
breakpoint is being hit there.

Can anyone tell me what is going on here?

Thanks in advance

Steve

[Alan Pretre]

"steveevets" <st********@discussions.microsoft.com> wrote in message
news:FC**********************************@microsof t.com...

public class Wierd
{
private int _MyInt = 0;
private bool _MyIntGot = false;
}

It's because you are using static initialization of _MyInt and _MyIntGot.
Move their initialization to the constructor and it should be fine.

public class Wierd {
private int _MyInt;
private bool _MyIntGot;

public int MyInt {
get {
_MyIntGot = true;
return _MyInt;
}
set {
_MyInt = value;
_MyIntGot = true;
}
}
public Wierd() {
_MyInt = 0;
_MyIntGot = false;
}
}

-- Alan

[Steve Walker]

In message <FC**********************************@microsoft.co m>,
steveevets <st********@discussions.microsoft.com> writes

Hi

I have written a class with a private member & public member with accessors
thus

In the debugger I can see that _MyWierd._MyIntGot becomes true before the line
int i = _MyWierd.MyInt;

The get accessor for _MyWierd.MyInt seems to be being called, though no
breakpoint is being hit there.

Can anyone tell me what is going on here?

Every time you view the value of the property, the code in the property
is being run.

Try this:

class Test
{
int i=0;
public int I
{
get{return i++;}
}
}

Instantiate it, set a breakpoint, and hover over some code using the
property. It goes up, rapidly.

--
Steve Walker

[Jon Skeet [C# MVP]]

Alan Pretre <al********@newsgroup.nospam> wrote:

"steveevets" <st********@discussions.microsoft.com> wrote in message
news:FC**********************************@microsof t.com...

public class Wierd
{
private int _MyInt = 0;
private bool _MyIntGot = false;
}

It's because you are using static initialization of _MyInt and _MyIntGot.
Move their initialization to the constructor and it should be fine.

No it's not - that's not static intiialization, and it won't invoke the
properties.

My guess is that it's using the debugger that's causing the
behaviour...

--
Jon Skeet - <sk***@pobox.com>
http://www.pobox.com/~skeet
If replying to the group, please do not mail me too

[Jon Skeet [C# MVP]]

steveevets <st********@discussions.microsoft.com> wrote:

I have written a class with a private member & public member with accessors
thus

<snip>

Are you by any chance watching this in a debugger? It may be executing
the property for you in order to show you the value. If that's not the
case, could you post a short but complete program which demonstrates
the problem?

See http://www.pobox.com/~skeet/csharp/complete.html for details of
what I mean by that.

--
Jon Skeet - <sk***@pobox.com>
http://www.pobox.com/~skeet
If replying to the group, please do not mail me too

[steveevets]

Hi
Yes it's the debugger that causes the effect.
Microsoft do (I have now read) specifically warn against modifing an object
within a get. I am guilty here of "Bad programming practice" & won't do it
again!

Thanks for your response.

Regards
Steve

"steveevets" wrote:

Hi

I have written a class with a private member & public member with accessors
thus

public class Wierd
{
private int _MyInt = 0;
private bool _MyIntGot = false;
public int MyInt
{
get
{
_MyIntGot = true;
return _MyInt;
}
set
{
_MyInt = value;
_MyIntGot = true;
}
}
public Wierd()
{

}
}

then on a windows form i have

private void button1_Click(object sender, System.EventArgs e)
{
Wierd _MyWierd = new Wierd();
string g = "g";
g = "hh"; // do something
g = "kk"; // do something
int i = _MyWierd.MyInt;
}

In the debugger I can see that _MyWierd._MyIntGot becomes true before the line
int i = _MyWierd.MyInt;

The get accessor for _MyWierd.MyInt seems to be being called, though no
breakpoint is being hit there.

Can anyone tell me what is going on here?

Thanks in advance

Steve

[steveevets]

Hi

No it's the debugger accessing the get inorder to show the value, & cleverly
bypassing the breakpoint.

Thanks though.

Regards
Steve

"Alan Pretre" wrote:

"steveevets" <st********@discussions.microsoft.com> wrote in message
news:FC**********************************@microsof t.com...

public class Wierd
{
private int _MyInt = 0;
private bool _MyIntGot = false;
}

It's because you are using static initialization of _MyInt and _MyIntGot.
Move their initialization to the constructor and it should be fine.

public class Wierd {
private int _MyInt;
private bool _MyIntGot;

public int MyInt {
get {
_MyIntGot = true;
return _MyInt;
}
set {
_MyInt = value;
_MyIntGot = true;
}
}
public Wierd() {
_MyInt = 0;
_MyIntGot = false;
}
}

-- Alan