我的 if else 语句似乎不起作用。如何防止向数据库发送空数据？

Question

 private void btnSubmitConsultation_Click(object sender, EventArgs e)
    {
        int medicalHistoryResult = insertMedicalHistory();

        if (medicalHistoryResult > 0)
        {
            MessageBox.Show("Document(s) submitted", "Success");
        }

        else
        {
            MessageBox.Show("Insert Fail");
        }


        int allergiesResult = insertAllergies();

        if (allergiesResult > 0)
        {
            if (txtNewAllergy.Text != null || txtReactions.Text != null)
            {
                if (txtNewAllergy.Text == null)
                {
                    MessageBox.Show("Please key in the Type of the allergy", "WARNING");
                }
                else if (txtReactions.Text == null)
                {
                    MessageBox.Show("Please key in the Description of the allergy", "WARNING");
                }
            }
            else
            {
                MessageBox.Show("Submitted, fool");
            }
        }
        else
        {
            MessageBox.Show("Not submitted, fool");
        }


    }

嗯，medicalHistoryResult 似乎工作正常，但 allergiesResult 根本没有做任何事情。

我的 insertAllergies 函数只是一个普通的 INSERT，没什么特别的。

这是我的 insertAllergies 函数：

private int insertAllergies()
    {
        int allergiesResult = 0;

        string strConnectionString = ConfigurationManager.ConnectionStrings["DBConnection"].ConnectionString;
        SqlConnection connection = new SqlConnection(strConnectionString);

        try
        {
            string strPatient = "SELECT patientID FROM PATIENT WHERE patientID=@searchPatientID";
            SqlCommand cmdPatient = new SqlCommand(strPatient, connection);
            cmdPatient.Parameters.AddWithValue("@searchPatientID", txtPatientID.Text);

            string strAllergies = "INSERT ALLERGIES (allergyType, allergyDesc, patientID) " +
                "VALUES (@insertType, @insertDesc, @insertPatient)";
            SqlCommand cmdAllergies = new SqlCommand(strAllergies, connection);

            connection.Open();

            cmdAllergies.Parameters.AddWithValue("@insertType", txtNewAllergy.Text);
            cmdAllergies.Parameters.AddWithValue("@insertDesc", txtReactions.Text);

            SqlDataReader readPatient = cmdPatient.ExecuteReader();
            if (readPatient.Read())
            {
                string addPatient = readPatient["patientID"].ToString();
                cmdAllergies.Parameters.AddWithValue("@insertPatient", addPatient);
            }
            readPatient.Close();

            allergiesResult = cmdAllergies.ExecuteNonQuery();


        }
        catch (Exception ex)
        {
            MessageBox.Show("Error: " + ex.Message);
        }
        finally
        {
            connection.Close();
        }

        return allergiesResult;
    }

--------------------------更新------ -------------------------------------------------------

好的，这是我的新逻辑：

       if (string.IsNullOrEmpty(txtNewAllergy.Text) || string.IsNullOrEmpty(txtReactions.Text))
       {
           if (string.IsNullOrEmpty(txtNewAllergy.Text) && txtReactions.Text != null)
           {
               MessageBox.Show("Please key in the Type of the allergy", "WARNING");
           }
           else if (string.IsNullOrEmpty(txtReactions.Text) && txtNewAllergy.Text != null)
           {
               MessageBox.Show("Please key in the Description  of the allergy", "WARNING");
           }
       }
       else if (txtNewAllergy.Text != null && txtReactions.Text != null)
       {
           int allergiesResult = insertAllergies();
       }

它似乎有效，但只有一个缺陷：当我将两个文本都提交为空时，会弹出“请输入过敏类型”。如果两个文本都是空的，我该怎么做，它什么都不做。

Answer 1

如果txtNewAllergy 和txtReactions 是TextBox，那么永远不会期望.Text 是null；您需要检查一个空的非空字符串。试试string.IsNullOrEmpty(...)：

if (!string.IsNullOrEmpty(txtNewAllergy.Text)
    || !string.IsNullOrEmpty(txtReactions.Text))

为了方便，我们倾向于使用扩展方法：

public static bool HasValue(this string value) {
    return !string.IsNullOrEmpty(value);
}

那么就是：

if (txtNewAllergy.Text.HasValue() || txtReactions.Text.HasValue())

另外：请注意有一个什么都不做的代码路径（请参阅“这里发生了什么？”）：

if (allergiesResult > 0)
{
    if (txtNewAllergy.Text != null || txtReactions.Text != null)
    {
        if (txtNewAllergy.Text == null)
        {
            MessageBox.Show("Please key in the Type of the allergy", "WARNING");
        }
        else if (txtReactions.Text == null)
        {
            MessageBox.Show("Please key in the Description of the allergy", "WARNING");
        }
        else
        {
            // WHAT HAPPENS HERE?
        }
    }
    else
    {
        MessageBox.Show("Submitted, fool");
    }
}
else
{
    MessageBox.Show("Not submitted, fool");
}

Answer 2

您的INSERT 查询缺少INTO 关键字。

尝试将strAllergies 更改为：

string strAllergies = @"INSERT INTO ALLERGIES (allergyType, allergyDesc, patientID)
                        VALUES (@insertType, @insertDesc, @insertPatient)";