同様のデータを取得するデータアクセスメソッドの重複を防ぐ方法は？

私がチームで取り組んでいるほぼすべてのプロジェクトで、同じ問題が侵入しているようです。誰かがデータを必要とするUIコードを書き、データアクセスメソッドを書きます。

AssetDto GetAssetById(int assetId)

1週間後、他の誰かがアプリケーションの別の部分で作業してAssetDtoいて、「承認者」を含む必要がありますが、次のように書きます：

AssetDto GetAssetWithApproversById(int assetId)

1か月後、誰かがアセットを必要としていますが、現在は「質問」（または「所有者」または「実行中のリクエスト」など）が含まれています。

AssetDto GetAssetWithQuestionsById(int assetId)
AssetDto GetAssetWithOwnersById(int assetId)
AssetDto GetAssetWithRunningRequestsById(int assetId)

そして、そのようなメソッドGetAssetWithOwnerAndQuestionsByIdが出現し始めると、さらに悪化します。

現れるパターンが表示されます。オブジェクトは大きなオブジェクトグラフにアタッチされており、このグラフのさまざまな部分をさまざまな場所に配置する必要があります。

もちろん、ほとんど同じことをするメソッドがたくさんあるのを防ぎたいです。それは単にチームの規律の問題ですか、それともこれを防ぐために使用できるパターンがありますか？場合によっては、個別のメソッドを使用することが理にかなっている可能性があります。つまり、実行中のリクエストでアセットを取得するのはコストがかかるため、常にこれらを含めたくありません。そのような場合の対処方法は？

構文的には、流動的なインターフェイスを備えた中間クエリ構築オブジェクトを作成します。

// all the basic, cheap to query fields
AssetDto a = AssetRetriever(asset_id).fetch() 

// some common expensive fields
AssetDto a = AssetRetriever(asset_id).withOwner().withQuestion().fetch() 

// numerous less common fields may not command dedicated methods
AssetDto a = AssetRetriever(asset_id).withFields("foo", "bar").fetch() 

// Better yet, use an enum and enjoy static checking
AssetDto a = AssetRetriever(asset_id).withFields(F_OWNER, F_QUESTION).fetch() 

実装するのに十分明白であることを願っています。実際にデータベースにアクセスする唯一の方法はfetch()です。

大きなオブジェクトを扱う場合、これは非常に一般的です。新しいメソッドを追加するとパフォーマンスが向上しますが、保守性が大幅に低下します。また、これら2つの中から選択する必要があります。

一般的に使用されるデータ（必ずしも最小ではない）を返すメソッド、オブジェクト全体を返すメソッド、そして最も高価なリソースの場合はさらにいくつかのメソッドがあることをお勧めします。

別のアプローチは、AssetQuestions GetAssetQuestionsById(int assetId)またはのように、オブジェクトの必要なフィールドのみを返すメソッドを持つこと Owners GetAssetOwnersById(int assetId)です。

これに加えて、データの取得に関するいくつかのルールを確立する必要があります。たとえば、誰かがオブジェクトの5つのフィールドを必要とし、8を返す既存のメソッドがある場合、既存のメソッドを使用する必要があります。

私は最近この同じ問題を経験し、次の解決策を採用しました：

データアクセスメソッドは、単一のリソース（データベーステーブルなど）からのみデータを取得する必要があり、プロセスがメインオブジェクトに追加された関連オブジェクトを必要とする場合、それぞれのオブジェクトを担当するメソッドを呼び出す必要があります。

このように、承認者のいるアセットが必要な場合は、オブジェクトを結合するファサードメソッドを作成する必要があります。

例：

public Class AssetFacade {

   public AssetDto getAssetWithQuestionsByAssetId(int assetId) { 

      AssetDto asset = AssetDao.getAssetById(assetId);
      List<QuestionDto> questions = AssetDao.getQuestionsByAssetId(assetId);
      asset.setQuestions(questions);

      return asset;
   };
 }

それは単にチームの規律の問題ですか、それともこれを防ぐために使用できるパターンがありますか？

はい、それはチームの命名パターンにおけるいくつかのガイドラインの問題です。GetEntityById（）、GetAllEntities（）、SetEntity（）、DeleteEntityById（）などのシンプルな4つのメソッドを設定できます。

さらに、名前が付いた2つのAssetSimpleDto GetAssetById(assetId)dtoとと呼ばれる別の詳細なdto がある場合がありますAssetDto GetAssetDetailById(assetId)。最初の方法とdtoは最小限に抑えるようにカスタマイズされていますが、2番目の方法は、機能に必要なすべての関連情報を提供します。