Erreur de syntaxe SQL (commande INSERT)

Question

J'ai un formulaire avec une zone de texte et un bouton, de sorte que lorsque l'utilisateur clique sur le bouton, le nom spécifié dans la zone de texte est ajouté à une table dans ma base de données sql. Le code du bouton est la suivante:

private void btnAddDiaryItem_Click(object sender, EventArgs e) 
{ 
    try 
    { 
     string strNewDiaryItem = txtAddDiaryItem.Text; 
     if (strNewDiaryItem.Length == 0) 
     { 
      MessageBox.Show("You have not specified the name of a new Diary Item"); 
      return; 
     } 
     string sqlText = "INSERT INTO tblDiaryTypes (DiaryType) VALUES = ('" + strNewDiaryItem + "');"; 
     cSqlQuery cS = new cSqlQuery(sqlText, "non query"); 
     PopulateInitialDiaryItems(); 
     MessageBox.Show("New Diary Item added succesfully"); 
    } 
    catch (Exception ex) 
    { 
     MessageBox.Show("Unhandled Error: " + ex.Message); 
    } 
} 

La cSqlQuery de classe est une classe simple qui exécute différentes actions T-SQL pour moi et son code est le suivant:

class cSqlQuery 
{ 
    public string cSqlStat; 
    public DataTable cQueryResults; 
    public int cScalarResult; 

    public cSqlQuery() 
    { 
     this.cSqlStat = "empty"; 
    } 

    public cSqlQuery(string paramSqlStat, string paramMode) 
    { 
     this.cSqlStat = paramSqlStat; 

     string strConnection = BuildConnectionString(); 
     SqlConnection linkToDB = new SqlConnection(strConnection); 

     if (paramMode == "non query") 
     { 
      linkToDB.Open(); 
      SqlCommand sqlCom = new SqlCommand(paramSqlStat, linkToDB); 
      sqlCom.ExecuteNonQuery(); 
      linkToDB.Close(); 
     } 

     if (paramMode == "table") 
     { 
      using (linkToDB) 
      using (var adapter = new SqlDataAdapter(cSqlStat, linkToDB)) 
      { 
       DataTable table = new DataTable(); 
       adapter.Fill(table); 
       this.cQueryResults = table; 
      } 
     } 

     if (paramMode == "scalar") 
     { 
      linkToDB.Open(); 
      SqlCommand sqlCom = new SqlCommand(paramSqlStat, linkToDB); 
      this.cScalarResult = (Int32)sqlCom.ExecuteScalar(); 
      linkToDB.Close(); 
     } 
    } 

    public cSqlQuery(SqlCommand paramSqlCom, string paramMode) 
    { 
     string strConnection = BuildConnectionString(); 
     SqlConnection linkToDB = new SqlConnection(strConnection); 
     paramSqlCom.Connection = linkToDB; 

     if (paramMode == "table") 
     { 
      using (linkToDB) 
      using (var adapter = new SqlDataAdapter(paramSqlCom)) 
      { 
       DataTable table = new DataTable(); 
       adapter.Fill(table); 
       this.cQueryResults = table; 
      } 
     } 

     if (paramMode == "scalar") 
     { 
      linkToDB.Open(); 
      paramSqlCom.Connection = linkToDB; 
      this.cScalarResult = (Int32)paramSqlCom.ExecuteScalar(); 
      linkToDB.Close(); 
     } 
    } 

    public string BuildConnectionString() 
    { 
     cConnectionString cCS = new cConnectionString(); 
     return cCS.strConnect; 
    }   
} 

La classe fonctionne bien tout au long de ma demande, donc je ne pense pas que l'erreur est dans la classe, mais je ne peux pas être sûr.

Lorsque je clique sur le bouton, je reçois le message d'erreur suivant:

syntaxe incorrecte = près

Ce qui est vraiment ennuyeux moi, parce que quand je lance exactement la même commande dans SQL Management Studio ça fonctionne bien. Je suis sûr qu'il me manque quelque chose d'assez simple, mais après avoir lu mon code à plusieurs reprises, j'ai du mal à voir où je me suis trompé.

Answer 1

Vous n'avez pas besoin =

Un insert valide ressemblerait

INSERT INTO table_name (column1, column2, column3,...) 
VALUES (value1, value2, value3,...) 

Source: http://www.w3schools.com/sql/sql_insert.asp

Answer 2

Supprimez le = après VALUES.

Answer 3

vous devez enlever = après les valeurs.

string sqlText = "INSERT INTO tblDiaryTypes (DiaryType) VALUES ('" + strNewDiaryItem + "');" 

et d'essayer d'utiliser des requêtes paramétrées pour éviter l'injection Sql. Utilisez votre code comme ça. Sql Parameters

string sqlText = "INSERT INTO tblDiaryTypes (DiaryType) VALUES (@DairyItem);" 
    YourCOmmandObj.Parameters.AddwithValue("@DairyItem",strNewDiaryIItem) 

Answer 4

Supprimer "=", et aussi je vous recommande d'utiliser string.format() au lieu de concaténation de chaînes.

sqlText = string.format(INSERT INTO tblDiaryTypes (DiaryType) VALUES ('{0}'), strNewDiaryItem);"

Answer 5

S'il vous plaît utiliser ce qui suit:

insert into <table name> Values (value);