Messages : 2,344
Sujets : 192
Inscription : Apr 2004
25/10/2014, 22:53
(Modification du message : 25/10/2014, 22:53 par CrazyCat.)
La seule remarque que je ferais, c'est que la procédure hostmask_process mériterait d'être sortie du namespace, et pourquoi pas dans un fichier annexe, car elle peut potentiellement être utile à d'autres scripts.
Sinon, rien à redire sur le script, à première vue il me semble plutôt propre.
Messages : 927
Sujets : 93
Inscription : Jul 2007
Niveau d’avertissement :
0%
26/10/2014, 00:26
(Modification du message : 26/10/2014, 01:17 par MenzAgitat.)
Tu as des & #92; dans ton code qui devraient être remplacés par des \
Les "set" dans le namespace eval devraient comporter des noms de variable pleinement qualifiés ( set ::antispams::why(advert) .... )
Les 2 foreach servant à arrêter les timers/utimers dans la proc de désinstallation ne te servent à rien
Messages : 299
Sujets : 8
Inscription : Dec 2008
Niveau d’avertissement :
0%
salut,
Les lignes
217 et 200 ont la même fonction " *viens* / *viens sur #* "
216 et 219 aussi " *http* / *http://* "
La ligne 229 " *irc* " me semble trop restrictive exemple " Salut, je suis nouveau ,qu'est-ce irc ? " ça match
De plus mettre un "antispam" sur le #salon, je trouve ça un peu fort, lors d'une discution, un utilisateur, peut etre amené dire a un autre "Viens sur #aide, un helper vas t'expliquer" et pour le coup sera "matché" .
Cordialement.
Messages : 2,344
Sujets : 192
Inscription : Apr 2004
(26/10/2014, 00:26)MenzAgitat a écrit : Tu as des & #92; dans ton code qui devraient être remplacés par des \
Les "set" dans le namespace eval devraient comporter des noms de variable pleinement qualifiés ( set ::antispams::why(advert) .... )
Les 2 foreach servant à arrêter les timers/utimers dans la proc de désinstallation ne te servent à rien
Tiens, je n'avais pas fait attention à cette erreur (due au forum) de & #92;, tellement habitué à la voir :)
Pour les "set", vu qu'ils sont dans le eval, quel intérêt de qualifier ?
Et concernant les kills de timers, je pense que ce n'est pas génant dans le sens où la désinstallation intervient rarement et qu'on ne sait pas trop comment évoluera le tcl. Là, ça ne sert à rien, mais un jour qui sait...
(26/10/2014, 12:30)cestlemien a écrit : salut,
Les lignes
217 et 200 ont la même fonction " *viens* / *viens sur #* "
216 et 219 aussi " *http* / *http://* "
La ligne 229 " *irc* " me semble trop restrictive exemple " Salut, je suis nouveau ,qu'est-ce irc ? " ça match
De plus mettre un "antispam" sur le #salon, je trouve ça un peu fort, lors d'une discution, un utilisateur, peut etre amené dire a un autre "Viens sur #aide, un helper vas t'expliquer" et pour le coup sera "matché" .
Cordialement.
Je ne pense pas que le choix des mots-clés soit à discuter mais plutôt à prendre comme exemple. C'est vraiment la partie qui devrait devenir la plus configurable.
Ah, ben voila une critique: il faudrait mettre une liste des mots clés facile à éditer, et que l'init du tcl crée les binds à partir de cette liste.
