[Débutant] Requête Erreur SqlException

3 pièce(s) jointe(s)

Requête Erreur SqlException

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:

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. :)

Pièce jointe 346828

Pièce jointe 346829

Pièce jointe 346833

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

1 pièce(s) jointe(s)

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:

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++;
                        }
                    }

Pièce jointe 346848

Citation:

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é

Citation:

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

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:

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 !

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 :ccool:

Citation:

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 :ccool:

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... !

Bonjour,

Citation:

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.

@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 ! :)

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:

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++;
      }
     }

Citation:
Envoyé par Addx_man
Code:

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:

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:

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

Citation:

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:

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:

requeteHangars += " AND z.Id IN (" + String.Join(", ", idHangars) + ")"
Citation:
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:

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 :lol:

Voilà, merci à vous :) !