C# と VB.NET の質問掲示板

わんくま同盟

ASP.NET、C++/CLI、Java 何でもどうぞ

C# と VB.NET の入門サイト


■92855 / )  Re[1]: 自作クラスからのフォームコントロール操作
□投稿者/ Azulean (1094回)-(2019/11/02(Sat) 23:07:07)
2019/11/02(Sat) 23:10:53 編集(投稿者)

No92854 (amanda102 さん) に返信
> "一般論としてコントロールをpublicメソッドに渡すような設計はあまり推奨できません。"
>
> 何が問題になるのでしょうか?

いろいろと切り口はありそうです。

・スレッドの考慮が忘れてしまいがち
・複数のクラスにコントロールのインスタンスを渡してしまっていると、コードの流れの把握が難しい、デバッグしづらくなる(誰がいつどこで書き換えたのか…?となる)
・誰がそのインスタンス(コントロール)を管理するのか曖昧になる
・寿命管理が弱くなる (フォームを閉じた後もそのクラスが生き残っているなど)
・クラスの責務が広くなる
・クラスの再利用性が下がる (少し設計を頑張れば、コントロールと癒着せずとも作れる→再利用性が上がるかも)
・コントロールに限らず、フィールド変数の多いクラスは理解する、修正するのが大変になりやすい
・コントロールに限らず、いろいろなクラスと密結合していると仕様変更に弱くなる


おそらく、下記のスレッドなのかと思いますが、この例であれば、「テキストボックス自体を渡さなくてもクラスは成り立つ」(責務を狭く・小さくできる)ということだと思います。
https://ja.stackoverflow.com/a/28646


> --
> 例@
> Formに貼り付けているボタンをクリック、
> 自作クラスのメソッドが呼び出され値をFormにあるテキストボックスに代入
> --

私もこの例の場合、テキストボックスをクラスに渡す…を第一候補とせずに考えますね。

1. クラスは文字列を生成するだけの責務にできないか?
2. 生成するタイミングが別だとしても、イベントで通知、コールバックで通知などの方式が良いと言うことはないか?
3. テキストボックスに代入するのはこのクラスに限定され、かつこのテキストボックスと同種のパターンがあるというように、部品化して問題なく、メリットがあるか?
 (例:ファイル名を D&D を受け取るヘルパークラスなど)

返信 編集キー/


管理者用

- Child Tree -