ASP.NET Web Forms表单提交无响应且数据库无更新问题求助

核心问题及修复方案 #

• Page_Load 未加回发判断导致下拉值重置
  Web Forms 每次页面回发都会触发 Page_Load 事件，你当前的部门下拉框绑定逻辑没有判断是否是首次加载，每次点击按钮提交都会重新绑定下拉选项，你选中的部门值会被直接覆盖为第一条选项，导致后续业务查询无匹配数据。
  修改 Page_Load 逻辑：

protected void Page_Load(object sender, EventArgs e)
{
    // 只有首次加载页面的时候绑定下拉框
    if (!IsPostBack)
    {
        string conn = ConfigurationManager.ConnectionStrings["GIMS_HIMS"].ConnectionString;
        SqlConnection sqlconn = new SqlConnection(conn);
        string depart = "select DeptCode, DepartmentName from Que_Master_Department where Active = 1";
        SqlDataAdapter sda = new SqlDataAdapter(depart, sqlconn);
        sqlconn.Open();
        DataTable dt = new DataTable();
        sda.Fill(dt);
        CounterName.DataSource = dt;
        CounterName.DataTextField = "DepartmentName";
        CounterName.DataValueField = "DepartmentName";
        CounterName.DataBind();
        sqlconn.Close();
    }
    btnShow.Click += btnShow_Click;
}

• 表单标签配置错误
  你手写的<form method="post" action="btnShow_Click">存在两个问题：

1. 通常 MasterPage 里已经存在带runat="server"的全局 form 标签，嵌套 form 在 HTML 里本身就是非法写法，会导致提交行为异常
2. action="btnShow_Click"会让表单提交到不存在的路径，不会触发当前页的后台逻辑
   解决方法：直接删除你自己写的这层 form 标签即可，服务器控件会自动用 MasterPage 的全局 form 提交。

• 服务器控件取值方式错误
  CounterName、CounterID、TerminalID 都是带runat="server"的HTML控件，你直接用控件名拼接SQL的话，实际拼接的是控件的ToString结果System.Web.UI.HtmlControls.HtmlSelect，根本不是你选中的实际值，需要用.Value属性获取用户输入/选中的值：
  比如SQL拼接部分修改为：

String userqry = "SELECT TOP(1) Token, ID FROM Que_Management2 Where Department = '" + CounterName.Value + "' AND Flag = 'False' ORDER BY  ID ASC";

所有用到这几个控件值的地方都要加.Value。

• 参数赋值大小写错误
  在CalcPatientRemain方法里，给@Date2参数赋值的代码是.value = Date2，C#区分大小写，属性名是大写的Value，这里会直接编译报错，修改为：

cmd.Parameters.Add("@Date2", SqlDbType.DateTime).Value = Date2;

• 前端标签未加runat属性导致后台访问异常
  你当前页面里的label1、label2是普通的span标签，没有加runat="server"，后台直接访问label1.Value会抛出空引用异常，把span标签修改为：

<p>Waiting Patients: <span id="label2" runat="server">100</span></p>
<p>Total Patients: <span id="label1" runat="server">100</span></p>

• 异常被吞导致无法定位问题
  你所有业务方法的catch块里只把异常赋值给了局部变量abc，没有任何日志或者抛出逻辑，代码运行出错会直接被你忽略，看起来就像方法没执行一样。先把所有catch块临时加一行输出异常的逻辑，就能定位具体报错点：

catch (Exception ex)
{
    Response.Write("<script>alert('" + ex.Message + "')</script>");
    // 或者直接抛出看堆栈 throw;
}

后续优化建议 #

所有SQL操作不要直接拼接用户输入内容，改为参数化查询，避免SQL注入风险。

内容的提问来源于stack exchange，提问作者Ayub Ghaziani