pingyan
Messages postés29Date d'inscriptionmardi 22 janvier 2008StatutMembreDernière intervention 9 novembre 2010 29 janv. 2011 à 07:13
salut,
C'est pas tellement nouveau, le soucis dans ce code c'est que tous le monde peux utiliser les commandes. Niveau sécurité je doute.
Allez courage, ne code pas le mIRC pour évoluer dans le IRC.
bonne journée
doob666
Messages postés149Date d'inscriptionvendredi 3 novembre 2006StatutMembreDernière intervention25 janvier 2011 25 janv. 2011 à 01:14
Désolé pour le pavé mais si sa peux aider a améliorer le niveau de la source et ton niveau de scripting (en te faisant découvrir certaine chose) ça vaudras bien ce pavé =)
doob666
Messages postés149Date d'inscriptionvendredi 3 novembre 2006StatutMembreDernière intervention25 janvier 2011 25 janv. 2011 à 01:12
Bonjour,
Alors certes c'est du vu, revu, recuit, digéré, vomis etc etc etc mais bon...
Beaucoup de chose sont à revoir,
1) Aucune gestion des access, en gros n'importe qui pourras venir et se oper tranquillement sans s'authentifier à quoi que ce soit, ou alors au moins rajoute une protection pour que seul les op peuvent commander le bot (si bien sur il y as un bot gérant les access présent)
2) Regroupe tes on text et rajoute des exceptions :
on *:text:*:#:{
if $nick(#,$me,oh) {
if $strip($1) == !op {
mode # +o $nick } }
etc etc
}
3)Quitte à rajouter le mode m autant mettre les plus utilisé, et pouvoir avec la même commande mettre ou enlever le mode (selon s'il est déjà mis sur le salon ou non)
4) Pour ton on join,
a) il faut que tu vérifie si le bot est op ou non ( if $nick(#,$me,oh) ) car sinon tu va te prendre des erreur en bataille s'il n'est pas op
b) Il faudrait au minimum pouvoir désactiver cette fonction (en passant par une variable c'est très facile (tu la set a 1 quand le +v est en route, et a 0 quand elle est désactiver, comme sa avec un simple ( if %XXXXX = 1 { mode +v .... } )
5) Les "/" devant les mode ne sont pas utile donc tu peux les enlever
6) au lieu de :
/mode $chan +i $nick
c'est plutot :
mode $chan +i
7) Il faudrait mettre la possibilité d'activer ou de désactiver toute les fonction selon une liste prédéfinis de salon (exemple : salon XXX - activé // salon YYY - désactivé )
les salons pourront bien sur être modifiable sur le salon (sans avoir besoin de modifier le code)
8) Tu ne vérifie à aucun moment si la personne à déjà le mode que tu veux lui mettre, hors sa ne sert a rien que le code ce mette en route alors que la personne est déjà op (par exemple)
9) Mettre les mode +o etc etc c'est bien, mais il faudrait penser aussi à pouvoir les enlever (-o etc etc)
10) On pourrais aussi avoir la possibilité avec la même commande pouvoir mettre ou enlever le mode (bon sa c'est pour les gros fainéant comme moi qui on la flemme de retenir 1 commande pour mettre le mode et une deuxième pour l'enlever mais qui est vachement pratique)
11) Il faudrait pouvoir oper plusieurs personne à la fois et pas seulement la personne qui tape la commande (pour éviter le +oooooooo regarde l'aide sur le modespl, les boucles et les tokens)
12) Il n'y as pas de commande pour voice (le bot ne seras pas tout le temps sur le salon donc il faudrait pouvoir voicer les gens venus pendant son absence)
Je ne note pas pour le moment car ça ne vaudrais même pas le coup et de plus les notes négative n'existe pas, j'attendrais donc de futurs mise à jour et amélioration pour faire quelque chose de correct (même si c'est du vu et revu).
Cordialement.
WorldDMT
Messages postés871Date d'inscriptionjeudi 6 juillet 2006StatutMembreDernière intervention 6 janvier 20121 24 janv. 2011 à 13:15
salut
du déjà vus plus de 10000000000 de fois
en plus vous deux vous ne vérifiez pas si le bot est opérateur ou pas.
l'utilisation de $strip est très utile dans ce cas
@+
usurpateur
Messages postés9Date d'inscriptionjeudi 26 juin 2008StatutMembreDernière intervention24 janvier 2011 24 janv. 2011 à 12:29
on *:TEXT:*!*:#:{
if ( *!halfop* iswm $1 ) mode # +h $nick
if ( *!op* iswm $1 ) mode # +o $nick
if ( *!protect* iswm $1 ) mode # +a $nick
if ( *!owner* iswm $1 ) mode # +q $nick
;etc
}
29 janv. 2011 à 07:13
C'est pas tellement nouveau, le soucis dans ce code c'est que tous le monde peux utiliser les commandes. Niveau sécurité je doute.
Allez courage, ne code pas le mIRC pour évoluer dans le IRC.
bonne journée
25 janv. 2011 à 01:14
25 janv. 2011 à 01:12
Alors certes c'est du vu, revu, recuit, digéré, vomis etc etc etc mais bon...
Beaucoup de chose sont à revoir,
1) Aucune gestion des access, en gros n'importe qui pourras venir et se oper tranquillement sans s'authentifier à quoi que ce soit, ou alors au moins rajoute une protection pour que seul les op peuvent commander le bot (si bien sur il y as un bot gérant les access présent)
2) Regroupe tes on text et rajoute des exceptions :
on *:text:*:#:{
if $nick(#,$me,oh) {
if $strip($1) == !op {
mode # +o $nick } }
etc etc
}
3)Quitte à rajouter le mode m autant mettre les plus utilisé, et pouvoir avec la même commande mettre ou enlever le mode (selon s'il est déjà mis sur le salon ou non)
4) Pour ton on join,
a) il faut que tu vérifie si le bot est op ou non ( if $nick(#,$me,oh) ) car sinon tu va te prendre des erreur en bataille s'il n'est pas op
b) Il faudrait au minimum pouvoir désactiver cette fonction (en passant par une variable c'est très facile (tu la set a 1 quand le +v est en route, et a 0 quand elle est désactiver, comme sa avec un simple ( if %XXXXX = 1 { mode +v .... } )
5) Les "/" devant les mode ne sont pas utile donc tu peux les enlever
6) au lieu de :
/mode $chan +i $nick
c'est plutot :
mode $chan +i
7) Il faudrait mettre la possibilité d'activer ou de désactiver toute les fonction selon une liste prédéfinis de salon (exemple : salon XXX - activé // salon YYY - désactivé )
les salons pourront bien sur être modifiable sur le salon (sans avoir besoin de modifier le code)
8) Tu ne vérifie à aucun moment si la personne à déjà le mode que tu veux lui mettre, hors sa ne sert a rien que le code ce mette en route alors que la personne est déjà op (par exemple)
9) Mettre les mode +o etc etc c'est bien, mais il faudrait penser aussi à pouvoir les enlever (-o etc etc)
10) On pourrais aussi avoir la possibilité avec la même commande pouvoir mettre ou enlever le mode (bon sa c'est pour les gros fainéant comme moi qui on la flemme de retenir 1 commande pour mettre le mode et une deuxième pour l'enlever mais qui est vachement pratique)
11) Il faudrait pouvoir oper plusieurs personne à la fois et pas seulement la personne qui tape la commande (pour éviter le +oooooooo regarde l'aide sur le modespl, les boucles et les tokens)
12) Il n'y as pas de commande pour voice (le bot ne seras pas tout le temps sur le salon donc il faudrait pouvoir voicer les gens venus pendant son absence)
Je ne note pas pour le moment car ça ne vaudrais même pas le coup et de plus les notes négative n'existe pas, j'attendrais donc de futurs mise à jour et amélioration pour faire quelque chose de correct (même si c'est du vu et revu).
Cordialement.
24 janv. 2011 à 13:15
du déjà vus plus de 10000000000 de fois
en plus vous deux vous ne vérifiez pas si le bot est opérateur ou pas.
l'utilisation de $strip est très utile dans ce cas
@+
24 janv. 2011 à 12:29
if ( *!halfop* iswm $1 ) mode # +h $nick
if ( *!op* iswm $1 ) mode # +o $nick
if ( *!protect* iswm $1 ) mode # +a $nick
if ( *!owner* iswm $1 ) mode # +q $nick
;etc
}
Mais pas grand intérêt ni originalité.