リポジトリ項目をリポジトリインタフェースの外に作成する必要がありますか？

Question

同僚は、エンティティとDtosの規約を使用して、アプリケーションでリポジトリパターンを実装しています。これは彼の考えであり、私はイディオムに精通していません。

は現在、エンティティとのDTOとの主な相違点は以下のとおりです。

• エンティティは、（DTOSがあまりにも、私はそれが臭い見つけるしている）IDフィールドを持っています。
• DtosはCloneの方法を有する。
• エンティティプロパティはシリアル化に適したデータ型が多く、変換はデータマッパーで実行されます。

質問です：

私はレポに送っ、クライアント側で自由に私のDTOを作成し、されるべきか？または、新しいDtoインスタンスだけをrepoインターフェイスでに要求する必要がありますか？

これは違い次のようになります。

TDto dto = new TDto(); 
// edit dto properties 
repo.Add(dto) 

と

TDto dto = repo.Add(); // repo : Repository<TDto> 
// edit dto properties 
repo.Update(dto); 

preferrable方法はありますか？それが好みの問題で、私が第二の選択肢を好むなら、私はあるコーナーに自分自身をペイントしないように気を付けるべきですか？

（免責事項：私はこのEntity/DtoはシンプルなシリアライゼーションベースのCRUDのニーズしかない比較的シンプルなクライアントデスクトップアプリケーションでは過度だと思っていますが、解決している限り「パターン」に従います。問題の代わりに邪魔になっての）

UPDATE：私は現在直面して解決しようとしています

一つの問題は、「臭いイドプロパティ」です。現在、Idは追加時にDtoに設定されます。コードは次のようなものです：メソッドの呼び出しはレポで作成された値に自身のIDが設定されただけで、今、同じDTOを返す

public Patient Add(Patient patient) 
    { 
     patient.Id = Guid.NewGuid(); 
     var entity = patient.ToEntity(); 
     _patientsCache.Add(entity); 
     this.Salvar(); 
     return patient; 
    } 

ていることに注意してください。私の心はこれを見て、「これは正しくない」と思っていますが、私はそれを説得力を持って正当化することはできません。

Answer 1

最終的に、複雑さとアーキテクチャ上の意思決定については「正しい」答えはありません。あなたが指摘したように、アプリケーションが小さければ、エンジニアリングするには余りにも多くのレイヤーが必要かもしれません。一方、アプリケーションのサイズが大きくなると、後でリファクタリングするのは難しくなります。

DTOのインスタンス化の問題については、インスタンス化だけです。デフォルトのコンストラクタを呼び出していてコンストラクタが非効率的な動作をしている場合、つまり複雑なロジックを実行したり、副作用のあるものを実行するのではなく、デフォルトの "空の"インスタンスを設定するだけです。 DTO - あなたがどこでそれを行うかは本当に問題ではありません。

クライアントの方が簡単ですし、1回の往復を節約できると思います。

クライアントにDTOの「ID」フィールドについて知らせたくない場合は、internalフィールドまたはプロパティにして、リポジトリコードに設定します。