2014-01-27 41 views
-1

我有一個類User包含這些屬性的設計模式,讓操作

private string GSM; 
private int ID; 
private string SessionID; 
private string SIM; 

此外,它具有以下靜態方法:

public static bool isExistedGSM(string GSM){ 
    return false; 
} 
public static bool isExistedSIM(string SIM){ 
    return false; 
} 
public static User CreateNewUser(int ID, string SIM, string GSM){ 
    return new User(ID, SIM, GSM); 
} 

在此外,它包含此本地功能

public void updateSesssionID(string newSessionID){ 
    this.SessionID = new SessionID; 
} 
get and set functions for all attributes 

有沒有更好的設計模式來幫助我實現相同的功能,但更好的架構?

任何建議,將不勝感激

謝謝

回答

1

我真的不能明白isExisted...手段。

您可能想要將CreateNewUser移動到某個用戶工廠(這是設計模式)。

另外,updateSessionID應該只是setSessionID--就像在JavaBeans中一樣。

順便說一句:是Java還是C#?堅持一個慣例 - 以大寫或小寫開始方法,但不要混合它們。字段名也是一樣。

+0

尊敬的@FilipMalczak先生,感謝您寶貴的答覆,我在我的服務器中同時使用了Java和C#。另外,我會關心大寫。總之,你是否建議我應該使用**(瞭解更多)**關於_user Factory_設計模式? – user2059935

+0

不,工廠是一種設計模式。另外,我的建議是離開更新...方法並堅持(is/get)/ set。 –