MaX_62 a dit :
Petits détails au niveau du code :
- la balise "full" a la fin ne sert a rien, elle n'est utilisée nulle part dans le code.
- les slash sont inutiles devant les commandes, tu peux mettre des "." devant les notices
- tu peux regrouper certaines parties du code :
if (%nbrdegens == 1) { set %1 $nick }
if (%nbrdegens == 2) { set %2 $nick }
if (%nbrdegens == 3) { set %3 $nick }
devient
- essaie de mettre des noms de variables plus longs car sinon tu pourrais avoir les memes noms de variables dans 2 scripts différents =/
Voila, j'aime pas trop ce genre de bots, je note pas.
Bonne continuation ;)
>> d'une chose l'une !
1 ! c'est quoi la diference entre taper un . ou un / ?
2 ! " if (%nbrdegens isnum 1-10) { set % [ $+ [ %nbrdgens ] ] $nick }
- .timer 1 3 /leave %chanpickup >> /leave ? j'connaissais pas... (/part)
" je trouve sa plus long a la compresion du code ou a la re-lecture pour gagner quelques lignes, de plus c'est souvent l'objet d'erreur !
Ca me fais penser que j'avais fait un script pas mal pour gather, j'avais copier le fonctionnement du bot de #cs.gather.fr (j'était a l'époque admin et le bot était souvent down), faudra que je la finisse un de ces 4.
pour clem54 un pickup c'est un match (en général 5 contre 5) ou ce sont des inconnues qui joue avec toi, en tappant '!add' ils s'ajoutent à la liste, et quand le compte est bon (10 joueurs en général), le bot t'envoie l'ip du serveur et le password.
Très optimisable.
Mais si c'est ta première source, c'est normal.
On utilise souvent au début la technique des variables pour le stockage des données, mais ensuite on passe aux fichiers txt, ini ou aux hash tables.
Pour ta source l'utilisation d'un petit fichier ini suffirait à l'optimiser grandement.
Sinon il manque des vérifications et des brides.
En l'état actuel :
- n'importe qui peut faire join ton bot sur un chan (gare aux debordements et abus).
- la même personne peut se !add 10 fois.
- regroupe tes evenements ON TEXT car par exemple pour ton !pickupon et !pickupoff il y a beaucoup de commandes similaires donc des doublons inutiles.
Je remarque de gros soucis niveau optimisation. Je vois que c'est ta première source donc je ne te critique pas, je te donne des conseils. Tu peux largement réduire le code grace à des boucles. Exemple :
set %1 ?
set %2 ?
set %3 ?
set %4 ?
set %5 ?
set %6 ?
set %7 ?
set %8 ?
set %9 ?
set %10 ?
Ici tu pourrais faire un alias qui sets les variable avec une boucle while :
;on déclare l'alias que l'on met en local ( -l )
alias -l setvar {
; on donne 11 à la variable temporaire %i
var %i 1
; ensuite on boucle jusqu'à que 1 soit inférieur ou égale à 10
while (%i <= 10) {
puis on déclare va variable %i à ?
set $+(%,%i) ?
; on n'oublie d'incrémenter %i de 1
inc %i
}
}
Voilà en faisant /setvar dans ton code ça va te mettre toutes tes variable %1 %2 %3 ... à ?
Oops jai deleter un truck je saver pas que sa server a ca dsl :x
->pour les réponses : je ne poste pas pour les points mais pour me faire corriger et apprendre ,j'ai vu les autres pickup script mais je voulais faire le mien ^^
Tant de set pour si peu d option possible, surtout qu un pick up bot a ete posté il y a peu. Je ne note pas non plus sinon on va te descendre en fleche...
Petits détails au niveau du code :
- la balise "full" a la fin ne sert a rien, elle n'est utilisée nulle part dans le code.
- les slash sont inutiles devant les commandes, tu peux mettre des "." devant les notices
- tu peux regrouper certaines parties du code :
if (%nbrdegens == 1) { set %1 $nick }
if (%nbrdegens == 2) { set %2 $nick }
if (%nbrdegens == 3) { set %3 $nick }
devient
if (%nbrdegens isnum 1-10) { set % [ $+ [ %nbrdgens ] ] $nick }
- .timer 1 3 /leave %chanpickup >> /leave ? j'connaissais pas... (/part)
- essaie de mettre des noms de variables plus longs car sinon tu pourrais avoir les memes noms de variables dans 2 scripts différents =/
Voila, j'aime pas trop ce genre de bots, je note pas.
Bonne continuation ;)
Petits détails au niveau du code :
- la balise "full" a la fin ne sert a rien, elle n'est utilisée nulle part dans le code.
- les slash sont inutiles devant les commandes, tu peux mettre des "." devant les notices
- tu peux regrouper certaines parties du code :
if (%nbrdegens == 1) { set %1 $nick }
if (%nbrdegens == 2) { set %2 $nick }
if (%nbrdegens == 3) { set %3 $nick }
devient
- essaie de mettre des noms de variables plus longs car sinon tu pourrais avoir les memes noms de variables dans 2 scripts différents =/
Voila, j'aime pas trop ce genre de bots, je note pas.
Bonne continuation ;)
>> d'une chose l'une !
1 ! c'est quoi la diference entre taper un . ou un / ?
2 ! " if (%nbrdegens isnum 1-10) { set % [ $+ [ %nbrdgens ] ] $nick }
- .timer 1 3 /leave %chanpickup >> /leave ? j'connaissais pas... (/part)
" je trouve sa plus long a la compresion du code ou a la re-lecture pour gagner quelques lignes, de plus c'est souvent l'objet d'erreur !
voila :) moi je te dit continue ramyskater !! :)
pour clem54 un pickup c'est un match (en général 5 contre 5) ou ce sont des inconnues qui joue avec toi, en tappant '!add' ils s'ajoutent à la liste, et quand le compte est bon (10 joueurs en général), le bot t'envoie l'ip du serveur et le password.
http://minilien.com/?v56O4YnZXi
:-p :-p