通用DAO和類型的業務對象 - 的方式來減少代碼

我已經在代碼中寫了下面GenericDAO如下通用DAO和類型的業務對象 - 的方式來減少代碼

abstract public class DAO <T extends AbstractBusinessObject> { 

// CRUD: create, read, update, delete 

public abstract T create(Class<T> abObj); 

public abstract T read(); 

public abstract void update(T abObj); 

public abstract void delete(T abObj);

但occasionaly當我實現自定義的DAO（如DAOinMemory）和它的方法，如create我必須寫下面的代碼：

public T create(Class<T> abstractObjectType) { 

    IDGenerator IDGenerator = IDGenerator.getInstance(); 

    if (BusinessEntityFirst.class.isAssignableFrom(abstractObjectType)) { 
     BigInteger id = IDGenerator.getID(); 
     BusinessEntityFirst bef = new BusinessEntityFirst(id); 
     return (T) bef; 
    } 

    if (BusinessEntitySecond.class.isAssignableFrom(abstractObjectType)) { 
     BigInteger id = IDGenerator.getID(); 
     BusinessEntitySecond bes = new BusinessEntitySecond(id); 
     return (T) bes; 
    } 

    if (BusinessEntityThird.class.isAssignableFrom(abstractObjectType)) { 
     BigInteger id = IDGenerator.getID(); 
     BusinessEntityThird bet = new BusinessEntityThird(id); 
     return (T) bet; 
    } 

    if (abstractObjectType == null) { 
     throw new NullPointerException("Can't create instance for:" + abstractObjectType); 
    } 
    return null; 
}

我看到create - 方法實施的一些地方看起來像麪條代碼（三個IF-CO與.isAssignableFrom方法），我無法做任何事情來克服它。你能否建議一些如何爲這部分代碼編寫一段代碼以擺脫Spaghetti代碼？

來源

2016-01-08 aestet

我會爲業務類製作單獨的DAO類（相應地鍵入）。正如你注意到你自己一樣，你試圖將它們全部塞進一個班級，你完全沒有辦法。 – Kayaman

您可以在類中擴展抽象類，並說C，然後擴展類C的類。然後在類C中定義該創建方法，並在擴展類中使用它們。這聽起來不錯嗎？ –

你可以看看Factory Design Pattern和Abstract Factory Design Pattern這應該有助於你設計你的代碼。在你的情況下，抽象工廠模式將是更好的設計選擇。

工廠設計模式根據需要處理對象的創建。所以在你的情況下，你將有一個對象生成器類，它將在工廠生成所需的對象。

來源

2016-01-08 19:58:17

通用DAO和類型的業務對象 - 的方式來減少代碼

回答

相關問題