Update-Tabelle von der Webseite

Question

Ich verwende eine gespeicherte Prozedur, um eine Tabelle zu aktualisieren, wenn der Benutzer auf Genehmigen klickt, aber ich bekomme die Nachricht, dass es erfolgreich genehmigt wurde, aber die Tabelle wurde nicht aktualisiert.

Aktualisieren Ich bin jetzt nur in der Lage, ein Element zu aktualisieren; Selbst wenn ich mehr als ein Element auswähle, wird nur eines aktualisiert.

Stored Procedure

Update RequisitionItem 
set [status] = 0, 
[approve_date] = @approve_date 
--[ApprovedBy]=@ApprovedBy 
where [status]=-1 and Req_No=@reqNumber and item_no=@item_no 

Methode

public void SetRequisitionStatus0(List <string> reqNumber,List <string> item_no) 
    { 
     SqlConnection connection = new SqlConnection(connectionString); 
     SqlCommand command = new SqlCommand(); 
     command.CommandText = "requisition_sp_setstatus0"; 
     command.CommandType = CommandType.StoredProcedure; 
     command.Parameters.Add("@reqNumber", SqlDbType.VarChar); 
     command.Parameters.Add("@item_no", SqlDbType.VarChar); 
     //command.Parameters.Add("@ApprovedBy", SqlDbType.VarChar); 
     command.Parameters.Add("@approve_date", SqlDbType.DateTime).Value = DateTime.Now; 

     using (command.Connection = connection) 
     { 
      try 
      { 
       connection.Open(); 
       foreach (var item in reqNumbers) 
       { 
        command.Parameters["@reqNumber"].Value = item; 

        foreach (var item1 in item_no) 
        { 
         command.Parameters["@item_no"].Value = item1; 
        } 

        command.ExecuteNonQuery(); 
       } 

      } 

      finally 
      { 
       connection.Close(); 
      } 
     } 
     return; 

    } 

Answer 1

Ich denke, die foreach-Schleifen Sie vermasselt. Die innere Schleife wird die gesamte Liste durchlaufen (item_no) und dann zurück zur äußeren Schleife, dann durchläuft die innere Schleife erneut die gesamte Liste und gibt Ihnen immer das letzte Element. Sie benötigen einen for Schleife, so etwas wie

for(int i = 0; i < reqNumbers.count; i++) 
{ 
    command.Parameters["@reqNumber"].Value = reqNumbers[i]; 
    command.Parameters["@item_no"].Value = item_no[i]; 

    command.ExecuteNonQuery(); 
} 

Sie nicht brauchen, um die Verbindung zu schließen, entweder - die using-Anweisung wird es automatisch schließen. und Sie brauchen nicht das letzte return.