Forum d'entraide PHPFrance

Bonjour à tous,

Je souhaite faire une barre de progression. En vrai, je l'ai déjà, mais je trouve mon code particulièrement long ... j'imagine qu'on peut faire mieux et c'est ce qui m'amène.

En gros, j'ai un formulaire de projets avec 19 étapes. Des cases à cocher permettent d'indiquer qu'une étape n'est pas utile dans ce projet par exemple. Donc mon script vérifie le nombre d'étapes valides ($compte) et compte celle qui ont une date d'enregistrée ($nbr). Ensuite on utilise la variable de largeur de l'espace ($largeur_cellule) dans lequel doit s'afficher l'image et on fait ($largeur_cellule / $compte)*$nbr

Je sais que je peux faire un for pour mon listing de if dans la seconde partie, mais ce que j'aimerai savoir, c'est si je peux faire des conditions directement dans ma requête (genre compter les étapes valides, et les dates remplies, ce qui allègerait mon code.

Qu'en pensez-vous ?

PS : ce code est à type d'exemple, l'affichage est géré différemment sur la "vraie" page, là je me contente d'agrandir une image de 1x1 pixel.

Erreur suite à mauvaise manip ! Les "as dateXX" dans ma requête comportent un espace, il va de soi qu'il n'y a pas d'espace ;o)

Bonjour,

Rien de méchant, mais si je peux me permettre, pense à :
- utiliser les boucles for pour faire ce genre de manipulation;
- Tu peux en mysql utiliser le IN( )

$nbRows = 19;
$fields = array(
	'p.project_id',
	'p.projet_nom',
	'e.entreprise_nom'
);
for($i=1;$i<=$nbRows ;$i++)
{
	$nb = str_pad($i , 2 , 0 , STR_PAD_LEFT );
	$fields[] = 'p.projet_checklist_'.$nb.'_check as check'.$nb;
	$fields[] = 'p.projet_checklist_'.$nb.'_date as date'.$nb;
}

$requeteProg = mysqli_query($link, "SELECT ".join(',',$fields)."
FROM BOG_PROJETS as p, BOG_ENTREPRISE as e
WHERE p.projet_entreprise_id=e.entreprise_id
AND (projet_statut_id in('51','52' ,'53' ,'54' ,'55' ,'56' ) )
ORDER BY p.projet_datedemande asc") or die ('ERREUR CHECKLIST<br>'. mysqli_error());

$largeur_cellule = 500;
echo '<table border="1">';
while ($prog = mysqli_fetch_array($requeteProg)){

	$compte = 0; // Nombre total d'étapes hors celles qui sont cochées
	$nbr = 0; // Nombre d'étapes qui sont faites (= une date est enregistrée)
	for($i=1;$i<=$nbRows ;$i++)
	{
		$nb = str_pad($i , 2 , 0 , STR_PAD_LEFT );
		if ($prog['check'.$i]!="1"){ $compte ++; if ($prog['date'.$i]!="0000-00-00"){ $nbr ++; }}
	}

	$size = ($largeur_cellule / $compte)*$nbr;
	echo '<tr>';
	echo '<td><strong>'.$prog['entreprise_nom'].'</strong>&nbsp;-&nbsp;'.$prog['projet_nom'].'</td>';
	echo '<td width="'.$largeur_cellule.'"><img src="images/progression.png" width="'.$size.'" height="20" alt=""></td>';
	echo '</tr>';
}
echo '</table>';

On passe de 76 lignes à 36 et on gagne en lisibilité.

Je suis sûr que ca peut encore être amélioré !

Bonne soirée