if用radiobuttons进行重构。

Question

有两个无线电按钮可以给出不同的价格。 示例：选择Elektric2和3天的租金。电动自行车= 200欧元 +每天30欧元（200 EU +（30*3））。

我该如何重构，因为我对所有选项都有60多行代码。

这是整个代码的一些：

if (radioButtonElectric2.Checked && radioButtonADay.Checked)
{
    electric2 = electric2 + 1 * 30;
    labelSubTotal.Text = "Subtotal:€" + electric2.ToString();
}
else if (radioButtonElectric2.Checked && radioButtonThreeDays.Checked)
{
    electric2 = electric2 + 3 * 30;
    labelSubTotal.Text = "Subtotal:€" + electric2.ToString();
}
else if (radioButtonElectric2.Checked && radioButtonSevenDays.Checked)
{
    electric2 = electric2 + 7 * 30;
    labelSubTotal.Text = "Subtotal:€" + electric2.ToString();
}
else if (radioButtonElectrical2.Checked && radioButtonFourteenDays.Checked)
{
    electric2 = electric2 + 14 * 30;
    labelSubTotal.Text = "Subtotal:€" + electric2.ToString();
}

Answer 1

您的方程式基本上归结为三分。

1. 基本价格
2. 租赁天
3. 数

将这三件事分为单个功能使您的代码变得更加简单。

public decimal GetBasePrice()
{
    return 
        radioButtonElectric1.Checked ? 60 :
        radioButtonElectric2.Checked ? 80 :
        radioButtonGas1.Checked ? 55 :
        0;
}
public int GetRentingDays()
{
    return
        radioButtonADay.Checked ? 1 :
        radioButtonThreeDays.Checked ? 3 :
        radioButtonSevenDays.Checked ? 7 :
        radioButtonFourteenDays.Checked ? 14 :
        0;
}
public decimal GetDailyRate()
{
    return 
        radioButtonElectric1.Checked ? 20 :
        radioButtonElectric2.Checked ? 30 :
        radioButtonGas1.Checked ? 25 :
        0;
}

最后，最终代码要简单得多：

decimal subTotal = GetBasePrice() + GetRentingDays() * GetDailyRate();