是否有重構此switch語句的好方法？

我現在有一個switch語句，看起來很笨重。它可以工作，但它非常笨重和重複性，因爲我實際上有另一個開關語句（它需要一個不同的變量）。是否有重構此switch語句的好方法？

if (time == 15) 
{ 
    percent = policyGroup[0]; 
} 
else 
{ 
    percent = policyGroup[1]; 
} 

var currentVal = Convert.ToDouble(profit[i]); 
switch (CSR[i]) 
{ 
    case "Buford": 
     Buford += currentVal * percent; 
     break; 
    case "Cain": 
     Cain += currentVal * percent; 
     break; 
    case "Young": 
     Young += currentVal * percent; 
     break; 
    case "Thomas": 
     Thomas += currentVal * percent; 
     break; 
    case "Armendt": 
     Armendt += currentVal * percent; 
     break; 
    case "Vanover": 
     Vanover += currentVal * percent; 
     break; 
    case "Phillips": 
     Phillips += currentVal * percent; 
     break; 
    case "Smalley": 
     Smalley += currentVal * percent; 
     break; 
    case "Pitman": 
     Pitman += currentVal * percent; 
     break; 
    case "Huse": 
     Huse += currentVal * percent; 
     break; 
    case "Nichols": 
     Nichols += currentVal * percent; 
     break; 
}

來源

2016-11-15 son rohan

字典或使用反射.... –

[重構的可能的複製長開關語句]（http://stackoverflow.com/questions/20586041/refactor-long-switch-statement） – Usman

如果這些變量是不可變的類型，那麼你只能用反射 – Steve

必須看起來真的矮胖以及部分是所有變量的聲明：

double Buford = 0, Cain = 0, Young, Thomas, Armendt, ..., Pitman;

這些變量不應該被硬編碼;相反，你應該使用存儲所有每名值的關聯容器：

IDictionary<string,double> valByName = new Dictionary<string,double> { 
    {"Buford", 0} 
, {"Pitman", 0} 
};

現在你可以這樣做：

valByName[CSR[i]] = currentVal * percent;

來源

2016-11-15 20:38:51 dasblinkenlight

是否有重構此switch語句的好方法？

回答

相關問題