PHPfrance

la communauté d'entraide francophone dédiée au PHP

If à rallonge .. comment puis-je faire plus propre ?

Répondre


Cette question est un moyen d’empêcher des soumissions automatisées de formulaires par des robots.
Smileys
:D :) :( :o :shock: :? 8-) :lol: :x :P :oops: :cry: :evil: :twisted: :roll: :wink: :!: :?: :idea: :arrow: :| :mrgreen: =D> #-o =P~ :^o :non: :priere: 8-|
Voir plus de smileys
  Revue du sujet
 

  Étendre la vue Revue du sujet : If à rallonge .. comment puis-je faire plus propre ?

par Sebe » 01 févr. 2008, 14:02

Salut,
Bon c'est du détail, mais quelques caractères de moins, ça fait aussi gagner en lisibilité :)
Je pense que c'est dû à mon coté matheux ... je me suis toujours dit qu'il n'y en avait jamais assez pour éviter les erreurs en math :roll:

Merci

par Ryle » 01 févr. 2008, 13:09

Tu as également beaucoup de parenthèses et d'apostrophes inutiles...

au lieu de 
if (($course->publier == '1') && ...
tu pourrais écrire 
if ( $course->publier == 1 && ...
Bon c'est du détail, mais quelques caractères de moins, ça fait aussi gagner en lisibilité :)

par Sebe » 01 févr. 2008, 12:43

Ok merci

par d0m » 01 févr. 2008, 12:27

déjà en séparant le code html pur du code PHP et en sautant des lignes et indentant tout simplement : 
<?php
if ($affichage == '1') {?>
  <tr bgcolor= "<?php echo $bgcolor;?>">
<?php if ( ($course->publier == '1')
            && ( ($course->pat_chal == 'C')
                 && ( ($class_Config['link_chal'] == '1')
                      || ($class_Config['link_chal'] == '2')))
            || (($course->pat_chal == 'P').....
     {
     <td><a href="<?php echo $link . $id;?>"><?php echo $date;?></a></td>
  ....

If à rallonge .. comment puis-je faire plus propre ?

par Sebe » 01 févr. 2008, 12:12

Bonjour,

Je voudrais savoir comment je peux rendre plus présentable cette série de condition: 
if ($affichage == '1') {
		echo '<tr bgcolor= "<?php echo $bgcolor;?>">';

		if (($course->publier == '1') && (($course->pat_chal == 'C') && (($class_Config['link_chal'] == '1') || ($class_Config['link_chal'] == '2'))) || (($course->pat_chal == 'P') && (($class_Config['link_pat'] == '1') || ($class_Config['link_pat'] == '2'))) || (($course->pat_chal == 'H') && (($class_Config['link_hors_chal'] == '1') || ($class_Config['link_hors_chal'] == '2')))){
		
		echo '<td><a href="' . $link . $id . '">' . $date . '</a></td>'
			. '<td><a href="' . $link . $id . '">' . $course->lieu . '</a></td>'
			. '<td><a href="' . $link . $id . '">' . $course->course . '</a></td>'
			. '<td><a href="' . $link . $id . '">' . $course->kilometrage . '</a></td>'
			. '<td><a href="' . $link . $id . '">' . $heure . '</a></td>'
			;			
		
			} else {
				echo '<td>' . $date . '</td>' 
					. '<td>' . $course->lieu . '</td>'
					. '<td>' . $course->course . '</td>'
					. '<td>' . $course->kilometrage . '</td>'
					. '<td>' . $heure . '</td>'
					;
				} // fin du if link
			echo '</tr>';
		} // fin if $affichage
Surtout le 2e if ... y-a-t-il un intérêt ?

Merci