2013-03-21 76 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

相关问题