Discussion :

[ninikkhuet]

Bonjour,

tout est dans le titre, j'ai récupéré une application, j'ai mon avis (grosse daube ) mais je voudrais connaître les avis de chacun sur une portion de code d'un fichier PHP (sachant que quasiment tout le code est comme ça..). Voilà le monstre :

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
38
39
40
41
42
43
<?
 
function showpagenav($page, $pagecount) {
?>
<table class="bd" border="0" cellspacing="1" cellpadding="4">
<tr>
<td><a href="../administrateur/<?php echo $_SESSION['fichier'];?>?a=add">Ajouter</a>&nbsp;</td>
<? 
   if ($_SESSION['page'] > 1) { ?>
<td><a href="../administrateur/<?php echo $_SESSION['fichier'];?>?page=<? echo $page - 1 ?>">&lt;&lt;&nbsp;Précédent</a>&nbsp;</td>
<? } ?>
<?
 
   if ($_SESSION['pagecount'] > 1) {
       if ($_SESSION['pagecount'] % $_SESSION['pagerange'] != 0) {
	    $rangecount = intval($_SESSION['pagecount'] / $_SESSION['pagerange']) + 1;
      } else {
	    $rangecount = intval($_SESSION['pagecount'] / $_SESSION['pagerange']);
      }
      for ($i = 1; $i < $rangecount +1; $i++) {
           $startpage = (($i -1) * $_SESSION['pagerange']) + 1;
           $count = min($i * $_SESSION['pagerange'], $_SESSION['pagecount']);
              if ((($_SESSION['page'] >= $startpage) && ($_SESSION['page'] <= ($i * $_SESSION['pagerange'])))) {
                  for ($j = $startpage; $j < $count +1; $j++) {
                       if ($j == $_SESSION['page']) {
?>
<td><b><? echo $j ?></b></td>
<?                    } else { ?>
<td><a href="../administrateur/<?php echo $_SESSION['fichier'];?>?page=<? echo $j ?>"><? echo $j ?></a></td>
<?                    } 
                  } 
               } else { ?>
<td><a href="../administrateur/<?php echo $_SESSION['fichier'];?>?page=<? echo $startpage ?>"><? echo $startpage ."..." .$count ?></a></td>
<?            } 
     } 
   } ?>
 
<? if ($_SESSION['page'] < $_SESSION['pagecount']) { ?>
<td>&nbsp;<a href="../administrateur/<?php echo $_SESSION['fichier'];?>?page=<? echo $_SESSION['page'] + 1 ?>">Suivant&nbsp;&gt;&gt;</a>&nbsp;</td>
<? } ?>
</tr>
</table>
<? } ?>

c'est parti, déchaînez-vous !

[metagoto]

C'est un professionnel qui a écrit ça ?

[cholopat]

je ne vois pas fort où est le problème ...

le code parle de lui même ... on dirait un programmeur novice en php qui essaye de mettre en place des choses différentes dans une structure de fonction, en jouant avec des sessions et des fonctions ...

ça me semble très bien !

mais oui, c'est une bicyclette à 4 roues ...

Si tu connais l'auteur du code et que tu as et le temps et les moyens, je pense que ce genre de démarche doit être soutenue, se serait sympa de l'aider .

[ninikkhuet]

C'est un professionnel qui a écrit ça ?

euh...oui

[ninikkhuet]

Si tu connais l'auteur du code et que tu as et le temps et les moyens, je pense que ce genre de démarche doit être soutenue, se serait sympa de l'aider .

je ne connais pas l'auteur, par contre je vais quand même essayer de lui donner des conseils
J'allais commencer par :
- séparer au maximum le PHP et le HTML
- éviter d'utiliser les variables superglobales (sécurité)

[Tillo]

Bonsoir,

Je trouve que l'on s'y perd un peu.
Il y a beaucoup de <? et de ?> et surtout de <? } ?>

A bientôt,
Alexis

[jabbounet]

cela manque de commentaires explicatifs tout ça je trouve.

quelqu'un d'exterieur au monde php comme moi, ne comprend pas trop ce qu'il se passe.

[Bebel]

Pour rajouter mon avis, le plus gros défaut est le manque de test.
Il n'y aucun test sur l'existence des valeurs en session, ni sur la valeur de $_SESSION['pagerange'] pour la division.

Apres le mélange de html, php dans le corps d'une fonction, je trouve ca horrible et peu pratique, car on n'a pas la main sur l'affichage.

En tout cas, si tu dois maintenir ce code, je te souhaite bien du courage.