2016-08-12 3 views
0

Ich versuche, eine gespeicherte Prozedur auszuführen und die Ausgabe zu drucken, aber wenn ich den folgenden Code ausführen bekomme ich Fehler wie "Prozedur oder Funktion 'SPInsertLocal' erwartet Parameter '@RES', die nicht geliefert wurde.""Prozedur oder Funktion erwartet Parameter, der nicht geliefert wurde."

private void InsertPdtLocal(string code, string PON,string Qty) 
     { 
      string str = Properties.Settings.Default.conLocal; 
      SqlConnection con = new SqlConnection(str); 
      SqlCommand cmd = new SqlCommand("Execute SPInsertLocal @PON,@TCode,@Qty,@Type", con); 
      try 
      { 
       con.Open(); 
       cmd.CommandTimeout = 150; 
       cmd.Parameters.AddWithValue("@PON", PON); 
       cmd.Parameters.AddWithValue("@Qty", Qty); 
       cmd.Parameters.AddWithValue("@TCode", code); 
       cmd.Parameters.AddWithValue("@Type", Globals.s_type); 
       SqlParameter output = new SqlParameter("@RES", SqlDbType.Int); 
       output.Direction = ParameterDirection.Output; 
       cmd.Parameters.Add(output); 
       cmd.ExecuteNonQuery(); 
       con.Close(); 
       int id = Convert.ToInt32(output.Value); 
      } 
      catch (Exception ex) 
      { 
       MessageBox.Show(ex.Message); 
      } 

Was mache ich hier falsch?

+4

Ihr SQLCommand enthält nicht @RES. Sie könnten auch einfach den neuen SqlCommand ("SPInsertLocal", con) verwenden und dann CommandType = CommandType.StoredProcedure; –

+1

Sie sollten wahrscheinlich verwenden Anweisungen sowie SQLCommand implementiert IDisposible –

Antwort

1
SqlCommand cmd = new SqlCommand("Execute SPInsertLocal @PON,@TCode,@Qty,@Type,@RES", con); 

ich den Parameter nicht vorbei, das Problem behoben

0

Sie können den Code Refactoring wie folgt, wo die Anweisung using für die automatische Verwaltung der Verbindung Schließen verwendet wird und vermeiden die hartzucodieren Execute-Anweisung in C# -Code ist eine schlechte Praxis

private void InsertPdtLocal(string code, string PON,string Qty) 
     { 
      string str = Properties.Settings.Default.conLocal; 
      try 
      { 

      using (SqlConnection con = new SqlConnection(str)) 
      { 
       using (SqlCommand cmd = con.CreateCommand()) 
       { 
        cmd.Parameters.AddWithValue("@PON", PON); 
        cmd.Parameters.AddWithValue("@Qty", Qty); 
        cmd.Parameters.AddWithValue("@TCode", code); 
        cmd.Parameters.AddWithValue("@Type", Globals.s_type); 
        var output = cmd.Parameters.Add("@RES" , SqlDbType.Int); 
        output.Direction = ParameterDirection.Output; 
        cmd.ExecuteNonQuery(); 
        int id = Convert.ToInt32(output.Value); 
       } 
      } 

      } 
      catch (Exception ex) 
      { 
       MessageBox.Show(ex.Message); 
      } 
     } 
Verwandte Themen