ExecuteNonQuery renvoie toujours 0

**BernardBouree** · 27/12/2017, 17h15

Bonjour

J'ai une requête sur une base Access définie de la façon suivante:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
var query = "UPDATE tblLR " +
                        "SET Location = @location " +
                        ", SortedLocation = @sortedLocation" +
                        ", Latitude = @latitude" +
                        ", Longitude = @longitude " +
                        " WHERE IDLR = @idLr";
 
            var dBcon = new OleDbConnection(Legacy.ConString);
            var cmd = new OleDbCommand(query, dBcon);
 
            cmd.Parameters.Add("@location", OleDbType.VarChar).Value = topLegacy.Location.Replace("'", "''");
            cmd.Parameters.Add("@sortedLocation", OleDbType.VarChar).Value = topLegacy.Location.Replace("'", "''");
            cmd.Parameters.Add("@shortName", OleDbType.VarChar).Value = topLegacy.ShortName.Replace("'", "''");
            cmd.Parameters.Add("@latitude", OleDbType.Double).Value = topLegacy.Latitude;
            cmd.Parameters.Add("@longitude", OleDbType.Double).Value = topLegacy.Longitude;
            cmd.Parameters.Add("@idLr", OleDbType.BigInt).Value = topLegacy.Idlr;
 
            var result = 0;
            using (dBcon)
            {
                try
                {
                    dBcon.Open();
                    result = cmd.ExecuteNonQuery();
                    if (result <= 0)
                    {
                        string msg = "Erreur dans MettreAJourLegacy avec comme arguments: " +
                        "\n topLegacy =  " + topLegacy.Idlr + " " + topLegacy.ToString() +
                         "\n topGeoDB = " + topGeoDb.ToString();
                        MessageBox.Show(msg, "Erreur fatale dans la méthode MettreAJourLegacy()", MessageBoxButtons.OK, MessageBoxIcon.Exclamation);
 
                    }
                    return result <= 0 ? false : true;
                }
                catch (OleDbException ex)
                {
                    string msg = ex.Message + "\n Erreur dans MettreAJourLegacy avec comme arguments: " +
                        "\n topLegacy =  " + topLegacy.Idlr + " " + topLegacy.ToString() +
                         "\n topGeoDB = " + topGeoDb.ToString();
                    //  Program.ReportCrash(ex, msg);
                    // ToponymeNonTraite.ListeToponymesNonTraites.Add(topLegacy);
                    MessageBox.Show(msg, "Erreur fatale dans la méthode MettreAJourLegacy()", MessageBoxButtons.OK, MessageBoxIcon.Exclamation);
                    return false;
                }
 
            }

La ligne result=cmd.ExecuteNonQuery();
renvoie systématiquement 0 et la requête n'est pas exécutée.

J'ai cherché sur le web mais n'ai pas trouvé la solution.

Merci pour votre aide

Bernard

**tatayo** · 27/12/2017, 17h42

Bonjour,
A mon avis il manque les quotes autours des valeurs de type chaine dans la requête, par exemple autour de @location.
Accessoirement, je pense qu'il serait utile (au moins en phase de debug) d'ajouter l'erreur retournée par Access dans le message d'erreur.
Tu auras ainsi des pistes dans ce genre de blocage.

Tatayo.

**BernardBouree** · 27/12/2017, 18h47

Bonjour

Je n'ai aucune erreur fournie par la requête!
Je suppose que si la syntaxe de la requête était erronée j'aurai une erreur.

Bernard

**popo** · 28/12/2017, 11h41

Envoyé par tatayo

Bonjour,
A mon avis il manque les quotes autours des valeurs de type chaine dans la requête, par exemple autour de @location.
Accessoirement, je pense qu'il serait utile (au moins en phase de debug) d'ajouter l'erreur retournée par Access dans le message d'erreur.
Tu auras ainsi des pistes dans ce genre de blocage.

Tatayo.

L'un des avantages des requêtes paramétrées est justement de se passer des quotes et des problèmes de format.

BernardBouree,
ExecuteNonQuery va déclencher une InvalidOperationException
OleDbException se produira si tu ne parviens pas a établir une connexion lors du Open().

Si tu passe dans le catch, cela signifie que la connexion ne s'est pas effectuée et dans ce cas, il est normal que la requête ne soit pas jouée.
Donc, en premier lieu :
- vérifie la chaine de connexion,
- vérifie la présence du lien ODBC (si ça existe toujours, il y a bien longtemps que j'ai pas tenté de me connecter à Access)
- regarde ce que te dis l'exception remontée.

Une fois cette étape passée et corrigée et seulement à ce moment, on pourra intervenir dans le code.
La façon dont tu codes ça est bizarre, j'essaierai plutôt comme ceci :

Code C# :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
using (var dBcon = new OleDbConnection(Legacy.ConString))
{
  using (var cmd = new OleDbCommand(query, dBcon))
  {
    cmd.Parameters.Add("@location", OleDbType.VarChar).Value = topLegacy.Location.Replace("'", "''");
    cmd.Parameters.Add("@sortedLocation", OleDbType.VarChar).Value = topLegacy.Location.Replace("'", "''");
    cmd.Parameters.Add("@shortName", OleDbType.VarChar).Value = topLegacy.ShortName.Replace("'", "''");
    cmd.Parameters.Add("@latitude", OleDbType.Double).Value = topLegacy.Latitude;
    cmd.Parameters.Add("@longitude", OleDbType.Double).Value = topLegacy.Longitude;
    cmd.Parameters.Add("@idLr", OleDbType.BigInt).Value = topLegacy.Idlr;
 
    try
    {
      dBcon.Open();
      var result = cmd.ExecuteNonQuery();
      if (result <= 0)
      {
         string msg = "Erreur dans MettreAJourLegacy avec comme arguments: " +
                        "\n topLegacy =  " + topLegacy.Idlr + " " + topLegacy.ToString() +
                         "\n topGeoDB = " + topGeoDb.ToString();
         MessageBox.Show(msg, "Erreur fatale dans la méthode MettreAJourLegacy()", MessageBoxButtons.OK, MessageBoxIcon.Exclamation);
         return false;
       }
       return true;
    }
    catch (OleDbException ex)
    {
        string msg = ex.Message + "\n Erreur dans MettreAJourLegacy avec comme arguments: " +
                        "\n topLegacy =  " + topLegacy.Idlr + " " + topLegacy.ToString() +
                         "\n topGeoDB = " + topGeoDb.ToString();
 
        MessageBox.Show(msg, "Erreur fatale dans la méthode MettreAJourLegacy()", MessageBoxButtons.OK, MessageBoxIcon.Exclamation);         
     }
  }
  return false;
}

**Noxen** · 28/12/2017, 12h33

Je suppose que les Replace("'", "''") servent à échapper les ', mais vous êtes sûrs que c'est nécessaire ? Est-ce-que ce n'est pas directement pris en charge par le constructeur de requête ?

**BernardBouree** · 28/12/2017, 15h23

Bonjour à tous et merci pour votre aide.

Non dans la version avec paramètres, je ne passe pas dans le catch, il n'y donc pas d'erreur fatale.
Mais j'ai systématiquement un result = cmd.ExecuteNonQuery qui me renvoit un 0 au lieu du 1 attendu qui est le nombre d'enregistrements concernés.
Je vais essayer votre proposition de code.

Pour les '' oui c'est obligatoire car sinon la requête est erronée.

Bernard

**Pol63** · 28/12/2017, 15h31

arretez de dire n'importe quoi
le but des DbParameters est entre autre de ne pas avoir de problème avec les ' donc faire des replaces ne peut que causer des problèmes
(et les dbparameters ne modifient pas la requete, ils sont passés comme variable, par exemple une date reste une date sur 8 octets au lieu d'être transformée en string)

outre le fait que se baser sur le retour d'executenonquery n'est pas toujours une bonne idée (sur access ca doit aller encore)
si ca te retourne 0 et que rien n'est modifié dans la base c'est que ta requete est mal écrite ou les valeurs que tu passes ne sont pas celles que tu penses ou les données dans la base ne sont pas celles que tu penses
donc y a juste à mettre un point d'arrêt pour vérifier, et tester la requete sur access, voire en select

ExecuteNonQuery renvoie toujours 0

C#

Vue hybride

Discussions similaires

Partager

Partager