c# - 如何在c#中使用switch将一行代码一分为二

Question

我有这行代码

acctStatusLabel.Text = acct.acctStatusCode == "APPR" || acct.acctStatusCode == "IACT"
                                       ? "Inactive"
                                       : "Closed";

我需要做的是更改它，以便 APPR 状态代码显示“已批准”，而 IACT 状态代码显示“非活动”

这是完整的代码集

if (acct.acctStatusCode == "DORM")
            acctStatusLabel.Text = "Dormant";
        else
            acctStatusLabel.Text = acct.acctStatusCode == "APPR" || acct.acctStatusCode == "IACT"
                                       ? "Inactive"
                                       : "Closed";

score 4 · Accepted Answer

像这样？

switch(acct.acctStatusCode)
{
    case "DORM":
        acctStatusLabel.Text = "Dormant";
        break;
    case "APPR":
        acctStatusLabel.Text = "Approved";
        break;
    case "IACT":
        acctStatusLabel.Text = "Inactive";
        break;
    default:
        acctStatusLabel.Text = "Closed";
        break;
}

更好的选择可能是使用静态字典：

{somewhere else}
private static Dictionary<string, string> Codes = new Dictionary<string, string>();

{in static constructor}
Codes.Add("DORM","Dormant");
Codes.Add("APPR","Approved");
Codes.Add("IACT","Inactive");

您的代码将类似于：

// set default value
acctStatusLabel.Text = "Closed";
// lookup decoded value
if(Codes.ContainsKey(code))
    acctStatusLabel.Text = Codes[code];

更好的方法是将代码/解码放在数据库表或其他可以更改而无需重新编译您的应用程序的数据源中。

score 2 · Accepted Answer

这应该可以解决问题：

switch (acct.acctStatusCode)
{
    case "DORM":
        acctStatusLabel.Text = "Dormant";
        break;
    case "APPR":
        acctStatusLabel.Text = "Approved";
        break;
    case "IACT":
        acctStatusLabel.Text = "Inactive";
        break;
    default:
        acctStatusLabel.Text = "Closed";
}

但是，最好将该逻辑移入acct对象本身。像这样的东西：

public string StatusCodeText
{
    get
    {
        switch (acctStatusCode)
        {
            case "DORM":
                return "Dormant";
            case "APPR":
                return "Approved";
            case "IACT":
                return "Inactive";
        }
        return "Closed";
    }
}

然后使用它变得简单得多：

acctStatusLabel.Text = acct.StatusCodeText;

“智能数据结构和愚蠢的代码比其他方式工作得更好。” - 埃里克·S·雷蒙德

score 1 · Accepted Answer

switch(acct.acctStatusCode)
{
    case "DORM":
        acctStatusLabel.Text = "Dormant";
        break;
    case "APPR":
    case "IACT":
        acctStatusLabel.Text = "Inactive";
        break;
    default:
        acctStatusLabel.Text = "Closed";
        break;
}

score 0 · Accepted Answer

像这样的东西

switch(acct.acctStatusCode) {
    case "DORM" : acctStatusLabel.Text = "Dormant"; break;
    case "APPR" : case "IACT" : acctStatusLabel.Text = "Inactive"; break;
    default : acctStatusLabel.Text = "Closed"; break;
}

score -1 · Accepted Answer

做就是了

if status =="xxxx"
   text="XXXX"
else if status == "yyy"
   text = "YYY"
else 
   text = "QQQ"

我知道这不是确切的代码。我的意思是我建议停止尝试用 ? 等等

c# - 如何在c#中使用switch将一行代码一分为二

5 回答 5

Related

Reference