MESSAGE D'ACCUEIL + RÉACTION À DES COMMANDES PERSONNALISABLE

Commentaire de RCA ArKanis le 11/08/2007 18:57:33

aie aie, très très mal codé !

Beaucoup de défauts :
- ON TEXT non regroupés
- utilisation de variables globales
- limité à 5 messages, pourquoi pas plus ?
- limité à un seul salon
- pas d'anti-flood
- fautes de grammaire (Met => Mets)
- Tu aurais pu faire en sorte qu'on puisse utiliser $nick ou $chan ainsi que d'autres identifieurs ($time, $date, ...)
- Si on tape "", vu la valeur par défaut des variables, on va avoir des notices envoyées ... utilise unset %msgcmg? sur le ON LOAD


if ($did(12).state == 1) devient : if ($did(12).state)
vu que $did().state renvoie une valeur booléenne

Pour regrouper tes ON TEXT, tu fais :

ON *:TEXT:*:%channelcmd:{
  var %i = 1
  while %i <= 7 {
    if (%msgcmd [ $+ [ %i ] ]) {
      .notice $nick $v1
      break
    }
    inc %i
  }
}
ça évite tes 21 lignes ...


Conclusion :
Assez inutile sous cette forme car trop peu d'options.
Très mal codé et pas du tout optimisé

Commentaire de RCA ArKanis le 11/08/2007 20:35:50

tant que tu es prêt à modifier ça va encore !
y'a des gens qui osent laisser des codes (encore pire que celui-ci !!!) tels quels

on t'aidera bien volontiers et tu pourras ainsi progresser plus rapidement :)

Commentaire de RCA ArKanis le 12/08/2007 02:11:06

Effectivement, j'ai pas pris le temps de relire

Voici la correction :
ON *:TEXT:*:%channelcmd:{
  var %i = 1
  while %i <= 7 {
    if ((%cmd [ $+ [ %i ] ] == $1) && (%msgcmd [ $+ [ %i ] ])) {
      .notice $nick $v1
      break
    }
    inc %i
  }
}

Commentaire de KoMa94 le 12/08/2007 04:28:36

encore merci pour votre aide c'est cool.
donc j'ai été voir un peu comment on utilisais la boucle while effectivement c'est très pratique. Arkanis j'ai donc changé mon code avec le code que tu m'as donné mais je l'ai pas copier bêtement j'ai biensur essayé de comprendre.
ce qui ma permi aussi de faire une autre boucle while pour les %msgaccueil que voici :

on 1:join:%channelcmd:{
  if (%checkaccueil = ON) {
    var %j = 1
    while %j <= 5 {
      if (%msgaccueil [ $+ [ %j ] ]) {
      .notice $nick $v1
    }
    inc %j
  }
}
}
Sinon Wims pour les did je ferais ça quand j'aurais du temps car la il est tard lol mais d'abord j'aimerais un peu plus comprendre les codes que j'ai modifié car j'ai compris leur fonctionnement mais je me pose encore quelques questions.
c'est sur cette ligne que j'ai un peu de mal : if ((%cmd [ $+ [ %i ] ] == $1) && (%msgcmd [ $+ [ %i ] ])) {
si quelqu'un pouvais ma la commenter pour que je comprenne comment on la lit et surtout d'ou sors ce $+
et pour finir pour ce soir dans ( .notice $nick $v1 )  le $v1 signifie quoi? (enfin j'ai bien remarqué que ça me renvoi la valeur de %msgcmd* mais j'ai pas compris comment)
je sais pas si je me suis bien exprimé enfin bref je vous remerci d'avance pour le temps que vous me consacré.

Commentaire de RCA ArKanis le 12/08/2007 11:51:17

tu as des problèmes de {} KoMa :
on *:join:%channelcmd:{
  if (%checkaccueil = ON) {
    var %j = 1
    while %j <= 5 {
      if (%msgaccueil [ $+ [ %j ] ]) .notice $nick $v1
      inc %j
    }
  }
}
faut que le /inc soit à l'intérieur de la while, sinon c'est une boucle infinie

comme l'a dit Wims, si tu peux, essaye de fixer la variable %checkaccueil soit à 1, soit à 0
tu aurais donc if (%checkaccueil) au lieu de if (%checkaccueil == ON)

Commentaire de KoMa94 le 12/08/2007 14:41:07

a oui aussi wims tu as dit plus haut que les on dialog n'était pas regroupé, j'ai essayé de voir si je pouvais le faire mais vu que le event n'est pas le meme je comprend pas comme je peux faire.
je sais je suis nul mais pour le moment ça reste compliqué tout ça pour moi xD. merci

Commentaire de RCA ArKanis le 12/08/2007 19:30:37

j'ai raison pour ma boucle, c'est bien $v1 qu'il faut utiliser ^^
en fait, j'ai l'impression que $v1 et $v2 dépendent du DERNIER if vérifié.

si on a par exemple :
if ((1) && (2 < 3)) echo -a $v1 - $v2
on aura "2 - 3" étant donné que le dernier if vérifié est 2 < 3

Commentaire de KoMa94 le 12/08/2007 20:59:40

effectivement avec $v1 mon script fonctionne très bien.
donc dans ton exemple :
if ((1) && (2 < 3)) echo -a $v1 - $v2
2 = $v1 et 3 = $v2 mais 1 = ??? lol je commence à comprendre mais ya toujours un petit mais.
sinon wims je me pencherais sur ce on dialog dans la semaine ;)

encore merci à vous 2 de vos explication clair =)

Commentaire de ISoKa le 13/08/2007 00:43:30

Bonjour,
en parlant de while et de reduction de code si je trompe pas :
#
# on *:dialog:KoMa:sclick:*: {
# if ( $did == 13 ) {
# set %msgaccueil1 $did(7)
# set %msgaccueil2 $did(8)
# set %msgaccueil3 $did(9)
# set %msgaccueil4 $did(10)
# set %msgaccueil5 $did(11)
# set %channelcmd $did(16)
# set %cmd1 $did(26)
# set %cmd2 $did(27)
# set %cmd3 $did(28)
# set %cmd4 $did(29)
# set %cmd5 $did(30)
# set %cmd6 $did(31)
# set %cmd7 $did(32)
# set %msgcmd1 $did(40)
# set %msgcmd2 $did(41)
# set %msgcmd3 $did(42)
# set %msgcmd4 $did(43)
# set %msgcmd5 $did(44)
# set %msgcmd6 $did(45)
# set %msgcmd7 $did(46)
# }
#
# if ( $did == 12 ) {
# if ($did(12).state) { did -n KoMa 7,8,9,10,11 | set %checkaccueil 1 | echo -a 7¤¤ 4Messages d'accueil 10activés }
# else { did -m KoMa 7,8,9,10,11 | set %checkaccueil 0 | echo -a 7¤¤ 4Messages d'accueil 10déactivés }
# }
# }
#
# on *:dialog:KoMa:*:*:{
# if $devent == init {
# if (%checkaccueil) { did -c KoMa 12 | did -n KoMa 7,8,9,10,11 }
# else { did -m KoMa 7,8,9,10,11 }
# did -a KoMa 7 %msgaccueil1
# did -a KoMa 8 %msgaccueil2
# did -a KoMa 9 %msgaccueil3
# did -a KoMa 10 %msgaccueil4
# did -a KoMa 11 %msgaccueil5
# did -a KoMa 16 %channelcmd
# did -a KoMa 26 %cmd1
# did -a KoMa 27 %cmd2
# did -a KoMa 28 %cmd3
# did -a KoMa 29 %cmd4
# did -a KoMa 30 %cmd5
# did -a KoMa 31 %cmd6
# did -a KoMa 32 %cmd7
# did -a KoMa 40 %msgcmd1
# did -a KoMa 41 %msgcmd2
# did -a KoMa 42 %msgcmd3
# did -a KoMa 43 %msgcmd4
# did -a KoMa 44 %msgcmd5
# did -a KoMa 45 %msgcmd6
# did -a KoMa 46 %msgcmd7
# }
# }
pourrai devenir ca si je m'a pas gourré :) :
on *:dialog:KoMa:sclick:*: {
if ( $did == 13 ) {
  set %channelcmd $did(16)
  var %i 1
  while (5 =< %i) { set %msgacceuil $+ %i  $did($calc(%i + 6)) | inc %i }
  var %i 1
  while (7 =< %i) { set %cmd $+ %i  $did($calc(%i + 25)) | set %msgcmd $+ %i  $did($calc(%i + 39)) | inc %i }
}
if ( $did == 12 ) {
if ($did(12).state) { did -n KoMa 7,8,9,10,11 | set %checkaccueil 1 | echo -a 7¤¤ 4Messages d'accueil 10activés }
else { did -m KoMa 7,8,9,10,11 | set %checkaccueil 0 | echo -a 7¤¤ 4Messages d'accueil 10déactivés }
}
}

on *:dialog:KoMa:*:*:{
if $devent == init {
if (%checkaccueil) { did -c KoMa 12 | did -n KoMa 7,8,9,10,11 }
else { did -m KoMa 7,8,9,10,11 }
var %i 1
while (5 =< %i) { did -a koma $calc(%i + 6) [ %msgacceuil $+ [ %i ] ] | inc %i }
did -a KoMa 16 %channelcmd
var %i 1
while (7 =< %i) { did -a koma $calc(25 + %i) [ %cmd $+ [ %i ] ] | did -a koma $calc(%i + 39) [ %msgcmd $+ [ %i ] ] | inc %i }
}
}

Commentaire de ISoKa le 13/08/2007 01:33:49

Ha! heu peut etre j'avou que j'ai pas testé le code que j'ai ecri donc fodrai que je verif ca. merci :)

Commentaire de KoMa94 le 14/08/2007 13:02:57

ah oui je vien de remarquer quelque chose aussi, quand j'ai changé les édit et que je clique sur validé ça prend bien en compte mes changements. par contre des que je reviens deçu tous mes édits ce sont effacé et on les chiffres comme je vous en ai parlé au dessus. je comprend vraiment pas pouquoi ça fait ça :s

Commentaire de KoMa94 le 14/08/2007 16:52:16

bon ba effectivement c'était ça, mon script marche nikel maintenant :D
c'est vrai que maintenant ça fait plus propre et j'ai gagné beaucoup de ligne.
sinon comme ça m'amuse pas mal de codé un peu, je vais encore améliorer mon script et y ajouter meme d'autre fonction au fur et à mesure. Pour l'instant j'ai pas encore d'idée car je cherchais a faire juste quelques chose dont j'avais besoin mais ça viendra.
donc pour le moment je vais essayé d'amélioré en terme d'option sur ce qu'on m'a dit plus haut (Arkanis) :

- utilisation de variables globales : que voulais tu dire par la car je comprend pas trop lol ^^
- limité à 5 messages, pourquoi pas plus ? je suis d'accord mais je suis donc obligé de rajouté des édits hein ?
- limité à un seul salon : ça effectivement ça pourais être bien mais vu mon utilité et mon niveau j'ai limité à 1 salon lol.
- pas d'anti-flood : effectivement yen a pas mais la va faloir que je me fasse mal pour codé ça ^^
- Tu aurais pu faire en sorte qu'on puisse utiliser $nick ou $chan ainsi que d'autres identifieurs ($time, $date, ...) : oui ça serait très utile d'ailleur j'aurais bien aimé que ça puisse ce faire mais quand je tape ces commandes dans un édit elles sont pas reconnu et je sais pas comment faire et puis je n'y ai pas trop réfléchi encore.

ah oui une derniere chose, quand je tape dans un édit je suis limité en nombre de caractère car quand j'arrive au bout de la ligne je ne peux plus continué, j'aurais bien aimé une solution.

Sinon merci à vous 3 pour vos aides, conseils et explications.

Commentaire de RCA ArKanis le 14/08/2007 16:59:58

- variables globales : ouvre ton remote, et va dans la section Variables. T'auras une grande liste de toutes les variables que tu utilises. La plupart d'entre elles sont temporaires et doivent donc être effacées.
Tu devrais inscrire les données les plus importantes dans un fichier .ini (/writeini et $readini)

- 5 messages : non, pas si tu de débrouilles pour en avoir une infinité (while)

- un salon : en utilisant des .ini, il aurait été très simple d'avoir autant de salons que tu voulais s'il se construit comme ceci :
[#salon1]
!commande=que dire ...
!site=l'url ...
etc
[#salon2]
...

- Anti-Flood : variables/timers : set -u3 %anti.flood 1, + d'autres qui comportent le nick dans le nom de la variable, par exemple

- En utilisant $eval(...,2), je pense que ça devrait fonctionner
sinon utilise 'nick', 'chan' et tu fais un mega $replace

une solution possible pour ton problème : utiliser mon GUI xD

Commentaire de ISoKa le 14/08/2007 18:51:48

Pour ne pas etre limiter par le nombre de salon tu peux tout à fait faire une listebox avec les noms de chans et quand tu selectionne un nom de chan ca te met dans tes edits tes differents trucs. aprés suffit de faire des variables qui prennent en compte les noms des salons genre %cmd $+ [ salon ] $+ [ %i ] . fin c'est un exemple ca car à mon avis si tu viens à gerer bcp de variables vaut mieu mettre tout ca en htables ca te pourira moins tes var. pour rajouter des edits tu peux faire un truc du meme principe avec des listbox ou combobox pour ne pas etre limiter sur le volume.

Commentaire de RCA ArKanis le 14/08/2007 19:50:22

tu plaisantes ? :(
il ne reste que le popup et le multiedit, tout le reste est opérationnel !

il ne fait pas bugguer le code et tu peux le trouver en téléchargeant ma météo xD