Disons que pour moi les fonctions de la DLL ne sont pas forcément très utiles quoique cela puisse un petit côté personnalisé aux dialogues et que je n'ai sûrement pas pensé à tous les côtés pratiques (tu aurais pu nous fournir un exemple).

Par contre fournir le code source serait, à mon avis, très utile pour tous ceux qui souhaitent développer un peu le côté DLL en mIRC.

Le commentaire de Pakofils_ me semble un peu mal venu.

Il n'en reste pas moins qu'il est assez pauvre tant au niveau du codage que des options. Tu pourrais grouper tes événements, utiliser un combo pour lister les serveurs, faire un système de connexion "profil" plutôt qu'un simple pseudo...

Bref, beaucoup de chose à retravailler comme il a été dit.

Tuni23 : tu crois vraiment que ces deux sites ont encore besoin de pub ?

A propos de la source :

- regroupe tes var sur une même ligne (var %a = bla, %b = bla)
- le "halt return" (dans l'aliase /bash) je ne vois pas trop à quoi ça sert.
- la série de if pourrait être optimisée il me semble
- travailler directement sur le code de la page serait une meilleure idée comme l'a dit ArKanis. Ceci dit, si tu restes avec un document texte, utilise le file handling plutôt que le /write vu que tu vas écrire pas mal de chose dans ledit doc'.

Sinon c'est un bon exemple d'utilisation des sockets,

bonne continuation

Pour le $iif, quelque chose du genre .msg salon $iif(%read,commande-si-vrai,commande-si-faux)

Le file handling fonctionne comme ça :

- ouverture du fichier
- écriture ligne 1
- écriture ligne 2
- fermeture

Alors que le /write fait : ouverture, écriture ligne 1, fermeture, ouverture, écriture ligne 2, fermerture.

C'est assez pratique quand tu cherches à écrire pas mal de choses dans un fichier. Si tu ne dois écrire qu'une liste, reste quand même sur le /write.

Regarde dans les commandes /fopen, /fclose, etc. pour le file handling. Il suffit juste de penser à ouvrir et à fermer le fichier, sinon, tout déconne.

Yep,

Alors voici quelques trucs :

- Groupe tes raws de cette manière  :

raw *:*:{
  if ($numeric == 311)
  elseif ($numeric == 401)
  etc.
}

- Le if ($2 == $null) devient if (!$2)

- Au niveau de l'utilisation des conditions :

if (%read != $null) et if (%read == $null) peuvent se simplifier avec l'identifiant $iif

- L'utilisation des variables globales n'est pas une très bonne idée, préfère les hashtables

Pour apporter quelques améliorations, tu pourrais envisager de faire un dialogue ou une @window où les différents événements seraient affichés (avec la possibilité de trier les logs/événements).

Bonne journée !

P.-S. : si tu es intéressé par les méthodes d'écriture dans les fichiers, je te conseille d'aller regarder du côté du file handling qui est une méthode beaucoup plus puissante pour effectuer ce genre d'opération.

Yop,

Pour les did -a à améliorer, j'ai pensé à faire une var pour stocker les données et une while pour  lister les données et les afficher dans les item. Après, c'est une technique comme une autre.

Yep,

Bon, c'est une source comme celle-là sera toujours utile, mais niveau codage c'est pas trop ça. Quelques trucs que tu peux modifier :

- Les var en cascade, tu changes et tu fais var %a = blabla, %b = blabla, %c = etc.
- tes événements dans le on *:dialog, tu peux utiliser les if ($devent == init), if ($devent == sclick), etc.
- Lorsque tu fais les "did -a $dname 4 $fullname" ça ne sert à rien. Laisse uniquement le "edit $fullname, 11, 42 57 60 10" (le fullname va se mettre directement dans l'édit).
- Les nombreux "did -a" peuvent être amélioré.
- Les /write en cascade aussi, c'est pas génial, pense au file handling (jete un oeil à l'aide).
- Les fautes d'orthographe sont trop nombreuses et un petit screen aurait été sympa.

Voilà, bonne continuation !

Salut,

Assez bien réalisé sauf quelques petits trucs à corriger :

- utilise les variables locales (/var) plutôt que les globales (pour éviter les /unset)
- lorsque tu utilises un .timer, nomme-le (ex .timertelnet) pour éviter des conflits
- lors de ta boucle avec le $findfile, pas besoin de mettre *.* seul le "*" suffit

Sinon, bon travail !

Quelle est l'utilité de poster la source d'un autre ?

Salut,

Voilà quelques astuces pour améliorer ton code :

- regroupe tes événements dans le on *:dialog: avec if ($devent == init), if ($devent == menu), etc.

- quand tu fais une boucle, pourquoi utilises-tu des variables globales ? Pense plutôt à utiliser la commande /var

- quand tu lances un timer, pour qu'il soit silencieux utilise un "." avant. Par exemple : .timerdialog

- pense à utiliser les "&&" pour préciser deux conditions sur une même ligne comme ceci :

if ($readini([ $chan $+ .ini ],SALON,caractéristique) == on) && ($readini([ $chan $+ .ini ],SALON,ZenLimite) == on) {

Et, pour terminer, essaye de mieux agencer ton dialogue. Il y a beaucoup trop d'espaces vides.

Sinon ta source à une utilité toute trouvée :)

Bonne continuation