Stack Overflow Error?

[Arjay]

Once you understand what is causing the stack overflow issue is to figure out how to prevent the problem from never happening again.

One way to do this is to adopt a consistent property/field naming convention.

One popular C# naming convention is to name Properties as Pascal case and Fields as camel case with a leading underscore.

public string State { get { return _state; } }
private string _state;

In this style, all local variables/ctor or method parameters get named as camel case (without the leading underscore)

This avoids having to use 'this.' to disambiguate params from class fields.

public Package( string state )
{
  _state = state; // no problem
}

vs (when leading underscore isn't used to denote class fields)

public Package( string state )
{
  this.state = state;  // this works, but it's ugly to some folks
}

Another way to avoid issues is to use an auto property

public string State { get; set; }

In fact, auto properties allow you to set access levels. For example, a public readonly property with a private setter would look like this:

public string State { get; private set; }

public Package( string state )
{
  State = state;
}

I generally try to use autoproperties whenever possible because it reduces code clutter (imo).