déjà vu des dizaines et des dizaines de fois, je me dmd même si ce code est de toi.. rien de nouveau par rapport aux autres, franchement si c'est posté juste pour poster c'est sans intérêt.
niveau code c'est vrment pas du tout optimisé, plein de variables globales inutiles.. etc etc
je reformule mon message :
C'est deja vu beaucoup de foix sur ce site ;)
sinon tu aurai pus mettre les options autres que oui et non (deja dit plus haut)
Sinon, ce qui est du code, tu aurai pu regrouper les on text, enleve les / et met des . devant les timers ;)
ca risque pas d'etre mal codé vu le temps qu'il fait chez toi...
> moi quand je script, je garde mes code pour moi et quand je les montre, c'est pour progresser.
quel commentaire ^^ franchement ce genre de commentaires on s'en apsserait vraiment... "vu le code, c'est mal codé"... ca risque pas d'etre mal codé vu le temps qu'il fait chez toi...
Juste au niveau du code :
-Regroupe les on text en un seul
-Utilise elseif au lieu des if
-Met des timer
-if (%vote != $null) { devient if (%vote) {
-if (%vote. [ $+ [ $address([ $nick ],2) ] ] == $null)
Devient
if (!%vote. $+ $address($nick,2))
-elseif (%vote.oui == %vote.non) { msg %vote.chan 3Egalitée ! }
utilise directement else ici
-Dans les $calc Je crois que les crochet sont inutiles
Et au lieu de faire deux section tu aurais pu mettre les alias comme ca : alias voteresult { ... }
Comme ca on mettait tout dans les remotes
bon, comme tu 'las dit, c'est pas très original ^^ fin je vais pas t'en tenir rigueur
- pense à regrouper tes events (plusieurs on text)
préfère on*:text:*:#: { if ($1 truc) { ... } | elseif ($1 machin) { ... } }
- pour les timers, le / n'est pas plus utile qu'ailleurs. je pense que ce que tu veux faire c'est les rendre invisibles, pour cela mets plutot .timer
- tu pourrais regroupper !oui et !non en faisant par exemple une commande !vote oui ou !vote non, puis ensuite :
# on *:text:!vote:%vote.chan:{
# if ($2 !isin ouinon) say $nick vote incorrect
# else {
# if (%vote != $null) {
# if (%vote. [ $+ [ $address([ $nick ],2) ] ] == $null) {
# inc %vote. [ $+ [ $2 ] ]
# set %vote. [ $+ [ $address([ $nick ],2) ] ] 1
# notice $nick 4Merci d'avoir voté =) !
# }
# else {
# notice $nick 4T'a deja voté !
# }
# }
# else {
# notice $nick 4Pas de vote en cours =) !
# }
# }
# }
je sais que !vote est déjà utilisé mais c'est à aménager (ps : if (%vote !=$null) devient if (%vote))
- pour que ton script diffère des autres, pourquoi ne pas proposer de faire un vote comprenant d'autres réponses que oui et non ? et si possible infinies (ce n'est pas très compliqué si tu maitrises les boucles)
sinon, pour une deuxième source c'est pas si mal, je note pas.
123410
pour tes info tu va dans :-Script editor
-remote
-file
-new
-copy_colly le script
-file
-save as...
-ok
et pi voila normalement c bon
Il y a rien a telecharger ???
On le mes ou ???
Merci d'avance