Error Checking with the Set Statement

[dipique]

private int Orientation
{
	get { return Orientation; }
	set
	{
		if ((value > 0) && (value < 5))
			Orientation = value;
	}
}

What I want to do is, if the variable is set to something that's not between 1 and 4, don't change the value of Orientation; however, when you try it, the "Orientation = value;" line triggers the "set" of the property again so I get a stack overflow. Any ideas? I'd like for the error checking to occur within the set statement if at all possible.

Dan

[Magiaus]

how about a private variable _orientation...?

private int _orientation = 0;
public int Orientation
{
	get { return _orientation; }
	set
	{
		if ((value > 0) && (value < 5) && (value != _orientation ))
			_orientation = value;
	}
}

Why do you have a private property anyway? a private variable is less code.... and it is bad practice to get or set a property value to/from it's self. Where is this value being stored. I would assume that it never works....

[nswan]

got to agree with Magiaus there. Surely your approach would just get stuck in a loop as it keeps trying to assign itself!