0

我是 C# 新手，我在使用 TextBox 验证等级字母（A、B、C、D、F）时遇到问题。现在，如果我输入一个完全符合其条件的等级字母，下面的代码将执行 if 语句而不是 else 语句，即使在单击 OK 按钮后小写然后大写也是如此。当我输入正确的等级字母时，它应该跳过 if 并继续执行 else 语句，但是我没有看到有什么问题。

    private void Button_Click(object sender, RoutedEventArgs e)
    {

        //automatically convert gradeLetter inputs to uppercase
        gradeLetter.Text = gradeLetter.Text.ToUpper();

        //check if gradeLetter entered is valid
        if (!string.IsNullOrWhiteSpace(gradeLetter.Text) || gradeLetter.Text != "A" || gradeLetter.Text != "B" || gradeLetter.Text != "C" || gradeLetter.Text != "D" || gradeLetter.Text != "F")
        {
            MessageBox.Show("Invalid grade letter or has an empty textbox!", "Caution!", MessageBoxButton.OK);
        }
        else
        {
            // switch statement to determine which 'gradeLetter' is being used 
            // and assign numerical numbers to 'gpa' to then be calculated.
            switch (gradeLetter.Text)
            {
                case "A": gradeW = 4.0;
                    break;
                case "B": gradeW = 3.0;
                    break;
                case "C": gradeW = 2.0;
                    break;
                case "D": gradeW = 1.0;
                    break;
                case "F": gradeW = 0.0;
                    break;
                default: // do nothing
                    break;
            }

            double result = (GPA += gradeW); //add to the gpa
            gCounter++; // increment the gpa entered
            result /= gCounter; // divide by the number of gpa entered
            result = Math.Round(result, 2, MidpointRounding.AwayFromZero); //round the result to two decimal places

            gpa.Text = result.ToString(); //convert result from int to string and display in 'gpa' TextBlock

            //append the input grade letters to 'gradeEntered' TextBlock
            gradeEntered.Text += gradeLetter.Text + System.Environment.NewLine;
       }
    }
4

3 回答 3

4

您可以使用包含有效成绩字母的列表，而不是使用许多条件（这会使阅读变得有些困难）：

string gradeLetter = "A";

List<string> gradeLetters = new List<string> { "A", "B", "C", "D", "F" };

if (!gradeLetters.Contains(gradeLetter))
{
    // invalid grade letter
}

else
{
    // carry on
}

你的代码：

if (!string.IsNullOrWhiteSpace(gradeLetter.Text) || gradeLetter.Text != "A" || gradeLetter.Text != "B" || gradeLetter.Text != "C" || gradeLetter.Text != "D" || gradeLetter.Text != "F")

永远都是真的。假设等级为 B。它将检查它是否与gradeLetter.Text != "A"短路，并且是真的，因为“B”！=“A”。

于 2014-02-07T03:17:02.583 回答
2

您当前的 if 语句的问题是有效输入仍然会显示为无效。如果输入是“B”，它将看到它不是“A”（在您的 if 语句中），这意味着整个标准都是真的。

将您的 if 语句更改为：

if (string.IsNullOrWhiteSpace(gradeLetter.Text) || !(gradeLetter.Text == "A" || gradeLetter.Text == "B" || gradeLetter.Text == "C" || gradeLetter.Text == "D" || gradeLetter.Text == "F"))

它本质上是在询问“您的输入是空的还是您的输入不是 A、B、C、D 或 F？

于 2014-02-07T03:16:11.237 回答
2

您还可以使用 Dictionary 方法：

gradeLetter.Text = gradeLetter.Text.ToUpper();
IDictionary<string, double> grades = new Dictionary<string, double>
{
    {"A", 4.0},
    {"B", 3.0},
    {"C", 2.0},
    {"D", 1.0},
    {"F", 0.0}
};

if (!grades.ContainsKey(gradeLetter.Text)){
    MessageBox.Show("Invalid grade letter or has an empty textbox!", "Caution!", MessageBoxButton.OK);
}
else{
    gradeW = grades[gradeLetter.Text];
    // other stuff
}
于 2014-02-07T16:00:32.007 回答