Tiré de prcs2svn: ,---- | def init_repositories(self): | [ ... ] | self.create_svn_repository() | [ ... ] | | [ ... ] | | def create_svn_repository(self): | if infos.svn_root[0] != 'f': | return | [ ... ] `----- Personnellement, je n'aime pas du tout ce style de programmation (par style, j'entend ici appeller inconditionnellement des méthodes qui peuvent ne rien faire): 1/ C'est peu lisible. Au minimum, ces méthodes devraient s'appeler maybe_... 2/ Ça fait des appels de fonction qui retournent tout de suite ce qui n'est pas beau (bien sûr je pinaille, mais c'est au niveau des principes). 3/ D'autant que le test est fait sur quelque chose de global (infos). 4/ Donc il est bien plus propre de sortir le test de la fonction elle-même. -- Didier Verna, didier@lrde.epita.fr, http://www.lrde.epita.fr/~didier EPITA / LRDE, 14-16 rue Voltaire Tel.+33 (1) 44 08 01 85 94276 Le Kremlin-Bicêtre, France Fax.+33 (1) 53 14 59 22 didier@xemacs.org