Re: [gulliver] [atelier python] Vous pouvez poser questions et code

[David MENTRE <dmentre at linux-france dot org>]

"Fred Lemasson" <djassper at gmail dot com> writes:

>> Qu'est-ce qui te pose soucis ?
> principalement le fait que l'on puisse planter le programme en se
> trompant dans la syntaxe de l'IpCode, pour moi ca va, mais pour
> quelqun qui découvre le logiciel ce n'est pas top ... je ne maitrise
> pas bien les try/except pour la gestion d'erreur, c'est ca qui manque
> au prog, et à mes skills python ...

En fait, je te demandais ce qui te posait problème avec le mécanisme des
exceptions. ;-) La syntaxe ? Juste un manque de pratique ? Une question
plus conceptuelle ?

>> Tu l'as mis en GPLv3. Rien à redire. :-) Par contre, je suppose que
>> c'est toi qui a ajouté la clause « If you Improve/Modify/Debug Pong, you
>> ["must" AND "will be happy to"] send me an email. » Je ne pense pas que
>> ce soit compatible avec la GPL. Si tu remplaces le « must » par un «
>> might », je pense que ce sera bon.
> Ahh ok, j'ai fait un fork alors ... humm et on ne peut pas faire de
> simples petits ajouts comme cela sans rompre la license ?

Il faut que les ajouts soient compatibles avec la licence. En
l'occurence, tu ne peux pas *obliger* quelqu'un à t'informer des modifs
sur le programme, ça rend le programme non libre.

Je ne suis pas expert dans ce domaine. La règle de base étant de prendre
tel quel la licence de la FSF et de ne rien retoucher.

>> Côté code, c'est très lisible. Néanmoins, j'aurais tendance à revoir le
>> formatage du code, notamment les espaces et la limitation des lignes à
>> 79 caractères. Les règles à suivre sont indiquées ici :
> hummm ok mais personnellement je trouve que des commandes coupées sur
> deux lignes sont moins lisibles, et en python avec l'indentation des
> blocs ca peut rogner vite fait les 80 caracteres, dans pong par
> exemple il y a 9 niveaux soit 36 caractères d'indentation sur
> certaines lignes

D'où l'importance d'avoir des fonctions pour découper tout ça et avoir
des lignes plus courtes ! ;-) 

Maintenant, tu fais comme tu veux, j'indiquais juste ce qui est
considéré comme une bonne pratique.

Pour moi, des expressions multi-lignes avec une bonne indentation sont
tout aussi lisibles. Par exemple, au lieu de 

    print "Pong Report : "+str(PingsUp+PingsDown+PingsWeird).rjust(8) +" IPs Ponged in :",TimeFrom(StartTime)

tu peux avoir

    print "Pong Report : " + str(PingsUp + PingsDown + PingsWeird).rjust(8) \
                           + " IPs Ponged in :", TimeFrom(StartTime)

Au passage, tu gagnerais en lisibilité en utilisant l'opérateur % pour
le print. Typiquement :

    print "Pong Report: %8d IPs Ponged in: %s" % (PingsUp + PingsDown + PingsWeird,
                                                  TimeFrom(StartTime))

Voir: http://docs.python.org/tut/node9.html#SECTION009100000000000000000

Amicalement,
d.
-- 
David Mentré <dmentre at linux-france dot org> -- http://gulliver.eu.org/