System.Data.SqlClient.SqlException: 'Syntaxe incorrecte près de'. '.'

Nov 24 2020

J'essaye quelque chose sur l'API .Net Core Wep avec SQL. J'ai une syntaxe incorrecte près de «.» ».»

ı utilisez postman pour api et essayez de savoir s'il est venu ou non avec json.

dans le code appsettings.json;

{
  "ConnectionStrings": {
    "EmployeeAppCon": "Data Source=.;Initial Catalog=EmployeeDB; Integrated Security=true"
  },
  "Logging": {
    "LogLevel": {
      "Default": "Information",
      "Microsoft": "Warning",
      "Microsoft.Hosting.Lifetime": "Information"
    }
  },
  "AllowedHosts": "*"
}

Et j'ai essayé la méthode get, post mais j'ai eu une erreur ici avec la méthode Uptade;

[HttpPut]
        public JsonResult Put(Department dep)
        {
            string query = @"
                       Uptade dbo.Department set 
                       DepartmentName='"+dep.DepartmentName+@"'
                       where DepartmentId="+dep.DepartmentId+@"
                       ";
            DataTable table = new DataTable();
            string sqlDataSource = _configuration.GetConnectionString("EmployeeAppCon");
            SqlDataReader myReader;
            using (SqlConnection myCon = new SqlConnection(sqlDataSource))
            {
                myCon.Open();
                using (SqlCommand myCommand = new SqlCommand(query, myCon))
                {
                    myReader = myCommand.ExecuteReader();
                    table.Load(myReader); ;
                    myReader.Close();
                    myCon.Close();
                }

            }
            return new JsonResult("Uptade Successfull");

        }

Où est ma faute, aidez-moi, merci beaucoup?

Réponses

2 MarcGravell Nov 24 2020 at 08:41

Il s'agit probablement d'un problème de concaténation dans le SQL; histoire courte: ne jamais concaténer les entrées dans SQL; l' opération correcte ressemble plus à:

update dbo.Department
set DepartmentName=@name
where DepartmentId=@id

@nameet @idsont des paramètres.

Ensuite, vous utiliseriez myCommand.Parameters.Add(...)pour inclure ces deux paramètres et leurs noms / valeurs , et utilisez ExecuteNonQuery(non ExecuteReader).

Toutefois! Ce serait beaucoup plus simple de demander à Dapper (gratuit, etc.) de faire tout le travail pour nous, alors nous pouvons simplement faire:

using var myCon = new SqlConnection(sqlDataSource); // don't even need to open it
myCon.Execute(@"
update dbo.Department
set DepartmentName=@name
where DepartmentId=@id",
    new { name = dep.DepartmentName, id = dep.DepartmentId });

new {...}ici définit nos paramètres nommés avec des valeurs.