OOP：この状況では、どのようなクラス設計が適切なはずですか？

Question

私はGUIから来るオブジェクトguiObjectを持っています。そのデータフィールドに基づいて、適切なクラスのdomainObjectをインスタンス化する必要があります。 DomainClassAまたはDomainClassBのいずれかになります。

DomainClassAは、1つの整数コンストラクタパラメータintParamA（guiObject.fieldAから）を持ちます。

DomainClassBには、1つの整数コンストラクタパラメータintParamB（guiObject.fieldBから）があります。 guiObjectから必要なフィールドをとり、私はAbstractFactoryを作った。この問題を解決するために

、GuiClassから適切なフィールドを持つDomainClassAFactoryまたはDomainClassBFactoryのいずれかをインスタンス化し、それをretuns。次に、これらのファクトリのいずれかcreate()は、domainObjectを正しくインスタンス化します。

しかし、今、guiObject.fieldCに応じて、私はdomainObjectをインスタンス化する前にintParamAとintParamB（1により、すなわち減少）を変更する必要があります。これを達成するためには、それぞれの異なるタイプの "パラメータ変更"のための別々のファクトリをそれぞれDomainClassに対して作成し、その後、正しいファクトリを生成する別々の抽象ファクトリを作成する必要がありました。それは醜いと思われ、それは醜いように見えます。

正しいデザインは何ですか？

Answer 1

がすべてである必要がない限り、guiObjectを呼び出すファクトリを持ち、正しいタイプを返します。

多くのアプリが必要としない過度の分析は、あまりにも多くのものにつながります。厳密に必要な場合にのみ抽象レイヤーを追加します。彼らは通常はありません。

Answer 2

私は個人的にはむしろあまりそれを広めるためにしようとするよりも、元の工場出荷時にすべてこのロジックを置くところ：

public DomainObj getDomainObj(GuiObject guiObject) { 
    int param = guiObject.someField ? guiObject.intParamA : guiObject.intParamB; 
    param = guiObject.fieldC ? param : param - 1; 
    return guiObject.someField ? new DomainClassA(param) : new DomainClassB(param); 
}