Ausführen einer update-gespeicherte Prozedur in C#

Muss ich wissen, wenn ich Schreibe, die gespeicherte Prozedur ordnungsgemäß und Wenn der C# - code für die Ausführung korrekt ist. aus irgendeinem Grund ist der Fehler zurückgegeben wird, da ist Falsche syntax in der Nähe 'c16b'. Alte Fehler

Der neue Fehler ist jetzt: Prozedur oder Funktion 'sptimeupdate' erwartet parameter '@Datum', der nicht angegeben wurde.

nvarchar-Zeichenfolge für die überprüfung und Aktualisierung der in der Spalte ClientID ist 3fc8ffa1-c16b-4d7b-9e55-1e88dfe15277, aber der Teil in Fett ist nur zu zeigen, in der debug-test-intel-Sinn in der Fehlerbehandlung

ALTER PROCEDURE sptimeupdate
    @id nvarchar(50),
    @date datetime
AS
BEGIN
    SET NOCOUNT ON;
    UPDATE ClientTable
    SET Today_Date=(@date)
    WHERE ClientID=(@id)
END

//--------------above stored procedure--------------------------------
//--------------Executing the stored procedure in C#

 IEnumerable<XElement> searchClientID =
 from clientid in main.XPathSelectElements("Network/ClientID")
 where (string)clientid.Attribute("id") == IntializedPorts[i].ToString()
 select clientid;

foreach (string clientid in searchClientID)
  {
     for (int up = 0; up < IntializedPorts.Count(); up++)
     {
          //Update the current time in the clientid tble.
          //Renames the table copy for groups
      try
       {
         string[] Clientid; //client id array
         Clientid = new string[IntializedPorts.Count()]; //Intialization of the array
         Clientid[up] = clientid.ToString();
         DateTime td = Convert.ToDateTime(toolDate.Text); //Just added a datetime object withdate
 SqlConnection sqlConnectionCmdString = new SqlConnection(@"Data=.\SQLEXPRESS;AttachDbFilename=C:\Users\Shawn\Documents\Visual Studio 2010\Projects\Server\database\ClientRegit.mdf;Integrated Security=True;User Instance=True");

   //EXECUTE THE STORED PROCEDURE sptimedate
   //string UpdateCommand = "sptimeupdate" + Clientid[up].ToString() + toolDate.Text;
   string UpdateCommand = "sptimeupdate" + "'" + Clientid[up].ToString() + "'" + "'" +td.ToString()+ "'"; //this is the new UpdateCommand string as to pass parameters to stored procedure 
    SqlCommand sqlRenameCommand = new SqlCommand(UpdateCommand, sqlConnectionCmdString);

    sqlConnectionCmdString.Open();
    sqlRenameCommand.ExecuteNonQuery();
    sqlConnectionCmdString.Close();
  }
  catch(DataException ex)
  {                                                                                                 MessageBox.Show("Failed to UpdateCurrentTime","DataError",MessageBoxButtons.OK,MessageBoxIcon.Error);
  }

 }

}
  • Wenn Sie könnten, auch könnten Sie bitte zeigen, im code für coverting die SQL-Daten-Verbindungs-string in eine Globale Zeichenfolge, so dass es gefunden werden kann auf jedem computer, anstatt nur von mir? Dank
  • Ok, ich änderte den code etwas und jetzt gibt mir einen anderen Fehler , Ihr zu sagen, die geben nicht die datetime in der datetime-parameter.
InformationsquelleAutor shawn | 2014-10-23
  1. 3

    Wenn Sie eine gespeicherte Prozedur aufrufen von code, den Sie brauchen, um einen Befehl zu erstellen mit Befehl-Typ StoredProcedure, ansonsten ist der Motor versucht, Ihren Befehl text wie es war ein sql-text: SELECT EINFÜGEN etc... Aber das mose wichtigste ist, dass Sie brauchen, um passieren die erforderlichen Parameter der gespeicherten Prozedur in der Parameters-Auflistung des command

    Also das könnte der code zum ersetzen der aktuellen

    string UpdateCommand = "sptimeupdate";
using(SqlConnection sqlConnectionCmdString = new SqlConnection(......))
using(SqlCommand sqlRenameCommand = new SqlCommand(UpdateCommand, sqlConnectionCmdString))
{
    DateTime td = Convert.ToDateTime(toolDate.Text);
    sqlRenameCommand.CommandType = CommandType.StoredProcedure;    
    sqlRenameCommand.Parameters.Add("@id", SqlDbType.NVarChar).Value = Clientid[up].ToString();
    sqlRenameCommand.Parameters.Add("@date", SqlDbType.DateTime).Value = td;
    sqlConnectionCmdString.Open();
    sqlRenameCommand.ExecuteNonQuery();
}

    Beachten Sie zwei Dinge. Die using-Anweisung ist die beste Praxis zu Folgen, wenn Sie eine Verbindung erstellen, um sicherzustellen, dass das korrekte schließen und die Entsorgung der Verbindung, zweitens, die parameter der DateTime erwartet von der sp verabschiedet werden sollte als ein DateTime-nicht als string - das bedeutet natürlich, dass Sie sollten sicher sein, dass die Inhalte der toolDate ist umwandelbar zu einem DateTime-Wert.

    • Ich habe versucht, genau so, wie Sie es haben, und ich bekomme das gleiche Ergebnis, beide Ihre und mein Weg wird beide arbeiten - aber ich bekomme immer noch eine Fehlermeldung, die sagt Prozedur oder Funktion 'sptimeupdate' erwartet parameter '@Datum', der nicht angegeben wurde.
    • Sorry, my bad, wird der parameter name ist das Datum, fixieren Sie es jetzt
    • Beachten Sie jedoch auch die skizzierten Ansatz von Bernd Linde in seiner Antwort. Wenn die Anzahl der loops ist klein, es gibt keinen großen Unterschied, aber viele Elemente halten den Bau der parameter-Sammlung außerhalb der Schleife und nur die Einstellung der Werte innerhalb der Schleife vorzuziehen ist
    InformationsquelleAutor Steve
  2. 1

    Ihre Fehler stammt aus dieser code-Zeile:

    string UpdateCommand = "sptimeupdate" + Clientid[up].ToString() + toolDate.Text;

    Gibt es Sie nur in der Verknüpfung der Clientid[up].ToString() als eine Zeichenkette in die andere Zeichenkette, das gleiche mit der toolDate.Text, sowohl ohne als auch sql-markup.

    Die resultierende SQL-Abfrage würde wie folgt Aussehen (unter der Annahme toolDate.Text ist '2014-10-23'):

    sptimeupdate3fc8ffa1-c16b-4d7b-9e55-1e88dfe152772014-10-23

    die, wie Sie sehen können, ist nicht eine korrekte SQL-Befehl.

    Sollten Sie immer verwenden parametrisierten Befehl-Anweisungen beim aufrufen von einfachen SQL-Befehlen.

    Jedoch in Ihrem Fall, Sie sind eigentlich das aufrufen einer gespeicherten Prozedur.

    So ändern Sie Ihren code so Griff es wie eine gespeicherte Prozedur, Beispiel unten.

    //Create the connection object once
using (SqlConnection sqlConnectionCmdString = new SqlConnection(@"Data=.\SQLEXPRESS;AttachDbFilename=C:\Users\Shawn\Documents\Visual Studio 2010\Projects\Server\database\ClientRegit.mdf;Integrated Security=True;User Instance=True"))
{
  //Same with the SqlCommand object and adding the parameters once also
  SqlCommand sqlRenameCommand = new SqlCommand("sptimeupdate", sqlConnectionCmdString);
  sqlRenameCommand.CommandType = CommandType.StoredProcedure;
  sqlRenameCommand.Parameters.Add("@id", SqlDbType.NVarChar);
  sqlRenameCommand.Parameters.Add("@datetime", SqlDbType.DateTime);

  //Open the connection once only
  sqlConnectionCmdString.Open();

  foreach (string clientid in searchClientID)
  {
    for (int up = 0; up < IntializedPorts.Count; up++)
    {
      try
      {
        //The below three lines seem redundant.
        //Clientid[up] will be equal to clientid after it all, so just use clientid
        //string[] Clientid;
        //Clientid = new string[IntializedPorts.Count];
        //Clientid[up] = clientid.ToString();

        sqlRenameCommand.Parameters["@id"].Value = clientid;
        sqlRenameCommand.Parameters["@datetime"].Value = toolDate.Text;
        sqlRenameCommand.ExecuteNonQuery();
      }
      //Might want to move this try..catch outside the two loops,
      //otherwise you will get this message each time an error happens
      //which might be alot, depending on the side of searchClientID
      catch (SqlException)
      {
        MessageBox.Show("Failed to UpdateCurrentTime", "DataError", MessageBoxButtons.OK, MessageBoxIcon.Error);
      }
    }
  }
}

    HINWEIS:
    Bitte Lesen Sie die Kommentare in den code oben für weitere Hinweise und Anregungen.
    Erstellen einer SqlConnection und SqlCommand, die für jede iteration wird die Leistung auswirken, auf Ihre Anwendung. Also lieber erstellen Sie Sie einmal und verwenden Sie Sie wieder, bis Sie fertig sind.

    Weiter Lesen kann man hier:

    P. S. dein sql-Prozedur-code, sieht gut aus, man könnte entfernen Sie die SET NOTCOUNT ON da nicht wirklich viel tun in diesem Szenario

    • Danke, Schau ich änderte die UpdateCommand-string, und es ist jetzt auf der Suche nur für den Datetime-parameter
    • Das problem ist die Art und Weise der string war gebaut zu werden, an die gespeicherte Prozedur übergeben, ich muss nur herausfinden, warum seine nicht immer die datetime -, alles andere ist so weit in Ordnung. Ich muss nur genügen die parameter nennen, die für das @DateTime-Parameter und es sollte wohl update
    • string UpdateCommand = "sptimeupdate" + "'" + Clientid[up].ToString() + "'" + "'" +td.ToString()+ "'"; //Das ist die neue
    • Haben Sie keine Leerzeichen in deinem Befehl text und auch keine Kommas trennen die verschiedenen Parameter.
    • Danke, Sie war beides Recht, aber ich habe die anderen Jungs Methode, es funktionierte perfekt, denn er hatte die Kommas im Ort und ich Schätze Euch beide vielen Dank
    InformationsquelleAutor Bernd Linde
Schreibe einen Kommentar