メソッドのパラメーターリストにオブジェクトまたはオブジェクト識別子を含める必要がありますか？

私たちのチームは次の議論をしています：

次の2つのメソッドがあるとします。

public Response Withdraw(int clubId, int terminalId,int cardId, string invoice, decimal amount);

public Response Withdraw(Club club, Terminal terminal,Card card, string invoice, decimal amount);

送信されたものは単なるIDです。

片方は、最初の方法が正しいと言います。ターミナルとクラブのIDしかないので、他に何もないことは明らかです。これが私のアプローチです。

反対側は、2番目の方法の方が柔軟性があるため、正しいと言います。

私たちはオブジェクトパラメータのアイデアに精通していますが、反対側もオブジェクトパラメータはオブジェクトをプロパティとして持つべきだと考えています。

正しいアプローチはどれですか？

たぶん3番目のさらに良いアプローチがありますか？

答えはコンテキストに依存します。

クライアントがそれらすべてのオブジェクトをすでに使用できると予想される場合は、オブジェクトパラメータを使用します。そうしないと、コードは必要以上に複雑に見えます。（例えば、彼らはのような呼び出しを持つでしょうclub.getId()。）

クライアントが簡単に利用できるIDしか持っていない場合は、実際にIDだけが必要な場合にクライアントがそれらすべてのオブジェクトをアセンブル/ロードする必要がない場合があるため、おそらく2番目のアプローチの方が適しています。

オプションは、両方のメソッドを提供することです。これにより、クライアントはどちらを使用するかを選択できます（これによりAPIが乱雑にならない場合）

一般に、オブジェクトパラメータはより拡張可能です。将来的に作業を行うために別のデータが必要になった場合、その追加情報を取得する別のメソッドを導入する必要がないためです。

最後に、メソッドのシグネチャは、メソッドが何をするか（あなたの場合、正確にはネットワークを介して行われること）の詳細によって指示されるべきではありません。APIは抽象的に意味をなす必要があるため、実装が変更されても、混乱することはありません。

最初のアプローチは、原始的な強迫観念を示しています。intとstringを渡すため、プログラマーが間違いを犯しやすい（たとえば、clubIdをterminalIdパラメーターに渡す）のは非常に簡単です。これはバグを見つけるのを難しくします。

2番目の例では、ターミナルが予想されるときにクラブを渡すことは不可能です。これはコンパイル時のエラーになります。

それでも、私はまだ見ますstring invoice。請求書は本当に文字列ですか？どういうamount意味ですか？これはおそらく通貨価値です。

「ネットワーク経由で送信されるものは単なるIDです」という質問で述べました。これは正しいですが、この要件によってドメインが不明瞭にならないようにしてください。

このアプローチを支持して私が見た最も良い説明は、Object Calisthenicsのルール3でした：

int自体は単なるスカラーであるため、意味がありません。メソッドがintをパラメーターとして取る場合、メソッド名はインテントを表現するすべての作業を行う必要があります。同じメソッドがHourをパラメーターとして取る場合、何が起こっているかを確認する方がはるかに簡単です。このような小さなオブジェクトは、Hourパラメーターを受け取るメソッドにYearを渡すことができないため、プログラムをより保守しやすくすることができます。プリミティブ変数を使用すると、コンパイラーは意味的に正しいプログラムを作成するのに役立ちません。オブジェクトを使用すると、たとえ小さなオブジェクトであっても、コンパイラとプログラマの両方に、値とは何か、なぜ使用されているのかについての追加情報が提供されます。

これには正解はありません。どちらのオプションもジョブに適している可能性があります。ほぼすべての方法で、請求書が私の額に溝を生みました。コードを読んだことから何がわかるのかわかりません。

IDを送信する場合は、両方のシステムをそれが表すものと密接に結合する必要があります。ClubIDは、clubsテーブルのキーです。さらに言えば、呼び出し元と呼び出し先の両方が、Clubsテーブルの呼び出し内容とそのデータベースを合意する必要があります。制約を課したくない、または課せない場合は、一般的な説明を使用してオブジェクトを渡します。ネイティブ、シリアル化、xml、名前=値は何でも、iniファイル:)

あなたが特定したように、それはあなたが「ネットワーク経由で」あなたを犠牲にするでしょう。識別子を送信するだけでそれを回避すると、別の場所でコストがかかります。だから、あなたに最も害を与えないのは、今（または後である可能性があります...）は、善対悪の指標です。