ListをreadonlyにしてもClearやAddができてしまう。

Question

public static readonly List<DataCollection> Messages = new();


//↓ほかのところ
Messages.Add(クラス);//できる
Messages.Clear(クラス);//できる

これを防ごうとしてprivateにすると読み取りもできなくなりました。
AddとClearをprivateの範囲でのみできるようにしたいのですが、どのようなコードにすればよいでしょうか。

Comments

[退会済みユーザー]

「privateの範囲でのみ」とはどういう意味ですか？　ごく部分的なコードだけでなく、「privateの範囲」とそれ以外の範囲が分かるように、もっと範囲を広げたサンプルコードを書くことはできませんか？

[Zuishin]

プライベートフィールドとパブリックプロパティで型を変えると良いでしょう。

Answer 1

そういった用途であれば、System.Collections.ObjectModel.ReadOnlyCollection<T>を使いましょう。

• ReadOnlyCollection<T> クラス

using System;
using System.Collections.ObjectModel;
using System.Collections.Generic;

class MyClass {

    //実際に要素を追加・削除するコレクション
    private List<int> innerList;
    //要素の追加・削除できない公開用コレクション
    public ReadOnlyCollection<int> Messages { get; }

    public MyClass() {
        //初期化
        this.innerList = new List<int>() { 1, 2, 3, 4, 5 };
        //公開用読み取り専用リスト初期化
        this.Messages = this.innerList.AsReadOnly();
        //こちらでも可
        //this.Messages = new ReadOnlyCollection<int>(this.innerList);
    }

    public void DoSomething() {
        this.innerList.Add(10);
    }
}

class Program {
    static void Main() {
        var data= new MyClass();

        data.DoSomething();
        foreach(var x in data.Messages) {
            Console.WriteLine(x);
        }
        
        //data.Add(10);   //コンパイルエラー
        //data.Clear();   //コンパイルエラー
    }
}

Answer 2

やりたい事はこういうことでしょう。readonly の意味を正確に理解していないものと推測しています。

using System;
using System.Collections.Generic;

class Program
{
    public static void Main(string[] args)
    {
        var x = new MyClass();
        Console.WriteLine(x.Numbers[3]);    // もちろん読み取りはできる

        // x.Numbers.Add(1);      // これは当然できない
        // x.Numbers.Remove(1);   // これは当然できない
        // x.Numbers.Clear();     // これは当然できない

        x.Numbers[0] = 0;         // 書き変えることもできない
        Console.WriteLine(x.Numbers[0]);    // チェックしてみると元の値から変わっていないことがわかる
    }
}

class MyClass
{
    List<int> _numbers = new List<int>() { 5, 3, 2, 8, 1 };

    public int[] Numbers
    {
        get { return _numbers.ToArray(); }
    }
}