Reprise du message précédent :

Non, c'est la internationale

si oui, je la trouve très drôle

Ouais bon, là n'est pas la question

[gatsu]T'as sûrement fait autant de merde dans tes requêtes SQL que dans ton HTML [/gatsu]

J'ai corrigé le coup du tr dans la condition, mais ça je l'ai vu, par contre au niveau de mes requêtes, je vois pas

 
Par hasard, ce serait pas parce que "date" est un mot clé mySQL?
Essaye en lui mettant des ` autour pour voir.

A savoir que j'ai aucune erreur SQL.

Question con mais t'as essayé de balancer les requêtes directement (dans la console mysql ou un client quelconque) avec des données de test chargées pour voir si ça renvoie bien ce que c'est censé renvoyer
 
Non parce que j'dis ça, j'dis rien, mais si tes requêtes SQL sont pas bonnes c'est pas dans blabla@web qu'il faut poser la question

Bah en tout cas mes requêtes renvoient ce qu'elles sont sensées renvoyer.

1/ il y a moyen de ne faire qu'une seule requete
2/ et si tu générait du html correct ?

Ben t'as du PHP moisi, mais ça m'étonne pas, je remarque qu'il y a un array_map(stripslashes) sur le résultat d'une requête db, ce qui n'a aucun sens

 
1/ Je veux bien mais je ne vois pas
2/ Je suis en train de tout refaire proprement sous Code Igniter, mais ça prend du temps...
 

 
Pourquoi le array_map n'a aucun sens ?

C'est le stripslashes qui n'a aucun sens, ça sert à rien de stripslasher sur la sortie d'une DB, sauf si t'as un double escaping en entrée qui est une erreur de ton côté. Et si tu l'as pas, tu risques d'altérer les données en enlevant des caractères qui sont censés être dans le texte.

 
Je fais un addslashes sur toutes mes données en entrée.
Je fais un stripslashes sur les données en sortie.
 
Je vois pas ce qu'il y a de bizarre en fait.

 
Et aussi le fait que normalement, c'est

.
La ca passe, mais c'est parce que la constante stripslashes n'est pas définie.

 
Ohlala

1. Le fait même d'utiliser addslashes est une connerie, addslashes ne doit jamais être utilisé, c'est une faille de sécurité béante dans ton truc
2. L'entrée d'une DB ce sont des requêtes, la sortie ce sont juste des données. L'échappement se fait sur les données pour les faire rentrer dans une requête sans générer d'erreur (et ouvrir des failles d'injection sql ou autres), quand les données sont rendues par la DB elles sont "déséchappées" puisque ce qui est renvoyé ce sont des données et non des requêtes (par contre si tu fais un dump SQL de la db, tout sera échappé comme il faut puisque tu auras récupéré des requettes et non des données)
3. Apprends à utiliser pdo/mysqli, s'il te plait
4. Tu sais pas coder, et tu ne connais pas tes outils. Je dis ça pour ton bien parce que tu as la possibilité d'apprendre, mais j'espère que tu ne bosses pas encore et que ce genre de trucs c'est pas du travail payé.

On s'en contrefout tu sais, tout le bout de code est à jeter alors un vieux warning dû à la moisissure de PHP j'avouerais que c'est pas la partie qui me choque le plus.

Pour le cas présent, je suis d'accord: le code est à jeter.
Cependant, ça ne ressemble pas à une faute de frappe (et grace a la magie de php, ca fonctionne meme), d'autant plus qu'en JS par ex., c'est la syntaxe correcte.
Du coup, c'est peut etre une erreur plus générale qu'il est intéressant de pointer.
Enfin bon...

Et sans utiliser mysqli/pdo, si je n'utilise pas addslashes, je passe donc par mysql_real_escape_string ?
Enfin j'ai pour souvenir que si je n'utilise pas stripslashes à la sortie de mes requêtes, j'ai des antislashes de partout, mais c'est peut-être du au fait que j'utilise addslashes en entrée.

J'ai jamais dis que je savais coder, je ne demande qu'à apprendre, et il n'y a que sur ce forum que je puisse apprendre, quand on voit les cours que j'ai en IUT.

Si tu as des antislashes partout, c'est
- soit que tu échappes 2 fois
- soit, plus probable, que ton php aie ces saloperies de magic_quotes activées (ce qui revient à échapper 2 fois)

Sinon pour ce qui est d'apprendre et comprendre, n'oublies tout de même pas "ta tête" et "la doc" (et qqs tutos bien foutus et lus consciencieusement) en plus du forum

De what?

 
La doc je m'en sers assez régulièrement, mais je vais pas non plus commencer à la lire en entier pour le fun.
Je ne pense pas échapper deux fois, donc oui, je dois avoir les magic_quotes d'activées, et donc mes stripslashes étaient tout à fait justifés

Vrai, faut faire la différence entre de vrais langages avec de vraies fonctions de première classe et des bouses genre PHP, mais il suffit de tourner tous les warnings au niveau max pour que PHP te le dise.

Donc encore une fois, OSEF.

Oui. Utilise mysqli/PDO et les requêtes préparées à la place s'il te plait, sauf si tu es en PHP4

Et au lieu de te demander pourquoi, tu fais un truc qui a l'air de marchouiller mais en fait t'es pas sûr et tu sais pas pourquoi

Sûrement pas, ce qui aurait été justifié c'est d'ajouter à la récupération de tes données un test sur magic_quotes_gpc, et de faire un stripslashes sur tes données en entrée si oui.

Dans une DB, on met des données correctes, pas une soupe merdique.

Non parce que là

• Les données que tu rentres à la main vont se faire stripslashes pour rien, et potentiellement détruire des trucs
• Si jamais ton hébergeur désactive les magic_quotes, voir au dessus
• Idem si il change la version de PHP et que ça passe avec une sans les magic quotes
• Ou que tu décides de changer d'hébergeur, que celui chez qui tu arrive connaît son boulot et qu'il a désactivé ce genre de merdes (ou ne l'a pas activé, me semble que c'est plus activé par défaut sur les dernières versions de PHP)

 
Le truc là c'est que comme je suis en train de tout ré écrire en utilisant code igniter, je vais pas commencer à rechanger mes dizaines de pages écrites comme un porc
 

 
Pour moi ça me paraissait tout à fait logique, on echap les données en entrés pour éviter des modifications de la requête, et on enlève les caractères d'échappement à la sortie.
Je ne me suis jamais posé de question parce que depuis le début, j'ai toujours du stripslashé mes données en sortie, donc rien d'anormal pour moi.
 

 
Super, lire la doc de centaines de fonctions dont tu n'auras jamais l'utilité pour tomber sur quelques dizaines de fonctions utiles.
 

Comme expliqué plus haut ( ) je ne connaissais pas du tout ce comportement

lis des tutos...parce qu'un langage ça permet de faire plein de choses, y compris de la merde...et donc lire un tuto bien foutu ça va te permettre de partir directement sur des bonnes pratiques de dev...
 
en java y'a un truc de Sun nommé The Java tutorial (tu le trouves sous google en cherchant "the really big index" )...en PHP je sais pas, mais je pense qu'on te conseillera facilement ici...

<?php
$sql = "SELECT date FROM ".TBL_PALMARES." GROUP BY DATE_FORMAT(date, '%Y')";
$res = mysql_query($sql) or die(mysql_error());
 
$i=1;
 
while ($year = mysql_fetch_assoc($res)) {
    $mon_annee = explode("-", $year['date']);
    echo '<img src="images/'.$mon_annee['0'].'.gif" alt="'.$mon_annee['0'].'" class="blockCenter" />';
?>
 
<table class="blockCenter" style="margin-top: 15px;">
    <tr>
        <th>Evenement</th><th>Localité</th><th>Date</th><th>Section</th><th>Place</th>
    </tr>
<?php
    $sql2 = "SELECT t1.*, DATE_FORMAT(t1.date, GET_FORMAT(DATE,'EUR')) AS date, t2.id, t2.nom AS nom_section
            FROM ".TBL_PALMARES." AS t1 
            LEFT JOIN ".TBL_ROSTER_SEC." AS t2 ON t1.section = t2.id
            WHERE DATE_FORMAT(t1.date, '%Y') = ".$mon_annee[0]."
            ORDER BY date DESC";
    $res2 = mysql_query($sql2) or die(mysql_error());
 
    while($palmares = mysql_fetch_assoc($res2))
    {
        $data = array_map(stripslashes, $palmares);
        
        if ($i%2 == 0) {
            echo '<tr class="gold">';
                echo '<td>'.$data['evenement'].'</td>';
                echo '<td>'.$data['ville'].'</td>';
                echo '<td>'.$data['date'].'</td>';
                echo '<td>'.$data['nom_section'].'</td>';
                echo '<td class="border_right">'.$data['place'].'</a></td>';
            echo '</tr>';
        }
        else {
            echo '<tr>';
                echo '<td>'.$data['evenement'].'</td>';
                echo '<td>'.$data['ville'].'</td>';
                echo '<td>'.$data['date'].'</td>';
                echo '<td>'.$data['nom_section'].'</td>';
                echo '<td class="border_right">'.$data['place'].'</a></td>';
            echo '</tr>';
        }
    }
}
?>
</table>

rhoo la vache  
 

si
parceque sinon, tu aura des centaines de pages a changer plus tard  

on échappe les données avec mysql_real_escape_string ou des requetes préparées
et c'est tout  

ca s'appelle l'apprentissage
c'est long, c'est chiant, mais il faut passer par la pour ne pas produire de la merde  

<?php
$req = "SELECT 
                t1.evenement,
                t1.ville,
                t1.place,
                DATE_FORMAT(t1.date, GET_FORMAT(DATE,'EUR')) AS date, 
                t2.nom AS nom_section,
                YEAR(t1.date) as annee
        FROM ".TBL_PALMARES." AS t1 
                        LEFT JOIN ".TBL_ROSTER_SEC." AS t2 ON t1.section = t2.id
        ORDER BY date DESC";
$res = mysql_query($req) or die(mysql_error());
 
$anneePrecedente = 0;
 
while(list($evenement,$ville,$place,$date,$nom,$annee) = mysql_fetch_array($res))
{
    if($annee != $anneePrecedente)
    {
        if($anneePrecedente != 0)
            echo '</table>';
            
        echo '<img src="images/'.$year.'.gif" alt="'.$year.'" class="blockCenter" />';
        echo '<table class="blockCenter" style="margin-top: 15px;">';
        echo '<tr>';
        echo '<th>Evenement</th><th>Localité</th><th>Date</th><th>Section</th><th>Place</th>';
        echo '</tr>';
    }
 
    echo ( ++$cpt %2 ) ? '<tr class="pair">'  : '<tr class="impair">' ;  // on utilise jamais class="unecouleur" , parceque le jour ou tu change ton theme, tu as class=orange sur une ligne bleue
    
    echo '<td>'.$evenement.'</td>';
    echo '<td>'.$ville.'</td>';
    echo '<td>'.$date.'</td>';
    echo '<td>'.$nom.'</td>';
    echo '<td class="border_right">'.$place.'</a></td>'; //la tu as un </a> sans le reste 
    echo'</tr>';
}
echo'</table>';
 
?>

 
perso, je ferai un truc comme ça :

• nommer les paramètres dont on a besoin dans la reuqete sql ( pas de select * , jamais )  
• pas de class=gold  ( le jour  ou tu changes  de theme , tu as l'air fin )  
• les noms de table variables, ca me semble pas utile, et ca complique le code  

 
Ouais, comme je fais du java aussi, je lis les tutos de Sun, qui sont super bien fait.
En PHP j'ai lu pas mal de tutos à une époque, et c'était d'un faible niveau, d'ailleurs c'est sur ces tutos que je me suis formé
 

<?php
$req = "SELECT 
                t1.evenement,
                t1.ville,
                t1.place,
                DATE_FORMAT(t1.date, GET_FORMAT(DATE,'EUR')) AS date, 
                t2.nom AS nom_section,
                YEAR(t1.date) as annee
        FROM ".TBL_PALMARES." AS t1 
                        LEFT JOIN ".TBL_ROSTER_SEC." AS t2 ON t1.section = t2.id
        ORDER BY date DESC";
$res = mysql_query($req) or die(mysql_error());
 
$anneePrecedente = 0;
 
while(list($evenement,$ville,$place,$date,$nom,$annee) = mysql_fetch_array($res))
{
    if($annee != $anneePrecedente)
    {
        if($anneePrecedente != 0)
            echo '</table>';
            
        echo '<img src="images/'.$year.'.gif" alt="'.$year.'" class="blockCenter" />';
        echo '<table class="blockCenter" style="margin-top: 15px;">';
        echo '<tr>';
        echo '<th>Evenement</th><th>Localité</th><th>Date</th><th>Section</th><th>Place</th>';
        echo '</tr>';
    }
 
    echo ( ++$cpt %2 ) ? '<tr class="pair">'  : '<tr class="impair">' ;  // on utilise jamais class="unecouleur" , parceque le jour ou tu change ton theme, tu as class=orange sur une ligne bleue
    
    echo '<td>'.$evenement.'</td>';
    echo '<td>'.$ville.'</td>';
    echo '<td>'.$date.'</td>';
    echo '<td>'.$nom.'</td>';
    echo '<td class="border_right">'.$place.'</a></td>'; //la tu as un </a> sans le reste 
    echo'</tr>';
}
echo'</table>';
 
?>

 
Merci pour le code
Donc toi, dès que tu récupère des données en SQL, au lieu d'utiliser le tableau de mysql_fetch_array (et pas assoc ), tu les mets directement dans des variables (d'où le "list" ), ou alors tu ne fais ça que parce qu'il y a peut de données ?
 
Par contre, je ne comprends pas comment anneePrecedente == 0 ?
Parce qu'il faut bien que la variable soit différente de 0 pour commencer une nouvelle année, mais d'après ce que j'ai compris, elle est tout le temps égale à 0 non

oui, ligne 29 il manque

il n'y a quasi pas de surcout a utiliser mysql_fetch_array au lieu de assoc , et on a les clé litteral ( date, ville ) et numérique
j'utilise le list dans 95% des cas ( en gros tant que je recupère moins d'une dizaine de variable d'un coup ) , je trouve que ca rend le code plus lisible et ca évite de passer par un tableau suplémentaire  
 

j'en ai oublié un bout

bien vu

 
Je me souviens bien, à mes tous débuts ici on m'avait dit d'utiliser assoc plutôt que array puisqu'on ne se servait jamais des clés numériques

 
pour le web y'a 2-3 concepts qui dépendent pas du langage : la séparation MVC par exemple (éviter que ton code soit le modèle (gestion de la BDD), le controleur (entrée et sortie de données de la BDD, gestion de ce qui doit etre affiché, gestion des inputs utilisateurs), et la vue (affichage et collecte des inputs utilisateurs)...
 
si tu lis un peu sur MVC, tu comprendras l'intéret de séparer ces éléments...

des personnes différentes ont des pratiques différentes ( surtout que là, il s'agit d'un point de détail )

alors que le MVC, par exemple, est une bonne pratique dans l'immense majorité des cas,bcp plus que d'avoir des tables avec des noms variables

Le MVC je le fais avec code igniter, parce que sinon là c'est vraiment le bordel, j'ai des fichiers dans tous les coins, et je dois repartir de zéro quasiment.

non, tu ne le fais pas

dans le meme fichier tu as du modele

$req = "SELECT 
                t1.evenement,
                t1.ville,
                t1.place,
                DATE_FORMAT(t1.date, GET_FORMAT(DATE,'EUR')) AS date, 
                t2.nom AS nom_section,
                YEAR(t1.date) as annee
        FROM ".TBL_PALMARES." AS t1 
                        LEFT JOIN ".TBL_ROSTER_SEC." AS t2 ON t1.section = t2.id
        ORDER BY date DESC";
$res = mysql_query($req) or die(mysql_error());

et de la vue

echo '<img src="images/'.$year.'.gif" alt="'.$year.'" class="blockCenter" />';
        echo '<table class="blockCenter" style="margin-top: 15px;">';
        echo '<tr>';
        echo '<th>Evenement</th><th>Localité</th><th>Date</th><th>Section</th><th>Place</th>';
        echo '</tr>';

I beg to differ. C'est tres utile et plus rapide quand tu selectionnes beaucoup de champs

ca depend des cas et de ta structure de base de donnée aussi...

 
j'ai un peu nuancé ma réponse après : dnas l'immense majorité des cas , j'utilsie des paramètre nommés
 
dans tout le code du portail, j'ai deux requêtes avec des  *, sur des tables qui contiennent des masses de champs ( une table de paramétrage et la table avec les données rh ) . Mais pour touttes les autres, je décris ce que je requete ( ca simplifie vachement le refactoring et la lisibilité du code, je trouve )

$req = "SELECT 
                t1.evenement,
                t1.ville,
                t1.place,
                DATE_FORMAT(t1.date, GET_FORMAT(DATE,'EUR')) AS date, 
                t2.nom AS nom_section,
                YEAR(t1.date) as annee
        FROM ".TBL_PALMARES." AS t1 
                        LEFT JOIN ".TBL_ROSTER_SEC." AS t2 ON t1.section = t2.id
        ORDER BY date DESC";
$res = mysql_query($req) or die(mysql_error());

echo '<img src="images/'.$year.'.gif" alt="'.$year.'" class="blockCenter" />';
        echo '<table class="blockCenter" style="margin-top: 15px;">';
        echo '<tr>';
        echo '<th>Evenement</th><th>Localité</th><th>Date</th><th>Section</th><th>Place</th>';
        echo '</tr>';

Je pense que quand tu apprend c'est bonne façon de comprendre le tout. Surtout si tu connais pas le HTML avant d'attaquer l'apprentissage du PHP
 
XaT