Requête Erreur SqlException

**Addx_man** · 30/01/2018, 10h47

Bonjour,

Je suis apprentis et je viens d'intégré une nouvelle entreprise. On m'a confié pour mission de travailler sur une interface qui tourne en ASP.Net avec une base SQL, et dans laquelle je dois corriger une erreur que je n'arrive a pas a résoudre.

J'ai également une question concernant la syntaxe du code suivant:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
  foreach (String idHangar in idHangars)
                    {
                        if (idHangar != "")
                        {
                            if (i == 0)
                                requeteHangars += " and (z.Id=" + idHangar + " ";
                            else
                                requeteHangars += " or z.Id=" + idHangar + " ";
                            i++;
                        }
                    }

A quoi sert la parenthèse en gras/souligné au niveau de z.id=" +idHangar +""; ? De séparateur ? j'ai eu l'impression qu'elle étais en trop mais il y a des parenthèses similaire à d'autres endroits du code.

Je vous poste l'erreur en capture d'écran ainsi que l'interface afficher avant l'erreur, c'est quand ont envoi les filtres que l'erreur apparaît, en espérant que vous pourriez m'apporter une aide !

Merci à vous.

Nom : ErreurSQLModif.png
Affichages : 197
Taille : 249,4 Ko

Nom : Message_erreurmodif.png
Affichages : 186
Taille : 122,4 Ko

Nom : Page_FiltreModif.png
Affichages : 196
Taille : 125,2 Ko

**Pol63** · 30/01/2018, 10h58

concernant la parenthèse elle sert à prioriser différement les and et les or, qui sont comme la multiplication et l'addition à ce niveau (la multiplication et les and sont prioritaires sur les additions et or)
5+3*8 étant différent de (5+3)*8

par contre c'est bien d'ouvrir des parenthèses mais il faut aussi les fermer, je pense que le problème vient de là, soit tu n'en fermes pas assez soit trop

après il faut apprendre à débugger, quand tu tombes sur l'erreur, tu peux regarder le détail de l'exception (le message au moins) ici problème de syntaxe vers )
après tu peux regarder la requête générée via les espions (dans le commandtext par exemple) pour l'analyser, tu trouveras alors quel est le problème de l'algo qui génère la requête

**Addx_man** · 30/01/2018, 11h23

Bonjour !

Merci pour cette réponse !

Et bien j'ai ajouté une parenthèse et j'ai une nouvelle erreur, plus haut dans le code cette fois ci ! Je pense que ce soit être un problème dans une requête SQL

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
                    // la liste des hangars est délimité par des tirets d'union "-"
                    // cette boucle va prendre l'ensemble des id hangars et les remplace par une requete sql
                    foreach (String idHangar in idHangars)
                    {
                        if (idHangar != "")
                        {
                            if (i == 0)
                                requeteHangars += " and (z.Id)=" + idHangar + " ";
                            else
                                requeteHangars += " or z.Id=" + idHangar + " ";
                            i++;
                        }
                    }

Nom : Message_Erreur_Ajout_Parent2.png
Affichages : 209
Taille : 206,1 Ko

**Pol63** · 30/01/2018, 11h28

Envoyé par Addx_man

Et bien j'ai ajouté une parenthèse

n'importe où et sans comprendre son utilité oui

en gros tu as fait (5)+3*8, ce qui n'a aucune utilité

Envoyé par Addx_man

Je pense que ce soit être un problème dans une requête SQL

il y a écrit SqlException et syntaxe incorrect vers ')' ... donc dire que ca ne vient pas de la requête me semble un peu raté ^^

l'ajout des parenthèses fermantes n'est pas sur le code que tu nous as mis au début, mais il doit être ailleurs, afin d'englober une opération de manière logique

**Addx_man** · 30/01/2018, 11h37

Ha désolé, c'est idiot de ma part, je n'ai pas pris le temps de bien étudié le code avant de faire ma modification...

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
                    foreach (String idHangar in idHangars)
                    {
                        if (idHangar != "")
                        {
                            if (i == 0)
                                requeteHangars += " and (z.Id=" + idHangar + " ";
                            else
                                requeteHangars += " or z.Id=" + idHangar + " ";
                            i++;
                        }
                    }
                    if (i != 0)
                    {
                        requeteHangars += ")";
                    }

Donc la parenthèse est a la fin pour englober le résultat du if.

Bref l'erreur n'est pas là, vue que la parenthèse est refermer. Je continue de creuser, et encore merci !

**Julien698** · 30/01/2018, 12h05

Salut,

Si tu as la possibilité d'accéder à la base, récupères la requête finalisée dans ta variable string et lances la directement l'éditeur SQL, tu y verras plus clair

**Addx_man** · 30/01/2018, 15h55

Envoyé par Julien698

Salut,

Si tu as la possibilité d'accéder à la base, récupères la requête finalisée dans ta variable string et lances la directement l'éditeur SQL, tu y verras plus clair

Je me sert de visual studio, après pour la base le problème c'est que c'est une base de prod sous windows server 2012 et je j'ai pas trop envie faire des tests dessus... !

**Prisson** · 30/01/2018, 18h27

Bonjour,

Je me sert de visual studio, après pour la base le problème c'est que c'est une base de prod sous windows server 2012 et je j'ai pas trop envie faire des tests dessus... !

Si tu veux vraiment retrouver ton erreur tu devrais faire certaines choses dont tu n'as pas envie de faire.
tu peux aussi faire une copie locale de la BD sur ton poste pour les tests ou bien encore commencer par supprimer la parenthèse qui te semble en trop et tester.

Une autre piste serait de vérifier la correspondance de toutes les parenthèses de fermeture en t'aidant du curseur. Try.......catch est également très utile pour la détection des exceptions.

**Addx_man** · 30/01/2018, 22h18

@Prisson

Tu a raison, je vais faire une copie de la BDD sur mon poste en locale et en faire une copie chez moi également.

Je vais tester les Try & Catch, merci pour la suggestion, je reviendrais vers vous pour les résultats.

Merci pour vos réponses, c'est encouragent !

**Addx_man** · 05/02/2018, 08h38

Bonjour, j'archive le sujet, le problème venait du fait que le i n'est pas remis a 0 lorsqu'il passe a la boucle des métiers. Je l'ai ajouté en gras en italique dans le code, donnez moi votre avis, et j'archiverais ensuite !

En attendant, je fais des tests

Merci à vous !

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
if (idHangar != "") {
      if (i == 0)
       requeteHangars += " and (z.Id=" + idHangar + " ";
      else
       requeteHangars += " or z.Id=" + idHangar + " ";
      i++;
     }
    }
    if (i != 0) {
     requeteHangars += ")";
    }
    //}
    // si on recoit une réponse d'un formulaire liste metiers
    if (lblMetiers.Text != null) {
     String[] idMetiers = new String[100];
     // on prend la réponse du formulaire la liste des métiers choisis
     idMetiers = lblMetiers.Text.Split(new [] {
      "-"
     }, StringSplitOptions.None);

     if (Request.Form["clos"] != null && Request.Form["clos"].ToString() == "true") {
      requeteMetiers += "";
     }
     // la liste des metiers est délimité par des tirets d'union "-"
     // cette boucle va prendre l'ensemble des id metiers et les remplace par une requete sql
	 i=0;
     foreach(String idMetier in idMetiers) {
      if (idMetier != "") {
       if (i == 0)
        requeteMetiers += " and (ac.Metier=" + idMetier + " ";
       else
        requeteMetiers += " or ac.Metier=" + idMetier + " ";
       i++;
      }
     }

**François DORIN** · 05/02/2018, 14h31

Envoyé par Addx_man

Code :

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

1
2
3
4
5
6
7
8
9
10
11
  foreach (String idHangar in idHangars)
                    {
                        if (idHangar != "")
                        {
                            if (i == 0)
                                requeteHangars += " and (z.Id=" + idHangar + " ";
                            else
                                requeteHangars += " or z.Id=" + idHangar + " ";
                            i++;
                        }
                    }

Parfois, comprendre ce qu'il y a derrière, et l'écrire de manière plus propre permet d'éviter de nombreux soucis

Ici, tu parcours une collection (idHangars) tout en gérant à côté une variable i qui sert d'indexeur. Bref, deux choses qu'il faut synchroniser pour que cela fonctionne comme attendu (i est bien incrémenté, mais est-il mis à 0 avant la boucle ?).

De plus, la requête est bien compliquée, sans compter le fait qu'elle omet la parenthèse fermante en SQL.

Ici, on cherche simplement à générer une requête SQL, et plus particulièrement une partie de la clause WHERE, qui prendra la forme suivante :

Code SQL :

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

AND (z.Id = 1 OR z.Id = 2 OR z.Id = 3)

Et donc, pour arriver à cela, on peut le faire en une ligne, et qui sera beaucoup plus lisible et maintenable :

Code C# :

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

requeteHangars += " AND z.Id IN (" + String.Join(", ", idHangars) + ")"

**Addx_man** · 05/02/2018, 16h14

Envoyé par François DORIN

Parfois, comprendre ce qu'il y a derrière, et l'écrire de manière plus propre permet d'éviter de nombreux soucis

Ici, tu parcours une collection (idHangars) tout en gérant à côté une variable i qui sert d'indexeur. Bref, deux choses qu'il faut synchroniser pour que cela fonctionne comme attendu (i est bien incrémenté, mais est-il mis à 0 avant la boucle ?).

De plus, la requête est bien compliquée, sans compter le fait qu'elle omet la parenthèse fermante en SQL.

Ici, on cherche simplement à générer une requête SQL, et plus particulièrement une partie de la clause WHERE, qui prendra la forme suivante :

Code SQL :

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

AND (z.Id = 1 OR z.Id = 2 OR z.Id = 3)

Et donc, pour arriver à cela, on peut le faire en une ligne, et qui sera beaucoup plus lisible et maintenable :

Code C# :

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

requeteHangars += " AND z.Id IN (" + String.Join(", ", idHangars) + ")"

Envoyé par Addx_man

Bonjour, j'archive le sujet, le problème venait du fait que le i n'est pas remis a 0 lorsqu'il passe a la boucle des métiers. Je l'ai ajouté en gras en italique dans le code, donnez moi votre avis, et j'archiverais ensuite !

En attendant, je fais des tests

Merci à vous !

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
if (idHangar != "") {
      if (i == 0)
       requeteHangars += " and (z.Id=" + idHangar + " ";
      else
       requeteHangars += " or z.Id=" + idHangar + " ";
      i++;
     }
    }
    if (i != 0) {
     requeteHangars += ")";
    }
    //}
    // si on recoit une réponse d'un formulaire liste metiers
    if (lblMetiers.Text != null) {
     String[] idMetiers = new String[100];
     // on prend la réponse du formulaire la liste des métiers choisis
     idMetiers = lblMetiers.Text.Split(new [] {
      "-"
     }, StringSplitOptions.None);

     if (Request.Form["clos"] != null && Request.Form["clos"].ToString() == "true") {
      requeteMetiers += "";
     }
     // la liste des metiers est délimité par des tirets d'union "-"
     // cette boucle va prendre l'ensemble des id metiers et les remplace par une requete sql
	 i=0;
     foreach(String idMetier in idMetiers) {
      if (idMetier != "") {
       if (i == 0)
        requeteMetiers += " and (ac.Metier=" + idMetier + " ";
       else
        requeteMetiers += " or ac.Metier=" + idMetier + " ";
       i++;
      }
     }

Bonjour !

Tout d'abord merci pour votre message !

C'est vrais que la requête est compliquer ! J'ai mis un peu de temps a tout comprendre, mais maintenant c'est plus clair !

Du coup j'ai remis mon dernier message en citation pour que ce soit plus clair, en gros, le même traitement est effectuer une deuxième fois pour Idmetier sauf que le i n'étais pas remis a 0 ! J'ai corrigé cela, sauf que j'ai un problème d'HTML il semblerait donc je me penche dessus

Voilà, merci à vous

!

Requête Erreur SqlException [Débutant]

ASP.NET

Discussions similaires

Partager

Partager