Discussion :

[tomtom59]

Bonjour à tous,

Voilà maintenant 2 à 3 mois que je développe en PERL, et j'ai réalisé un script en PERL d'environ 400 lignes. Bon je dois dire que j'en ai bien bavé pour le faire mais je pense qu'après l'avoir réalisé il est maintenant temps de l'optimiser.

Bon là ca se gate. Je n'ai pas trop d'idées sur ce qui se fait ou ce qui n'est pas conseillé en PERL.

Y aurait-il une personne à qui je pourrais envoyer le code source du code pour qu'il me dise d'un oeil extérieur ce qui pourrait clocher et ainsi ce que je pourrais améliorer dans le code?

Je vous remercie beaucoup par avance.

Thomas

[djibril]

Tu pourrais déjà nous dire le but de ton script.
Pourquoi penses tu qu'il n'es pas optimal!
Sinon si ce n'est pas du code confidentiel, tu peux toujours le mettre sur le forum.

[tomtom59]

Salut Djibril, il s'agit d'un code assez confidentiel, en fait il s'occupe d'analyser des lignes de logs en continu (à peu près 150 millions de lignes ajoutées par jour), et sort un rapport statistique chaque jour.

C'est vrai que le code n'est pas commenté mais en fait je cherche a l'optimiser parce qu'au bout de 4 ou 5h, il monte en consommation CPU et plafonne à 100%... Donc j'ai beau chercher d'où cela pourrait venir je sèche...

Ca t'interesse d'y jeter un mini coup d'oeil? Peut-être y trouveras tu au premier regard une étourderie de débutant! Enfin c'est comme tu le sens, sinon avis aux amateurs!

Merci par avance.

Tom

[djibril]

Alors le premier conseil que je te donne est de le relire ligne à ligne à tete reposé.
Ensuite tu essayes de le commenter au maximum.
Ca te permettra de te re-poser des questions sur ta logique lors de sa conception.
C'est un bon moyen de déceler des erreurs de conception.
De plus, dit toi que tu le commentes pour quelqu'un d'autre ou même pour toi dans 1 an quand tu auras oublié la logique de ton script. Tu verras de suite ce qui va ou pas. Et ton script sera déjà plus propre.

[Jedai]

As-tu lu la section de notre FAQ sur le style en Perl ? Essaie d'appliquer tous les conseils (en particulier j'espère que tu utilises déjà le pragma "strict"). Les divers outils de Profiling ainsi que Devel::Size pourrait également t'être utile pour déterminer où est la problème.

--
Jedaï

[tomtom59]

Tout d'abord merci pour vos conseils!

Je viens de passer l'aprem à commenter le code et à le nettoyer. Pour répondre à ta question Jedai, oui j'utilise le pragma strict =)

Je vais essayer d'utiliser Devel::Size ce soir, en attendant si vous voulez jeter un coup d'oeil au code, il est maintenant propre (tout du moins plus propre qu'avant), et commenté donc avis aux amateurs

Merci,

Thomas