ASV SIMPLE ET CLAIRE

Commentaire de atlantisfr le 08/09/2006 17:34:20

Les dialogs d'ASV c'est du vu et revu. Si tu approtes des nouveauter comme un ASV en picwin, skinnable et encore le systeme d'ASV c'est pas trés utile. Niveau code, utilise des identifieurs $devent et $did, quand tu as une seule commande par $did retire les { }.

Exemple :
on *:dialog:Lasv:*:*:{
  if ($devent == edit) {
    if ($did == 13) set %nom $did(13)
    if ($did == 14) set %prenom $did(14)
    if ($did == 15) set %age $did(15)
    if ($did == 17) set %ville $did(17)
    if ($did == 18) set %poids $did(18)
    if ($did == 19) set %cheveux $did(19)
    if ($did == 20) set %yeux $did(20)
    if ($did == 23) set %portable $did(23)
    if ($did == 22) set %mail $did(22)
    if ($did == 25) set %asv $did(25)
    if ($did == 21) set %taille $did(21)
    if ($did == 36) set %caractere $did(36)
  }
}

Au lieu de :
on *:dialog:Lasv:edit:13:{ set %nom $did(13) }
on *:dialog:Lasv:edit:14:{ set %prenom $did(14) }
on *:dialog:Lasv:edit:15:{ set %age $did(15) }
on *:dialog:Lasv:edit:17:{ set %ville $did(17) }
on *:dialog:Lasv:edit:18:{ set %poids $did(18) }
on *:dialog:Lasv:edit:19:{ set %cheveux $did(19) }
on *:dialog:Lasv:edit:20:{ set %yeux $did(20) }
on *:dialog:Lasv:edit:23:{ set %portable $did(23) }
on *:dialog:Lasv:edit:22:{ set %mail $did(22) }
on *:dialog:Lasv:edit:25:{ set %asv $did(25) }
on *:dialog:Lasv:edit:21:{ set %taille $did(21) }
on *:dialog:Lasv:edit:36:{ set %caractere $did(36) }

Voilà donc essaye d'optimiser ton code, modifie ta source sur le site et on revoit ta note aprés.

Commentaire de Dragounix le 08/09/2006 17:40:11

ok atlantisfr merci je le modifi dessuite :)

Commentaire de atlantisfr le 08/09/2006 21:27:33

Dragounix, de bons tutoriaux sur les picwins on était fait par KiNdErMaXi sur ce site.

Commentaire de atlantisfr le 08/09/2006 21:52:09

Tu optimises que une partie de tes "On dialog". Cadeau pour cette foi si :

on *:dialog:Lasv:*:*:{
  if ($devent == edit) {
    elseif ($did == 13) set %nom $did(13)
    elseif ($did == 14) set %prenom $did(14)
    elseif ($did == 15) set %age $did(15)
    elseif ($did == 17) set %ville $did(17)
    elseif ($did == 18) set %poids $did(18)
    elseif ($did == 19) set %cheveux $did(19)
    elseif ($did == 20) set %yeux $did(20)
    elseif ($did == 23) set %portable $did(23)
    elseif ($did == 22) set %mail $did(22)
    elseif ($did == 25) set %asv $did(25)
    elseif ($did == 21) set %taille $did(21)
    elseif ($did == 36) set %caractere $did(36)
  }
  if ($devent == sclick) {
    elseif ($did == 16) set %sexe homme
    elseif ($did == 27) {
      elseif (%sexe == homme) msg $active Je suis Un Homme
      elseif (%sexe == femme) msg $active Je suis Une Femme
    }
    elseif ($did == 26) msg $active Je m'appelle %nom %prenom jai %age ans je suis un/une %sexe j'abite a %ville je pèse %poids la couleur de mes yeux %yeux la couleur de mes cheveux %cheveux je mesure %taille je suis %caractere mon e-mail %mail mon numero de portable %portable
    elseif ($did == 28) msg $active Mon asv est %asv
    elseif ($did == 29) msg $active Je m'appelle %nom %prenom
    }
    elseif ($did == 32) set %sexe femme
    elseif ($did == 33) msg $active Mon adresse e-mail est %mail
    elseif ($did == 34) {
      set %notice $$?="Pseudo ou Salon ?"
      notice %notice $me Je m'appelle %nom %prenom jai %age ans je suis un/une %sexe j'abite a %ville je pèse %poids a couleur de mes yeux %yeux a couleur de mes cheveux %cheveux je mesure %taille je suis %caractere mon e-mail %mail mon numero de portable %portable
    }
    elseif ($did == 38) {
      set %pseudo $$?="Pseudo ?"
      msg %pseudo Je m'appelle %nom %prenom jai %age ans je suis un/une %sexe j'abite a %ville je pèse %poids la couleur de mes yeux %yeux la couleur de mes cheveux %cheveux je mesure %taille je suis %caractere mon e-mail %mail mon numero de portable %portable
    }
    elseif ($did == 39) {
      set %salon $$?="Salon ?"
      msg %salon Je m'appelle %nom %prenom jai %age ans je suis un/une %sexe j'abite a %ville je pèse %poids la couleur de mes yeux %yeux la couleur de mes cheveux %cheveux je mesure %taille je suis %caractere mon e-mail %mail mon numero de portable %portable
    }
  }
  if ($devent == init) {
    elseif (%sexe == homme) did -c Lasv 16
    elseif (%sexe == femme) did -c Lasv 32
  }
}

Commentaire de atlantisfr le 08/09/2006 22:03:24

Derien Dragounix

Commentaire de Dragounix le 08/09/2006 23:00:10

atlantisfr ma source veu plu marché :(

Commentaire de S333 le 09/09/2006 11:51:36

set %notice $$?="Pseudo ou Salon ?"
       notice %notice $me Je m'app...

Notice $$?="Pseudo ou Salon ?" $me Je m'appelle...
Ca revient au même :) et au moins, tu ne crées pas une variable.

Comme l'a dit arkanis, il y a souvent des elseif qui commencent une série de if...
C'est if truc, sinon(elseif) truc,sinon etc...

Il y a aussi une erreur dans le if $devent = sclick, à l'endroit de elseif ($did == 29)
Tu as mis une accolade fermée } alors que tu ne l'as pas ouverte dans le elseif(ce qui était d'ailleurs bon). Il faut la supprimmée, parce que tout ce qui suis ne fait pas parti du on click. Même si ça marche comme ça... puisque, on ne peut edit un boutton ou scroll, enfin... un dclick, un uclick, rclick, drop, réagissent aussi dans ce cas. Donc autant les intégrer au sclick.

--- Encore un rappel ---
Lorsque vous utilisez des variables globales(set) dans un add-on... il serait préférable que ces variables ne soient pas déjà utilisées par un autre add-on... Donc mettez des choses plus spécifiques... %pseudo %salon... ce sont des choses qui ont grande chance d'être déjà utilisées... même si on essaie de moins en moins utiliser les variables globales.

Commentaire de Dragounix le 09/09/2006 20:43:40

je ne comprend pas trop rca arkanis je suis nul je sui vrment un ti debutent qui c'est rien faire :(( snif

Commentaire de Dragounix le 09/09/2006 21:00:21

ok merci rca arkanis je relit plain de foi comme ça jai pigé merci pour toute votre aides =))))

Commentaire de Dragounix le 13/09/2006 15:37:46

ok merci CestFortEnChocolat ^^

Commentaire de Keanu85 le 22/09/2006 11:08:35

addonds basique ^^ et en plus le dialogue y a plein de perte de place :o tu tes pas cassé le ***