C#: Initializing class variables [duplicate]

Question

Possible Duplicate:
C# member variable initialization; best practice?

Which is the right way to initialize class variables. What is the difference between [1] and [2].

//[1]
public class Person
{
   private int mPersonID = 0; 
   private string mPersonName = "";
}

OR

//[2]
public class Person
{
     private int mPersonID = 0; 
     private string mPersonName = "";

     public Person()
     {
         InitializePerson();
     }

     private void InitializePerson()
     {
          mPersonID = 0;
          mPersonName = "";
     }
}

There is similar question here: stackoverflow.com/questions/298183/… — RandomWebGuy
– RandomWebGuy, Commented Jun 9, 2011 at 3:21

BrokenGlass · Accepted Answer · 2011-06-09 03:24:24Z

6

Instance variables that are assigned a default value as part of their declaration will get this value assigned right before the constructor is run, from the outside there is no perceptible difference in behavior between 1) and 2), it's mostly a matter of style.

You also introduce an additional InitializePerson() method in your approach 2) - this can be beneficial if you have multiple constructors that then all can use the same common initialization method (which keeps the code DRY).

Edit in response to comment, see MSDN:

Fields are initialized immediately before the constructor for the object instance is called. If the constructor assigns the value of a field, it will overwrite any value given during field declaration.

edited Jun 9, 2011 at 3:24

answered Jun 9, 2011 at 3:18

BrokenGlass

161k28 gold badges297 silver badges339 bronze badges

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

6 Comments

Alex Aza Over a year ago

How is it possible to have fields assigned right before the constructor?

BrokenGlass Over a year ago

@Alex: that's how it works, also you check out msdn.microsoft.com/en-us/library/ms173118%28v=vs.80%29.aspx for a reference

Alex Aza Over a year ago

I wish the person who wrote this explained how code can be called just before the constructor. Take a look at my answer, the code is definitely inside of the constructor.

BrokenGlass Over a year ago

true that, can't argue with the IL. I guess semantically in this context there is not much difference between the field initializer code being put at the beginning of the constructor or saying they are executed right before the constructor call. There are limitations in field initializers though (i.e can't access other instance fields) that make them different from "other" constructor code.

Anthony Pegram Over a year ago

You'd see a difference when there's a base class. The derived members will be initialized, then the base constructor will be called, then the derived constructor will complete.

|

Alex Aza · Accepted Answer · 2011-06-09 03:29:42Z

2

In both cases it will generate IL that will look almost the same. Saying 'almost', because you introduced extra method in the second approach. If you just put it to constructor, than it would be THE same IL code.

So it is better to use approach #1 as it is less code.

Also you don't need to assign default values (0 for int, null for reference types).

These two examples generate identical IL:

public class Test
{
    private string test1 = "";
}

public class Test
{
    private string test1;

    public public Test()
    {
        test1 = "";
    }
}

IL:

.method public hidebysig specialname rtspecialname instance void .ctor() cil managed
{
    .maxstack 8
    L_0000: ldarg.0 
    L_0001: ldstr ""
    L_0006: stfld string ConsoleApplication1.Test::test1
    L_000b: ldarg.0 
    L_000c: call instance void [mscorlib]System.Object::.ctor()
    L_0011: nop 
    L_0012: ret 
}

edited Jun 9, 2011 at 3:29

answered Jun 9, 2011 at 3:17

Alex Aza

78.8k26 gold badges157 silver badges134 bronze badges

7 Comments

smatter Over a year ago

So I need to assign string to "", right?

Andrew Savinykh Over a year ago

yes, if you want it to be equal "", which is not the same as null

Khepri Over a year ago

Is there some specific reason you'd want to default a string to an empty string instead of null? If not, then no, you don't have to.

Cos Callis Over a year ago

You don't "need" to assign "" to string, as that is the default value for a string type, but if you are going to it is better to use string.Empty than quotes (almost imperceptively so, but still)

Anthony Pegram Over a year ago

@Cos, you are incorrect. null is the default for a string as well as other reference types. An empty string must be set explicitly.

|

alexD · Accepted Answer · 2011-06-09 03:21:03Z

0

In the 2nd case, if you try referencing mPersonName in the constructor before you assign it in InitializePerson, you'll get a null reference exception since String is NULL by default. int default value is 0 so there is no issue there. In the first case, you don't have to worry about where you use them in your constructor since you assign values on declaration at the class level. Otherwise, there is no difference.

answered Jun 9, 2011 at 3:21

alexD

2,3742 gold badges32 silver badges43 bronze badges

Collectives™ on Stack Overflow

C#: Initializing class variables [duplicate]

3 Answers 3

6 Comments

7 Comments

Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

6 Comments

7 Comments

Comments

Linked

Related