Discussion :

[maaxiiimee]

Bonjour à tous!
Je vous sollicite aujourd'hui pour obtenir vos avis, conseils concernant le besoin ci-dessous.

J'utilise en PHP une boucle qui me permet d'interroger une table contenant une liste de figurines.
Dans ma boucle j'affiche la date de sortie de chacune d'elles.

Ce que je souhaite faire : afficher "Bientôt disponible" lorsque la date de sortie FR est supérieure à la date du jour.

Je ne sais pas trop comment faire, utiliser une condition dans ma boucle me semble périlleux. Pouvez-vous m'aider svp ?

Voici le code utilisé :

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
36
37
<?php
include 'connect_db.php';
 
// Si tout va bien, on peut continuer
 
// On récupère tout le contenu de la table figurine et image
$reponse = $pdo->query(
	'SELECT * FROM figurine AS f, image AS i, serie AS s
	WHERE fk_i_id=i_id
	AND fk_s_id=s_id
	GROUP BY f_id');
 
$current_date = date("d-m-Y");
echo "<p>Nous sommes le " . $current_date . "</p>" ;
 
// On affiche chaque entrée une à une
while ($donnees = $reponse->fetch())
{
?>
    <p>
		ID : <?php echo $donnees['f_id'];?>
		<strong>Figurine</strong> : <?php echo $donnees['f_name']; ?><br />
    Cette figurine fait parti de la série suivante : <?php echo $donnees['s_name']; ?>.<br />
		Date de sortie FR : <?php echo $donnees['f_release_date_fr'];?><br />
		Date de sortie USA : <?php echo $donnees['f_release_date_usa'];?><br />
		Date de sortie JP : <?php echo $donnees['f_release_date_jp'];?><br />
		Date de sortie AUS : <?php echo $donnees['f_release_date_aus'];?><br />
		ID de l'image associée : <?php echo $donnees['i_id'];?><br />
		<img src="http://localhost:8888/test/<?php echo $donnees['i_location'].$donnees['i_name']; ?>"><br />
		<a href="edit_figurine.php?id=<?php echo urlencode($donnees['f_id']); ?>;">éditer</a>
		</p>
<?php
}
 
$reponse->closeCursor(); // Termine le traitement de la requête
 
?>

[badaze]

Pourquoi ce serait périlleux de faire une condition dans une boucle ?

[Songbird]

Code :

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

figurine AS f, image AS i, serie AS s

Pourquoi détériorer les noms des tables alors qu'ils étaient parfaitement lisibles ?

[maaxiiimee]

Pourquoi ce serait périlleux de faire une condition dans une boucle ?

Je me demandais si en terme de performances il y avait autre chose à faire... Mais c'est pour cela que je poste ici, pour vous le demander.
Mon idée étant d'utiliser la méthode optimale.

Tu me recommandes donc de faire ma condition dans la boucle ?

Code :

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

figurine AS f, image AS i, serie AS s

Pourquoi détériorer les noms des tables alors qu'ils étaient parfaitement lisibles ?

Exact, j'ai changé le nommage entre temps, je peux les retirer maintenant.

[badaze]

Tu as deux solutions.

1 - une condition dans la boucle.
2 - une condition dans la requête SQL. Ca devrait ressembler à : case when curdate() > date_table then ‘Bientôt disponible’ else date_table end as mon_champ.

La condition étant simple je la mettrais dans le while.

[Songbird]

Je me demandais si en terme de performances il y avait autre chose à faire... Mais c'est pour cela que je poste ici, pour vous le demander.
Mon idée étant d'utiliser la méthode optimale.

Boucle ne rime pas forcément avec perte de performance, cela signifie simplement qu'il faut répéter des choses, ni plus ni moins. Si tu ne vois pas du premier coup d'oeil ce qu'il faut fixer pour améliorer les performances (et que tes utilisateurs ne se plaignent pas), ça sera (très généralement) quelque chose de tordu qui te fera gagner 200 ns là où tu n'en as pas forcément besoin, mais cela reste un avis très personnel.

[maaxiiimee]

Ok merci à tous pour vos retours.

Je mets ce sujet à "résolu"

[foetus]

Si il y a un truc à faire ... mais cela va te demander de dupliquer du code (il n'y a pas de macros) et niveau performance cela doit être négligeable (mais à tester)

Il faut récupérer ta liste de figurines par date française croissante ("ORDER BY f_release_date_fr ASC")

Et ensuite, coder en pseudo code :

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
    $donnees = $reponse->fetch();
 
    while(($donnees !== false) && ($donnees['f_release_date_fr'] < now))  {
//      ...
        Date de sortie FR : <?php echo $donnees['f_release_date_fr'];?><br />
//      ...
 
        $donnees = $reponse->fetch();
    }
 
    while(($donnees !== false) && ($donnees['f_release_date_fr'] == now)) {
//      ...
        Date de sortie FR : aujourd’hui<br />
//      ...
 
        $donnees = $reponse->fetch()
    }
 
    while($donnees !== false) {
//      ...
        Date de sortie FR : bientôt disponible<br />
//      ...
 
        $donnees = $reponse->fetch();
    }