Discussion :

[pomoxp]

Hello

Je me pose une question métaphysique : j'ai un script toto.pl comme suit

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
 
use Mymodule::Init;
print File::Spec->catfile('c:\a','essai.txt') . "\n";

Ce code fonctionne car le module File::Spec est chargé à l'intérieur du module MyModule::Init;
Est-ce que c'est bien ?
Devrais-je ajouter tout de même un

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

use File::Spec;

dans mon script toto.pl ? pour les Best Practices ?
Ou autrement dit, comment pourrais-je faire en sorte que ce qui est fait dans mon module MyModule::Init ne soit pas "transmis" à mon script toto.pl à part les fonctions que j'exporte ?

C'était la question du matin de milieu de semaine

[djibril]

Pour une question de meilleure lisibilité, je suggère de rajouter use File::Spec;.
A la relecture de ton programme, tu comprendras de suite pourquoi il fonctionne. Ce n'est qu'un avis personnel.

[pomoxp]

Bonjour Djibril

A la relecture de ton programme, tu comprendras de suite pourquoi il fonctionne.

Je ne comprends pas pourquoi tu dis cela.

Si mon module Mymodule::Init contient use File::Spec;
alors oui mon script fonctionne.

Si derrière un développeur décide de modifier le script et de ne plus utiliser le module Mymodule::Init alors le script ne fonctionnera plus car il n'y aura plus le use File::Spec implicitement fait.

Donc ma compréhension est que chaque script devrait charger explicitement le module dont il a besoin mais parfois en testant on peut en oubliant et pourtant le code fonctionnera si un module inclut lui-même un autre module utilisé dans le script.

Suis-je clair ?


D'où ma question : avez-vous été confronté à ce genre de problème et si oui comment faites-vous ?

[djibril]

Bonjour Djibril
Si mon module Mymodule::Init contient use File::Spec;
alors oui mon script fonctionne.

Si derrière un développeur décide de modifier le script et de ne plus utiliser le module Mymodule::Init alors le script ne fonctionnera plus car il n'y aura plus le use File::Spec implicitement fait.

D'où ma suggestion de charger le module pour éviter ce genre de soucis

[hauru]

Ce que veut dire Djibril c'est que ce serait mieux d'ajouter dans toto.pl:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

use File::Spec;

Les autres developpeurs ne savent pas forcement où se trouvent les differents modules.
Cela permet d'avoir une meilleur lisibilité dans le programme.

De plus, tout définir permet d'eviter certaines supprises lorsque l'on veut supprimer certains use.

[pomoxp]

Ok merci à tous les deux.
Nous sommes donc d'accord et c'est mieux quand c'est dit.