C# 奖金计算器

Question

创建一个 Bonus Calculator 程序，它有两个重载方法——一个接受以双精度表示的工资和奖金，另一个接受以双精度表示的工资和整数形式的奖金。

我写了这个程序，我可以得到作为 int 的奖金来工作，但它们都作为 double 不起作用

namespace BonusCalculation
{
 class Bonus
 {
   static void Main(string[] args)
   { 
        double salary;
        int bonus;
        double bonusPercent;

        WriteLine("What is your salary?");
        salary = Convert.ToDouble(ReadLine());
        WriteLine("What is your bonus?");
        string bonusString = Console.ReadLine();
        if (int.TryParse(bonusString, out bonus))
        { CalcBonus(salary, bonus); }
        else if((double.TryParse(bonusString, out bonusPercent)))
        { CalcBonus(salary, bonusPercent); }

        WriteLine( "Your new salary is {0:c2}", CalcBonus(salary,bonus));
    }
  static double CalcBonus(double s,double b)
    {
        s = (s * b) + s;
        return s;
    }
 static double CalcBonus(double s, int b)
    {
        s = s + b;
        return s;
    }
  }
}

当我以双倍奖励运行该程序时，它不会进行数学运算。 感谢您的任何帮助。

Answer 1

问题在这里：

    if (int.TryParse(bonusString, out bonus))
    { CalcBonus(salary, bonus); }
    else if((double.TryParse(bonusString, out bonusPercent)))
    { CalcBonus(salary, bonusPercent); }

    WriteLine( "Your new salary is {0:c2}", CalcBonus(salary,bonus));

如果bonusString不是有效整数，则永远不会设置bonus ，并且WriteLine对CalcBonus的最后一次调用使用0作为bonus 值。

与其尝试推断奖金类型，不如让用户指定他们输入的是百分比还是值，并且只做一次数学运算而不是再次执行WriteLine调用。

Answer 2

 public partial class Form1 : Form
{
    // Constant field for the contribution rate
    private const decimal CONTRIB_RATE = 0.05m;

    public Form1()
    {
        InitializeComponent();
    }

    // The InputIsValid method converts the user input and stores
    // it in the arguments (passed by reference). If the conversion
    // is successful, the method returns true. Otherwise it returns
    // false.
    private bool InputIsValid(ref decimal pay, ref decimal bonus)
    {
        // Flag variable to indicate whether the input is good
        bool inputGood = false;

        // Try to convert both inputs to decimal.
        if (decimal.TryParse(grossPayTextBox.Text, out pay))
        {
            if (decimal.TryParse(bonusTextBox.Text, out bonus))
            {
                // Both inputs are good.
                inputGood = true;
            }
            else
            {
                // Display an error message for the bonus.
                MessageBox.Show("Bonus amount is invalid.");
            }
        }
        else
        {
            // Display an error message for gross pay.
            MessageBox.Show("Gross pay is invalid.");
        }

        // Return the result.
        return inputGood;
    }

    private void calculateButton_Click(object sender, EventArgs e)
    {
        // Variables for gross pay, bonus, and contributions
        decimal grossPay = 0m, bonus = 0m, contributions = 0m;

        if (InputIsValid(ref grossPay, ref bonus))
        {
            // Calculate the amount of contribution.
            contributions = (grossPay + bonus) * CONTRIB_RATE;

            // Display the contribution.
            contributionLabel.Text = contributions.ToString("c");
        }
    }

    private void exitButton_Click(object sender, EventArgs e)
    {
        // Close the form.
        this.Close();
    }
}

}