C# - StackOverflowException instanciating class over constructor [duplicate]

Question

im writing a c# application. im pretty new to c#.

I got a StackOverflowException (yes! :D) trying to set the class properties in the constructor like this:

namespace WindowsUpdateOnLan
{
    public class NetworkAdapter
    {
        public NetworkAdapter(PropertyDataCollection properties)
        {
            String value = null;
            foreach (PropertyData pd in properties)
            {
                if (pd.Name.Equals("GUID"))
                    Id = Guid.Parse(pd.Value.ToString());

                if (pd.Name.Equals("Name"))
                    Name = pd.Value.ToString();

                if (pd.Name.Equals("NetConnectionID"))
                {
                    value = Regex.Replace(pd.Value.ToString(), @"\s+", "");
                    adapterType = (AdapterTypeEnum)Enum.Parse(typeof(AdapterTypeEnum), value);
                }

                if (pd.Name.Equals("NetEnabled"))
                {
                    value = Regex.Replace(pd.Value.ToString(), @"\s+", "");
                    adapterStatus = (AdapterStatusEnum)Enum.Parse(typeof(AdapterStatusEnum), value);
                }
            }
        }

        /// <summary>
        /// Contains the GUID that is used to identify the adapter
        /// </summary>
        public Guid Id
        {
            get { return this.Id; }
            private set { Id = value; }
        }

And Visual Studio tells me to make sure i dont have an infinite loop.

I must have forgotten something important or maybe the syntax is not right.

Could anybody take a look at it?

i was not asking about the exception so its no duplicate

Master Azazel
– Master Azazel

2016-03-18 20:21:20 +00:00
Commented Mar 18, 2016 at 20:21 — Master Azazel
– Master Azazel, Commented Mar 18, 2016 at 20:21

Andrey · Accepted Answer · 2016-03-18 16:10:42Z

10

public Guid Id
{
    get { return this.Id; }
    private set { Id = value; }
}

You call same property getter/setter in getter/setter. Thus recursion.

Solutions:

Add field id and use it.
Use autopropery: public Guid Id { get; private set; }

answered Mar 18, 2016 at 16:10

Andrey

60.3k12 gold badges127 silver badges169 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Master Azazel Over a year ago

thanks! autoproperty works awesome! its still not fully clear to me how this happens is it because i have no 'this.' in the setter?

Andrey Over a year ago

@MikeAzazel doesn't matter. this is optional basically compiler adds it for you .

Collectives™ on Stack Overflow

C# - StackOverflowException instanciating class over constructor [duplicate]

1 Answer 1

2 Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Linked

Related