SQL-оператор SqlCommand выполняется дважды - PullRequest
Новая
       0

SQL-оператор SqlCommand выполняется дважды

3 голосов
/ 14 июля 2011

Я создаю приложение ASP.NET, используя C #, и каждый раз, когда я пытаюсь выполнить SQL INSERT, используя SqlCommand, команда выполняется дважды.Я реализовал то, что было в этом посте ( Запрос SQL-вставки выполняется дважды ), но это все равно не помогает.

Вот код: 

protected void btn_Add_Click(object sender, EventArgs e)
{
        error.InnerHtml = " ";

        SqlConnection MyConn = new SqlConnection("Data Source=.\\SQLEXPRESS;"
            + "Initial Catalog=SIMON;"
            + "Persist Security Info=True;"
            + "User ID=username;Password=password");

        string sqlStatement = @"INSERT INTO [Network_Equipment] "
            + "([Network_Equipment_Name], [Network_Equipment_Type_ID], [IP_Address], [FQDN], [NetBIOS_Name], [Building_ID], [Description])"
            + " VALUES "
            + "(@Network_Equipment_Name, @Network_Equipment_Type_ID, @IP_Address, @FQDN, @NetBIOS_Name, @Building_ID, @Description)";

            SqlCommand sqlCmd = new SqlCommand(sqlStatement, MyConn);
            sqlCmd.Connection = MyConn;

            sqlCmd.Parameters.AddWithValue("@Network_Equipment_Name", ((in_Add_Equipment.Value == null) ? (object)DBNull.Value : (object)in_Add_Equipment.Value));
            sqlCmd.Parameters.AddWithValue("@Network_Equipment_Type_ID", ((ddl_Equipment_Type.SelectedValue == null) ? (object)DBNull.Value : (object)ddl_Equipment_Type.SelectedValue));
            sqlCmd.Parameters.AddWithValue("@IP_Address", ((in_IP_Address.Value == null) ? (object)DBNull.Value : (object)in_IP_Address.Value));
            sqlCmd.Parameters.AddWithValue("@FQDN", ((in_FQDN.Value == null) ? (object)DBNull.Value : (object)in_FQDN.Value));
            sqlCmd.Parameters.AddWithValue("@NetBIOS_Name", ((in_NetBIOS.Value == null) ? (object)DBNull.Value : (object)in_NetBIOS.Value));
            sqlCmd.Parameters.AddWithValue("@Building_ID", ((ddl_Building.SelectedValue == null) ? (object)DBNull.Value : (object)ddl_Building.SelectedValue));
            sqlCmd.Parameters.AddWithValue("@Description", ((ta_Description.Value == null) ? (object)DBNull.Value : (object)ta_Description.Value));


            try
            {
                MyConn.Open();
            }
            catch (Exception er)
            {
                error.InnerHtml = er.ToString();
                return;
            }

            try
            {
                sqlCmd.ExecuteNonQuery();
            }
            catch (Exception er)
            {
                error.InnerHtml = er.ToString();
                return;
            }

            error.InnerHtml = "Successfully added: " + in_Add_Equipment.Value;

            sqlCmd.Dispose();
            sqlCmd = null;
            MyConn.Close();

            in_Add_Equipment.Value = "";
            ddl_Equipment_Type.SelectedIndex = 0;
            in_IP_Address.Value = "";
            in_FQDN.Value = "";
            in_NetBIOS.Value = "";
            ddl_Building.SelectedIndex = 0;
            ta_Description.Value = "";

            GridView1.DataBind();
}

Я запустил его в режиме отладки, и sqlCmd.ExecuteNonQuery () вызывается только один раз, но у меня добавляются две идентичные записи.

Что я пропустил и / или сделал неправильно?

-------- РЕДАКТИРОВАТЬ --------

Я использовал модифицированную версию Джон Райан 's, но он все еще был добавлен дважды.

Вот ВСЕ код, который идет с этим:

Equipment.aspx 

<%@ Page Language="C#" AutoEventWireup="true" CodeBehind="Equipment.aspx.cs" Inherits="SIMON.Equipment" %>

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">

<html xmlns="http://www.w3.org/1999/xhtml">
<head runat="server">
    <title></title>
    <script language="javascript" type="text/javascript">
        // disables the button specified and sets its style to a disabled "look".
        function disableButtonOnClick(oButton, sButtonText, sCssClass) {
            oButton.disabled = true; // set button to disabled so you can't click on it.
            oButton.value = sButtonText; // change the text of the button.
            oButton.setAttribute('className', sCssClass); // IE uses className for the css property.
            oButton.setAttribute('class', sCssClass); // Firefox, Safari use class for the css property. (doesn't hurt to do both).
        }
    </script>
</head>
<body>
    <form id="form1" runat="server">
    <div>

        <asp:ScriptManager ID="ScriptManager1" runat="server"></asp:ScriptManager>
        <asp:UpdatePanel ID="UpdatePanel2" runat="server">
            <ContentTemplate>
                <div id="error" runat="server">&nbsp;</div>
                <div>
                    <fieldset style="display: inline-block; padding: 5px; -moz-border-radius: 8px; border-radius: 8px;">
                    <legend style="text-decoration: underline;">[ Add Equipment ]</legend>
                        <table cellpadding="3px" style="border: 0px; width: 300px;">
                            <tbody>
                                <tr>
                                    <td style="text-align: left;"><label for="in_Add_Equipment" style="white-space: nowrap;">Equipment Name</label></td>
                                    <td style="text-align: right;"><input id="in_Add_Equipment" runat="server" size="20" /></td>
                                    <td style="text-align: left;"><label for="ddl_Equipment_Type" style="white-space: nowrap;">Equipment Type</label></td>
                                    <td style="text-align: right;">
                                        <asp:DropDownList ID="ddl_Equipment_Type" 
                                            runat="server" DataSourceID="Network_Equipment_Type" DataTextField="Network_Equipment_Type" 
                                            DataValueField="Network_Equipment_Type_ID"></asp:DropDownList>
                                    </td>
                                </tr>
                                <tr>
                                    <td style="text-align: left;"><label for="in_IP_Address" style="white-space: nowrap;">IP Address</label></td>
                                    <td style="text-align: right;"><input id="in_IP_Address" runat="server" size="20" class="OnlyNumeric" /></td>
                                    <td style="text-align: left;"><label for="in_FQDN" title="Fully Qualified Domain Name" style="white-space: nowrap;">FQDN</label></td>
                                    <td style="text-align: right;"><input id="in_FQDN" runat="server" size="20" /></td>
                                </tr>
                                <tr>
                                    <td style="text-align: left;"><label for="in_NetBIOS" style="white-space: nowrap;">NetBIOS Name</label></td>
                                    <td style="text-align: right;"><input id="in_NetBIOS" runat="server" size="20" /></td>
                                    <td style="text-align: left;"><label for="ddl_Building" style="white-space: nowrap;">Building</label></td>
                                    <td style="text-align: right;">
                                        <asp:DropDownList ID="ddl_Building" runat="server" 
                                            DataSourceID="Buildings" DataTextField="Building_Name" 
                                            DataValueField="Building_ID"></asp:DropDownList>
                                    </td>
                                </tr>
                                <tr>
                                    <td style="text-align: left;"><label for="ta_Description" style="white-space: nowrap;">Description</label></td>
                                    <td style="text-align: right;" colspan="3"><textarea id="ta_Description" runat="server" cols="50" rows="3"></textarea></td>
                                </tr>
                                <tr>
                                    <td colspan="4" style="text-align: right;">
                                        <asp:Button ID="btn_Add" Text="Add" CssClass="ui-state-default ui-corner-all" runat="server" OnClick="btn_Add_Click" />
                                    </td>
                                </tr>
                            </tbody>
                        </table>
                    </fieldset>
                </div>
            </ContentTemplate>
        </asp:UpdatePanel>
        <p></p>
        <fieldset style="display: inline-block; padding: 5px; -moz-border-radius: 8px; border-radius: 8px;">
        <legend style="text-decoration: underline;">[ Update Equipment ]</legend>
           <asp:UpdatePanel ID="UpdatePanel1" runat="server">
                <ContentTemplate>
                    <asp:SqlDataSource ID="SqlDataSource1" runat="server" 
                        ConnectionString="<%$ ConnectionStrings:SIMON-Writer %>" 
                        DeleteCommand="DELETE FROM [Network_Equipment] WHERE [Network_Equipment_ID] = @original_Network_Equipment_ID" 
                        OldValuesParameterFormatString="original_{0}" 
                        SelectCommand="SELECT * FROM [All_Network_Equipment-Simplified]" 

                        UpdateCommand="UPDATE [Network_Equipment] SET [Network_Equipment_Name] = @Network_Equipment_Name, [Network_Equipment_Type_ID] = @Network_Equipment_Type_ID, [IP_Address] = @IP_Address, [FQDN] = @FQDN, [NetBIOS_Name] = @NetBIOS_Name, [Building_ID] = @Building_ID, [Description] = @Description, [Active] = @Active WHERE [Network_Equipment_ID] = @original_Network_Equipment_ID">
                        <DeleteParameters>
                            <asp:Parameter Name="original_Network_Equipment_ID" Type="Int32" />
                        </DeleteParameters>
                        <InsertParameters>
                            <asp:Parameter Name="Network_Equipment_Name" Type="String" />
                            <asp:Parameter Name="Network_Equipment_Type_ID" Type="Int32" />
                            <asp:Parameter Name="IP_Address" Type="String" />
                            <asp:Parameter Name="FQDN" Type="String" />
                            <asp:Parameter Name="NetBIOS_Name" Type="String" />
                            <asp:Parameter Name="Building_ID" Type="Int32" />
                            <asp:Parameter Name="Description" Type="String" />
                            <asp:Parameter Name="Active" Type="Boolean" />
                        </InsertParameters>
                        <UpdateParameters>
                            <asp:Parameter Name="Network_Equipment_Name" Type="String" />
                            <asp:Parameter Name="Network_Equipment_Type_ID" Type="Int32" />
                            <asp:Parameter Name="IP_Address" Type="String" />
                            <asp:Parameter Name="FQDN" Type="String" />
                            <asp:Parameter Name="NetBIOS_Name" Type="String" />
                            <asp:Parameter Name="Building_ID" Type="Int32" />
                            <asp:Parameter Name="Description" Type="String" />
                            <asp:Parameter Name="Active" Type="Boolean" />
                            <asp:Parameter Name="original_Network_Equipment_ID" Type="Int32" />
                        </UpdateParameters>
                    </asp:SqlDataSource>
                    <asp:GridView ID="GridView1" runat="server" AutoGenerateColumns="False" 
                        CellPadding="4" DataKeyNames="Network_Equipment_ID" 
                        DataSourceID="SqlDataSource1" ForeColor="#333333" GridLines="None">
                        <AlternatingRowStyle BackColor="White" ForeColor="#284775" />
                        <Columns>
                            <asp:BoundField DataField="Network_Equipment_ID" 
                                HeaderText="Network_Equipment_ID" ReadOnly="True" 
                                SortExpression="Network_Equipment_ID" Visible="False" />
                            <asp:BoundField DataField="Network_Equipment_Name" HeaderText="Equipment Name" SortExpression="Network_Equipment_Name" />
                            <asp:BoundField DataField="Network_Equipment_Type_ID" HeaderText="Network_Equipment_Type_ID" SortExpression="Network_Equipment_Type_ID" Visible="False" />
                            <asp:TemplateField HeaderText="Equipment Type" SortExpression="Network_Equipment_Type">
                                <EditItemTemplate>
                                    <asp:DropDownList ID="DropDownList1" runat="server" 
                                        DataSourceID="Network_Equipment_Type" DataTextField="Network_Equipment_Type" 
                                        DataValueField="Network_Equipment_Type_ID" 
                                        SelectedValue='<%# Bind("Network_Equipment_Type_ID") %>'>
                                    </asp:DropDownList>
                                </EditItemTemplate>
                                <ItemTemplate>
                                    <asp:Label ID="Label1" runat="server" Text='<%# Bind("Network_Equipment_Type") %>'></asp:Label>
                                </ItemTemplate>
                            </asp:TemplateField>
                            <asp:BoundField DataField="IP_Address" HeaderText="IP Address" SortExpression="IP_Address" />
                            <asp:BoundField DataField="FQDN" HeaderText="FQDN" SortExpression="FQDN" />
                            <asp:BoundField DataField="NetBIOS_Name" HeaderText="NetBIOS Name" SortExpression="NetBIOS_Name" />
                            <asp:BoundField DataField="Building_ID" HeaderText="Building_ID" SortExpression="Building_ID" Visible="False" />
                            <asp:TemplateField HeaderText="Building" SortExpression="Building_Name">
                                <EditItemTemplate>
                                    <asp:DropDownList ID="DropDownList2" runat="server" DataSourceID="Buildings" 
                                        DataTextField="Building_Name" DataValueField="Building_ID" 
                                        SelectedValue='<%# Bind("Building_ID") %>'>
                                    </asp:DropDownList>
                                </EditItemTemplate>
                                <ItemTemplate>
                                    <asp:Label ID="Label2" runat="server" Text='<%# Bind("Building_Name") %>'></asp:Label>
                                </ItemTemplate>
                            </asp:TemplateField>
                            <asp:TemplateField HeaderText="Description" SortExpression="Description">
                                <EditItemTemplate>
                                    <asp:TextBox ID="TextBox3" runat="server" Height="50px" Text='<%# Bind("Description") %>' TextMode="MultiLine" Width="200px"></asp:TextBox>
                                </EditItemTemplate>
                                <ItemTemplate>
                                    <asp:Label ID="Label3" runat="server" Text='<%# Bind("Description") %>'></asp:Label>
                                </ItemTemplate>
                            </asp:TemplateField>
                            <asp:CheckBoxField DataField="Active" HeaderText="Active" SortExpression="Active" />
                            <asp:CommandField ShowEditButton="True" />
                        </Columns>
                        <EditRowStyle BackColor="#999999" />
                        <FooterStyle BackColor="#5D7B9D" Font-Bold="True" ForeColor="White" />
                        <HeaderStyle BackColor="#5D7B9D" Font-Bold="True" ForeColor="White" />
                        <PagerStyle BackColor="#284775" ForeColor="White" HorizontalAlign="Center" />
                        <RowStyle BackColor="#F7F6F3" ForeColor="#333333" />
                        <SelectedRowStyle BackColor="#E2DED6" Font-Bold="True" ForeColor="#333333" />
                        <SortedAscendingCellStyle BackColor="#E9E7E2" />
                        <SortedAscendingHeaderStyle BackColor="#506C8C" />
                        <SortedDescendingCellStyle BackColor="#FFFDF8" />
                        <SortedDescendingHeaderStyle BackColor="#6F8DAE" />
                    </asp:GridView>
                    <br />
                    <asp:SqlDataSource ID="Network_Equipment_Type" runat="server" ConnectionString="<%$ ConnectionStrings:SIMON-Reader %>" 
                        SelectCommand="SELECT DISTINCT * FROM [Network_Equipment_Types] ORDER BY [Network_Equipment_Type]">
                    </asp:SqlDataSource>
                    <asp:SqlDataSource ID="Buildings" runat="server" ConnectionString="<%$ ConnectionStrings:SIMON-Reader %>" 
                        SelectCommand="SELECT DISTINCT * FROM [Buildings] ORDER BY [Building_Name]">
                    </asp:SqlDataSource>
                </ContentTemplate>
            </asp:UpdatePanel>
        </fieldset>
    </div>
    </form>
</body>
</html>

Equipment.aspx.cs 

namespace SIMON
{
    public partial class Equipment : System.Web.UI.Page
    {
        protected void Page_Load(object sender, EventArgs e)
        {
            if (!Page.IsPostBack)
            {
                PostBackOptions optionsSubmit = new PostBackOptions(btn_Add);
                btn_Add.OnClientClick = "disableButtonOnClick(this, 'Please wait...', 'disabled_button ui-corner-all'); ";
                btn_Add.OnClientClick += ClientScript.GetPostBackEventReference(optionsSubmit);
            }
        }

        protected void btn_Add_Click(object sender, EventArgs e)
        {
            error.InnerHtml = "&nbsp;";

            SqlTransaction trans = null;
            using (SqlConnection MyConn = new SqlConnection(ConfigurationManager.ConnectionStrings["SIMON-Writer"].ToString()))
            {

                MyConn.Open();
                trans = MyConn.BeginTransaction();

                string sqlStatement = "INSERT INTO [Network_Equipment] "
                    + "([Network_Equipment_Name], [Network_Equipment_Type_ID], [IP_Address], [FQDN], [NetBIOS_Name], [Building_ID], [Description])"
                    + " VALUES "
                    + "(@Network_Equipment_Name, @Network_Equipment_Type_ID, @IP_Address, @FQDN, @NetBIOS_Name, @Building_ID, @Description)";

                using (SqlCommand sqlCmd = new SqlCommand(sqlStatement, MyConn))
                {

                    sqlCmd.Transaction = trans;
                    sqlCmd.Connection = MyConn;

                    sqlCmd.Parameters.AddWithValue("@Network_Equipment_Name", ((in_Add_Equipment.Value == null) ? (object)DBNull.Value : (object)in_Add_Equipment.Value));
                    sqlCmd.Parameters.AddWithValue("@Network_Equipment_Type_ID", ((ddl_Equipment_Type.SelectedValue == null) ? (object)DBNull.Value : (object)ddl_Equipment_Type.SelectedValue));
                    sqlCmd.Parameters.AddWithValue("@IP_Address", ((in_IP_Address.Value == null) ? (object)DBNull.Value : (object)in_IP_Address.Value));
                    sqlCmd.Parameters.AddWithValue("@FQDN", ((in_FQDN.Value == null) ? (object)DBNull.Value : (object)in_FQDN.Value));
                    sqlCmd.Parameters.AddWithValue("@NetBIOS_Name", ((in_NetBIOS.Value == null) ? (object)DBNull.Value : (object)in_NetBIOS.Value));
                    sqlCmd.Parameters.AddWithValue("@Building_ID", ((ddl_Building.SelectedValue == null) ? (object)DBNull.Value : (object)ddl_Building.SelectedValue));
                    sqlCmd.Parameters.AddWithValue("@Description", ((ta_Description.Value == null) ? (object)DBNull.Value : (object)ta_Description.Value));


                    try
                    {
                        sqlCmd.ExecuteNonQuery();
                        trans.Commit();
                    }
                    catch (Exception er)
                    {
                        error.InnerHtml = er.ToString();
                        return;
                    }
                    finally
                    {
                        MyConn.Close();

                        error.InnerHtml = "Successfully added: " + in_Add_Equipment.Value;

                        in_Add_Equipment.Value = "";
                        ddl_Equipment_Type.SelectedIndex = 0;
                        in_IP_Address.Value = "";
                        in_FQDN.Value = "";
                        in_NetBIOS.Value = "";
                        ddl_Building.SelectedIndex = 0;
                        ta_Description.Value = "";

                        GridView1.DataBind();
                    }
                }
            }
        }
    }
}

-------- РЕДАКТИРОВАТЬ 2 --------

Я обнаружил, что вызывает "двойное выполнение""оператора SQL к этому коду: 

    protected void Page_Load(object sender, EventArgs e)
    {
        if (!Page.IsPostBack)
        {
            PostBackOptions optionsSubmit = new PostBackOptions(btn_Add);
            btn_Add.OnClientClick = "disableButtonOnClick(this, 'Please wait...', 'disabled_button ui-corner-all'); ";
            btn_Add.OnClientClick = ClientScript.GetPostBackEventReference(optionsSubmit);
        }
    }

Когда я закомментирую его, информация добавляется один раз, но кнопка не отключается, как я хочу.Отключение кнопки предотвратит нетерпение и многократное нажатие кнопки.

Предложения?

Вот что я получаю:

aspx 

<asp:Button ID="btn_Add" Text="Add" CssClass="ui-state-default ui-corner-all" runat="server" OnClick="btn_Add_Click" />

aspx.cs 

if (!Page.IsPostBack)
{
    PostBackOptions optionsSubmit = new PostBackOptions(btn_Add);
    btn_Add.OnClientClick = "disableButtonOnClick(this, 'Please wait...', 'disabled_button ui-corner-all'); ";
    btn_Add.OnClientClick += ClientScript.GetPostBackEventReference(optionsSubmit);
}

выполняет SQL-оператор дважды

aspx 

<asp:Button ID="btn_Add" Text="Add" CssClass="ui-state-default ui-corner-all" runat="server" />

aspx.cs 

if (!Page.IsPostBack)
{
    PostBackOptions optionsSubmit = new PostBackOptions(btn_Add);
    btn_Add.OnClientClick = "disableButtonOnClick(this, 'Please wait...', 'disabled_button ui-corner-all'); ";
    btn_Add.OnClientClick += ClientScript.GetPostBackEventReference(optionsSubmit);
}

кратко меняет "btn_Add"желаемым образом, но не выполняет код для" btn_Add_Click "

aspx 

<asp:Button ID="btn_Add" Text="Add" CssClass="ui-state-default ui-corner-all" runat="server" OnClick="btn_Add_Click" />

aspx.cs 

if (!Page.IsPostBack)
{
    PostBackOptions optionsSubmit = new PostBackOptions(btn_Add);
    btn_Add.OnClientClick = "disableButtonOnClick(this, 'Please wait...', 'disabled_button ui-corner-all'); ";
    //btn_Add.OnClientClick += ClientScript.GetPostBackEventReference(optionsSubmit);
}

меняет" btn_Add "вжелаемым образом, не выполняет код для "btn_Add_Click" и обновляет всю страницу до Equipment.aspx (обычно Equipment.aspx загружается в div через событие jquery .load)

Ответы [ 4 ]

3 голосов
/ 14 июля 2011

Несколько предложений ...

  • строки подключения должны храниться в вашем файле web.config.

  • можно ловить исключения для каждого шага на этом пути, но используйте оператор "using". Таким образом, вам не нужно беспокоиться об утилизации вашего объекта.

  • Я бы порекомендовал добавить файл Global.asax в ваш проект и отлавливать там ошибки приложения. В своих выражениях catch вы можете поместить строку и сказать «выбросить новое исключение ...» и передать свое конкретное исключение, чтобы его можно было легко отследить и описать ошибку. Тогда я бы порекомендовал настроить блок приложения «Ведение журнала» в Enterprise Library 5.0 Наличие глобальной переменной «error», как у вас, вероятно, не очень хорошая идея. Если вы хотите легко изучить Enterprise Library, загрузите примеры Лаборатории Enterprise Library.

  • Я бы не использовал команды SQL непосредственно в вашем коде. Я хотел бы использовать хранимые процедуры и сделать SQL на самом сервере SQL. Если у вас есть переменные, настройте параметры хранимой процедуры. Хранимые процедуры более безопасны для вашего веб-приложения. Возможно, было бы неплохо использовать блок приложения базы данных из корпоративной библиотеки Microsoft. У них есть весь этот код для вас, и вам просто нужно импортировать библиотеку и знать команды. Они справляются со всеми последствиями для вас.

  • при использовании оператора using нет необходимости закрывать соединение или удалять. Я удалил это в коде.

  • если вы генерируете исключение, нет необходимости в операторе «return» ... следующий блок вверх по стеку с оператором catch (или оператор finally), который отвечает требованиям, перехватит его. Если их нет, Application_Error в Global.asax перехватит его. Убедитесь, что вы правильно обрабатываете необработанные исключения в .NET Framework. Microsoft нарушила архитектуру необработанных исключений .NET 3.5, поэтому вам нужно либо добавить свой собственный модуль, либо вернуться к архитектуре 1.1. Или обновите до .NET 4.0.

  • Фактический вопрос: Ваше событие click может вызываться дважды, потому что вы используете обработчик И у вас есть атрибут события «onclick», указанный в вашем теге

Вот # 1: <Жерех: кнопка ID = "btn_Add" Text = "Добавить" CssClass = "ui-state-default-ui-corner-all" Runat = "сервер" <strong> OnClick = "btn_Add_Click" />

Вот # 2: btn_Add.OnClientClick = ClientScript.GetPostBackEventReference (optionsSubmit);

Если у вас есть атрибут onclick на вашем элементе управления, вам не нужно добавлять специальный код (например, OnClientClick или обработчики событий). Просто используйте 1 или другой.

Удачи тебе! 

        error.InnerHtml = "&nbsp;";

        string connStr = "Data Source=.\\SQLEXPRESS;"
            + "Initial Catalog=SIMON;"
            + "Persist Security Info=True;"
            + "User ID=username;Password=password");

        using (SqlConnection MyConn = new SqlConnection(connStr))
        {
            string sqlStatement = @"INSERT INTO [Network_Equipment] "
                + "([Network_Equipment_Name], [Network_Equipment_Type_ID], [IP_Address], [FQDN], [NetBIOS_Name], [Building_ID], [Description])"
                + " VALUES "
                + "(@Network_Equipment_Name, @Network_Equipment_Type_ID, @IP_Address, @FQDN, @NetBIOS_Name, @Building_ID, @Description)";

            SqlCommand sqlCmd = new SqlCommand(sqlStatement, MyConn);
            sqlCmd.Connection = MyConn;

            sqlCmd.Parameters.AddWithValue("@Network_Equipment_Name", ((in_Add_Equipment.Value == null) ? (object)DBNull.Value : (object)in_Add_Equipment.Value));
            sqlCmd.Parameters.AddWithValue("@Network_Equipment_Type_ID", ((ddl_Equipment_Type.SelectedValue == null) ? (object)DBNull.Value : (object)ddl_Equipment_Type.SelectedValue));
            sqlCmd.Parameters.AddWithValue("@IP_Address", ((in_IP_Address.Value == null) ? (object)DBNull.Value : (object)in_IP_Address.Value));
            sqlCmd.Parameters.AddWithValue("@FQDN", ((in_FQDN.Value == null) ? (object)DBNull.Value : (object)in_FQDN.Value));
            sqlCmd.Parameters.AddWithValue("@NetBIOS_Name", ((in_NetBIOS.Value == null) ? (object)DBNull.Value : (object)in_NetBIOS.Value));
            sqlCmd.Parameters.AddWithValue("@Building_ID", ((ddl_Building.SelectedValue == null) ? (object)DBNull.Value : (object)ddl_Building.SelectedValue));
            sqlCmd.Parameters.AddWithValue("@Description", ((ta_Description.Value == null) ? (object)DBNull.Value : (object)ta_Description.Value));

            try
            {
                MyConn.Open();
            }
            catch (Exception er)
            {
                //error.InnerHtml = er.ToString();
                //return;
                throw new Exception(...
            }

            try
            {
                sqlCmd.ExecuteNonQuery();
            }
            catch (Exception er)
            {
                //error.InnerHtml = er.ToString();
                //return;
                throw new Exception(...
            }

            error.InnerHtml = "Successfully added: " + in_Add_Equipment.Value;
        }

        in_Add_Equipment.Value = "";
        ddl_Equipment_Type.SelectedIndex = 0;
        in_IP_Address.Value = "";
        in_FQDN.Value = "";
        in_NetBIOS.Value = "";
        ddl_Building.SelectedIndex = 0;
        ta_Description.Value = "";

        GridView1.DataBind();
1 голос
/ 14 июля 2011

удаляет «return», который в двух зацеплениях

и

добавляет оператор finally для каждой попытки

и перемещает 

 sqlCmd.Dispose();             sqlCmd = null;             MyConn.Close();

в finally

при отладке вы можете увидеть оператор вставки, правильно ли это.затем посмотрите базу данных.

0 голосов
/ 14 июля 2011

Помимо слишком большого количества кода в одной функции, без использования операторов using и т. Д., Единственная причина, по которой эта функция обработчика событий будет вызываться более одного раза, заключается в том, что она была подключена более одного раза.Проверьте настройку AutoEventWireUp, код впереди для определения OnClick на кнопке и явное соединение событий в вашем коде сзади (вероятно, будет выглядеть как btn_Add.Click += btn_Add_Click;.

0 голосов
/ 14 июля 2011

@ Иисус Рамос прав.

Сделай что-нибудь подобное. 

private _constring = new SqlConnection("Data Source=.\\SQLEXPRESS;"
        + "Initial Catalog=SIMON;"
        + "Persist Security Info=True;"
        + "User ID=username;Password=password");


protected void btn_Add_Click(object sender, EventArgs e)
   {

        SqlTransaction trans = null;
        using (var con = new  SqlConnection(_constring))
        {
            con.Open();
            trans = con.BeginTransaction();
            using (

                 string sqlStatement ="//SqlStuff";

                var  sqlCmd = new  SqlCommand(sqlStatement,con))
            {
                 sqlCmd.Transaction = trans;
                 sqlCmd.Connection = con;

                try
                {


                    //sqlcmd.Parameters Stuff


                    sqlCmd.ExecuteNonQuery();
                    trans.Commit();

                   //SUCCESS
                }
               catch (Exception er)
               {
               error.InnerHtml = er.ToString();
               return;
                }
                finally
                {
                    con.Close();
                }

            }
        }
    }

Привет

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.

Нет похожих вопросов

...