2013-03-21 87 views
-1

我有所有這些代碼塊的所有工作,但我想濃縮他們只是爲了看起來更好。有任何想法嗎?有沒有辦法壓縮這段代碼?

public static AccountEnum Direct() 
    { 
     const AccountEnum direct = AccountEnum .DIRECT; 
     return direct; 
    } 

    public static AccountEnum Partner() 
    { 
     const AccountEnum partner = AccountEnum .PARTNER; 
     return partner; 
    } 

    public static AccountEnum Resold() 
    { 
     const AccountEnum resold = AccountEnum .RESOLD; 
     return resold; 
    } 

    public static AccountEnum Referral() 
    { 
     const AccountEnum referral = AccountEnum .REFERRAL; 
     return referral; 
    } 
+3

http://codereview.stackexchange.com/可能會更好嗎? – 2013-03-21 15:15:27

+2

1.爲什麼你有一個函數來訪問一個枚舉值? 2.爲什麼你首先做一個常量而不是直接返回值? 3.也許這是codereview的問題。 – antonijn 2013-03-21 15:15:38

+0

我不認爲我明白你爲什麼不直接引用枚舉而不是靜態包裝? – Rikon 2013-03-21 15:15:39

回答

1

在這裏使用const沒有意義。你可以這樣做:

public static AccountEnum Direct() 
{ 
    return AccountEnum.DIRECT; 
} 

public static AccountEnum Partner() 
{ 
    return AccountEnum.PARTNER; 
} 

public static AccountEnum Resold() 
{ 
    return AccountEnum .RESOLD; 
} 

public static AccountEnum Referral() 
{ 
    return AccountEnum .REFERRAL; 
} 

雖然我懷疑有這些方法有很多的意義。

14

只允許人們直接訪問Enum。沒有理由擁有這些方法。

+0

該枚舉可能已經公開,編譯器會抱怨不一致的可訪問性,否則,不? – antonijn 2013-03-21 15:17:36

+0

我只是喜歡upvotes .. – 2013-03-21 15:17:38