更新不起作用C＃ADO.net

我一直在這個小時，並找不到問題在這裏。我有3頁，其中2個有更新，工作正常，但這不是我希望有人能夠找到問題所在。更新不起作用C＃ADO.net

一切「出現」工作，我看到'成功更新'的消息，但它只是不更新數據。

這裏是代碼中的相關片段：

ASCX：

<asp:Repeater ID="TestDataList" runat="server" onItemCommand="Item_Command"> 
     <ItemTemplate> 
      <div class='<%# Container.ItemIndex % 2 == 0 ? "list-wrap" : "list-wrap alternate" %>'> 
       <a href="#" class="edit-list icon-pencil icon-large"></a> 
       <div class="update"> 
        <span class="dl-content"> 
         <h2><asp:Label ID="TestNameDisplay" runat="server" CssClass="name" Text='<%# Eval("TestName") %>'/></h2> 
        </span> 
        <asp:LinkButton ID="deleteTestCase" CssClass="delete-list icon-trash icon-large" runat="server" CommandName="deleteTestCase" CommandArgument='<%#Eval("TestCaseID")%>'/> 

        <%--edit form--%> 
        <span class="dl-update">  
         <asp:TextBox ID="TxtUpdateTestName" runat="server" CssClass='textEntry' Text='<%#Eval("TestName")%>'></asp:TextBox> 
         <asp:LinkButton ID="EditTestNameButton" runat="server" Text="Save" CommandName="SelectTestName" CommandArgument='<%#Eval("TestCaseID")%>' ValidationGroup='<%# "UpdateTestCaseName" + Eval("TestCaseID") %>' /> 
         <asp:RequiredFieldValidator ID="UpdateTestNameRequired" runat="server" ControlToValidate="TxtUpdateTestName" CssClass="formValidation" ErrorMessage="What good is a test case with no name?" ValidationGroup='<%# "UpdateTestCaseName" + Eval("TestCaseID") %>'/> 
        </span> 
       </div> 
</ItemTemplate> 
    </asp:Repeater>

代碼背後：

protected void Item_Command(Object sender, RepeaterCommandEventArgs e) 
    { 
     if (e.CommandName == "SelectTestName") 
     { 
      string setTestNameSQL = "UPDATE TestCases SET TestName = @TestName WHERE TestCaseID = " + e.CommandArgument; 

      SqlConnection conn = new SqlConnection(GetConnectionString()); 
      SqlCommand cmdUpdateTestName = new SqlCommand(setTestNameSQL, conn); 

      TextBox tb = (TextBox)e.Item.FindControl("TxtUpdateTestName"); 

      SqlParameter u1 = new SqlParameter("TestName", tb.Text); 
      SqlParameter u2 = new SqlParameter("TestCaseID", e.CommandArgument); 

      cmdUpdateTestName.Parameters.Add(u1); 
      cmdUpdateTestName.Parameters.Add(u2); 

      try 
      { 
       conn.Open(); 
       cmdUpdateTestName.ExecuteNonQuery(); 
       PopulateTestList(); 
       lblUserFeedbackMessage.Text = "Sucessfully updated!"; 
      } 
      catch (System.Data.SqlClient.SqlException ex) 
      { 
       string msg = "Update Error:"; 
       msg += ex.Message; 
       throw new Exception(msg); 
      } 
      finally 
      { 
       conn.Close(); 
      } 
     }

來源

2013-06-01 Goose

嘿好友！別擔心。你只是錯過了sqlparameter對象名稱中的@符號 – 2013-06-01 10:42:21

試試這樣說：

string setTestNameSQL = "UPDATE TestCases SET TestName = @TestName WHERE TestCaseID = @TestCaseID"; 

SqlConnection conn = new SqlConnection(GetConnectionString()); 
SqlCommand cmdUpdateTestName = new SqlCommand(setTestNameSQL, conn); 

TextBox tb = (TextBox)e.Item.FindControl("TxtUpdateTestName"); 

SqlParameter u1 = new SqlParameter("@TestName", tb.Text); 
SqlParameter u2 = new SqlParameter("@TestCaseID", e.CommandArgument); 

cmdUpdateTestName.Parameters.Add(u1); 
cmdUpdateTestName.Parameters.Add(u2);

否則你請勿提供參數的值，因爲缺少@，並且您的命令易受到sql injection attack的攻擊。

來源

2013-06-01 10:35:15 Kai

啊謝謝你。我已經接受你的答案，因爲它很好的詳細。我想我在開始的時候錯過了@，然後在試圖修復它的時候慢慢地變得更糟：P – Goose

實際上，它仍然沒有保存 - 我的代碼現在符合你的建議，但它不更新測試的名稱案件..？？ – Goose

@Goose您的數據庫實際上是否與您的'TestCaseId'匹配？請檢查'e.CommandArgument'是否是一個字符串。如果不添加'.ToString（）' – Kai

你已經錯過了@（符號）在sqlparamete對象名稱，對於你的查詢參數對象的名字是@TestName。但是你已經使用了SqlParameter Name是@TestName。發現了@符號

試試下面行，而不是該行代碼

 SqlParameter u1 = new SqlParameter("**@TestName**", tb.Text);

來源

2013-06-01 10:32:40

您正在使用SqlParameters的這是一件好事，因爲它避免了SQL注入但後來你爲什麼直接傳遞e.CommandArgument更新查詢。所以將其更改爲

string setTestNameSQL = "UPDATE TestCases SET TestName = @TestName WHERE TestCaseID = @TestCaseID";

，並添加參數是這樣的（不要忘了在參數名稱前添加@）

SqlParameter u1 = new SqlParameter("@TestName", tb.Text); 
SqlParameter u2 = new SqlParameter("@TestCaseID", e.CommandArgument)

來源

2013-06-01 10:33:22 Sachin

嗯，我會看看謝謝你的隊友，感謝它 – Goose

更新不起作用C＃ADO.net

回答

相關問題