VOTE SYSTEM BY [SIMON]
calopsfr
Messages postés
137
Date d'inscription
mardi 4 novembre 2003
Statut
Membre
Dernière intervention
22 août 2005
-
26 juin 2005 à 15:07
cs_hunter38 Messages postés 46 Date d'inscription samedi 12 mai 2007 Statut Membre Dernière intervention 12 juin 2007 - 25 mai 2007 à 20:23
cs_hunter38 Messages postés 46 Date d'inscription samedi 12 mai 2007 Statut Membre Dernière intervention 12 juin 2007 - 25 mai 2007 à 20:23
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.
https://codes-sources.commentcamarche.net/source/32335-vote-system-by-simon
https://codes-sources.commentcamarche.net/source/32335-vote-system-by-simon
25 mai 2007 à 20:23
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
16 févr. 2006 à 15:59
Il y a rien a telecharger ???
On le mes ou ???
Merci d'avance
21 juil. 2005 à 15:07
30 juin 2005 à 18:50
27 juin 2005 à 08:53
niveau code c'est vrment pas du tout optimisé, plein de variables globales inutiles.. etc etc
26 juin 2005 à 19:36
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.
26 juin 2005 à 16:28
26 juin 2005 à 16:17
sinon A+ [SIMON] :p
26 juin 2005 à 15:44
Devient
if (!%vote. $+ $address($nick,2))"
je pense pas... vaut mieux laisser les crochets :$
Dans tes on TEXT, $address($nick,2) devient $wildsite
Déjà vu des dizaines de fois sur ce site, donc si c'est juste parce que tu tenais a le faire toi même, je vois pas pourquoi tu postes...
26 juin 2005 à 15:40
26 juin 2005 à 15:08
-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
26 juin 2005 à 15:07
- 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.