1

这是 C# 程序我在这行代码中有一个问题：

我输入大于或等于 400 的=数字它运行我输入的数字小于或等于 500=它运行我输入介于 400 和 500 之间的数字，例如 401 到 499=它运行我清空输入 = 错误我输入字母 = 错误

请注意，我运行的代码仅单独接受整数，它运行的代码以及不接受空输入的代码也运行

你觉得哪里不对？

        int parsedValue;
        if (int.Parse(txtVacate.Text) <= 400 || int.Parse(txtVacate.Text) >= 500)
            MessageBox.Show("Romms provided is not vacant or does not exist at all.");

        else if (txtVacate.Text == " ")
            MessageBox.Show("You provide empty");

        else if (!int.TryParse(txtVacate.Text, out parsedValue))
            MessageBox.Show("Please provide right info");
        else
        {
            MySqlConnection connection = null;
            string hostname = "localhost";
            string database = "aparece_hoteldb";
            string username = "root";
            string password = "";
            connection = new MySqlConnection("host=" + hostname +
                                            ";database=" + database +
                                            ";username=" + username +
                                            ";password=" + password + ";");


            string table = "reservations";
            string query = "DELETE FROM reservations WHERE RoomNumber = " + txtVacate.Text;
            connection.Open();
            MySqlDataAdapter da_res = null;
            DataSet ds_res = null;
            ds_res = new DataSet();
            da_res = new MySqlDataAdapter(query, connection);
            da_res.Fill(ds_res, table);
            MessageBox.Show("Room" + " " + txtVacate.Text + " " + "is now Vacant please reload!");
            dataGridView2.DataSource = ds_res.Tables[table];
            this.Close();
4

3 回答 3

3

恐怕你的逻辑是错误的。如果txtVacate.Text不包含整数，您的代码将会崩溃。这是评估不同条件的正确顺序。

int parsedValue = 0;
bool isValid = int.TryParse(txtVacate.Text, out parsedValue));
if (txtVacate.Text == " ") // Better: if (txtVacate.Text.Trim() == string.Empty)
{
    MessageBox.Show("You provide empty");
}
else if (!isValid)
{
    MessageBox.Show("Please provide right info");
}
else if (parsedValue <= 400 || parsedValue >= 500)
{
    MessageBox.Show("Romms provided is not vacant or does not exist at all.");   
}
else
{
    ...
}
于 2013-03-14T02:37:47.273 回答
1

您可以简化这几个步骤。

首先，检查以确保输入不为 null、空格或空。

如果有一个值，解析它以确保它是一个数值。

if (!string.IsNullOrWhiteSpace(txtVacate.Text))
{
    in parsedValue = 0;
    bool isValid = int.TryParse(txtVacate.Text, out parsedValue));

    if (isValid) 
    {
        if (parsedValue <= 400 || parsedValue >= 500)
            MessageBox.Show("Romms provided is not vacant or does not exist at all.");
        else 
        {
            // main stuff here
        }
    }
    else 
    {
        MessageBox.Show("Please provide right info");
    }
else
{
    MessageBox.Show("You provide empty");
}

虽然很复杂，但这应该可以正确处理所有用例

于 2013-03-14T02:48:29.013 回答
1

您的第一个 if 语句使用

int.Parse(txtVacate.Text)

对于空或非数字（文本）值，这将引发 FormatException，因此您的“else if”案例将永远不会被评估。

您应该在第一个“if”中使用 TryParse，而不是 Parse，或者捕获 FormatException 并相应地处理它。

于 2013-03-14T02:38:48.503 回答