C#で、別のクラスを経由して複数の値を渡したい

Question

前提・実現したいこと

Dataクラスに格納された値を、Managerクラスを経由し、Formクラスに渡すプログラムを作成しています。
ManagerはDataとFormの値を仲介するだけで、値の加工は行いません。

FormがDataから値を取得する際に、以下の様な3つの方法を思いついたのですが、どの方法が正しいのか判断が付きません。
どの方法が正しいのか、もしくは、他に正しい値の渡し方があれば教えて下さい。

１、ManagerでDataの値を取得するメソッドを個別に作成し、Formから呼び出す。
２、outパラメータを使って、一括で呼び出す。
３、DataのインスタンスをFormに渡して処理する。

発生している問題・エラーメッセージ

それぞれで以下のような問題がありました。

１、『ManagerでDataの値を取得するメソッドを個別に作成し、Formから呼び出す』方法
→Managerクラスで、個別にメソッドを作るのが大変。

２、『outパラメータを使って、一括で呼び出す』方法
→引数が膨大になる。

３、『DataのインスタンスをFormに渡して処理する』方法
→一番楽だったが、この方法が正しいのかわからない。

実際のコードではDataクラスのフィールドが10~20程度あるので、１と２の方法はかなり難しいのではないかと思いました。

該当のソースコード

    // メインフォーム
　　public partial class Form1 : Form
    {
        Manager manager = new Manager(3);

        int resultInt;           // int型の結果
        string resultString;     // string型の結果
        bool resultBool;         // bool型の結果

        public Form1()
        {
            InitializeComponent();
            ShowData();
        }

        // 結果を表示する処理
        public void ShowData()
        {
            // 個別にメソッドを呼び出す方法（方法1）
            for(int i=0; i<3; i++)
            {
                resultInt = manager.GetDataInt(i);
                resultString = manager.GetDataString(i);
                resultBool = manager.GetDataBool(i);
                Console.WriteLine($"【{i}】{resultInt},{resultString},{resultBool}");
            }
　　　　　　　// outを使用して一括で値を取得する方法（方法2）
            for (int i = 0; i < 3; i++)
            {
                manager.GetAllData(i, out resultInt, out resultString, out resultBool);
                Console.WriteLine($"【{i}】{resultInt},{resultString},{resultBool}");
            }
            // インスタンスを受け取る方法（方法3）
            for (int i = 0; i < 3; i++)
            {
                var data = manager.GetClass(i);

                resultInt = data.DataInt;
                resultString = data.DataString;
                resultBool = data.DataBool;
                Console.WriteLine($"【{i}】{resultInt},{resultString},{resultBool}");
            }

        }
    }

    // DataクラスとFormクラスを仲介するクラス
    class Manager
    {
        List<Data> data = new List<Data>();       // Dataインスタンスの格納用リスト
        
        // コンストラクタ
        public Manager(int value)
        {
            for(int i=0; i<value; i++)
            {
                data.Add(new Data(i));
            }
        }

        // int型の値を取得する処理(方法1)
        public int GetDataInt(int number)
        {
            return data[number].DataInt;
        }

        // string型の値を取得する処理(方法1)
        public string GetDataString(int number)
        {
            return data[number].DataString;
        }

        // bool型の値を取得する処理(方法1)
        public bool GetDataBool(int number)
        {
            return data[number].DataBool;
        }

        // outを使用して一括で値を取得する処理(方法2)
        public void GetAllData(int number, out int dataInt, out string dataString, out bool dataBool)
        {
            dataInt = data[number].DataInt;
            dataString = data[number].DataString;
            dataBool = data[number].DataBool;
        }

        // Dataのインスタンスを渡す処理(方法3)
        public Data GetClass(int number)
        {
            return data[number];
        }

    }

    // データを格納するクラス
    class Data
    {
        public int DataInt { get; set; }
        public string DataString { get; set; }
        public bool DataBool { get; set; }

        // コンストラクタ
        public Data(int number)
        {
            DataInt = number;
            DataString = number.ToString();
            DataBool = number > 0;
        }
    }

試したこと

特になし。

Comments

[gentaro]

> どの方法が正しいのか、もしくは、他に正しい値の渡し方があれば教えて下さい。

回答不能問題です。
「正しさ」の基準を「第三者が理解できる形で」明示してください。

特に基準がない場合、プログラムにバグがなく完全にあなたの意図通りに動いている限り、どのような実装をしても「正しい」です。

[退会済みユーザー]

コードを書く前に、何故Managerを挟んでいるかとか、具体的にどういうデータをやり取りしているか自分で正しくイメージ出来て人に伝えられないとダメです。

[K_Hirata]

変身が遅くなり申し訳ありません。

>>gentaro様
初心者なので、「正しさ」をうまく表現出来ないのですが、このプログラムは今後、別のフォームや、値を加工するクラスを追加する予定です。

ですので、「追加や変更に伴う修正をできる限り減らしたい」、「後からコードを見て値の処理の構造が理解しやすい」の２つの視点から、正しい値の受け渡しの方法を教えていただきたいです。

>>radian様
Managerクラスを挟んでいるのでは、将来的に別のフォームやクラスを追加して、値のやり取りを増やす際に、「Managerクラスを挟んでおけば、処理が複雑にならないのでは？」と考えたためです。
(フォームが3つある時に、それぞれが個別にDataクラスにアクセスしたら混乱するという考えです)

具体的なデータのやり取りについてですが、すみません、「具体的なデータ」の意味がよく分かりません。
ソフトで行いたいことは、ファイルから数値と文字列を取り出してDataクラスに格納、フォームからの操作でそれらを呼び出して表示するといったものです。

[gentaro]

「正しい」という表現を使わず、その２つの視点における最適解（あるいは最適解に至るための考え方）を知りたい、という形に質問文を修正すれば、質問方法として納得はできます。

[退会済みユーザー]

> 具体的なデータのやり取りについてですが、すみません、「具体的なデータ」の意味がよく分かりません。
> ソフトで行いたいことは、ファイルから数値と文字列を取り出してDataクラスに格納、フォームからの操作でそれらを呼び出して表示するといったものです。

まずファイルという情報が後付けで出てきたので、貴方以外の見ている人にはここでやっとDataの元のデータがファイルなのかというのが判った訳です。しかし、ファイルの形式も、フォームにどう対応させたいのかも、どういうデータが入っているかも具体的な事が何も明らかになっていません。（ファイルが複数か、Managerはそれをまとめ上げないといけないのか）
この状態で、正しいかとうかという漠然とした質問を投げられても、誰も適切な回答は返せないでしょう。要は、足りない情報で他人に設計を丸投げしているのです。

1、2、に問題があると判って居るなら、とりあえず3でやってみればよいのではないでしょうか。

Answer 1

変数アクセス用のクラスを定義して、そのインスタンスを各フォームに渡してアクセスさせればいいでしょ。
その変数の実体はどこにあるかなんか意識する必要もないですし。