c# - 単一行のアクセサーは、メソッドまたは変数のように記述する必要があります

Question

現在のコーディング標準では、単純なアクセサーでさえもメソッドのように記述する必要があります。

すなわち

string x 
{
  get 
    { 
      return _x;
    }
}

私にはそれを次のように書くことは理にかなっていますが、

string x { get { return _x; } }

アクセサーが 10 個になると、コードのページをスクロールして、全体がうまくはまっているのではなく、どのように組み合わされているのか疑問に思うでしょう。

どちらがより理にかなっていますか?

score 1 · Accepted Answer

私にとっては、次のように書くことは理にかなっています。

string x 
{
    get { return _x; }
    set { _x = value; }
}

しかし、2 年前に働いていた場所では、次のように書くように求められました。

string x {
    get { return _x; }
    set { _x = value; }
}

そして、私はそれに慣れ、3か月で自然になりました。

私にとって、これは 1 行で 2 つの異なることを宣言するようなものですが、ほとんどの場合、私はそれに慣れていなかったので、好きではありません。

public string X { get { return _x; } }

同時に、自動プロパティは正常に機能します。

public string X { get; set; }

それはすべてあなたのグループ次第です。選択した標準を決定して進めなければなりません。良いも悪いもありません。マイクロソフトは何かを推奨するかもしれませんが、これはほとんどの場合、すべての開発者がコードを簡単に理解できるようにするためです。

1 に答える 1