Как вставить данные в DB

Question

Я искал онлайн ответы разных людей о том, как вставлять данные с помощью asp.net и C#, но до сих пор все разные вещи, которые я пробовал, ничего не делают, почему моя кнопка не делать что-либо, когда я нажимаю, чтобы вставить данные в мою БД? также, что я использую, чтобы различать текстовое поле и раскрывающийся список, мог бы я использовать ddl.selecteditem, который я сейчас использую, и это лучший способ? большинство исходных текстов онлайн и учебники, которые я рассматривал, имеют дело с gridviews, и я просто не смог эффективно перевести это на то, что мне нужно.

вот мой HTML

  </td> 
      <td style="height: 51px; width: 377px; font-weight: bold;"> 
       Project Tester 
      </td> 
      <td> 
       <asp:DropDownList ID="ddlTester" runat="server" 
        DataTextField="" Width="203px" DataBound="ddlTester_DataBound" 
        AutoPostBack="False"> 
       </asp:DropDownList> 
       &nbsp; 
       <asp:SqlDataSource runat="server" ID="SqlDataSource1" ConnectionString='<%$ ConnectionStrings:ProjectsAndTasksTestConnectionString %>' 
        SelectCommand="SELECT [TesterName] FROM [Users] ORDER BY [TesterName]"> 
       </asp:SqlDataSource> 
      </td> 
     </tr> 
     <tr> 
      <td> 
       &nbsp; 
      </td> 
      <td style="height: 51px; width: 377px; font-weight: bold;"> 
       Project Name&nbsp; 
      </td> 
      <td> 
       <asp:TextBox ID="txtProjectName" runat="server" Width="200px"></asp:TextBox> 
       &nbsp; 
      </td> 
     </tr> 
     <tr> 
      <td style="height: 51px"> 
       &nbsp; 
      </td> 
      <td style="height: 51px; width: 377px; font-weight: bold;"> 
       Project Description</td> 
      <td style="height: 51px"> 
       <asp:TextBox ID="txtProjectDesc" runat="server" TextMode="MultiLine" Height="105px" Width="237px"></asp:TextBox> 
       &nbsp; 
      </td> 
     </tr> 
     <tr> 
      <td> 
       &nbsp;</td> 
      <td style="height: 51px; width: 377px; font-weight: bold;"> 
       Assigned Date</td> 
      <td> 
       <asp:TextBox ID="txtStartDate" runat="server" Width="200px"></asp:TextBox> 
      </td> 
     </tr> 
     <tr> 
      <td> 
       &nbsp;</td> 
      <td style="height: 51px; width: 377px; font-weight: bold;"> 
       Due Date</td> 
      <td> 
       <asp:TextBox ID="txtEndDate" runat="server" Width="200px"></asp:TextBox> 
      </td> 
     </tr> 
     <tr> 
      <td> 
       &nbsp;</td> 
      <td style="height: 51px; width: 377px; font-weight: bold;"> 
       Project Platform</td> 
      <td> 
       <asp:TextBox ID="TxtProjectPlatform" runat="server" Width="200px"></asp:TextBox> 
      </td> 
     </tr> 
     <tr> 
      <td> 
       &nbsp;</td> 
      <td style="height: 51px; width: 377px; font-weight: bold;"> 
       BIOS</td> 
      <td> 
       <asp:TextBox ID="TxtBios" runat="server" Width="200px"></asp:TextBox> 
      </td> 
     </tr> 
     <tr> 
      <td> 
       &nbsp;</td> 
      <td style="height: 51px; width: 377px; font-weight: bold;"> 
       PCH</td> 
      <td> 
       <asp:TextBox ID="TxtPch" runat="server" Width="200px"></asp:TextBox> 
      </td> 
     </tr> 
     <tr> 
      <td> 
       &nbsp;</td> 
      <td style="height: 51px; width: 377px; font-weight: bold;"> 
       PROC</td> 
      <td> 
       <asp:TextBox ID="TxtProc" runat="server" Width="200px"></asp:TextBox> 
      </td> 
     </tr> 
     <tr> 
      <td> 
       &nbsp;</td> 
      <td style="height: 51px; width: 377px; font-weight: bold;"> 
       Forward To</td> 
      <td> 
       <asp:DropDownList ID="employeeEmailDropDownList" runat="server" 
      Visible="True" 
        DataSourceID="SqlDataSource2" DataTextField="EmailAddress" DataValueField="EmailAddress" Width="203px"> 
     </asp:DropDownList> 
       <asp:SqlDataSource ID="SqlDataSource2" runat="server" ConnectionString="<%$ ConnectionStrings:ProjectsAndTasksTestConnectionString %>" 
        SelectCommand="SELECT [EmailAddress] FROM [Users] ORDER BY [TesterName]"></asp:SqlDataSource></td>  
     </tr> 
     <tr> 
      <td> 
       &nbsp;</td> 
      <td> 
       &nbsp;</td> 
      <td> 
       &nbsp;</td> 
     </tr> 
     <tr> 
      <td> 
       &nbsp;</td> 
      <td style="width: 377px"> 
       &nbsp;</td> 
      <td> 
       <asp:Button ID="Button1" runat="server" onclick="CreateProject_Click" 
        Text="Create Project" Font-Bold="true" Height="45px" Width="198px" /> 
       <asp:Label ID="errorLabel" runat="server" ForeColor="Red"></asp:Label> 
       <asp:Label ID="successLabel" runat="server" ForeColor="Green"></asp:Label> 
      </td> 
     </tr> 
     <tr> 
      <td style="height: 29px"> 
       </td> 
      <td style="height: 29px; width: 377px;"> 
       </td> 
      <td style="height: 29px"> 
       </td> 
     </tr> 
    </table> 
    </asp:Panel> 

</asp:Content> 

и здесь является C# У меня есть

using System; 
using System.Collections.Generic; 
using System.Configuration; 
using System.Data; 
using System.Data.SqlClient; 
using System.Linq; 
using System.Web; 
using System.Web.UI; 
using System.Web.UI.WebControls; 

namespace Project_Set 
{ 

    public partial class Projects_AddProject : System.Web.UI.Page 
    { 

     protected void Page_Load(object sender, EventArgs e) 
     { 
      if (!IsPostBack) 
      { 
       Populate1(); 
      } 
     } 

     protected void ddlTester_DataBound(object sender, EventArgs e) 
     { 
      //Inserting an item in the 0 index of the DDL named "-Select-" 
      //which will navigate the user to select an item 
      ddlTester.Items.Insert(0, new ListItem("-Select-")); 
     } 

     public void Populate1() 
     { 
      if (!IsPostBack) 
      { 
       string connetionString = "Data Source=JVANCEX-MOBL2;Initial Catalog=ProjectsAndTasksTest; Trusted_Connection=True;"; 
       SqlCommand cmd = new SqlCommand("SELECT TesterName AS FullName FROM [Users]", new SqlConnection(ConfigurationManager.ConnectionStrings["ProjectsAndTasksTestConnectionString"].ConnectionString)); 

       using (SqlConnection cnn = new SqlConnection(connetionString)) 
       { 

        cmd.Connection.Open(); 

        SqlDataReader ddlValues; 
        ddlValues = cmd.ExecuteReader(); 

        ddlTester.DataSource = ddlValues; 
        ddlTester.DataValueField = "FullName"; 
        ddlTester.DataTextField = "FullName"; 
        ddlTester.DataBind(); 

        cmd.Connection.Close(); 
        //cmd.Connection.Dispose(); 
       } 
      } 

     } 

     public void CreateProject_Click(object sender, EventArgs e) 
     { 
      if (!IsPostBack) 
      { 
       string connetionString = "Data Source=JVANCEX-MOBL2;Initial Catalog=ProjectsAndTasksTest; Trusted_Connection=True;"; 
       string insStmt = "INSERT INTO Projects (TesterName, ProjectName, ProjectDescription, DueDate, DateAssigned, Platform) values (@TesterName, @ProjectName,@ProjectDesc, @DueDate, @DateAssigned, @Platform)"; 

       using (SqlConnection cnn = new SqlConnection(connetionString)) 
       { 
        cnn.Open(); 
        SqlCommand insCmd = new SqlCommand(insStmt, cnn); 
        insCmd.Parameters.AddWithValue("@ProjectName", txtProjectName); 
        insCmd.Parameters.AddWithValue("@ProjectDesc", txtProjectDesc); 

        insCmd.Parameters.AddWithValue("@TesterName", ddlTester.SelectedItem); 

        insCmd.Parameters.AddWithValue("@DateAssigned", txtStartDate); 
        insCmd.Parameters.AddWithValue("@DueDate", txtEndDate); 
        insCmd.Parameters.AddWithValue("@Platform", TxtProjectPlatform); 
        //insCmd.Parameters.AddWithValue("@Bios", TxtBios.Text); 
        //insCmd.Parameters.AddWithValue("@Pch", TxtPch.Text); 
        //insCmd.Parameters.AddWithValue("@Proc", TxtProc.Text); 
        insCmd.ExecuteNonQuery(); 
        cnn.Close(); 
        //MessageBox.Show(affectedRows + " Project Created!"); 

       } 

      } 

     } 

Я знаю, что нужно использовать .Parameters для защиты от возможного введения MySQL, но помимо этого я новый для asp.net и C#.

Answer 1

У вас есть if (!IsPostBack) вокруг вашего кода, который в противном случае вставлял бы данные в базу данных, но единственный способ, которым это событие могло быть достигнуто (кроме прямого вызова его из C#), было бы, если бы вы нажали кнопку и вызвали обратную передачу.

Вам нужно удалить эту инструкцию if из этого метода, и она должна работать.

Вы бы быстро заметили это, если бы у вас в коде был set a breakpoint, а затем шагнул. Если у вас возникнет вопрос о потоке программы, установите точку останова и пройдите по коду по строкам.

Некоторые другие примечания:

• Вы должны переместить строку подключения в файл конфигурации, такого как web.config. Если вам когда-либо понадобится обновить его, это не даст вам перестроить веб-сайт. Вы можете получить к нему доступ программно через ConfigurationManager.ConnectionStrings["connection string name here"].ConnectionString

• Я бы переместил код, который создает SqlCommand, и задает параметры. вне используемого блока, затем выполните insCmd.Connection = cnn;. Мне нравится, чтобы моя связь открывалась как можно меньше времени. Это также позволит вам использовать конструктор, который принимает командную строку в качестве параметра, поэтому нет необходимости в переменной insStmt.

• Для веб-форм нет классов MessageBox, например, для WPF или WCF. Вместо этого вы можете запустить JavaScript, когда страница завершит загрузку на клиенте. Пока что простенький alert() будет делать, но в конце концов вы можете собрать его с чем-то вроде noty.

• Вы можете использовать неявно типизированную переменную var вместо того, чтобы явно указывать для переменных. Это делает код короче и легче читается, и проще реорганизовать, если вы в конечном итоге меняете типы позже. И на самом деле у него нет недостатков.

• Вы должны обернуть попытку/уловку вокруг выполнения команды вставки, чтобы поймать какие-либо проблемы и, возможно, зарегистрировать ошибку где-нибудь, а затем показать общий «Произошла непредвиденная ошибка» для клиента.

• Переменные, которые вы передаете в свои параметры, имеют тип TextBox по большей части.Это означает, что вам необходимо получить доступ к свойству .Text, чтобы получить введенный текст.

• Если DateAssigned и DueDate являются типами даты в вашей базе данных, вам необходимо использовать DateTime.TryParse, чтобы убедиться, что они являются действительными строками, которые представляют дату. Если они не соответствуют формату, отобразите сообщение об ошибке клиенту и не выполните команду insert. Клиент должен использовать некоторый JavaScript, чтобы принудительно ввести введенную дату в правильный формат (но не полагайтесь на нее, если пользователь отключил JavaScript).

Суммируя все это, вот как я бы написать обработчик события:

protected void CreateProject_Click(object sender, EventArgs e) 
{ 
    var insCmd = new SqlCommand("INSERT INTO Projects (TesterName, ProjectName, ProjectDescription, DueDate, DateAssigned, Platform) values (@TesterName, @ProjectName,@ProjectDesc, @DueDate, @DateAssigned, @Platform)"); 
    insCmd.Parameters.AddWithValue("@ProjectName", txtProjectName.Text); 
    insCmd.Parameters.AddWithValue("@ProjectDesc", txtProjectDesc.Text); 
    insCmd.Parameters.AddWithValue("@TesterName", ddlTester.SelectedItem); //This should probably be .SelectedValue instead 
    insCmd.Parameters.AddWithValue("@DateAssigned", txtStartDate.Text); 
    insCmd.Parameters.AddWithValue("@DueDate", txtEndDate.Text); 
    insCmd.Parameters.AddWithValue("@Platform", TxtProjectPlatform.Text); 

    using (var cnn = new SqlConnection(ConfigurationManager.ConnectionStrings["TestDatabase"].ConnectionString)) 
    { 
     insCmd.Connection = cnn; 
     cnn.Open();    
     insCmd.ExecuteNonQuery(); 
     cnn.Close(); 
    } 

    ClientScript.RegisterStartupScript(this.GetType(), "insert-success", "alert('Project created');", true); 
}