C# のメンバー変数の書き方に関しての質問です。

現在、前担当者から引き継いだC#のプログラムを見ているのですが、一部気になる記述を見かけました。

サンプルとして以下のコードを貼らせていただきます。

Program.cs
1using System;
2
3namespace HelloWorld
4{
5    class Program
6    {
7        static void Main(string[] args)
8        {
9            UserA userA1 = new UserA();
10            UserB userB1 = new UserB();
11            UserA userA2 = new UserA(2);
12            UserB userB2 = new UserB(3);
13            Console.WriteLine("userA1:" + userA1.Age);
14            Console.WriteLine("userB1:" + userB1.Age);
15            Console.WriteLine("userA2:" + userA2.Age);
16            Console.WriteLine("userB2:" + userB2.Age);
17            /*
18            userA1.Age = 4; //code error
19            userB1.Age = 5; //code error
20            */
21            Console.WriteLine("Hello World!");
22        }
23    }
24
25    class UserA
26    {
27        public UserA()
28        {
29        }
30
31        public UserA(int age) {
32            _age = age;
33        }
34
35        private int _age = 0;
36        public int Age
37        {
38            get { return _age; }
39        }
40    }
41
42    class UserB
43    {
44        public UserB()
45        {
46        }
47
48        public UserB(int age) {
49            Age = age;
50        }
51
52        public int Age
53        {
54            get;
55            private set;
56        } = 0;
57    }
58}
59

現在は「UserA」の「_age」を使う書き方となっており、個人的には「UserB」の「_age」を使わず、初期化子やアクセス修飾子を使う書き方の方が良いのではと考えております。
※自分は２ヶ月程度しかC#に触れたことがなく、そのあたりの良し悪しがよくわかってないです。

特に懸念・デメリットなどなければ「UserB」の書き方に修正しようと考えているのですが、何か問題などありますでしょうか。

なお、環境につきましては以下のようになっております。

- Unity 2018.4.3f
    - Scripting Runtime Version:.NET 4.x Equivalent
    - Scripting Backend: IL2CPP
    - .NET Standard 2.0

行動規範の内容に同意します

回答4件

こんにちは。

UserA の書き方は、プロパティに自動実装機能が存在しなかった C# 3.0 未満の時代のテンプレであり、遺物です。

UserB の書き方は C# 3.0 で搭載された「プロパティの自動実装」と、C# 6.0 から利用可能な「プロパティのデフォルト値代入」を同時に利用した書き方です。

今どきの C# を利用しているのであれば、UserA の書き方を採用する理由はほぼありません。

ちなみに、自分だったら以下のように書きます。趣味ですが。

csharp
1    class UserC
2    {
3        public UserC() : this(age: 0) // age 未設定を許可したくないならこのコンストラクタ自体を削除する。
4        { }
5
6        public UserC(int age) {
7            Age = age;
8        }
9
10        public int Age { get; } // getter-only は C# 6.0 から。クラス内でこの値を「変更」する必要があるなら、`private set;` を追加する。
11    }