WANTISPAM V3.0

Commentaire de PaDa le 14/05/2008 21:11:49

"LE MEILLEUR ANTISPAM DE LA SAISON PAR WORLDDMT"
Tu fais aussi dans l'humour ?

C'est un petit remote intéressant, surtout pour un débutant puisqu'il est lisible, mais ce n'est pas parce que tu as pondu une petite regex assez basique que tu peux dire ce genre de chose..

Bonne continuation

Commentaire de WorldDMT le 14/05/2008 21:29:27

salut Arkanis

tu a vu ça ou??? "Gaffe sur l'ortho (d'avoir installé, la liste des nicks protégés, d'avoir utilisé) progégé ? loooooooooool :pp

et pour le

if (($nick isop $chan) || ($readini(protect_user.ini,prot,$nick)) || ($wildtok($3-,#*,1,32) == $chan)) return

pourquoi pas aussi ça

if (($2 isop $1) || ($readini(protect_user.ini,prot,$nick)) || ($wildtok($3-,#*,1,32) == $1) || (!$regex($3-,/(\#\w+\b|(?:[-_\.]\w+)?@\w+([-_\.]?\w+)?\.\w+\b|(www\.|http:\/\/)\w+([-_\.]?\w+)?\.\w{2,4})/i))) return | ban -k $1-2 2 4Spam Protection 1by WorldDMT 14 trouve moi en pv en cas d'erreur  

;) c'est cool nn :p

Commentaire de RCA ArKanis le 14/05/2008 21:56:49

le but, c'est de ne pas avoir de | (c'est super moche et ça ralenti inutilement le code !!!)
ne pas non plus abuser des lignes super longues
le regex est déjà assez long en lui même pour que tu puisses lui attribuer une ligne à lui tout seul

les fautes d'orthographe que je t'ai données sont bien entendu corrigées ... à toi de les trouver dans ta description qui fait partie de la source.

le unload peut lui aussi tenir en une ligne :
on *:unload:echo * Merci d'avoir utiliser WorldDMT Anti Spam (Wantispam V3.0)
(en corrigeant cette horrible faute au passage)

tu ne perds pas tellement en clarté en plus.

Commentaire de CsDarkman le 14/05/2008 22:28:15

bein ta corrigé le max d'erreur
ta mis tt sous l alias
ta version comém a encore besoin de mise a jour
Pour ta action
ban -k $1-2 2 4Spam Protection 1by WorldDMT 14 trouve moi en pv en cas d'erreur  
utilise aussi un halt apré l'action
Pour les multi version essaye de faire des mise a jour o méme code
Pas créer chaque jour une nouvelle v$
:)

Commentaire de PaDa le 14/05/2008 23:01:20

@WorldDMT: non, je n'ai pas testé : je n'ai pas de doute sur le fait que ton code s'exécute bien, c'est ce que tu as mis en commentaire, juste au dessus du mien, qui ne pouvait pas marcher, relis-toi ;o)

Commentaire de PaDa le 15/05/2008 00:43:02

Rah...

if (($2 isop $1) || ($readini(protect_user.ini,prot,$nick)) || ($wildtok($3-,#*,1,32) == $1) || (!$regex($3-,/(\#\w+\b|(?:[-_\.]\w+)?@\w+([-_\.]?\w+)?\.\w+\b|(www\.|http:\/\/)\w+([-_\.]?\w+)?\.\w{2,4})/i))) return | ban -k $1-2 2 4Spam Protection 1by WorldDMT 14 trouve moi en pv en cas d'erreur  

Voici le code que tu as écrit dans un de tes commentaires : là-dedans, le "ban -k" ne peut pas être exécuté, puisqu'il est mis après un "return" qui arrête l'exécution du code.

Ouvre ton éditeur de remotes.
Tape ceci :
alias blah {
  echo -s avant
  if ($true) return | echo -s return!
  echo -s après
}

Et amuse-toi :-)

Ensuite, je ne tiens pas à être méchant, mais je n'ai pas vraiment besoin de tester ton code pour savoir ce qu'il fait, s'il marche en gros, et s'il m'intéresse. Comme je l'ai dit plus haut, je le trouve intéressant pour l'aspect pédagogique. Il est relativement original, et plutôt bien écrit pour un débutant qui passerait, et pourrait s'en inspirer. Les autres t'ont déjà fait des remarques constructives sur ta manière de coder, tu vas pouvoir progresser avec cela, et petit à petit tu pourras arriver à une source vraiment sympa ! Mes commentaires ne sont pas méchants ou négatifs, je pense sincèrement que ta source est intéressante, je faisais juste remarquer que l'un de tes codes DANS TES COMMENTAIRES était un peu mal foutu.

Et puis, pour mon premier comment, je te taquine juste, désolé si ça t'a été désagréable, ça n'était pas le but.

Bonne continuation !

Commentaire de WorldDMT le 15/05/2008 10:44:47

merci PADA
mais je voulais t'eclairsir quelque choses :)
pour que tu sois mechant ou pas je ne te juge pas et je prefaire un commentaire mechant qu'une flatterie :)

en plus pour

if (($2 isop $1) || ($readini(protect_user.ini,prot,$nick)) || ($wildtok($3-,#*,1,32) == $1) || (!$regex($3-,/(\#\w+\b|(?:[-_\.]\w+)?@\w+([-_\.]?\w+)?\.\w+\b|(www\.|http:\/\/)\w+([-_\.]?\w+)?\.\w{2,4})/i))) return | ban -k $1-2 2 4Spam Protection 1by WorldDMT 14 trouve moi en pv en cas d'erreur  


dsl PADA mais j'ai bien dis que j'ai testé alors si ça ne ne s'execute pas qui m'a kick alors ??? regarde le "|" :) ce qui veu dire moi le debutan je te dis que si le nick est op ou est ds la liste proteger ou a ecrit le nom du chan ou n'est pas ds le regex return mais si nn ça serait ban -k ;)

et j'ai tester et je me suis touver ak un kick ban :D mais t'a raison apres le return ya pas d'action mais si c'etait sans le "|" :)

Commentaire de PaDa le 15/05/2008 13:27:49

Ok, (au temps|autant) pour moi, mon cerveau était éteint hier, désolé. Comme quoi je suis pas encore en osmose avec mIRC..




(saloperie de mIRC)

Commentaire de PaDa le 15/05/2008 15:12:08

C'est surtout que je fais plus de mIRC assez souvent, et si je ne dis pas de bêtises, dans d'autres langages, la condition porte sur la ligne entière quand tu mets un séparateur type ";" (en php, perl, ruby par exemple). Enfin, ce serait à vérifier !

Pour la regex, tu as repris telle quelle la regex de TropNul sur ta source précédente ? A vue de nez elle doit pouvoir se rendre un peu plus lisible en la construisant avec des variables, puisque le même motif se répète plusieurs fois.

Commentaire de PaDa le 15/05/2008 20:05:14

Troll detected :-)

L'important quand tu fais du dev à un niveau un peu sérieux, c'est souvent (en entreprise notamment) que ton code 1/ remplisse la fonction pour lequel il est écrit avec un certain nombre de contraintes (performance, modularité, montée en charge, interfaçage avec d'autres systèmes), 2/ soit lisible/maintenable/possible à reprendre par un autre développeur.

Un petit exemple pour la route ?
Voici du Perl :
perl -le '$|=$_=1; (1x$_)!~/^(11+)\1+$/o && print while ++$_'

C'est court, c'est sûr. Mais je doute que tu comprennes instantannément à quoi ça sert (le premier qui trouve sans le lancer a droit à mon admiration éternelle :-)) Et je doute également qu'un développeur "normal" arrive à l'adapter à ses besoins facilement...

My 2 cents...

Commentaire de PaDa le 15/05/2008 21:11:04

Ok pour la regex :) Elle est assez bien foutue pour détecter un maximum de choses sans faire trop de faux-positifs, je pense que si tu te plonges dedans tu arriveras à la rendre plus lisible mais pas l'améliorer de beaucoup. En tout cas ton amélioration me parait intéressante.

Et je ne cherche pas à fusionner d'autres langages avec mIRC (bien que ce soit possible), je donnais juste un exemple pour illustrer mon propos, à savoir : "coder en un minimum de lignes ne me parait pas être un objectif raisonnable" :)

Bonne continuation, à bientôt :)