S'ENREGISTRER AUPRÈS DE SON BOT

Commentaire de PaDa le 23/07/2008 00:26:52

Regarde les critiques sur plus de la moitié des sources récentes. Rien qu'à regarder ta source, ça me pique les yeux tellement il y a de répétitions et de choses inutiles. Comme ce n'est pas non plus minable je te mets 3. Bonne continuation...

Commentaire de uaip le 23/07/2008 10:25:24

Comme PaDa l'a dit, ta source n'est pas "minable" (mieux que tes anciennes, donc il y a une progression) cependant le code pète et se répète. Essaye de n'utiliser qu'un événement "TEXT".
Et un détail aussi :
- $null signifie "rien", donc "if ($2 == $null)" devient "if (!$2)"
- les commandes n'ont pas de / devant (tes "/mode" deviennent donc "mode")
- n'oublie pas les parenthèses pour les if -> if ((condition_1) && (condition_2))
- pas besoin des accolades lorsqu'il n'y a qu'une seule commande -> if (condition) commande

Maintenant si tu veux passer à un niveau supérieur, remplace les variables et le fichier.txt par des hashtables :)

Commentaire de WorldDMT le 23/07/2008 11:30:52

PADA si il met "0" donc $2 != $null
!$2 veu dire ya pas de $2 alors ça revien $null

donc uaip a raison "if ($2 == $null)" devient "if (!$2)"

Commentaire de uaip le 23/07/2008 15:17:17

Exact, j'avais conscience que "!$2" n'étais pas forcément égal à "$2 == $null" quand j'ai écrit le topic (mais alors pour quoi avoir posté ? je ne sais pas...)

//var %a 0 | echo -a $iif(%a == $null,nul,pas nul) retourne 'pas nul'
//var %a 0 | echo -a $iif(!%a,nul,pas nul) retourne 'nul'

Admirez le beau paradoxe.

Je ne sais pas trop comment ça marche en mIRC (étant donné que les variables ne sont pas vraiment des variables en tant que tel) mais en C par exemple, tester si une variable = NULL ne vérifie pas si son contenu est vide mais si cette variable existe.

Donc :

//var %a 0 | echo -a $iif(%a == $null,nul,pas nul) retourne 'pas nul'
;Logique, car la variable %a existe

//var %a 0 | echo -a $iif(!%a,nul,pas nul) retourne 'nul'
;le '!' vérifie si %a possède un contenu, et comme $false = 0 (en binaire) là, comme le dit PaDa, ça retourne 'nul'.

En fait, jouer avec des variables stokant 1 ou 0, c'est un peu jouer avec des booléens.

Mais pour en revenir au problème, dans son cas je pense que c'est peu utile...

(PS: mon discours ne s'appuie que sur mes expériences personnelles, donc tout n'est peut-être pas juste).

Commentaire de PaDa le 23/07/2008 15:26:51

Sisi je pense que tu vois juste. mIRC est un langage de script sans typage, et pour des raisons de simplicité 0, $null et $false sont traités pareils dans les structures conditionnelles. Inversement, 1, "une valeur quelconque" ou $true seront traités pareil, c'est à dire seront "VRAI".

Commentaire de WorldDMT le 23/07/2008 15:35:30

ah cool j'ai apris quelque chose merci :)

Commentaire de PaDa le 23/07/2008 15:40:49

Ok merci ;)

Commentaire de WorldDMT le 23/07/2008 21:15:24

t'a plutot gaté :/
au moin au debut ça pouvait marché mais là c'est grave :/

tu met entete "On *:text: register*:?:{" ce qui est pour PV rien que

et les elseif n'ont rien a y voir :/

faut mettre au debut

On *:text:*:?:{
  if ($1 == register) {
   if ($2 == $null) && ...

et c'est pas tout les commandes en PV
alors il va falloir aussi

On *:text:*:#:{

stp au lieu de mettre 5/10 sur mon tutorial sans bien lire essaye de faire ce que j'ai expliqué dedans au moins..

Commentaire de WorldDMT le 24/07/2008 09:50:04

salut

minebre, ne regarde pas les autres phrases restont sur ton code
ce que tu a fais ne marche pas selon ta derniere mise a jour
tout les elseif son apparament bien mais l'entete doit etre comme ça

On *:text:*:#:{
  if ($1 == !v) {
    If (%ol [ $+ [ $address($nick,2) ] ] == OFF) { .notice $nick [ERREUR] <~~> Vous devez être identifier pour utiliser cette commande ( /msg $me login MotDePasse ) | /return }
    If (%ol [ $+ [ $address($nick,2) ] ] == ON) && (%axx [ $+ [ $address($nick,2) ] ] > 0) && ($2 == $null) { /mode $chan +v $nick }
    If (%ol [ $+ [ $address($nick,2) ] ] == ON) && (%axx [ $+ [ $address($nick,2) ] ] > 4) { /mode $chan +v $2 }
  }
  elseif ($1 == !dv) {
    If (%ol [ $+ [ $address($nick,2) ] ] == OFF) ...
    ...
    ect...
  }
}

tu fais pareil pour PV

au lieu de 3 on text fais 1 seule avec joker

On *:text:*:?:{
  if ($1 == register) {
    ....
  }
  elseif ($1 == login) {
    ...
  }
}

Commentaire de WorldDMT le 24/07/2008 10:54:26

je ne sais pas si c'est moi qui m'exprime mal :/

regarde l'entete que tu a fais

On *:text: register*:?:{ <== l'entete si prend rien que pour "register"
  if (!$2) && ($1 == register) { ...
...
}
elseif ($1 == login) {

quand il y a des if ($1 == ...) il faut que l'entete soit avec un joker " * "

On *:text:*:?:{ <== TU REMARQUE ICI L'ENTETE CONTIEN UN JOKER " * "
  if ($1 == register) { <=== ICI LE JOKER " * " EST SOUS CONDITION QU'IL SOIT "REGISTER"
    if ($2 == $null) && ($1 == register) {
    ...
    ect..
}


et tu a tjr (%axx [ $+ [ $address($nick,2) ] ] < 10) c'est quoi %axx les niveau des users?

Commentaire de uaip le 24/07/2008 12:15:49

Simple parenthèse...

"Ok, je continue a dire que l'mirc sa donneré mal a la tete a un cachet d'aspirine ..."

Personnellement, je n'ai jamais rien vu de plus facile et simpliste que le mIRC scripting.

Commentaire de uaip le 24/07/2008 13:00:47

Le html ca n'a rien à voir ! C'est juste une syntaxe. Ne mélange pas tout.

Commentaire de PaDa le 24/07/2008 13:08:26

Le html fait juste de l'affichage, c'est un langage de mise en forme, pas un langage de traitement / "intelligent"

Commentaire de minerbe le 24/07/2008 13:11:06

Ben en tout cas merci a tout ceux qui on participé pour m'aider =]

Commentaire de WorldDMT le 24/07/2008 13:18:47

dans bientot 30 cmmentaires et le codes ne fonctionne tjr pas :/

Commentaire de WorldDMT le 24/07/2008 13:53:59

ah wé t'a testé??
je ne pense pas
déjà le %axx n'a pas d'existance
et le syntaxe est mal ecrit
quesqui marche dans ton code tu peu m'expliquée stp?

Commentaire de minerbe le 24/07/2008 15:43:56

Bonjour isoka tu entend quoi par

On *:text: !v*:#:{
If (%ol [ $+ [ $address($nick,2) ] ] == OFF) { .notice $nick [ERREUR] <~~> Vous devez être identifier pour utiliser cette commande ( /msg $me login MotDePasse ) | /return }
If (%ol [ $+ [ $address($nick,2) ] ] == ON) && (%axx [ $+ [ $address($nick,2) ] ] > 0) && ($2 == $null) { /mode $chan +v $nick }
If (%ol [ $+ [ $address($nick,2) ] ] == ON) && (%axx [ $+ [ $address($nick,2) ] ] > 4)  /mode $chan +v $2 }
}
elseif ($1 == !dv) {
...
tu voudrais dire que entre $address($nick,2) ] ] > 4)  /mode $chan +v $2 } et elseif ($1 == !dv) {  je dois pas fermer O_o ?

Commentaire de ISoKa le 24/07/2008 16:21:12

je ve dire que :
-si tu regroupe tes on text enleve des conditions imcompatible : On *:text: !v*:#:{ et elseif ($1 == !dv) {
- ensuite tu veux regrouper tes on text mais tu le referme :
On *:text: !v*:#:{
If (%ol [ $+ [ $address($nick,2) ] ] == OFF) { .notice $nick [ERREUR] <~~> Vous devez être identifier pour utiliser cette commande ( /msg $me login MotDePasse ) | /return }
If (%ol [ $+ [ $address($nick,2) ] ] == ON) && (%axx [ $+ [ $address($nick,2) ] ] > 0) && ($2 == $null) { /mode $chan +v $nick }
If (%ol [ $+ [ $address($nick,2) ] ] == ON) && (%axx [ $+ [ $address($nick,2) ] ] > 4)  /mode $chan +v $2 }
}
quand tu fais ca tu referme carrement l'evenement donc ca va pas du tout.
essais de revoir ca

Commentaire de ISoKa le 24/07/2008 16:27:56

en plus, ton code est long pour rien, y a plein de regroupements possible, le premier qui me saute aux yeux:
# elseif ($1 == !h) {
# If (%ol [ $+ [ $address($nick,2) ] ] == OFF) { .notice $nick [ERREUR] <~~> Vous devez être identifier pour utiliser cette commande ( /msg $me login MotDePasse ) | /return }
# If (%ol [ $+ [ $address($nick,2) ] ] == ON) && (%axx [ $+ [ $address($nick,2) ] ] > 3) && ($2 == $null) { /mode $chan +h $nick }
# If (%ol [ $+ [ $address($nick,2) ] ] == ON) && (%axx [ $+ [ $address($nick,2) ] ] > 4) { /mode $chan +h $2 }
# }
# elseif ($1 == !dh) {
# If (%ol [ $+ [ $address($nick,2) ] ] == OFF) { .notice $nick [ERREUR] <~~> Vous devez être identifier pour utiliser cette commande ( /msg $me login MotDePasse ) | /return }
# If (%ol [ $+ [ $address($nick,2) ] ] == ON) && (%axx [ $+ [ $address($nick,2) ] ] > 3) && ($2 == $null) { /mode $chan -h $nick }
# If (%ol [ $+ [ $address($nick,2) ] ] == ON) && (%axx [ $+ [ $address($nick,2) ] ] > 4) { /mode $chan -h $2 }
# }

devient
elseif ($1 == !dh) || ($1 == !h) {
If (%ol [ $+ [ $address($nick,2) ] ] == OFF) { .notice $nick [ERREUR] <~~> Vous devez être identifier pour utiliser cette commande ( /msg $me login MotDePasse ) | /return }
If (%ol [ $+ [ $address($nick,2) ] ] == ON) && (%axx [ $+ [ $address($nick,2) ] ] > 3) && ($2 == $null) { /mode $chan $replace($right($1,-1),d,-) $nick }
If (%ol [ $+ [ $address($nick,2) ] ] == ON) && (%axx [ $+ [ $address($nick,2) ] ] > 4) { /mode $chan -h $2 }
}
en faisant ca à chak fois tu réduis ton code de moitié ce qui serai sympa quand meme, et encore, je suis sur qu'on peu factoriser bcp plus ce type d'action avec par exemple des regex et regsub

Commentaire de minerbe le 24/07/2008 16:31:15

au lieu de On *:text: !v*:#:{ faut mettre On *:text:*:#:{

donc au lieu de

On *:text: !v*:#:{
If (%ol [ $+ [ $address....

tu met

On *:text:*:#:{
  if ($1 == !v) {
  If (%ol [ $+ [ $address...

Compris =]

Commentaire de minerbe le 24/07/2008 16:33:30

si tu parle de sa

- $null signifie "rien", donc "if ($2 == $null)" devient "if (!$2)"


si je l'ai prix en compte mais on ma dit de rechanger =/

Commentaire de minerbe le 24/07/2008 16:35:57

enfin je suis même pas sur de savoir exactement ce que c'est il faudra que je regarde.

Maintenant si y a quelque chose que j'ai oublier de faire dit le moi j'arrenge sa =]

Commentaire de minerbe le 24/07/2008 16:38:30

ok toute façon tout est dit. J'ai encore pas mal de chose a apprendre avant de venir a ton niveau ou au niveau de wims, isoka, worldDMT ...

Voila bon scripting a tous

Commentaire de minerbe le 24/07/2008 18:22:26

Euh ben pour l'instant sa me le cale pas donc voila =P lol
pour le On *:text:*:?:{
          if ($1 == register) {
je voulais le faire mais j'ai oublier

Merci beaucoup =)

Commentaire de minerbe le 24/07/2008 18:36:52

ouai c'est vrai ... ou sinon y doit avoir un moyen pour que se soit que les admin owner du salon qui y touche on pourrait faire un truc du genre:

if (!$istok(%protect,$nick,32) && !$istok(%owner,$nick,32) && !$istok(%master,$nick,32)) { notice $nick Accès refusé. | return }

Non ? O_o

Commentaire de minerbe le 24/07/2008 18:44:19

Mais laisse mes parenthaises elle t'on rien fait mdrr.

bon ben j'attend un avis pour mette

if (!$istok(%protect,$nick,32) && !$istok(%owner,$nick,32) && !$istok(%master,$nick,32)) { notice $nick Accès refusé. | halt }

comme sécurité dans ma commande ...

Commentaire de minerbe le 24/07/2008 18:51:27

c'été qu'une suggestion hein O_O

Commentaire de WorldDMT le 24/07/2008 19:37:28

plus de 50 commantaires sur un code et on se trouve sans les dernier acolades "}" :/

au lieu de tout ces lignes tu pouvais faire un fichier.ini contien les host autorisé
et un autre fichier.txt il y a les commandes en ligne avec separateurs
qui sera executé avec $gettok

soi plus pro au lieu des fichier tu fais en hach tables

bonne chance pour tes futur codes @+

Commentaire de devorZ le 27/07/2008 04:38:45

retirer cette sources elle ne le fait pas avancer et elle est d'un mauvaise exemple pour les autres..

Commentaire de WorldDMT le 27/07/2008 09:54:57

oui UAIP l'acolade au niveau de la ligne 20

mais le veritable probleme que celui qui utilise ce code, donne le pouvoir a n'importe qui, je dis bien N'IMPORTE QUI d'avoir possibilité de faire un masse kick et foutre le bordel dans un chan et finalement kické le bot lui meme avec son owner :/

moi je trouve que 3/10 c'est trop...