2012-01-12 7 views
6

Otrzymuję ten komunikat o błędzie, gdy próbuję wykonać następujący kod.ExecuteNonQuery wymaga polecenia do transakcji

ExecuteNonQuery requires the command to have a transaction when the connection assigned to the command is in a pending local transaction 

Czy ktoś może doradzić, gdzie jest problem? Domyślam się, że źródłem problemu jest część, w której próbuję wykonać procedurę przechowywaną.

Procedura składowana jest tworzy własny transakcji podczas wykonywania

using (SqlConnection conn = new SqlConnection(connStr)) 
      { 
       conn.Open(); 

       SqlCommand command = conn.CreateCommand(); 
       SqlTransaction transaction; 

       // Start a local transaction. 
       transaction = conn.BeginTransaction("createOrder"); 

       // Must assign both transaction object and connection 
       // to Command object for a pending local transaction 
       command.Connection = conn; 
       command.Transaction = transaction; 

       try 
       { 
        command.CommandText = "INSERT INTO rand_resupply_order (study_id, centre_id, date_created, created_by) " + 
         "VALUES (@study_id, @centre_id, @date_created, @created_by) SET @order_id = SCOPE_IDENTITY()"; 

        command.Parameters.Add("@study_id", SqlDbType.Int).Value = study_id; 
        command.Parameters.Add("@centre_id", SqlDbType.Int).Value = centre_id; 
        command.Parameters.Add("@date_created", SqlDbType.DateTime).Value = DateTime.Now; 
        command.Parameters.Add("@created_by", SqlDbType.VarChar).Value = username; 

        SqlParameter order_id = new SqlParameter("@order_id", SqlDbType.Int); 
        //study_name.Value = 
        order_id.Direction = ParameterDirection.Output; 
        command.Parameters.Add(order_id); 

        command.ExecuteNonQuery(); 
        command.Parameters.Clear(); 

        //loop resupply list 
        for (int i = 0; i < resupplyList.Count(); i++) 
        { 
         try 
         { 
          SqlCommand cmd = new SqlCommand("CreateOrder", conn); 
          cmd.CommandType = CommandType.StoredProcedure; 

          cmd.Parameters.Add("@study_id", SqlDbType.Int).Value = study_id; 
          cmd.Parameters.Add("@centre_id", SqlDbType.Int).Value = centre_id; 
          cmd.Parameters.Add("@created_by", SqlDbType.VarChar).Value = username; 
          cmd.Parameters.Add("@quantity", SqlDbType.VarChar).Value = resupplyList[i].Quantity; 
          cmd.Parameters.Add("@centre_id", SqlDbType.Int).Value = centre_id; 
          cmd.Parameters.Add("@depot_id", SqlDbType.VarChar).Value = depot_id; 
          cmd.Parameters.Add("@treatment_code", SqlDbType.Int).Value = centre_id; 
          cmd.Parameters.Add("@order_id", SqlDbType.Int).Value = (int)order_id.Value; 
          cmd.ExecuteNonQuery(); 
         } 
         catch (SqlException ex) 
         { 
          transaction.Rollback(); 
          ExceptionUtility.LogException(ex, "error"); 
          throw ex; 
         } 
         catch (Exception ex) 
         { 
          transaction.Rollback(); 
          ExceptionUtility.LogException(ex, "error"); 
          throw ex; 
         } 
         finally 
         { 
          conn.Close(); 
          conn.Dispose(); 
         } 

        } 

        return (int)order_id.Value; 

       } 
       catch (Exception ex) 
       { 
        transaction.Rollback(); 
        ExceptionUtility.LogException(ex, "error"); 
        throw ex; 
       } 
       finally 
       { 
        // Attempt to commit the transaction. 
        transaction.Commit(); 

        conn.Close(); 
        conn.Dispose(); 
        command.Dispose(); 
       } 
+0

drobna uwaga - nie należy próbować popełnić w 'finally' - które powinny być raczej w' try' po operacji –

+0

może być nieco stary, ale mogą Państwo potwierdzić, że moja odpowiedź rozwiązała twój problem? – Afshin

Odpowiedz

2

użyciu Connection String transakcja nie tak popularny far.you możesz usuwać co rzeczy, które związane SqlTransaction a następnie owinąć kod z TransactionScope

+1

Należy zachować szczególną ostrożność w przypadku transakcji TransactionScope. Wydaje się całkiem miło, gdy działa. Ale okazało się, że jest bardzo delikatna podczas kierowania na nielokalne serwery SQL. Potrzebujesz rzeczy, które trzeba skonfigurować "po prostu". –

11

podczas korzystania z transakcji powinieneś używać go wszędzie.

cmd.Transaction = transaction; 
+0

ma znaczenie, gdy sama procedura składowana zawiera własną transakcję? – pothios

+0

Tak naprawdę to robi z punktu widzenia .net. – Afshin

+0

, więc pytanie, w jaki sposób można uzyskać uruchomienie transakcji, która wywoła procedurę przechowywaną z własnej transakcji :( – pothios

Powiązane problemy