[Débutant] SqlException

Résolu

addx_man

Messages postés: 19
Date d'inscription: lundi 5 décembre 2011
Statut: Membre
Dernière intervention: 7 mars 2018

- Modifié le 30 janv. 2018 à 10:40
nagaD.scar

Messages postés: 4270
Date d'inscription: samedi 8 septembre 2007
Statut: Membre
Dernière intervention: 4 mars 2021

- 13 mars 2018 à 11:45

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:

  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.

A voir également:

Index de colonne non valide
Java.sql.sqlexception: index de colonne non valide - Meilleures réponses
Echec de conversion dans la représentation interne - Meilleures réponses
Java.sql.sqlexception: index de colonne non valide - Forum - Java
Projet php débutant ✓ - Forum - PHP
Java.sql.sqlexception: operation not allowed after resultset closed - Forum - Java
Mastermind python débutant - Forum - Python
[ms sql server] - débuter avec les triggers - Conseils pratiques - SQL

2 réponses

Réponse 1 / 2

nagaD.scar

Messages postés: 4270
Date d'inscription: samedi 8 septembre 2007
Statut: Membre
Dernière intervention: 4 mars 2021

16
30 janv. 2018 à 13:52

Salut,

En l'état tu donnes trop peu d'information: on est pas en mesure de voir toute la construction de la requete.

La parenthèse peu ou non être une erreur, ca va dépendre de tes conditions .. mais vu qu'on ne voit pas toute la construction de la clause, il n y a pas de réponses à donner.

D'après le message d'erreur, il semblerai justement que le problème provienne d'une parenthèse fermante qui en trop.

Bref au final: peux-tu nous montrer toute la construction de ta requête ? (à mininma la construction du where pour au moins vérifier la syntaxe)

naga

addx_man

Messages postés: 19
Date d'inscription: lundi 5 décembre 2011
Statut: Membre
Dernière intervention: 7 mars 2018

30 janv. 2018 à 14:23

Sur la requête que j'ai posté, il y a une parenthèse fermente


                    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 += ")";
                    }

Je pense que le problème viens de la requête SQL, mais il y a trop de données a traité, j'ai vraiment du mal.

Je peu poster le SQL mais je ne sais pas si ça peu aider

Merci pour la réponse en tous cas !

nagaD.scar

Messages postés: 4270
Date d'inscription: samedi 8 septembre 2007
Statut: Membre
Dernière intervention: 4 mars 2021

16
30 janv. 2018 à 14:30

encore une fois en l'état c'est assez limité, mais ce que l 'on peut surtout voir c'est que dans le cas ou il n'y a pas d'ID dans ta liste, ta requete aura une parenthèse fermante sans avoir été ouverte.

Je te conseil d'essayer de faire ceci:

bool bFilter = false ; 
foreach (String idHangar in idHangars)
{
	if (idHangar != "")
	{
		bFilter = true;
		if (i == 0)
			requeteHangars += " and (z.Id=" + idHangar + " ";
		else
			requeteHangars += " or z.Id=" + idHangar + " ";
		i++;
	}
}
if (bFilter)
{
	requeteHangars += ")";
}

redis moi.

naga

addx_man

Messages postés: 19
Date d'inscription: lundi 5 décembre 2011
Statut: Membre
Dernière intervention: 7 mars 2018

30 janv. 2018 à 15:05

Merci pour tes réponses !

Toujours le même résultat, je perd la tête moi avec tout ça moi !

        

protected void Page_Load (object sender, EventArgs e)
        {

            using (SqlCommand scmd = new SqlCommand())
            {
                if (Request.Form["idHangars"] != null && lblHangars.Text == "")
                {
                    lblHangars.Text = Request.Form["idHangars"];
                }
                if (Request.Form["idMetiers"] != null && lblMetiers.Text == "")
                {
                    lblMetiers.Text = Request.Form["idMetiers"];
                }
                // se connecter à la base de donnée
                scmd.Connection = connectionString();
                scmd.CommandType = CommandType.Text;
                scmd.CommandText = "";
                String requeteHangars = "", requeteMetiers = "", risqueChantier = "", risqueChantierWhere = "", toutaction = "";                    // Traitement requete : recherche par hangars selectionnee
                String visitea = "", visiteb = "", visitec = "", visitez = "", visitedepanage = "", visitevip = "";
                // si on recoit une réponse d'un formulaire liste hangars
                if (lblHangars.Text != null)
                {
                    String[] idHangars = new String[100];

                    // on prend la réponse du formulaire la liste des hangars choisis
                    idHangars = lblHangars.Text.Split(new[] { "-" }, StringSplitOptions.None);
                    int i = 0;

                    // 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++;
                        }
                    }
                    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
                        foreach (String idMetier in idMetiers)
                        {
                            if (idMetier != "")
                            {
                                if (i == 0)
                                    requeteMetiers += " and (ac.Metier=" + idMetier + " ";
                                else
                                    requeteMetiers += " or ac.Metier=" + idMetier + " ";
                                i++;
                            }
                        }
                        if (i != 0)
                        {
                            requeteMetiers += "or ac.Metier is null)";
                        }
                    }


                    if (chkbA.Checked)
                    {
                        visitea = " and Visite_a=1";
                    }
                    else
                    {
                        visitea = "";
                    }

                    if (chkbB.Checked)
                    {
                        visiteb = " and Visite_b=1";
                    }
                    else
                    {
                        visiteb = "";
                    }

                    if (chkbC.Checked)
                    {
                        visitec = " and Visite_c=1";
                    }
                    else
                    {
                        visitec = "";
                    }

                    if (chkbZ.Checked)
                    {
                        visitez = " and Visite_z=1";
                    }
                    else
                    {
                        visitez = "";
                    }

                    if (chkbDepanage.Checked)
                    {
                        visitedepanage = " and Depanage=1";
                    }
                    else
                    {
                        visitedepanage = "";
                    }

                    if (chkbVIP.Checked)
                    {
                        visitevip = " and Visite_vip=1";
                    }
                    else
                    {
                        visitevip = "";
                    }

                    if (ddlRapport.SelectedValue == "2")
                    {
                        toutaction = " and Clos is null";
                    }
                    else if (ddlRapport.SelectedValue == "1")
                    {
                        toutaction = " and Clos is not null";
                    }

                    if (chkbRisque.Checked)
                    {
                        risqueChantierWhere = " and ch.numChantierId is not null ";
                        risqueChantier = " and cc.Insatisfaction_client is not null and cc.Impact_da is not null and cc.Risque_cout is not null and cc.Risque_delai is not null ";
                    }

                    // Requête génération de tous les chantiers
                    scmd.CommandText = "select c.id_chantier, immatriculation, ta.type_avion, ns.numero_serie, c.numeroChantierId as numChantier, Date_arrivee, Nature_chantier, Point_fixe, Vol_control, c.Date_livraison as datelivraison, Niveau_chantier, h.Nom, case z.emplacement WHEN 1 THEN 'EXT' else '' end as emplacement, r2.Nom as responsableTechniqueNom, SUBSTRING(r2.Prenom, 1, 1) as responsableTechniquePrenom, r1.Nom as responsableCommercialNom, SUBSTRING(r1.Prenom, 1, 1) as responsableCommercialPrenom, fin_inspection, cc.Travaux_peinture from Chantiers c left join commentaires_chantiers cc on cc.[numChantierId]=c.id_chantier inner join Avions av on av.id_avion = c.idAvion inner join typeAvions ta on ta.Id = av.type_avionId inner join numeroSeries ns on ns.Id = av.numero_serieId inner join Zones z on z.Id = c.idZoneHangar inner join Hangars h on h.Id = z.idHangar left join commentaires_chantiers ch on ch.numChantierId=c.id_chantier left join Responsables r1 on r1.Id=c.idResponsableCommercial left join Responsables r2 on r2.Id = c.idResponsableTechnique where Date_livraison is null " + risqueChantier + " " + requeteHangars + " " + visitea + " " + visiteb + " " + visitec + " " + visitez + " " + visitedepanage + " " + visitevip + " " + risqueChantier + " order by z.preference asc, c.numeroChantierId asc, immatriculation asc";


                    Label8.Text = scmd.CommandText;
                    sqlConnection1.Open();
                    SqlDataReader articleReader = scmd.ExecuteReader();

nagaD.scar

Messages postés: 4270
Date d'inscription: samedi 8 septembre 2007
Statut: Membre
Dernière intervention: 4 mars 2021

16
30 janv. 2018 à 15:14

mmh ce que j'avais proposé n'était pas utile en fait, bref x)

Je check ca tout à l'heure !

addx_man

Messages postés: 19
Date d'inscription: lundi 5 décembre 2011
Statut: Membre
Dernière intervention: 7 mars 2018

30 janv. 2018 à 15:20

Haha pas de soucis merci ^^

nagaD.scar

Messages postés: 4270
Date d'inscription: samedi 8 septembre 2007
Statut: Membre
Dernière intervention: 4 mars 2021

16
30 janv. 2018 à 16:53

c'est bizarre ... dans ta requête,

risqueChantier

apparait deux fois et il y plusieurs filtres qui ne sont pas utilisés :

risqueChantierWhere
toutaction
requeteMetiers
requeteHangars

mais n'ayant pas la fermeture du using, je suppose qu'il y a d'autres requetes qui suivent 0> en l'état, outre le doublon risqueCahntier (qui ne va pas poser de problème, juste rajouter des tests inutiles, petit impact de perf mais bon), il n'y a pas d'erreur sur cette requete (les seules parentèses sont dans le select et sont corrects).

Du coup il faudrai que tu me fournisse les requetes qui suivent, et tout particulièrement celles qui utilisent requeteMetiers et requeteHangars.

Aussi dans le cas ou il y aurai un autre paramètre qui se construit ensuite, tu peux l'envoyer.

dans tous les cas, ce qui serai pas mal serai d'ajouter une ecriture des requetes dans un log avant leur execution histoire de voir réellement ce qui est envoyé :


 File.AppendAllText("[chemin+nomfichier]", "content here");

en mettant a chaque fois un nom différents. ensuite tu pourras tester les requetes et les paster aussi ici pour qu'on puisse voir.

n'hésites pas si je suis pas clair ^^

naga

addx_man

Messages postés: 19
Date d'inscription: lundi 5 décembre 2011
Statut: Membre
Dernière intervention: 7 mars 2018

Modifié le 30 janv. 2018 à 23:07

Tu est très clair et je remercie pour ton aide !

Je te met la suite du code ci bas:

String numChantier2 = "", nomHangar = "", emplacement = "", nomMetier = "";
                    int cptHangar = 0, cptAction = 0, cptImmat = 0;

                    // Si la requête retoure des informations
                    if (articleReader.HasRows)
                    {

                        // Lire les informations
                        while (articleReader.Read())
                        {
                            SqlDataReader reader2;
                            // Récupérer l'ensemble des informations sur le chantier
                            String id_chantier = (articleReader["id_chantier"] == DBNull.Value ? "" : articleReader.GetInt32(0).ToString());
                            String numChantier = articleReader["numChantier"].ToString();
                            String dateArrivee = (articleReader["Date_arrivee"] == DBNull.Value ? "" : articleReader.GetDateTime(5).ToString("dd/MM/yy"));
                            String pointFixe = (articleReader["Point_fixe"] == DBNull.Value ? "" : articleReader.GetDateTime(7).ToString("dd/MM/yy"));
                            String volControl = (articleReader["Vol_Control"] == DBNull.Value ? "" : articleReader.GetDateTime(8).ToString("dd/MM/yy"));
                            String finInspection = (articleReader["fin_inspection"] == DBNull.Value ? "" : articleReader.GetDateTime(17).ToString("dd/MM/yy"));

                            // Si changement de hangar, nouveau bloc d'affichage
                            if (nomHangar != articleReader["Nom"].ToString() || emplacement != articleReader["emplacement"].ToString())
                            {
                                if (cptHangar != 0)
                                {
                                    htmlTable.Append("<div class='page-break'></div>");
                                }
                                cptHangar++;
                                htmlTable.Append("<table>");
                                nomHangar = articleReader["Nom"].ToString();
                                emplacement = articleReader["emplacement"].ToString();
                                htmlTable.Append("<tr><td height='14px' colspan='10'></td></tr>");
                                htmlTable.Append("<tr><th class='tresgrand' style='color:black'>" + articleReader["Nom"] + " " + articleReader["emplacement"] + "</th></tr>");
                                htmlTable.Append("</table>");

                            }

                            // Petit espacement entre l'immat et le hangar
                            htmlTable.Append("<table width='100%'>");
                            htmlTable.Append("<tr><td height='6px' colspan='10'></td></tr>");
                            htmlTable.Append("</table>");


                            // Remplir les informations concernant le chantier
                            htmlTable.Append("<table width='100%'>");

                            // Récupération des valeurs dans des variables
                            String nomTechnique = null, prenomTechnique = null, nomCommercial = null, prenomCommercial = null, respCom = "", respTech = "";
                            if (articleReader["responsableTechniquePrenom"].ToString() != "")
                                prenomTechnique = articleReader["responsableTechniquePrenom"].ToString();
                            if (articleReader["responsableTechniqueNom"].ToString() != "")
                                nomTechnique = articleReader["responsableTechniqueNom"].ToString();
                            if (articleReader["responsableCommercialPrenom"].ToString() != "")
                                prenomCommercial = articleReader["responsableCommercialPrenom"].ToString();
                            if (articleReader["responsableCommercialNom"].ToString() != "")
                                nomCommercial = articleReader["responsableCommercialNom"].ToString();

                            if (nomCommercial != null && prenomCommercial != null)
                                respCom = prenomCommercial + "." + nomCommercial;
                            if (nomTechnique != null && prenomTechnique != null)
                                respTech = prenomTechnique + "." + nomTechnique;


                            nomMetier = null;
                            cptImmat++;
                            htmlTable.Append("<tr><td><table width='100px' border='0' class='grand trborder' colspan='2' cellpadding='4' cellspacing='0'><tr><td bgcolor='#ffffff'>" + articleReader["immatriculation"] + "</td></tr></table></td></tr>");
                            htmlTable.Append("<tr class='noBorder'><td height='5px'  class='noBorder' colspan='10'></td></tr>");
                            htmlTable.Append("<tr class='trborder'><th class='nomhangar couleurhangar moyen couleurtexte'>Type</th><th class='nomhangar couleurhangar moyen couleurtexte'>SN</th><th class='nomhangar couleurhangar moyen couleurtexte'>Dossier</th><th class='nomhangar couleurhangar moyen couleurtexte'>Nature Chantier</th><th class='nomhangar couleurhangar moyen couleurtexte'>Chef A/C</th><th class='nomhangar couleurhangar moyen couleurtexte'>Commercial</th><th class='nomhangar couleurhangar moyen couleurtexte'></th></tr>");
                            htmlTable.Append("<tr class='trborder'>");
                            htmlTable.Append("<td class='trborder petit' width='5%'>" + articleReader["type_avion"] + "</td>");
                            htmlTable.Append("<td class='trborder petit' width='5%'>" + articleReader["numero_serie"] + "</td>");
                            htmlTable.Append("<td class='trborder petit' width='10%'>" + articleReader["numChantier"] + "</td>");
                            htmlTable.Append("<td class='trborder petit' width='55%'>" + articleReader["Nature_chantier"] + "</td>");
                            htmlTable.Append("<td class='trborder petit' width='10%' >" + respTech + "</td>");
                            htmlTable.Append("<td class='trborder petit' width='10%'>" + respCom + "</td>");

                            // Changement de la couleur de la case en fonction du niveau de chantier
                            if (articleReader["Niveau_chantier"].ToString() == "1")
                                htmlTable.Append("<td class='trborder couleurVerte petit' bgcolor='green' width='5%'>" + articleReader["Niveau_chantier"] + "</td>");
                            else if (articleReader["Niveau_chantier"].ToString() == "2")
                                htmlTable.Append("<td class='trborder couleurJaune petit' bgcolor='yellow' width='5%'>" + articleReader["Niveau_chantier"] + "</td>");
                            else if (articleReader["Niveau_chantier"].ToString() == "3")
                                htmlTable.Append("<td class='trborder couleurRouge petit' bgcolor='red' width='5%'>" + articleReader["Niveau_chantier"] + "</td>");
                            else
                                htmlTable.Append("<td></td>");

                            htmlTable.Append("</tr>");
                            htmlTable.Append("</table>");

                            // Affichage des informations sur le tableau
                            htmlTable.Append("<table  width='100%'>");
                            htmlTable.Append("<tr class='noBorder'><td height='5px'  class='noBorder' colspan='5'></td></tr>");
                            htmlTable.Append("<tr class='trborder'><th class='nomhangar couleurhangar moyen couleurtexte' width='10%%'>Arrivée</th><th class='nomhangar couleurhangar moyen couleurtexte' width='20%%'>Fin Inspection</th><th class='nomhangar couleurhangar moyen couleurtexte' width='10%%'>PF</th><th class='nomhangar couleurhangar moyen couleurtexte' width='10%%'>VdC</th><th class='nomhangar couleurhangar moyen couleurtexte' width='50%%'>Travaux Peinture</th></tr>");
                            htmlTable.Append("<tr class='trborder'>");
                            htmlTable.Append("<td class='trborder petit'>" + dateArrivee + "</td>");
                            htmlTable.Append("<td class='trborder petit'>" + finInspection + "</td>");
                            htmlTable.Append("<td class='trborder petit'>" + pointFixe + "</td>");
                            htmlTable.Append("<td class='trborder petit'>" + volControl + "</td>");
                            htmlTable.Append("<td class='trborder petit'>" + articleReader["travaux_peinture"] + "</td>");
                            htmlTable.Append("</tr>");
                            htmlTable.Append("</table>");

                            // Espacement entre les informations du chantier et les actions
                            htmlTable.Append("<table width='100%'>");
                            htmlTable.Append("<tr><td height='6px' colspan='10'> </td></tr>");
                            htmlTable.Append("</table>");

                            // Liste des actions du chantier    
                            // Requete lister actions selon chantier
                            
                            cmd2.Connection = connectionString2();
                            cmd2.CommandText = "SELECT id, c.numeroChantierId, date_reunion, nom_action, clos, priorite, andon, nom_metier from Actions ac full outer join Chantiers c on ac.numeroChantierId=c.id_chantier inner join Avions av on av.id_avion=c.idAvion inner join Metiers m on m.id_metier=ac.Metier where ac.numeroChantierId=" + id_chantier + " " + requeteMetiers + " " + toutaction + " and (Andon <= 2 or Andon is null)";
                            sqlConnection2.Open();
                            reader2 = cmd2.ExecuteReader();
                            htmlTable.Append("<table width='100%'>");

                            String dateReunion = null, clos = null, id_action = null;
                            //scmd.CommandText = "select c.id_chantier, immatriculation, ta.type_avion, ns.numero_serie, c.numeroChantierId as numChantier, Date_arrivee, Nature_chantier, Point_fixe, Vol_control, c.Date_livraison as datelivraison, Niveau_chantier, ac.Date_reunion as datereunion, nom_action, nom_metier, h.Nom, case z.emplacement WHEN 1 THEN 'EXT' else '' end as emplacement, Andon, ac.Id as id_action, ac.Clos, r2.Nom as responsableTechniqueNom, SUBSTRING(r2.Prenom, 1, 1) as responsableTechniquePrenom, r1.Nom as responsableCommercialNom, SUBSTRING(r1.Prenom, 1, 1) as responsableCommercialPrenom, fin_inspection, cc.Travaux_peinture, LEAD(immatriculation) OVER (ORDER BY z.preference asc, emplacement asc, c.numeroChantierId asc, immatriculation asc, CASE WHEN Clos IS NULL THEN 0 ELSE 1 END DESC, nom_metier asc, Clos desc, Date_reunion asc) immat2 from Chantiers c left join commentaires_chantiers cc on cc.[numChantierId]=c.id_chantier inner join Avions av on av.id_avion = c.idAvion inner join typeAvions ta on ta.Id = av.type_avionId inner join numeroSeries ns on ns.Id = av.numero_serieId left join Actions ac on  c.id_chantier = ac.numero_chantierId left join Metiers m on m.id_metier = ac.Metier inner join Zones z on z.Id = c.idZoneHangar inner join Hangars h on h.Id = z.idHangar left join commentaires_chantiers ch on ch.numChantierId=c.id_chantier left join Responsables r1 on r1.Id=c.idResponsableCommercial left join Responsables r2 on r2.Id = c.idResponsableTechnique where 1=1 " + risqueChantierWhere + " " + requeteHangars + " " + requeteMetiers + " " + visitec + " " + risqueChantier + " and Date_livraison is null " + toutaction + " and (Andon <=2 or Andon is null) order by z.preference asc, emplacement asc, c.numeroChantierId asc, immatriculation asc, CASE WHEN Clos IS NULL THEN 0 ELSE 1 END DESC, nom_metier asc, Clos desc, Date_reunion asc";
                            //scmd.CommandText = "select c.id_chantier, immatriculation, ta.type_avion, ns.numero_serie, c.numeroChantierId as numChantier, Date_arrivee, Nature_chantier, Point_fixe, Vol_control, c.Date_livraison as datelivraison, Niveau_chantier, ac.Date_reunion as datereunion, nom_action, nom_metier, h.Nom, case z.emplacement WHEN 1 THEN 'EXT' else '' end as emplacement, Andon, ac.Id as id_action, ac.Clos, r2.Nom as responsableTechniqueNom, SUBSTRING(r2.Prenom, 1, 1) as responsableTechniquePrenom, r1.Nom as responsableCommercialNom, SUBSTRING(r1.Prenom, 1, 1) as responsableCommercialPrenom, fin_inspection, cc.Travaux_peinture, LEAD(immatriculation) OVER (order by z.preference asc, c.numeroChantierId asc, immatriculation asc, CASE WHEN Clos IS NULL THEN 0 ELSE 1 END DESC, nom_metier asc, Clos desc, Date_reunion asc) immat2 from Chantiers c left join commentaires_chantiers cc on cc.[numChantierId]=c.id_chantier inner join Avions av on av.id_avion = c.idAvion inner join typeAvions ta on ta.Id = av.type_avionId inner join numeroSeries ns on ns.Id = av.numero_serieId full outer join Actions ac on  c.id_chantier = ac.numero_chantierId left join Metiers m on m.id_metier = ac.Metier inner join Zones z on z.Id = c.idZoneHangar inner join Hangars h on h.Id = z.idHangar left join commentaires_chantiers ch on ch.numChantierId=c.id_chantier left join Responsables r1 on r1.Id=c.idResponsableCommercial left join Responsables r2 on r2.Id = c.idResponsableTechnique where 1=1 " + risqueChantierWhere + " " + requeteHangars + " " + requeteMetiers + " " + visitea + " " + visiteb + " " + visitec + " " + visitez + " " + visitedepanage + " " + visitevip + " " + risqueChantier + " and Date_livraison is null " + toutaction + " and (Andon <=2 or Andon is null) order by z.preference asc, c.numeroChantierId asc, immatriculation asc, CASE WHEN Clos IS NULL THEN 0 ELSE 1 END DESC, nom_metier asc, Clos desc, Date_reunion asc";

                            if (reader2.HasRows)
                            {
                                htmlTable.Append("<tr>");
                                htmlTable.Append("<tr class='trborder'><th class='nomhangar couleurhangar moyen couleurtexte'>N°</th><th class='nomhangar couleurhangar moyen couleurtexte'>Métier</th><th class='nomhangar couleurhangar moyen couleurtexte'>Action</th><th class='nomhangar couleurhangar moyen couleurtexte'>OUV.</th><th class='nomhangar couleurhangar moyen couleurtexte'>CLÔT.</th></tr>");

                                while (reader2.Read())
                                {
                                    dateReunion = (reader2["date_reunion"] == DBNull.Value ? "" : reader2.GetDateTime(2).ToString("dd/MM/yy"));
                                    clos = (reader2["Clos"] == DBNull.Value ? "" : reader2.GetDateTime(4).ToString("dd/MM/yy"));
                                    id_action = (reader2["id"] == DBNull.Value ? "" : reader2.GetInt32(0).ToString());


                                    htmlTable.Append("<tr class='trborder couleur'>");
                                    //htmlTable.Append("<td style='text-align:center' class='trborder spec couleur trespetit' width='5%'>" + id_action.Substring(id_action.Length, 3) + "</td>");
                                    htmlTable.Append("<td style='text-align:center' class='trborder spec couleur trespetit' width='5%'>" + id_action + "</td>");
                                    if (reader2.GetInt32(6).ToString() == "1")
                                        htmlTable.Append("<td class='trborder spec couleurRouge trespetit' width='15%'> " + reader2["NOM_METIER"].ToString() + "</td>");
                                    else if (reader2.GetInt32(6).ToString() == "2")
                                        htmlTable.Append("<td class='trborder spec couleurVerte trespetit' width='15%'>" + reader2["NOM_METIER"].ToString() + "</td>");
                                    else
                                        htmlTable.Append("<td class='trborder spec couleur trespetit' width='15%'>" + reader2["NOM_METIER"].ToString() + "</td>");
                                    htmlTable.Append("<td class='trborder spec couleur trespetit' width='70%'>" + reader2["nom_action"] + "</td>");
                                    htmlTable.Append("<td class='trborder datereunion couleur trespetit' width='5%'>" + dateReunion + "</td>");
                                    htmlTable.Append("<td class='trborder datereunion couleur trespetit' width='5%'>" + clos + "</td>");
                                    htmlTable.Append("</tr>");
                                }
                            }
                            htmlTable.Append("</table>");
                            sqlConnection2.Close();

                            // réinitialisation des paramètres de la requête
                            cmd2 = new SqlCommand();
                            cmd2.Connection = connectionString2();
                            sqlConnection2.Open();
                            cmd2.CommandText = "SELECT numeroChantierId, Date_depart, motif from Departs where numeroChantierId=" + id_chantier;
                            reader2 = cmd2.ExecuteReader();
                            //*****************************partie ajoutée****************************************

                            cmd3 = new SqlCommand();
                            SqlDataReader reader3;
                            cmd3.Connection = connectionString3();
                            //requéte pour afficher les faits marquants dans chaque chantier en fonction de la date du jour
                            cmd3.CommandText = "SELECT faitMarquants.titreFaitMarquant,faitMarquants.Date,faitMarquants.idAvion,Chantiers.numeroChantierId from faitMarquants  " +
                                "INNER JOIN Chantiers ON faitMarquants.idAvion=Chantiers.idAvion  and  Chantiers.numeroChantierId =" + articleReader["numChantier"] + "and  convert(varchar(10),faitMarquants.Date,131)=convert(varchar(10),GETDATE(),131)";

                            sqlConnection3.Open();
                            reader3 = cmd3.ExecuteReader();



                            htmlTable.Append("<div id=faits marquants>");
                            htmlTable.Append("<table width = '100%' style='text-align:left;text-decoration:none; border:1'>");
                            //htmlTable.Append("<p><font color=red>Faits marquants</font></p>");
                            htmlTable.Append("<th class='trborder datereunion couleur trespetit' style='text-align:left;' width='60%';text-decoration:none;><p style='text-decoration:none;'><b><u><font color=red >Faits marquants:</font></b></u></p>");
                            if (reader3.HasRows)
                            {
                                // Lire les informations  
                                while (reader3.Read())
                                {
                                    htmlTable.Append("<br>" + reader3["titreFaitMarquant"] + "<br>");
                                    

                                }
                            }
                            htmlTable.Append("</th>");
                            htmlTable.Append("</table>");
                            htmlTable.Append("</div>");

                            foreach (String idHangar in idHangars)

                            {
                                //table pour afficher les commentaire -> ajout d'un textarea pour saisir les commentaires  
                                htmlTable.Append("<div id=commentaire'><table width='100%' ><tr height='136px'><th class='nomhangar moyen sousligne' style='text-align:left;text-decoration:underline;' width='60%'><p>Commentaires</p> <form ID='formCommentaire' runat='server'><textarea style='border:none' ID=commentaire name='commentaire' rows='1' cols='150'  runat='server' placeholder='-'  onkeyup='AutoGrowTextArea(this)' ></textarea></form> </th><th width='40%'></div> ");
                                htmlTable.Append("<table width='100%' border='1'>");
                                htmlTable.Append("<tr class='trborder'><th width='20%' class='nomhangar couleurhangar moyen couleurtexte'>Départ</th><th width='80%' class='nomhangar couleurhangar moyen couleurtexte'>Motif</th>");
                            }

                            if (reader2.HasRows)
                            {
                                while (reader2.Read())
                                {
                                    htmlTable.Append("<tr><td style='text-align:center' class='trborder spec trespetit'>" + reader2.GetDateTime(1).ToString("dd/MM/yyyy") + "</td>");
                                    htmlTable.Append("<td class='trborder spec trespetit'>" + reader2["motif"].ToString() + "</td></tr>");
                                }
                            }

                            htmlTable.Append("</table></th></tr></table>");
                            sqlConnection2.Close();

                        }
                    }
                    SqlConnection.ClearAllPools();
                    PlaceHolder1.Controls.Add(new Literal { Text = htmlTable.ToString() });
                    articleReader.Close();
                    SqlConnection.ClearAllPools();
                    articleReader.Dispose();
                }
            }

nagaD.scar

Messages postés: 4270
Date d'inscription: samedi 8 septembre 2007
Statut: Membre
Dernière intervention: 4 mars 2021

16
31 janv. 2018 à 07:55

Salut !

Je devais être fatigué ...je me suis pas rendu compte que la variable i n'était pas remise à 0 ^^

du coup:


int i = 0;

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

Juste avant le second foreach, il faut remettre "i" à 0 (en gras souligné) => dans le cas ou il y avait des idHangars, i était incrémenté ... et forcément ensuite lors du parcours des metiers, soit il n'y en a pas et tu te retrouver avec un

or ac.Metier is null)

donc parenthèse fermante sans qu'elle ai été ouverte, soit il y en avait mais comme i>0m tu n'as toujours pas la parenthèse ouvrante.

Je sais pas pourquoi je l'ai pas vu avant x) bref du coup l'erreur devait être remonté lors de l'execution de la seconde commande pour le remplissage du tableau.

redis moi ;)

naga

nagaD.scar

Messages postés: 4270
Date d'inscription: samedi 8 septembre 2007
Statut: Membre
Dernière intervention: 4 mars 2021

16
31 janv. 2018 à 10:08

bien !
Alors pour les explications vu que je ne connais pas ton niveau je vais expliquer globalement, ensuite si tu as des questions spécifiques demandes.

Donc dans un premier temps, on a trois filtres : les id d'hangars, ceux de metiers et "clos"
je ne sais pas ce "clos" représente mais il sert à rien vu qu'on en fait rien :

   if (Request.Form["clos"] != null && Request.Form["clos"].ToString() == "true") {
    requeteMetiers += "";
   }

à partir des listes d'id (qui à l'origine sont des chaines avec '-' comme separateur, d'ou le split), on construit des sous requetes:
- requeteHangars pour la première séléction ( chaque itération du résultat représentant une ligne du tableau sur ta page web)
- requeteMetiers qui va charger ce qui semble être un descriptifs des metiers, utilisé pour filtrer les metiers de chaque hangar.

De ce que je vois je dirai que sur la page au final, chaque hangar génère un tableau, et que chaque metier du hangar représente une ligne de tableau.
On peut aussi voir qu'on souhaite ajouter les faits marquants du jours, mais je penses que cette partie n'est pas terminée car on ne fait qu'ajouter son titre :

     while (reader3.Read()) {
      htmlTable.Append("<br>" + reader3["titreFaitMarquant"] + "<br>");
     }

'question hors code => tu travails chez airbus? ^^

s'il y a des parties précises qui t'échappent, demande ;)

naga

Réponse 2 / 2

Whismeril

Messages postés: 17345
Date d'inscription: mardi 11 mars 2003
Statut: Modérateur
Dernière intervention: 26 mai 2022

596
31 janv. 2018 à 07:43

Bonjour à tous les 2, je me permets un petit rappel sur la coloration syntaxique.
En l’état, vous l’utilisez mal tous les 2.
Il faut préciser de quel langage il s’agit, en cliquant sur la petite flèche vers le bas.
Exemple, vous:

String numChantier2 = "", nomHangar = "", emplacement = "", nomMetier = "";
                    int cptHangar = 0, cptAction = 0, cptImmat = 0;

                    // Si la requête retoure des informations
                    if (articleReader.HasRows)
                    {

                        // Lire les informations
                        while (articleReader.Read())
                        {
                            SqlDataReader reader2;
                            // Récupérer l'ensemble des informations sur le chantier
                            String id_chantier = (articleReader["id_chantier"] == DBNull.Value ? "" : articleReader.GetInt32(0).ToString());
                            String numChantier = articleReader["numChantier"].ToString();
//...

En précisant qu’il s’agit de c#

String numChantier2 = "", nomHangar = "", emplacement = "", nomMetier = "";
                    int cptHangar = 0, cptAction = 0, cptImmat = 0;

                    // Si la requête retoure des informations
                    if (articleReader.HasRows)
                    {

                        // Lire les informations
                        while (articleReader.Read())
                        {
                            SqlDataReader reader2;
                            // Récupérer l'ensemble des informations sur le chantier
                            String id_chantier = (articleReader["id_chantier"] == DBNull.Value ? "" : articleReader.GetInt32(0).ToString());
                            String numChantier = articleReader["numChantier"].ToString();
//....

C’est mieux non?

Plus de détails ici
http://codes-sources.commentcamarche.net/faq/10686-le-nouveau-codes-sources-comment-ca-marche#balises-code

addx_man

Messages postés: 19
Date d'inscription: lundi 5 décembre 2011
Statut: Membre
Dernière intervention: 7 mars 2018

31 janv. 2018 à 08:21

Ha oui, beaucoup mieux merci :)

[Débutant] SqlException

2 réponses

Votre réponse

sur les mêmes thèmes