Question

I have a very simple C# class:

namespace mybox
{
    public class userAccount : IMyInterface
    {
        public userAccount()
        {
        }
        private int _userId;
        private string _userName;

        public int userId
        {
            get { return _userId; }
            set { userId = value; }
        }
        public string userName
        {
            get { return _userName; }
            set { userName = value; }
        }

        public string list(int myUserId)
        {
            ...
           myOutPut = string.Format("{0} {1} {2}", u.userId, u.userName);
           return myOutPut.ToString();
        }

    public void add()
    {
           pillboxDataContext db = new pillboxDataContext();
           userAccount newUser = new userAccount();
           newUser.userName = "test123";

           db.SubmitChanges();
       }
    }
}

In my default.aspx.cs in the Page_Load event I’m trying to call the list method:

protected void Page_Load(object sender, EventArgs e)
{
    pillbox.userAccount myUA = new pillbox.userAccount();
    myUA.add();

// Console.WriteLine(myUA.list(1));

}

When I call the add method I can see that it is trying to assign the value test123 to the property but I get the following message:

An unhandled exception of type ‘System.StackOverflowException’ occurred in App_Code.1zm0trtk.dll

Any ideas of what I’m doing incorrectly?

Answer 1

The problem is with the way you defined your properties.

You are trying to refer to the property to which you are assigning the value in the setter
which is resulting in an infinte recursion (to be specific this line is triggering it newUser.userName = "test123";).

Change them to:

public int userId         
{             
    get { return _userId; }             
    set { _userId = value; }         
}         

public string userName         
{             
        get { return _userName; }             
        set { _userName = value; }         
}