problème if, else ..

Version imprimable

Voici mon code me permettant de visualiser les appels par rapport à une date rentrée dans une formulaire. J'aimerais afficher un script alert du genre : aucun résultat vérifiez le format souhaité etc quand la requête n'a pas d'enregistrement...

Je ne vois pas où le placer, j'ai essayé mais ce script alert vient dès que je vais sur la page !

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
 
 
// si clic sur rechercherDateAppel et champ fonctionDateAppel rempli
                    if(isset($_POST['rechercherDateAppel']) AND isset($_POST['fonctionDateAppel'])){
                            $fonctionDateAppel=$_POST['fonctionDateAppel'];?><br /><?php
                            $afficheDateAppel = $bdd->query("SELECT numAppel,heureAppel,raisonAppel,dureeAppel,
                                                            suiteAppel,nomPrenomAppelant,commentaire, 
                                                            DAY(dateAppel) AS jour, MONTH(dateAppel) AS mois, YEAR(dateAppel) AS annee
                                                            FROM appels WHERE dateAppel='".$fonctionDateAppel."'
                                                            ORDER BY numAppel DESC")
                                                            or die(print_r($afficheDateAppel->errorInfo()));?><br /><?php
                        while ($donnees = $afficheDateAppel->fetch())
                        {
                            $afficheFonctionDateAppel = array (
                            '<tr>Appel datant du '.$donnees['jour'] . '/' . $donnees['mois'] . '/' . $donnees['annee'].'',
                            'Numéro de l\'appel : '.$donnees['numAppel'].'',
                            'Heure de l\'appel : '.$donnees['heureAppel'].'',
                            'Raison de l\'appel : '.$donnees['raisonAppel'].'',
                            'Durée de l\'appel : '.$donnees['dureeAppel'].'',
                            'Suite de l\'appel : '.$donnees['suiteAppel'].'',
                            'Collaborateur : '.$donnees['nomPrenomAppelant'].'',
                            'Commentaire : '.$donnees['commentaire'].'</tr>');?><br /><?php
                        foreach($afficheFonctionDateAppel as $element)
                        {
                            echo $element . '<br />';
 
                        }    
                        }
                            $afficheDateAppel->closeCursor(); // Termine le traitement de la requête
                        }

15/06/2012, 16h30
Benjamin Delespierre

Code:

or die(print_r($afficheDateAppel->errorInfo()))

T'aurais rien pu faire de plus horrible ? :cry:

Pour connaitre le nombre de lignes retournées par une requête MySQL, utilise PDOStatement::rowCount.

salut,

décidément... :aie:

as tu lu la doc des objets PDOStatement?

il te suffit de tester la méthode rowCount()

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
if(isset($_POST['rechercherDateAppel']) AND isset($_POST['fonctionDateAppel'])){
                            $fonctionDateAppel=$_POST['fonctionDateAppel'];?><br /><?php
                            $afficheDateAppel = $bdd->query("SELECT numAppel,heureAppel,raisonAppel,dureeAppel,
                                                            suiteAppel,nomPrenomAppelant,commentaire, 
                                                            DAY(dateAppel) AS jour, MONTH(dateAppel) AS mois, YEAR(dateAppel) AS annee
                                                            FROM appels WHERE dateAppel='".$fonctionDateAppel."'
                                                            ORDER BY numAppel DESC")
                                                            or die(print_r($afficheDateAppel->errorInfo()));?><br /><?php
if($afficheDateAppel->rowCount()==0)
{
  echo'aucun résultat<br/>';
}
else
{
while ($donnees = $afficheDateAppel->fetch())
                        {
                            $afficheFonctionDateAppel = array (
                            '<tr>Appel datant du '.$donnees['jour'] . '/' . $donnees['mois'] . '/' . $donnees['annee'].'',
                            'Numéro de l\'appel : '.$donnees['numAppel'].'',
                            'Heure de l\'appel : '.$donnees['heureAppel'].'',
                            'Raison de l\'appel : '.$donnees['raisonAppel'].'',
                            'Durée de l\'appel : '.$donnees['dureeAppel'].'',
                            'Suite de l\'appel : '.$donnees['suiteAppel'].'',
                            'Collaborateur : '.$donnees['nomPrenomAppelant'].'',
                            'Commentaire : '.$donnees['commentaire'].'</tr>');?><br /><?php
                        foreach($afficheFonctionDateAppel as $element)
                        {
                            echo $element . '<br />';
 
                        }    
}

tu devrais éviter de mixer html et php même si c'est pensé pour être fait car la façon dont tu le fait et l'indentation engendrée risque de générer des erreurs et une difficulté de relecture par toi ou d'autre plus tard...

il vaut mieux un

Code:

echo'<br/>';

pour ce genre de truc et pas faire une sortie de bloc php

le "or die" ne doit être réserver que tant que tu développe, il vaut mieux utiliser un système de messages qui ne donne que les infos techniques si besoin et n'arrête pas ton script violemment... pourquoi pas un enregistrement du problème dans une table log en bd par exemple et un simple message d'erreur pour l'utilisateur... à toi de voir

:aie:

15/06/2012, 17h03
Benjamin Delespierre

Citation:

tu devrais éviter de mixer html et php même si c'est pensé pour être fait car la façon dont tu le fait et l'indentation engendrée risque de générer des erreurs et une difficulté de relecture par toi ou d'autre plus tard...

Tu peux mixer les deux si tu utiliser intelligement la syntaxe alternative. Par intelligement j'entends bien sûr qu'il faut faire les traitements AVANT l'affichage et surtout pas pendant !
16/06/2012, 15h29
nolookpass8

Merci ,ça fonctionne !

Par contre, pour les requêtes SQL avec SELECT j'ai lu sur le net qu'il était préférable d'utiliser fetchColumn() plutôt que rowCount() ! les 2 fonctionnent
16/06/2012, 16h35
ericd69

les 2 n'ont rien à voir... fetchColumn sert à ne retourner qu'une seule colonne de la ligne, il ne te retourne pas le nombre de ligne ni ne permet de faire ce que tu veux...
18/06/2012, 10h46
Benjamin Delespierre

En fait la solution la plus portable est d'effectuer une requête COUNT(*) et d'aller cherche le résultat avec fetch column.