按钮触发两次 - 在数据库中插入2条记录

时间:2011-01-31 23:50:14

标签: c# asp.net events button handler

使用AutoEventWireup = true在MasterPage中运行。我无法改变这一点。 按钮在数据库中插入新记录 当按钮命中时,插入两个记录。

  <asp:Button ID="AddLab" Text="Add Lab" OnClick="AddLab_OnClick" CssClass="btn" runat="server" />

PageDirective:

    AutoEventWireup="true"  <- I have tried removing this, setting to false.

PageLoad / PreRender显示使用按钮显示此处没有任何操作:

    protected void Page_Load(object sender, EventArgs e)
    {
        if (!IsPostBack)
        {
            DbDataReader ddrGrp = rdoGroups();
            if (ddrGrp.HasRows)
            {
                rdoGroup.DataSource = ddrGrp;
                rdoGroup.DataBind();
            }
            ddrGrp.Close();
            // preset filter by lab to true
            CheckBox FiltLabs = (CheckBox)CommonUI.FindCtrlRecursive(this.Master, "FiltLabs");
            FiltLabs.Checked = true;
        }

    }

    protected override void OnPreRender(EventArgs e)
    {
        base.OnPreRender(e);
        // nothing yet
        rdoLabs.Items.Clear();
        DbDataReader ddrLab = rdoUserLabs();
        if (ddrLab.HasRows)
        {

            rdoLabs.DataSource = ddrLab;
            rdoLabs.DataBind();
            if (CommonUI.strTest((string)Session["rdoLabs"]))
            {
                if (Convert.ToInt32(Session["rdoLabs"]) > 0)
                {
                    rdoLabs.SelectedValue = (string)Session["rdoLabs"];
                }
            }
        }
        ddrLab.Close();
        // get group-lab mappings
        cboGroupLab.Items.Clear();
        DbDataReader ddrGroupLab = cboGroupLabsMap();
        if (ddrGroupLab != null)
        {
            if (ddrGroupLab.HasRows)
            {

                cboGroupLab.DataSource = ddrGroupLab;
                cboGroupLab.DataBind();
            }
        }
    }

    protected void AddLab_OnClick(object sender, EventArgs e)
    {
        LabAdmn labAdd = new LabAdmn();
        TextBox txtLab = (TextBox)CommonUI.FindCtrlRecursive(this.Master, "txtLab");
        CheckBox Active = (CheckBox)CommonUI.FindCtrlRecursive(this.Master, "cboLabActive");
        string[] strArr = new string[] { "lab_id" };
        labAdd.LabName = txtLab.Text;
        labAdd.Active = (bool)Active.Checked;
        // AddLab is where record is fired...see below...
        Hashtable ht = labAdd.AddLab(labAdd, strArr);
        ht = labAdd.AddLab(labAdd, strArr);
        if (ht != null)
        {
            HiddenField hLabId = (HiddenField)CommonUI.FindCtrlRecursive(this.Master, "hLabId");
            hLabId.Value = Convert.ToString(GetHTParm(ht, strArr[0]));
        }


    }
    #endregion AddLab_OnClick

    #region AddLab
    public Hashtable AddLab(LabAdmn labAdmn, string[] colOutputNames)
    {
        DAL myDal = new DAL(DBType, DB);
        Hashtable ht = new Hashtable();
        DAL.Parameters[] parms = new DAL.Parameters[]
        {
            new DAL.Parameters("driver_id","A",ParameterDirection.Input),
            new DAL.Parameters("lab_id",labAdmn.LabId,ParameterDirection.InputOutput),
            new DAL.Parameters("active",labAdmn.Active,ParameterDirection.Input),
            new DAL.Parameters("lab_name",labAdmn.LabName,ParameterDirection.Input)
        };
        CommandType cmdType = CommandType.StoredProcedure;
        string cmdText = "asp_Labs_Admin";
        ht = myDal.ExecuteQueryOutput(cmdType, cmdText, colOutputNames, parms, true);
        return ht;

    }
    #endregion AddLab

我已经读过onclick和runat导致双重处理。如果我删除runat,该按钮将不会显示,因为它是服务器端按钮。如果删除OnClick事件,则无论AutoEventWireup是true,false还是已删除,事件都不会触发。

3 个答案:

答案 0 :(得分:2)

看起来你正在两次调用“AddLab”方法:

Hashtable ht = labAdd.AddLab(labAdd, strArr);
        ht = labAdd.AddLab(labAdd, strArr);

答案 1 :(得分:1)

如果您没有将处理程序附加到按钮两次,请查看代码中的某处。也许在Designer中有一个附加的事件处理程序。也许在您的初始化代码中的某处。

尝试使用按钮删除代码并重新放置,而不在标记中使用onclick属性。

中添加您的页面或控制到“oninit”
AddLab.Click += this.AddLab_OnClick;

但首先,尝试删除标记的属性,然后重试,如果它仍然会触发两次

答案 2 :(得分:0)

您可以将AddLab_OnClick()方法重命名为其他内容并在OnClick =“”处理程序中分配吗?我在想AutoEventWireup正在触发第一个事件,第二个事件是由于OnClick再次分配...

另外,试试Luke建议的内容。这将证明事件的分配..