Discussion :

[jbaudin]

Bonjour à tous !!

Je suis en train de faire un script php qui s'allonge de plus en plus.

Comme tout le script n'est jamais utilisé à chaque fois, je voudrais avoir vos avis sur le fait de séparer le code en plusieurs fichiers distinct qui seront appeler par un fichier principal.

Est-ce que cela améliorerait les performances de mon script ?

Merci pour vos réponses !

[Mr N.]

oui ca améliorera les performances si tu ne charges le fichier inclus que quand tu en as besoin
oui ca améliorera la lisibilité de ton code si tu découpes les parties de ton script en "modules" cohérents

[Johnny P.]

Déjà peux-tu mettre ton code si tu veux ! sinon si c'est vraiment long alors coupe ton script en plusieurs fichiers ! si tu utilises le même fichier pour connexion base de donnée et autre ben utilise include ou require

[jbaudin]

Merci pour vos réponses.
Actuellement j'utilise switch pour décider de ce qui doit être fait.
Mais certaines conditions commencent à être longue.
J'utilise déjà des classes qui sont dans 2 fichiers différents.

Mais je me pose encore des questions sur l'anatomie de mon code.
Les [un switch] sont là pour vous montrer vaguement l'anatomie du code.

[un switch par défaut] - Pour le moment j'ai un fichier index.php qui gère tout ce qui se passe sur la page principale (à savoir la liste des en-têtes des mails enregistrée dans ma base de données).

[un switch] - J'ai aussi la gestion de la récupréation des en-têtes des mails.
C'est un code qui appel une classe (qui en appel une autre qui gere les flux imap) qui se charge de récupérer les en-têtes des mails sur le serveur et me renvoie le nombre récupérés (avec l'insertion dans la bases de données des en-tête serializées) .

[un switch] - Là, le code affiche un message avec le nb de mails récupérés et se relance de lui-même au bout de 3s avec des variables en paramètres.

[un switch] - A partir de ce moment, le code vérifie si il y des en-tête serializées dans une table et les unserialize dans une autre tables (en vidant la table serializée bien sûr).
Ensuite le code affiche un tableau de la table des en-têtes unserializées.

Vous me suivez jusque là ?

[un switch] - Il y a aussi un code de suppression des mails.
Ce code appel aussi une classe (la même en fait) qui se charge de vider la table de la base de données qui contient les en-têtes.
En premier je demande validation de l'action à faire

[un switch] - Si oui, alors j'efface et j'affiche la liste avec les verifs déjà vues plus haut.

Le code le plus long est celui qui affiche la liste (avec toute les vérifs).
C'est ce code qui grossi de plus en plus.

Qu'en pensez-vous ?

Toutes critiques sont les bienvenues.
(surtout les positives )

Merci

[Ajout]
Pour le moment mon script ne fait que 177 lignes (merci au moteur de template Smarty !!)
Mais il va très certainement grossir !!
[/Ajout]

[Johnny P.]

177 lignes ? c'est pas grand je connais des fichiers qui font 500 à 1000 lignes ? ben comme j'ai dit regarde ton code et essaies de trouve ce qui peut-être soit coupé ou raccourci n'utilise pas des if-elseif-else quand tu peux utiliser l'instruction switch ! à toi de voir comment raccourcir le code

[Mr N.]

On peut avoir un aperçu de cet index.php ?

[ofostier]

Hello,


Effectivemment ca peux améliorer les performances si tu fait appel
a de grosse bibliothèques.

Ca peut aider aussi a la lisibilité.


Mais bon c'est vrais que pour une centaine de ligne je ne suis pas sur que ca vaille le coup tout de suite .... mais c'est bien d'y penser.
Surtout si tu pense que tes scripts vont s'allonger et que le nombre de visiteurs en silmultanés va s'accroitre.

Moi je fait ca sur un programme qui appel plusieurs librairies de plusieurs milliers de ligne.


Bye et bon code

O.FOSTIER