PANNEAU DE CONFIGURATION POUR CHANGER D'ÉCRITURE

Commentaire de uaip le 05/02/2009 18:44:48

Salut, pas besoin de { } quand il n'y a qu'une seule instruction, manque un peu de rigueur dans la syntaxe (selon moi, d'autres diront que non) au niveau de la logique if/elseif et des parenthèses de conditions.
Tu aurais pu optimiser ton code pour éviter les if en cascade (c'est très moche).
Bref, niveau débutant, et non expert.

(aucune froideur dans mon message, c'est simplement qu'il y a peu de chose à dire de ta source, car elle est assez "pauvre". Si ta source marche, tant mieux, mais ce n'est pas exactement le but de ce site).
Bonne continuation.

Commentaire de ScriptingBen le 15/02/2009 13:35:14

Citation:
je mets le code dans ma remote
j'apuille pour la couleur de la premiere lettre je choisi la couleur BLEU et je n'apuille pas sur un autre bouton j'aurai toute la phrase BLEU!!!

Non parce que après [...] $+ %graspl $+ $left($1-,1)) $+ j'ai mit  (le petit L qui stop la couleur).

Commentaire de ScriptingBen le 15/02/2009 13:39:58

Et pour la menu bar c'est exacte j'ai oublié de mettre le code merci de le rappeler :)

Commentaire de WorldDMT le 16/02/2009 11:47:28

tu a peut etre modifier le code
j'ai bien testé avantde posté mon commentaire :)

et c'est quoi ce truc que tu as fais dans ton input???

tu peux expliquer pourquoi ou a quoi ça sert

tu n'a tjr pas changer le "$right($1-,$calc($len($1-) - 1)))" pourquoi cette complication tu as la possibilité de faire rien que "$mid($1-,2)"

bref j'attend que tu m'explique pourauoi ces 2 lignes dans ton input

@+

Commentaire de ScriptingBen le 18/02/2009 16:56:05

Au faite, j'ai compris ce que $mid ! :) Merci c'est très pratique ^^

Commentaire de uaip le 21/02/2009 18:48:19

Salut,
Non mais, tu ne trouves pas que de la ligne 41 à 120 (tout l'événement dialog, quoi), c'est très moche ?
C'est là qu'on voit que cette source est "débutant". Tu attribues bêtement des id à tes contrôles puis tu enchaines avec des if en cascade. Ce n'est pas du scripting, ça, c'est de la mocheté.
De plus, après tests, le dialog est affreux, pas du tout fluide. Déjà c'est peu intuitif, puis on ne sait pas si les boutons sont cochés ou pas.

if ($did == 1) { /dialog -m couleurs couleurs }
-> ça sert à quoi, vu qu'aucun id n'est 1 ?
Fais aussi gaffe à tes noms de dialog ou variables, %text peut déjà être utilisé par un autre script.
Dernière chose, l'attribut size n'a besoin que de 4 paramètres (et non 5) et il faut une virgule après les values des contrôles (/help /dialog), donc ton "garantis 100% testé..."

Voilà un petit exemple d'optimisation :

;Pour l'ouvir allez ensuite dans votre menubar ;)
menu menubar {
  Couleur d'écriture:dialog -m ec_couleurs ec_couleurs
}
dialog -l ec_couleurs {
  title "Couleur: Première lettre"
  size 1 -1 250 225
  tab "Première lettre", 100, 5 5 242 200
  tab "Texte", 101

  box "Effet de police", 102, 10 30 230 50
  check "Gras", 20, 20 50 70 15
  check "Souligné", 21, 100 50 70 15

  box "Couleur", 103, 10 90 230 100
  radio "Rouge" 4, 20 110 60 19
  radio "Bleu" 12, 20 130 60 19
  radio "Vert" 3, 20 150 60 19
  radio "Noir" 1, 100 110 60 19
  radio "Gris" 14, 100 130 60 19
  radio "Orange" 7, 180 110 60 19
  radio "Violet" 6, 180 130 60 19

  button "Réinitialiser" 22, 25 202 80 20
  button "Tester" 23, 107 202 50 20
  button "OK" 24, 160 202 60 20, ok

}
on *:dialog:ec_couleurs:*:*:{
  if ($devent == init) {
    did -u $dname 20,21
    did -c $dname 1
  }
  elseif ($devent == sclick) {
    if (!$did) return
    ;Mise à jour du title
    dialog -t $dname Couleur: $iif($dialog($dname).tab == 100,Première lettre,Texte)
    ;Choix de la couleur
    if ($did <= 14) set % $+ $iif($dialog($dname).tab == 100,ec_premierelettre,ec_text) $+(,$did)
    ;Choix du gras
    if ($did == 20) set % $+ $iif($dialog($dname).tab == 100,ec_grasp1,ec_grastext) $iif($did($did).state,)
    ;Choix du souligné
    if ($did == 21) set % $+ $iif($dialog($dname).tab == 100,ec_soulettre,ec_soutext) $iif($did($did).state,)


    if ($did == 22) {
      did -u $dname 20,21,4,12,3,1,14,7,6
      did -c $dname 1
      did -f $dname 100
      unset %ec_*
      echo -at Couleurs réinitialisées - Couleur par défaut
    }
    if ($did == 23) {
      echo -at 4(Test de la couleur) %ec_soulettre $+ %ec_premierelettre $+ %ec_graspl $+ N $+ %ec_soulettre $+ %ec_graspl $+ %ec_text $+ %ec_grastext $+ %ec_soutext $+ at0_Script Color
    }
  }
}
on *:input:*: {
  if ($left($1-,1) != /) && ($left($1-,1) != !) && ($left($1-,1) != =) && ($left($1-,1) != &) && ($left($1-,1) != ^) && ($left($1-,1) != -) && ($left($1-,1) != +) && ($left($1-,1) != :) && ($left($1-,1) != ;) {
    say $+(%soulettre $+ %premierelettre $+ %graspl $+ $left($1-,1)) $+  $+ %soulettre $+ %graspl $+ %text $+ %grastext $+ %soutext $+ $mid($1-,2)) $+  | halt
  }
}

Je n'ai pas touché à l'événement input, et il reste quelques trucs à améliorer, mais j'ai surtout voulu te montrer comment jouer avec les id au lieu d'utiliser des if en cascade tous moches.

Bonne continuation.

Commentaire de ScriptingBen le 22/02/2009 19:15:16

deja les -u -f et -c je comprend pas trop , ensuite ça par exemple:

if ($did == 21) set % $+ $iif($dialog($dname).tab == 100,ec_soulettre,ec_soutext) $iif($did($did).state,)

les $dname , $iif je comprend pas trop ce que c'est , voila =) et merci pour les conseils.

Commentaire de WorldDMT le 23/02/2009 08:11:35

pour l'input si je tape ";salut les amis" ou "^Salut les amis" j'aurais pas de couleur je pense si c'est pour les emoicons il va faloir autre methode

Commentaire de KiMoW le 03/03/2009 04:48:22

il n'y a pas moyen de sorte que quand on tape un nick
il ne fais qu'une seule couleur :x
Prcq quand je fais couleur de la première lettre vert/gras
et la suite orange
quand je tape un pseudo il n'a pas d'HL vu le code de couleur
il n'y a pas moyen de le developper pour regler ce petit porblème svp =)
Merci d'avance ^^

Commentaire de ScriptingBen le 07/01/2010 00:13:42

Bonjour,

Il y a longtemps que je n'ai pas mit à jour ma source, mais je vais le faire prochainement toujours pour aider ceux qui en ont besoin, tout en gardant le code de base, c'est à dire chaque événement ou $did prend un paragraphe, je trouve cela plus claire, personnellement.

Voila, je vous souhaite de bien en profiter, en attendant la mise à jour ;)

Concernant la question de KiMoW, non je n'ai pas fait pour ma part cette fonction, il suffit de rajouter une condition "Si ce mot fait parti des pseudo, blabla".