Boucle FOR [résolu]

Marsh Posté le 10-06-2004 à 18:01:13

Bonjour,

J'aurai besoin d'aide pour remplir un tableau à partir d'une requête sql, il s'agit de voir quelle reunion est organisée dans quelle salle.
Le problème c'est que la requête en question ne retourne rien... ou du moins je ne vois rien affiché donc difficile de voir ce qui ne va pas...

Voyez plutôt le résultat:

Je suis dessus depuis déjà 2 jours et ça me rend fou (je tiens à préciser que mon niveau en prog php laisse à désirer :sweat: )
Voici la partie du code censé afficher "Libre" ou "Occupé" dans chaque cellule:

Code :

// Recherche des dates de reunion pour chaque salle
for ( $j=1; $j <= 5; $j++ ) {
for ( $k=1; $k <= 2; $k++ ) {
if ($k == 1) {
$demi = "M";
} else {
$demi = "A";
}
echo "<TD width=\"51\" align=\"center\">";
$sql2 = 'SELECT id_reunion, date_reunion, demi_journee, objet, nom_demandeur, service_demandeur FROM reunions
WHERE no_salle = '.$data['id_salle'].'
AND date_reunion = '.date("Y-m-d", '.mktime(0,0,0,$y_year,$m_month,$d_day).').'
AND demi_journee = "'.$demi.'";';
$req2 = mysql_query($sql2) or die ('Erreur SQL ! '.$sql2.' '.mysql_error());
while($data2 = mysql_fetch_array($req2))
{
if ($data2["objet"] == "" ) {
echo "Libre";
} else {
echo "Oqp";
}
}
echo "</TD>";
}
$d_day = $d_day+1;
}

Merci d'avance pour votre aide! :jap:

Message édité par Doudos le 11-06-2004 à 18:16:27

---------------
A+,

Reply

Marsh Posté le 10-06-2004 à 18:01:13

Reply

Marsh Posté le 10-06-2004 à 18:03:45

PS: Evidemment les reunions le samedi & dimanche sont inexistantes... faut juste que je supprime l'affichage de ces 2 jours, mais ça c'est un autre problème!

---------------
A+,

Reply

Marsh Posté le 10-06-2004 à 18:35:23

1er erreur :
mktime(0,0,0,$m_month,$d_day,$y_year)

Tu peux aussi gérer la demin journée dans la date : minuit pour le matin, midi pour l'après midi

matin : mktime(0,0,0,$m_month,$d_day,$y_year)
après midi : mktime(12,0,0,$m_month,$d_day,$y_year)
date doit alors prendre en 1er paramètre Y-m-d auquel il faut ajouter G et i pour l'heure et les minutes

---------------
fermez vos topics résolus avec le tag [Résolu] en fin de titre. Merci !

Reply

Marsh Posté le 10-06-2004 à 18:39:55

Ensuite, plutôt que de faire n requêtes, utilise date_reunion BETWEEN (date_debur_semaine, date_fin_semaine) ORDER BY date_reunion ASC

Cela te permet de récupérer toutes les positions occupées en 1 seule fois et ensuite tu gères l'affichage

---------------
fermez vos topics résolus avec le tag [Résolu] en fin de titre. Merci !

Reply

Marsh Posté le 10-06-2004 à 18:46:13

En passant à mktime(0,0,0,$m_month,$d_day,$y_year) rien n'a changé... du moins au niveau de ce qu'il retourne...

Le truc c'est que cette page existe déjà et fonctionne mais en ASP et il faudrait la même en PHP.
Pour ce qui est des demi-journées, celles-ci sont gérées via la base de donnée par un champ "demi_journee" avec comme valeur M ou A (pour Matin ou Après-midi) et je n'ai pas la possibilité de changer ça, mais c'est vrai que ta soluce paraît + pratique je n'y avais jamais pensé.

Si vous voulez, j'ai cette partie du code en ASP mais j'y comprends pas grand chose...

---------------
A+,

Reply

Marsh Posté le 10-06-2004 à 18:53:38

Moi non plus mais envoie toujours ...

Tu peux garder la demi journée dans ta table et gérer cela avec les dates quand même puisque tu as ce champs date. ASP continue avec ce champs là, PHP non ...

---------------
fermez vos topics résolus avec le tag [Résolu] en fin de titre. Merci !

Reply

Marsh Posté le 10-06-2004 à 18:59:47

La requête récupère plus d'infos et les résultats retournés sont mieux exploités, mais sur le principe je pense que ça correspond...
J'affinerai plus tard ce que j'ai fait en PHP, mais j'aimerai bien que ça fonctionne déjà avec ce qu'il y a :crazy:

Code :

'
' On affiche les réservations de la salle traitée
'
err = 0
If request("InitDate" ) = "" then
Date_du_jour= DateValue(now())
else
Date_du_jour=Cdate(Request("InitDate" ))
end if
For Icol = 1 to 5
Do while weekday(Date_du_jour,vbmonday) > 5
Date_du_jour = Date_du_jour+ 1
loop
Ddate = Month(Date_du_jour) & "/" & Day(Date_du_jour) & "/" & Year(Date_du_jour)
For IDemi = 1 to 2
Response.write ("<td width='51' align='center'>" )
Select Case IDemi
Case 1
Demi = "M"
Case 2
Demi = "A"
End select
szSQL = "SELECT DISTINCTROW Salles.[Nom de la salle], Salles.[Identifiant de la salle], Salles.[N° téléphone], "
szSQL = szSQL & "Réunions.[IdRéunion], Réunions.[Date de la réunion], Réunions.[Demi-journée], "
szSQL = szSQL & "Réunions.[Objet], Réunions.[Nom du demandeur], Réunions.[Service du demandeur] "
szSQL = szSQL & "FROM Réunions INNER JOIN Salles ON Réunions.[N° de salle] = Salles.[Identifiant de la salle] "
szSQL = szSQL & "WHERE (Réunions.[Date de la réunion]) = #" & Ddate & "# "
szSQL = szSQL & "AND Réunions.[Demi-journée] = '" & Demi & "' "
szSQL = szSQL & "AND Salles.[Nom de la salle] = '" & Salles (Isal) & "' "
szSQL = szSQL & "ORDER BY Salles.[Nom de la salle], Réunions.[Date de la réunion], Réunions.[Demi-journée] DESC;"
Set oRS1=oconn.execute(szSQL)
if err <> 0 and err <> 3021 then
Response.write (" Erreur lecture des réservations "& err.description )
Response.end
end if
if not oRS1.EOF then
Objet=oRS1("Objet" )
Demandeur=oRS1("Nom du demandeur" )
response.write ("<a href='Reservation_modification.asp?Id=" & oRS1("IdRéunion" ) & "'>" )
response.write ("<img src='images/reunions1.gif' border='0' alt='Objet : " & Objet & " // Organisé par : " & Demandeur & " Identifiant : " & oRS1("IdRéunion" ) & "'>" )
response.write ("</a>" )
else
response.write "<a href='javascript:Reserver(" & quote & Salles(ISal) & quote
response.write (", " & day(date_du_jour) & ", " & month(date_du_jour) & ", " & year(date_du_jour))
response.write (", " & quote & Demi & quote & " )';>Libre</a>" )
end if
response.write "</td>"
Next
Date_du_jour=Date_du_jour + 1
Next
Response.write("</td>" )

---------------
A+,

Reply

Marsh Posté le 10-06-2004 à 19:47:33

Ta requete cherche 1salle, 1date et 1 demi journée données. Donc il faudrait remplacer ta boucle while par

Code :

if (mysql_num_rows($req2) == 1){
//on a un enregistrement alors occupé
$date2 = mysql_fetch_array($req2);
//affichage
}
elseif (mysql_num_rows($req2) == 0) {
//c'est libre
//affichage
}
else {
// c'est pas normal, il y a un bug
}
mysql_close();

autre proposition de fonctionnement

Code :

<?php
$aujourdhui = time();
$jour_actuel = date("w", $aujourdhui);
if ($jour_actuel <> 1) {
switch ($jour_actuel) {
case 0 :
//on est dimanche donc on passe au lundi suivant
$deb_semaine = $aujourdhui + 86400;
break;
case 6 :
//on est samedi donc idem -> lundi suivant
$deb_semaine = $aujourdhui + (86400 * 2);
break;
default :
//on se place sur le lundi de la semaine en cours
$debut_semaine = $aujourdhui - (86400 * ($jour_actuel - 1));
break;
}
}
else {
$debut_semaine = $aujourdhui;
}
$fin_semaine = $debut_semaine + (86400 * 5);
$requete = "SELECT `date_reunion`, `no_salle`, `demi_journee`, `tes autres champs` FROM `ta table`
WHERE (date_reunion BETWEEN ".date("Y-m-d", $debut_semaine)." AND ".date("Y-m-d", $fin_semaine)." )
ORDER BY date_reunion ASC, demi_journee DESC, no_salle ASC;";
$reponse_SQL = mysql_query($requete, $link_db) or die("Erreur requete ".mysql_error());
//récupérer les enregistrements dans 1 tableau
//parcourir la semaine par demi journées
//vérifier si la salle est occupée ou non
//afficher en conséquence

---------------
fermez vos topics résolus avec le tag [Résolu] en fin de titre. Merci !

Reply

Marsh Posté le 11-06-2004 à 13:37:57

j'ai remplacé la boucle while par un if comme tu l'as dit

Code :

if (mysql_num_rows($req2) == 1) {
//on a un enregistrement alors occupé
$data2 = mysql_fetch_array($req2);
//affichage
echo "Oqp";
}
elseif (mysql_num_rows($req2) == 0) {
//c'est libre
echo "Libre";
}
else {
// c'est pas normal, il y a un bug
echo "Bug";
}

maintenant, toutes les cellules sont remplies par "Libre" pourtant j'ai bien créer quelques "pseudo" reunions dans la bdd...
je me demande si l'incrémentation de la date du jour fonctionne, ou tout simplement si le format de la date correspond au format de celui du champ "date_reunion" (YYYY-MM-DD)

je comprend que moyennement ton autre proposition :sweat:
pour virer le samedi et le dimanche je pense que je pourrai le faire en récupérant le jour de la semaine, et en supprimant 0 (dimanche) et 6 (samedi)

---------------
A+,

Reply

Marsh Posté le 11-06-2004 à 13:51:10

Je commence par me positionner sur le lundi de la semaine en cours. Sauf si nous sommes samedi ou dimanche où je me positionne sur le lundi suivant.

Ensuite je détermine la date fin de semaine qui correspond au vendredi.

l'etape suivante est de faire la requete sur le base en demandant la liste des salles occupées, des dates et de l'heure pour les dates comprises entre le debut et la fin de semaine (calculé auparavant)

---------------
fermez vos topics résolus avec le tag [Résolu] en fin de titre. Merci !

Reply

Marsh Posté le 11-06-2004 à 13:51:10

Reply

Marsh Posté le 11-06-2004 à 13:53:18

Commence par vérifier ce que te retourne ta requete avec phpmyadmin par exemple. Si c'est OK, c'est que cela vient du traitement de la réponse.

---------------
fermez vos topics résolus avec le tag [Résolu] en fin de titre. Merci !

Reply

Marsh Posté le 11-06-2004 à 18:15:08

wouhouhou!! ça marche!! :bounce:

Apparemment ct juste un problème de date qu'il fallait mettre entre " "

Code :

$jour_n = date("Y-m-d", mktime(0,0,0,$m_month,$d_day,$y_year));
$sql2 = 'SELECT id_reunion, date_reunion, demi_journee, objet, nom_demandeur, service_demandeur FROM reunions
WHERE no_salle = '.$data['id_salle'].'
AND date_reunion = "'.$jour_n.'"
AND demi_journee = "'.$demi.'";';
$req2 = mysql_query($sql2) or die ('Erreur SQL ! '.$sql2.' '.mysql_error());
if (mysql_num_rows($req2) == 1) {
//on a un enregistrement alors occupé
$data2 = mysql_fetch_array($req2);
//affichage
echo "<IMG SRC=\"images/reunions1.gif\">";
}
elseif (mysql_num_rows($req2) == 0) {
//c'est libre
echo "Libre";
}
else {
// c'est pas normal, il y a un bug
echo "Bug";
}

Merci pour ton aide T509 vraiment!

Message édité par Doudos le 11-06-2004 à 18:16:02

---------------
A+,

Reply

Marsh Posté le 11-06-2004 à 19:21:42

Il ne te reste plus qu'à optimiser ta boucle pour éviter de faire n requêtes sur le serveur.

1ere étape : autant de requetes que de salles
2eme etape : 1 seule et unique requete pour la semaine.

---------------
fermez vos topics résolus avec le tag [Résolu] en fin de titre. Merci !

Reply

Marsh Posté le 11-06-2004 à 19:25:04

Tout ce que tu as pu voir du code se trouve dans une boucle while qui retourne le nom des différentes salles existantes... donc je pense que le nombre de requête est déjà limité non?

---------------
A+,

Reply

Marsh Posté le 11-06-2004 à 19:48:08

oui mais tu fais une requete par salle et par date et par demi journée.

sur 5 jours, cela fait 10 requetes par jour fois le nombre de salles.

En 1 requete par salle, tu peux arriver à avoir les 5 jours de la semaines. Cela divise par 10 le nombre de requetes.

Il y a moyen de faire une seule requete pour tout aussi

relis un de mes post plus haut

---------------
fermez vos topics résolus avec le tag [Résolu] en fin de titre. Merci !

Reply

Boucle FOR [résolu]

Sujets relatifs:

Leave a Replay