Bon, un troisième topic, dsl mais j'ai beaucoup de choses à dire....

| if ($did(cur,2).seltext == $null)
devient :
| if (!$did(cur,2).seltext)

Et sinon, j'ai trouvé encore de nombreuses optimisations...
Pour cela par exemple:
| if ($did(cur,2).seltext == Bleu) {
| set %cur Curseur\Bleu.cur ... Curseur\bleu.cur
| }
| if ($did(cur,2).seltext == Vert) {
| set %cur Curseur\Vert.cur ... Curseur\vert.cur
| }
| if ($did(cur,2).seltext == Noir) {
| set %cur Curseur\Noir.cur ... Curseur\noir.cur
| }
| if ($did(cur,2).seltext == Gris) {
| set %cur Curseur\Gris.cur ... Curseur\gris.cur
| }
| if ($did(cur,2).seltext == Jaune) {
| set %cur Curseur\Jaune.cur .... Curseur\jaune.cur
| }
je fais :
| set %cur $+(curseur\,$did(cur,2).seltext,.cur)

Et tu peux aussi mettre des Hashs Tables...
Et quand tu auras fait tout ceci, tu pourras faire un addon de configuration visuel complet pour mIRC et rajouter pleins d'autres options.

Celà peut devenir un bon truc...
6/10 pour t'encourager, nah!

Ah, le code n'ai pas parfait non plus :

| did -a cur 2 Bleu
| did -a cur 2 Vert
| did -a cur 2 Noir
| did -a cur 2 Gris
| did -a cur 2 Jaune
Renseigne toi sur la commande /didtok
| didtok cur 2 44 Bleu, Vert, Noir, Gris, Jaune
Je ne sais pas si celà  marche. Depuis le temps que je ne scripte plus, ba j'ai oublié... Faut que je me remettes dans le bain.

| if (%onstartcur == on)
Renseigne toi sur $true et $false...
Si tu fais :
| set %onstartcur 1
à la place de :
| set %onstartcur on
ton if deviendra :
| if (%onstartcur)
Plus court, plus agréable... plus pro

J'ai mis 1 qui a le même effet que $true pour optimisé la suite de ton code:
| if ($did(cur,6).state == 1) { set %onstartcur on }
| if ($did(cur,6).state == 0) { set %onstartcur off }
qui devient pour moi :
| set %onstartcur $did(cur,6).state

Regardes :
| if ($did(cur,2).seltext == Bleu) {
| set %cur Curseur\Bleu.cur ... Curseur\bleu.cur
| }
| if ($did(cur,2).seltext == Vert) {
| set %cur Curseur\Vert.cur ... Curseur\vert.cur
| }
| if ($did(cur,2).seltext == Noir) {
| set %cur Curseur\Noir.cur ... Curseur\noir.cur
| }
| if ($did(cur,2).seltext == Gris) {
| set %cur Curseur\Gris.cur ... Curseur\gris.cur
| }
| if ($did(cur,2).seltext == Jaune) {
| set %cur Curseur\Jaune.cur .... Curseur\jaune.cur
| }
Réfléchis, et tu pourras faire ce gros code en 1 seule ligne...


Mes codes ne sont sûrement pas encore correcte mais celà te donnes une idée d'optimisation. Essaye et si tu n'y arrives pas, je t'aiderai avec plaisir :-).

Cela doit faire... au moins 6 mois que je suis pas venu...

Bon pour ton addon, je ne l'ai pas testé, je ne télécharge pas moi monsieur ! Mais j'ai regardé ton aperçu.

Pour la liste, je verrais plûtot un combo avec image (fesable avec mdx)...
Comme l'a dit KinderMaxi, pouvoir rajouter des curseurs dans la liste serait bien aussi.

Je vais télécharger rien que le curseur.mrc et te donner mes commentaires sur ton code ensuite.

Bonne continuation.

Bonjour

>ouép merci itello, et pour te dire j'ai pas mis unset %q*
>en pensant aux autres variables de mIRC qui n'ont pas de
>rapport à celles de l'add-on.
Ha ba alors...
Tu as le choix de remplacer tes vars par des plus complexex (ex: %quoteparanneau_nymeA) ou bien de laisser comme celà.

>Pour les | c'est l'habitude, et je savais pas que ça
>ralentissait le code.
Et bien tu le sais maintenant :-)

>pour la window au chargement je trouve ça plus joli
>qu'un simple /echo, c'est tout.
Et aussi plus 'lourd', en plus on a même pas le bouton X et il faut lire jusqu'à la fin de ta win avant de savoir comment fermer...

>euh je vois pas pk tu dis que j'ai fait cet add-on vite,
>qu'il soit horrible d'accord c'est vrai que ça peut manquer
>d'icon. pour le bug sur le button "Annuler" je vois pas
>d'où il vient si tu pourrais me donner un peu plus de
>détails je pourrai faire une mise à jour.
Bouton "annuler" ??? Je n'en vois pas.
Et c'est le code qui est 'à la va vite'.

Pour ton histoire de mdx, je n'utilises pas de dll, donc je ne peux rien te dire.

Aurevoir

Bonjour.
Mouais, je trouves celà légerement 'horrible', bugué, imcomplet, à la vite fait... Je préféres encore quelqu'un qui fait un petit truc et qui le fait bien que quelqu'un qui en fait un gros et qui le fait mal.

1/ Déjà la @win au 'on load' est vraiment inutile, un simple /echo suffit et encore... Car, tu n'imformes en rien le 'client' avec ce message, donc il est inutile.

2/ Calmos les | qui ralentissent ton code.

3/ Simplifie ton code:
>unset %qc %ql %qmod
devient:
>unset %q*
-
>if ($exists($qc) == $false)
devient:
>if (!$exists($qc))
-
>if ($qf(options,strip) == on)
Si tu mets $true et $false comme réponse à $qf, ton code devient:
>if $qf(options,strip)
-
>if (?quote iswm %x)
Un problème est, pourquoi mettre iswm qui recherche un * (joker) hors que isin suffit.
-
Ton alias $qs est simplifiable avec une certaine logique...
Si ton $1 changait de valeur...
-
>menu menubar,channel,query,status
devient:
>menu *
qui marche en plus pour les @wins.
-
>$iif($1 == false,Quotes.ini)
>$iif($2 == false,mdx.dll)
>$iif($3 == false,dialog.mdx)
>$iif($4 == false,views.mdx)
>$iif($5 == false,bars.mdx)
>$iif($6 == false,popups.dll)
Reflexion...

4/ Marche pas avec mirc 6.14. Tes dlls sont t'elles vraiment utiles? Sauf pour rendre beau, je pense pas....

5/ Très belle langue que tu utilises. Est-ce que c'est du franglais?
> System devient Système
> Strip devient ... je sais pas
Mais vraiment, tu fais tout en anglais où bien tout en français.

6/ Bugs:
Titlebar
Dès qu'on touche la titlebar, le message de fermeture s'affiche.
Et dés qu'on clique sur cancel, mIRC bug à mort
-
Menu
Quand on prend "Lancer cet quote", si on en a pas séléctionné celà s'éxecute quand même.
Il faudrait grisé les 3 premiéres fonctions du menu quand on n'a pas séléctionné de quote.

Enfin bon, pour ton code, la simplification est encore large mais je ne vais pas y passer la journée. Je laisse un autre scripteur le faire.
Sinon, c'est quand même très bien, je vais donner un léger 7/10.

Aurevoir, et bonne continuation pour ton addon qui peut s'amélioré.
Pour le niveau initié, on va dire que c'est limite mais correcte.

Bonjour.

Ou mieu encore:
-$convertis(nombre,monnaie1,monnaie2)
Donc nombre sera monnaie1 et se convertira en monnaie2, ce serai vraiment parfait, lol.

Sinon tu utilises des prop beaucoups trop compliqués à comprendre....
Donc tu devrais les 'clarifiés'....

Et pour ton code, je ne vais pas dire grand chose, il y-a quasi RIEN!

Bonjour.
Mouais, bon... Très simpliste quand même.

Correction:
1/ Regroupes tout les on dialog possible, tu peux même faire ceci:
>on *:dialog:*:*:*: {
>if $dname == dialog1 { ... }
>if $dname == dialog2 { ... }
>}
Oui bon, pour celà c'est toi qui vois.
-
2/ Simplifier son code !!!
>($did($dname,3) == $null)
Ne pourrait t'il pas devenir:
>!$did(3)
-
3/ Ecore simplifier son code:
Tu fais ceci pour ton dialog:
>text "", 1, 5 7 159 30
Et tu rajoutes ceci:
>on 1:dialog:generalerte:init:*: {
>/did -a generalerte 1 %tention
>}
A la place, il suffirait de faire:
>text %tention, 1, 5 7 159 30
-
4/ Et encore:
$did($dname,3)
Pourquoi un $dname ?
-
Ton code n'étant  pas très riche en scripting, je ne peux pas beaucoup te donner de conseils de simplification, néanmois les conseils d'améliorations eux, seront plus nombreux.

AMELIORATION:
1/ Et voici une liste:
< Un répondeur en pv quand on est away et un quand on ne l'est pas.
< Possibilité de multiserveur ou non.
< Message d'away personnalisable.
< Stopper away dés qu'on parle. Configurable.
< Enregistrer les messages pv dans une fenêtre @window.
< Beau design!

Bonsoir.

Voici un 'long' addon qui pourrait vite devenir très court avec une certaine logique dans l'emplacement de tes objets dans ton dialog et une amélioration du code.
Je vais te donner quelques petits conseils utiles, mais la simplification, tu la feras toi même.

1/ Regroupement de tes "on *:dialog" de la façon suivante:
> on *:dialog:ghost:*:*: {
> if $devents == edit {
> if $did == 1
> }
> }
Enfin, tu dois connaitre et sinon il suffit de lire le tutorial suivant:
- http://www.scriptsdb.org/tutar.php?id=263

2/ Remplacement de $null.
>if ($did(31) != $null) {
Devint:
>if $did(31) {
-
Et donc logiquement:
>if ($did(31) == $null) {
Devient:
>if !$did(31) {

3/ Compatible à plus de serveurs:
>NickServ ghost ...
Certains serveurs n'ayant pas cette commande, tu devrais rajouter cette ligne:
>alias nickserv msg nickserv $1-
Enfin, c'est toi qui voit.

4/ Dans certaines commandes tu mets un / devant, hors qu'il est inutile, donc l'enlever serait utile.

5/ Simplification:
> if (!$hget(pseudolist)) {
> /hmake pseudolist 100 | .hadd pseudolist pseudo1 $did(2)
> }
> else { .hadd pseudolist pseudo1 $did(2) }
Devient:
>hadd -m pseudolist pseudo1 $did(2)

Enfin bon, tu vois un peu...
Moi j'attends de voir un code 2 fois plus petit, au moin.

_____________________________________________
_____________________________________________

La simplification finie, tu pourras plus simplement améliorer ton addon. Et voici mes conseils d'améliorations:

1/ Faire une liste de pseudo et de mot de passe infinie en remplaçant tes edits par des listes, et en utilisant les tokens. Plus complexe, mais plus agréable pour l'utilisateur.
Un très bon tutorial sur les tokens est:
-http://www.scriptsdb.org/tutar.php?id=2

2/ Enlever tes boutons flats et les mettre normales, et moin gros.

3/ Faire un dialog visuellement plus beau (il l'est déjà beaucoup). Enfin plus simple à dire qu'à faire.
Pour faire de beaux dialog, tu trouveras des logiciels sur:
-http://www.tismania.org/prog.php

Bonne continuation, bravo. Un beau 7/10 .

Bonjour.

1/ Déjà, dsl de te le dire, mais ton dialog est affreux. Il faudrait que tu le refasses, et si il le faut prend un logiciel fait express pour ceci.
Tu pourras en trouver sur tismania.org:
http://www.tismania.org/prog.php

Sinon, n'aillant quasiment pas de code dans ton addon, je te conseillerai pas grand chose, simplement de regrouper tout tes on dialog.
>on *:dialog:gf:sclick:*: {
>if $did == 5 { ... }
>if $did == 18 { ... }
>}

Puis voila, bonne continuation...
Je ne notes pas, j'attends le nouveau dialog.

Bonjour.
Ce script est simple, mais au moins il marche bien, effectivement. Je vais donc ne pas être trop sévére et te donner un petit 6/10 (pour remonter ton 1/10).

<CORRECTION<<
1/Alors, je commence à dezipper cet addon avec WinZip et voilà:
-ERREUR
-passe: Shell32 089.ico: ce fichier n'est pas au format standard Zip 2.0
Donc, voilà bon ...
-
2/Ensuite, ton fichier est en .tip hors que le format par défaut de mirc est le .mrc. Et celà pose un léger problème pour ceux qui ont un éditeur de script autre que mIRC (moi, par exemple!!).
-
3/Regardons ceci:
>menu menubar {
>Se logger en tant que CSop:dialog -dm csop csop
>}
Pourquoi ne pas simplement mettre <Csop> qui n'aggrandira pas le dialog inutilement, et je pense que ce sera quand même clair.
Et faire menu * { te permettrais de le mettre dans tous les menus de mIRC.
-
4/Je transforme ceci:
>set %nick $did(csop,2)
>set %pass $did(csop,5)
>oper %nick %pass
>unset %pass
>unset %nick
en:
>oper $did(2) $did(5)
Plus court, je pense.
-
5/Regrouper tes 2 dialogs comme ceci:
>on *:dialog:csop:*:*: {
>if $devent == init {  ... }
>if $did == 7 { ... }
>}
Non ?
-
6/Orthographe (niark) et comprehension:
Celà ne doit pas être ton point fort mais ce n'est pas grave, je suis là et je ferais ma correction un autre jour.

PS: Dans ce topic, je n'ai pas fait attention à l'orthographe, je suis trop fatigué pour réfléchir.