Question

我的代码是允许将重复的电子邮件添加到数据库中。我在允许进入数据库之前添加了一行代码，以防止添加重复的电子邮件地址，但是使用此代码我仍然会收到重复的电子邮件。我在asp.net＆amp;中提供了两个表单代码。 c＃代码。请帮忙。

最初我收到了错误＆amp;我在VS＆amp;中调试了代码意识到我实际上有错误的电子邮件条目规范，已经纠正。在VS调试器中，我看到TextBox1.Text＆amp;的值。 TextBox2.Text，我也看到通过字符串查询的电子邮件地址=然而问题是，即使电子邮件已经在数据库中，它仍然会再次添加。我的代码修复此问题的任何改进？我的逻辑错了吗？

c＃code：

protected void Button1_Click(object sender, EventArgs e)
    {
        OleDbConnection con = new OleDbConnection();
        con.ConnectionString = ConfigurationManager.ConnectionStrings["northwind"].ToString();
        con.Open();
        string query = "SELECT COUNT(ID) FROM Table1 WHERE pEmail= '" + TextBox2.Text +"'";
        OleDbCommand cmd = new OleDbCommand(query, con);

        var count = cmd.ExecuteNonQuery();

        if (count > 0)
        {
            Label1.Text = "email is already in use";
        }
        else {
        cmd.CommandText = "insert into[Table1](pName, pEmail)values(@nm,@em)";
        cmd.Parameters.AddWithValue("@nm", TextBox1.Text);
        cmd.Parameters.AddWithValue("@em", TextBox2.Text);
        cmd.Connection = con;
        int a = cmd.ExecuteNonQuery();
        if (a>0)
            {
                Label1.Text = "Inserted Sucessfully!";
            }
        }
    }
}

表格代码：

<form id="form1" runat="server">
<div style="height: 138px">

    Enter Name:<asp:TextBox ID="TextBox1" runat="server" style="margin-left: 12px"></asp:TextBox>

    <asp:RequiredFieldValidator 
    id="reqName" 
    ControlToValidate="TextBox1"
    Style="color:Red"   
    ErrorMessage="Please enter your name!"
    runat="server"  />

    <br />
    Enter Email:&nbsp;&nbsp;&nbsp;
    <asp:TextBox ID="TextBox2" runat="server"></asp:TextBox>

    <asp:RegularExpressionValidator 
    id="ValidEmail" 
    ControlToValidate="TextBox2" 
    Style="color:Red"
    ValidationExpression="^([a-zA-Z0-9_\-\.]+)@((\[[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\.)|(([a-zA-Z0-9\-]+\.)+))([a-zA-Z]{2,4}|[0-9]{1,3})(\]?)$"
    ErrorMessage="Invalid Email Entry" 
    runat="server" />

    <br />
    <asp:Button ID="Button1" runat="server" OnClick="Button1_Click" Text="Submit" />
    <br />
    <asp:Label ID="Label1" runat="server"></asp:Label>

</div>
</form>

Answer 1

您应该使用ExecuteScalar代替ExecuteNonQuery来获取行数

原件：

var count = cmd.ExecuteNonQuery();

建议改变：

var count = cmd.ExecuteScalar();

请参阅https://stackoverflow.com/a/4269651/1050927

ExecuteNonQuery方法返回受INSERT，UPDATE或DELETE影响的行数。此方法用于执行前面所述的DML（数据操作语言）语句。

ExecuteScalar方法将在SELECT语句的第一行，第一列中返回单个值。当您希望只返回查询中的一个值时，将使用此方法。

Answer 2

使用ExecuteScalar并将结果转换为int。我还建议您将sql查询并置更改为参数，如果您使用的是ASP.NET验证器，则必须检查页面的属性IsValid，因为它将告诉您控件是否已通过验证（请记住用户可以禁用javascript并发布表单。）

protected void Button1_Click(object sender, EventArgs e)
{
    if (IsValid)
    {
        using (var con = new OleDbConnection())
        {
            con.ConnectionString = ConfigurationManager.ConnectionStrings["northwind"].ToString();
            con.Open();
            using (var cmd = con.CreateCommand())
            {
                cmd.CommandText = "SELECT COUNT(ID) FROM Table1 WHERE pEmail= @em";
                cmd.Parameters.AddWithValue("@em", TextBox2.Text);
                int count = Convert.ToInt32(cmd.ExecutScalar());
                if (count > 0)
                {
                    Label1.Text = "email is already in use";
                }
                else
                {
                    cmd.CommandText = "insert into[Table1](pName, pEmail)values(@nm, @em)";
                    cmd.Parameters.AddWithValue("@nm", TextBox1.Text);
                    // not need to add @em parameter, it was added previously
                    int insertedRows = cmd.ExecuteNonQuery();
                    if (insertedRows > 0)
                    {
                        Label1.Text = "Inserted Sucessfully!";
                    }
                }
            }
        }
    }
}

代码不能防止数据库中的重复电子邮件地址

2 个答案: