例に示すパターンのとき、メソッドをコールすべき適切な場所はどこでしょうか？

Question

###前提・実現したいこと
お世話になります。
クラスZのメソッドに引数を与えてデータを取得してクラスAのDataプロパティを更新したいです。
クラスZのGetDataMethodをコールするにはクラスBのNumberプロパティとValueプロパティとクラスAのCodeプロパティの値が必要になります。
クラスBではクラスAのリストを持ちクラスAのCodeは変更されません。
NumberまたはValueが変更されたタイミングでクラスAのDataプロパティを更新する感じです。

私はパターン①、パターン②の方法を考えました。
この場合、どちらが適切なのかまた別の方法があれば教えてください。

※コードはVBで記載しましたが言語は問いません。

###該当のソースコード

       Public Class Z
            Public Shared Function GetDataMethod(code As String, number As Integer, value As Integer) As Object
                Return Nothing
            End Function
        End Class

        Public Class A
            Public ReadOnly Property Code As String
            Public Property Data As Object
            Public Sub New(code As String)
                Me.Code = code
            End Sub
            Public Sub GetDataMethod(number As Integer, value As Integer)
                'パターン①
                Data = Z.GetDataMethod(Code, number, value)
            End Sub
        End Class

        Public Class B
            Public Property Number As Integer
            Public Property Value As Integer
            Public Property Items As List(Of A)
            Public Sub GetDataMethod()
                For Each Item In Items
                    'パターン①
                    Item.GetDataMethod(Number, Value)
                    'パターン②
                    Item.Data = Z.GetDataMethod(Item.Code, Number, Value)
                Next
            End Sub
        End Class

###補足情報(言語/FW/ツール等のバージョンなど)
Microsoft Visual Studio 2015 Community(VB)
.Net Framework 4.7

Answer 1

このような記事を見つけたため、備忘録的に記載します。
どのメソッドをどのクラスで実装すべきか

この記事では2つのクラスがある場合、
「両方の属性を参照しながら処理するメソッドはどちらのクラスに実装すべきなのか？」
という問いの答えが記載されています。

原則は以下の２つ
・ヘッダと明細のような上下関係がクラスにある場合、必要な属性の最上位に当たるクラスのメソッドとする
・クラスの関係が対等または並列である場合、第3のクラスを作ってそのメソッドとする(Mediatorパターンの利用)

今回私が疑問に思っていたことは1つ目の原則に当たりそうです。
よって、Zuishin様とozwk様のご指摘通り、パターン②のクラスBにメソッドを実装するほうが良いようです。

Answer 2

クラスZをなくしてAのメソッドにします。

Public Class B
    Public Property Number As Integer
    Public Property Value As Integer
    Public Property Items As List(Of A)
    Public Sub GetDataMethod()
        For Each Item In Items
            Item.Update(Number, Value)
        Next
    End Sub
End Class

Answer 3

違いがわかりません。
メリット・デメリットが仮にあるとしてもわずかでしょう。
直すなら A B Z が互いに依存しあっている構造自体を直したいです。

追記

あえて言うならパターン②でしょうか。
余計なメソッドが一つ減ってわずかに読みやすくなります。

Comments

[mikupedia]

Zuishin 様
早々にご回答していただきありがとうございました。
＞直すなら A B Z が互いに依存しあっている構造自体を直したいです。
と回答していただきましたが、その構造に興味があります。
もし面倒でなければコードを記載していただけないでしょうか？

[Zuishin]

与えられた条件だけでは無理です。
コードではなく仕様の問題です。