RÉPONDEUR + ANTI PV

Commentaire de RCA ArKanis le 01/05/2007 10:55:26

C'est du Déjà Vu, et le code est assez moche (pas optimisé)
Rien qu'en dessous de ton code, dans la case "Sources qui peuvent êreen rapport avec celle-ci", tu n'as que des anti-pv, et y'en a beaucoup plus ...
Utilise else quand c'est possible et évite les variables globales. Prends des .ini, c'est largement mieux.


  if ($did(3).state == 1) { did -e optionpv 4 | set %pvmessage on }
  if ($did(3).state == 0) { did -b optionpv 4 | set %pvmessage off }
  if ($did(6).state == 1) { set %pvferme on }
  if ($did(6).state == 0) { set %pvferme off }

devient :
if ($did(3).state) { did -e opionpv4 | set %pvmessage 1 }
else { did -b optionpv 4 | set %pvmessage 0 }
if ($did(6).state) set %pvferme 1
else set %pvferme 0

Les 0 et 1 sont booléens, tu gagnes donc du temps (à l'écriture) si tu veux faire un if
Les { } ne sont pas nécessaires si tu n'as qu'une action, le tout en une ligne
Les / ne sont JAMAIS nécessaires

Ensuite,
  if (%pvmessage == on) { /msg $nick %pvmsg }
  if (%pvferme == on) { /notice $nick 14Je n'accèpte pas de PV désolé | close -m $nick }
devient :
if (%pvmessage) msg $nick %pvmsg
if (%pvferme) { notice $nick 14Je n'accèpte pas de PV désolé | close -m $nick }

Et enfin, regroupe tes ON DIALOG
on *:dialog:nom:*:*:{
  if ($devent == init) {
    ...
  }
  elseif ($devent == sclick) {
    if ($did == 1) ...
    elseif ($did == 2) ...
    etc ...
  }
}

Corrige déjà tout ça :)

Commentaire de Showcase le 01/05/2007 18:41:16

  if ($did(3).state == 1) { did -e optionpv 4 | set %pvmessage on }
  if ($did(3).state == 0) { did -b optionpv 4 | set %pvmessage off }
  if ($did(6).state == 1) { set %pvferme on }
  if ($did(6).state == 0) { set %pvferme off }

devient

  if ($did(3).state == 1) { did -e optionpv 4 | set %pvmessage on }
  else { did -b optionpv 4 | set %pvmessage off }
  if ($did(6).state == 1) { set %pvferme on }
  else { set %pvferme off }

ça marche mieux comme ça :)

Commentaire de Panthouffle le 01/05/2007 21:45:11

Salut,

dans la première ligne, "alias Option PV dialog -m optionpv optionpv"

ton aliase a un truc de trop, colle "Option" avec "PV" ou vire le "PV" ou le "Option", parce qu'il y a un erreur.

pour les variables, quand tu veux les mettre à "OFF" tu peux les supprimer, ensuite tu n'as qu'à regarder si la variable existe pour savoir si elle est activée, ex :

  if ($did(3).state == 1) { did -e optionpv 4 | set %pvmessage on }
  if ($did(3).state == 0) { did -b optionpv 4 | set %pvmessage off }
  if ($did(6).state == 1) { set %pvferme on }
  if ($did(6).state == 0) { set %pvferme off }

devient :

  if (%pvmessage) { did -e optionpv 4 | unset %pvmessage }
  else { did -b optionpv 4 | set %pvmessage on }
  if (%pvferme) unset %pvferme
  else set %pvferme on

pour la vérification des variables dans les conditions j'suis pas certain qu'il n'y ait aucun risques : si tu double clic vite dans le check si les variables se changent bien...

ensuite tu peux adapter pour le "on join"
Le groupe quand est-ce que tu l'active ? ;(

Commentaire de Nitrite59 le 07/05/2007 13:31:50

Ca à l'air de fonctionner même si les scripts plus ou moins récents comportent tous un anti-pv :) Bonne continuation.