大規模で膨大なクラスをより小さなクラスに分割する方法を教えてください。

Question

私はあまりにも多くのことをやっていると思う大きな「マネージャー」クラスを持っていますが、それをより論理的な単位に分割する方法はわかりません。

一般クラスには、基本的には、以下の方法で構成されて話す：

 
class FooBarManager 
{ 
    GetFooEntities(); 
    AddFooEntity(..); 
    UpdateFooEntity(..); 
    SubmitFooEntity(..); 
    GetFooTypes(); 
    GetBarEntities(); 
} 

マネージャークラスは私のビジネスロジックの一部であり、すべてのCRUDが含まれているデータ・アクセス・レベルで別の「マネージャ」クラスのインスタンスをconstainsすべてのエンティティの操作

データアクセスレイヤーから異なるエンティティを取得しているため、データエンティティをビジネスエンティティに変換するためのマネージャクラス以外の場所にコンバーターがあります。

マネージャクラスの理由は、私がunittestingを行うときに、それぞれの "マネージャ"クラスを模擬したいと思ったからです。各マネージャクラスは現在1000以上のlocであり、それぞれ40〜50のメソッドを含んでいます。私はそれらがかなり肥大していると考えて、すべてのデータアクセスロジックを単一のクラスに入れるのは難しいと感じています。私は何をすべきですか？

どのように私はそれらを分割について行くと私は使用する必要があります特定のデザインパターンはありますか？

Answer 1

一般的でない限り、すべてのデータアクセスを1つのクラスに入れるべきではありません。まず、データアクセスクラスを、オブジェクトまたは関連するオブジェクトグループ（CompanyManager、CustomerManagerなど）ごとに1つのマネージャに分割します.1つの「godクラス」を使用してマネージャにアクセスする必要がある場合は、各マネージャのインスタンスを使用できますあなたの真のマネージャークラスで

Answer 2

 /FooManager 
Manager     (derive from Manager) 
     \ BarManager 

は自己説明

Answer 3

私は組成物を使用してお勧めであるべき。マネージャーがやっている機能について考えてみましょう。単一の責任の行に沿ってそれらを分割します。 FooBarManagerのほとんどは、Fooとbarエンティティのコレクションです。だから、最低でも、FooBarManager

public class EntityCollection<T> : IList<T> 
where T : BaseEntity 
{ /* all management logic here */} 
public class FooCollection : EntityCollection<foo> {} 
public class BarCollection : EntityCollection<bar> {} 
public class FooBarManager 
{ 
public FooCollection { /*...*/ } 
public BarCollection { /*...*/ } 
public FooBarManager() : this(new FooCollection(), new BarCollection()){} 
public FooBarManager(FooCollection fc, BarCollection bc) { /*...*/ } 
} 

Answer 4

あなたFooBarManagerから収集ロジックを抜け出すにはGod Objectアンチパターンによく似ています。

あなたのような状況では、Patterns of Enterprise Application Architectureに掘り下げて検討してください、Martin Fowler一見すると、Data Mapperを作成するように見えます。しかし、あなたのニーズに十分かもしれないActive Recordのような代替案を検討してください。

また、お使いのプラットフォームにORM library/softwareを使用することを検討してください。正当な理由なしに自分自身を構築することは、これらのツールによってすでに多かれ少なかれ解決済みの多くの問題に直面するだけです。