.NETプロパティ-プライベートセットまたは読み取り専用プロパティを使用しますか？

プロパティでプライベートセットを使用し、ReadOnlyプロパティにする場合はどのような状況ですか？以下の非常に単純な2つの例を考慮してください。

最初の例：

Public Class Person

    Private _name As String

    Public Property Name As String
        Get
            Return _name
        End Get
        Private Set(ByVal value As String)
            _name = value
        End Set
    End Property

    Public Sub WorkOnName()

        Dim txtInfo As TextInfo = _
            Threading.Thread.CurrentThread.CurrentCulture.TextInfo

        Me.Name = txtInfo.ToTitleCase(Me.Name)

    End Sub

End Class

// ----------

public class Person
{
    private string _name;
    public string Name
    {
        get { return _name; }
        private set { _name = value; }
    }

    public void WorkOnName()
    {
        TextInfo txtInfo = System.Threading.Thread.CurrentThread.CurrentCulture.TextInfo;
        this.Name = txtInfo.ToTitleCase(this.Name);
    }
}

2番目の例：

Public Class AnotherPerson

    Private _name As String

    Public ReadOnly Property Name As String
        Get
            Return _name
        End Get
    End Property

    Public Sub WorkOnName()

        Dim txtInfo As TextInfo = _
            Threading.Thread.CurrentThread.CurrentCulture.TextInfo

        _name = txtInfo.ToTitleCase(_name)

    End Sub

End Class

// ---------------

public class AnotherPerson
{
    private string _name;
    public string Name
    {
        get { return _name; }
    }

    public void WorkOnName()
    {
        TextInfo txtInfo = System.Threading.Thread.CurrentThread.CurrentCulture.TextInfo;
        _name = txtInfo.ToTitleCase(_name);
    }
}

どちらも同じ結果をもたらします。これは善悪の状況ではなく、単に好みの問題ですか？

使用する理由はいくつかありますprivate set。

1）バッキングフィールドをまったく使用せず、読み取り専用の自動プロパティが必要な場合：

public string Name { get; private set; }   

public void WorkOnName()
{
    TextInfo txtInfo = Thread.CurrentThread.CurrentCulture.TextInfo;
    Name = txtInfo.ToTitleCase(Name);
}  

2）クラス内の変数を変更するときに追加の作業を行い、それを単一の場所でキャプチャする場合：

private string _name = string.Empty;
public string Name 
{ 
    get { return _name; }
    private set 
    {
        TextInfo txtInfo = Thread.CurrentThread.CurrentCulture.TextInfo;
        _name = txtInfo.ToTitleCase(value);
    }
}

ただし、一般的には、個人的な好みの問題です。私が知っている限りでは、どちらかを使用するパフォーマンス上の理由はありません。

外部からセッターにアクセスできないようにする場合は、プライベートセットを使用します。

プロパティを1回だけ設定する場合は、読み取り専用を使用します。コンストラクターまたは変数初期化子。

これをテスト：

void Main()
{
    Configuration config = new Configuration();
    config.ResetConfiguration();

    ConfigurationReadOnly configRO = new ConfigurationReadOnly();
    configRO.ResetConfiguration();
}

public class Configuration
{
    public Color BackgroundColor { get; private set; }
    public Color ForegroundColor { get; private set; }
    public String Text { get; private set; }

    public Configuration()
    {
        BackgroundColor = Color.Black;
        ForegroundColor = Color.White;
        Text = String.Empty;
    }

    public void ResetConfiguration()
    {
        BackgroundColor = Color.Black;
        ForegroundColor = Color.White;
        Text = String.Empty;
    }
}

public class ConfigurationReadOnly
{
    public readonly Color BackgroundColor;
    public readonly Color ForegroundColor;
    public readonly String Text;

    public ConfigurationReadOnly()
    {
        BackgroundColor = Color.Black;
        ForegroundColor = Color.White;
        Text = String.Empty;
    }

    public void ResetConfiguration()
    {
        BackgroundColor = Color.Black; // compile error: due to readonly keyword
        ForegroundColor = Color.White; // compile error: due to readonly keyword
        Text = String.Empty; // compile error: due to readonly keyword
    }
}

3番目のオプションを提案できますか？

public class Person
{
    public string Name { get; protected set; }

    public void SetName(string name)
    {
        TextInfo txtInfo = System.Threading.Thread.CurrentThread.CurrentCulture.TextInfo;
        this.Name = txtInfo.ToTitleCase(name);
    }
}

これにより、Nameプロパティがすべての外部コードに対して事実上読み取り専用になり、明示的なSetメソッドが提供されます。設定時に値を変更するため、Nameプロパティのセットを単に使用するよりも、明示的なSetの方が好きです。通常、プロパティ値を設定すると、後でgetを呼び出したときに同じ値が返されることが期待されます。これは、setで ToTitleCaseを実行した場合には発生しません。

しかし、あなたが言ったように、正しい答えはありません。

C＃6.0以降、ゲッターのみの自動プロパティが言語に追加されました。ここを参照してください：https://github.com/dotnet/roslyn/wiki/New-Language-Features-in-C%23-6#getter-only-auto-propertiesを。

以下に例を示します。

public class SomeClass
{
    public int GetOnlyInt { get; }

    public int GetOnlyIntWithInitializer { get; } = 25;

    public SomeClass(int getOnlyInt)
    {
        GetOnlyInt = getOnlyInt;
    }
}

2番目の例を使用しないでください。プロパティを使用する全体のポイントは、ゲッターの取得とセッターの設定以外に何も起こっていない場合でも、そのゲッターとセッターを介してすべてのアクセスを集中させることです。一箇所。

2番目の例は、プロパティを設定する場合にそれを放棄します。大規模で複雑なクラスでそのアプローチを使用し、後でプロパティの動作を変更する必要がある場合は、1つの場所（プライベートセッター）で変更を行うのではなく、検索と置換の土地にいることになります。

セッターのアクセスレベルを変更する必要があるたびに、通常、Protected（このクラスと派生クラスのみが値を変更できます）またはFriend（アセンブリのメンバーのみが値を変更できます）に変更しました。

ただし、セッターでバッキング値を変更する以外のタスクを実行する場合は、Privateを使用するのが最適です。先に指摘したように、バッキング値を直接参照せず、代わりにそれらのプロパティを介してのみアクセスすることをお勧めします。これにより、後でプロパティに加えた変更が、内部だけでなく外部にも適用されます。また、プロパティとそのバッキング変数を参照しても、パフォーマンスの低下はほとんどありません。

また、パフォーマンスの低下はほとんどありません...

しかし、明確にするために、プロパティにアクセスすることで、その裏変数にアクセスするよりも遅いです。プロパティのgetterおよびsetterはCallおよびReturnを必要とするメソッドですが、プロパティのバッキング変数は直接アクセスされます。

そのため、プロパティのgetterがコードブロック内で何度もアクセスされる可能性がある場合、プロパティの値が最初にキャッシュされ（ローカル変数に保存され）、代わりにローカル変数が使用されることがあります。もちろん、ブロックの実行中にプロパティを非同期に変更できないことを前提としています。