[共通クラス]複数プロジェクトのフォームと共有するクラスの各フォームとの繋げ方

前提・実現したいこと

■ 前提条件 ■
1つのソリューションで複数のWindowsフォームアプリケーションを作っています。
具体的には、2つの機能が違うフォームアプリケーションを作ります。
その為、ソリューションの中に、複数プロジェクトを作り、各プロジェクトごとでフォームを管理しようと考えています。（この構造が客先希望の為）

また、フォームを作るのは、1つのフォームに対して1人です。
それぞれのフォームから共通で使うものは、共通プロジェクトとして作り、共通で使えるフォーム・クラスを作る方向で進めています。

今回は、自分が共通クラスを作る役回りになりました。
マージ時の不具合を防ぐため、出来るだけ共通クラスは共通で使えるようにしたいと考えています。

■ 実現したい事 ■
共通クラスを使用する際に、クラス内で処理を行い、各フォームへデータを渡したいと考えています。

各フォームのインスタンスを取得し、値を渡す...ということを考えましたが、そうすると、各フォーム毎に入り口を作る必要があるので、そこをスマートに出来ないかと、皆様にお知恵を借りたいと考え、投稿しました。

よろしくお願いいたします。

該当のソースコード

フォームからインスタンスを取得する部分

VBNET
1    'フォームとの連携用
2    Private _conectForm As System.Windows.Forms.Form
3    ''' <summary>
4    ''' フォームとの連携用
5    ''' </summary>
6    ''' <returns></returns>
7    Public Property FrmInstance() As FrmMain1
8        Get
9            Return _conectForm
10        End Get
11        Set(value As FrmMain1)
12            _conectForm = value
13        End Set
14    End Property
15
16    ''' <summary>
17    ''' フォームとの連携設定
18    ''' </summary>
19    ''' <param name="form"></param>
20    Public Sub setForm(ByVal form As FrmMain1)
21        FrmInstance = form
22    End Sub

下記のように共通クラスからリストボックスにデータを渡したい。

VBNET
1    ''' <summary>
2    ''' 伝達監視
3    ''' </summary>
4    ''' <param name="token"></param>
5    Private Sub LinkLoop(ByVal token As CancellationToken)
6            If System.IO.Directory.Exists(FolderName1) Then
7                'ファイル確認作業(SearchFile: フォルダからファイルを検索して列挙)
8                Dim sarchMae As IEnumerable(Of String) = SearchFile(FolderName1)
9                If sarchMae IsNot Nothing Then
10                    'リストボックスへ反映
11                    FrmInstance.AddListBoxSelect(sarchMae)
12                End If
13            End If
14    End Sub

補足情報（FW/ツールのバージョンなど）

OS：Windows10
開発環境：VisualStudio2017 / .NET Framework4.0

行動規範の内容に同意します

回答3件

共通処理というのであれば、他のものを意識しなければならない形は良くないと思います。

リストボックスにデータを表示したいのであれば、フォーム側が共通処理を呼び出して、戻り値などでデータを受け取ってフォーム側で表示処理を行うべきかと思います。

投稿2021/02/09 10:53

YAmaGNZ

総合スコア10469

TEC_S

2021/02/10 01:09

ご回答、ありがとうございます。共通処理の役割、確かにおっしゃられる通り、他のものを意識しなければいけない形は良くないというのは、回答いただいたことで、納得しました。あくまで、共通クラスはデータを返す、、、までを役割とすれば良いですね。

行動規範の内容に同意します

ベストアンサー

YAmaGNZさんと同じです

そもそも、共通クラス側で呼出元のフォームを参照しようとすると循環参照でビルド出来ないはずです

また、質問本文の共通クラスのメソッドですが、なぜリストボックスへ設定までやるのでしょうか？
共通
クラスの担当すべき責務はデータ作成まで
リストボックスへ設定というのはデータの利用方法なので共通クラスではなくフォームクラスで行うべきです

持たすべき機能と責務はどこまでなのかをきちんと分けてそこまでにとどめておくようにすべきだと思います
同じ事やってるから全部押し付けろってのは激重万能クラスというスパゲティを作る悲劇の始まりにしかなりません

投稿2021/02/09 11:39

len_souko

総合スコア1363

TEC_S

2021/02/10 01:15

確かに、ビルド出来ないですよね。すみません。。。クラスの担当すべき責務、データに関してのみですね。リストボックスへの設定は、呼び出すフォーム側でやってもらう方が、使う側にとっても良いですね。自分の設計時に意識すべきところが、ちょっとあいまいだったと思います。今後は、そこら辺を分けて考えるようにしたいと思います。ありがとうございました。

行動規範の内容に同意します

ふつうに「複数のプロジェクト」で使われてる標準ライブラリ等がどういうインターフェイスになってるかを見て考えりゃわかると思うけど