SCAN IRCOP BY DEVORZ

Commentaire de PaDa le 05/08/2008 07:46:47

Tu peux faire les commentaires toi-même. Dès qu'on dépasse les commentaires bidon sur la manière de coder, il faut se poser des questions à chaque truc que tu fais.

Exemples :
- "who 0 o" => est-ce que c'est la seule manière de repérer un ircop ? Réponse, non. Donc regarde par exemple le scanner d'ircop que j'avais fait, ça te donnera une piste pour améliorer la méthode
- "raw 352:*:{ did -a IRCOP 1 $6 | HALT }" => est ce que le dialog "ircop" est forcément ouvert ? Réponse, non. Donc si quelqu'un recoit un raw 352 sans que ton dialogue soit ouvert, ca va lui faire une erreur alors que ca ne devrait pas. En plus méchant, ton code va pourrir un script propre.
- "dialog Ircop { " => est-ce que c'est un bon nom pour un dialogue comme ça ? Ca ne va pas être pris par un autre dialogue ? Idem pour l'alias

Sinon rien d'extraordinaire, mais sur une petite source comme ça, il faut se concentrer à peaufiner pour ne pas déranger les autres addons, et pour être le plus propre possible... Bonne continuation !

Commentaire de WorldDMT le 05/08/2008 12:31:38

salut

alias Ircop { $iif($dialog(Ircop),dialog -x Ircop,dialog -m Ircop Ircop) }

si le dialog est ouvert ça te servira a qoi de taper /ircop pour fermer??

sauf si c'est "v" au lieu de "x"

alias Ircop { $iif($dialog(Ircop),dialog -v Ircop,dialog -m Ircop Ircop) }

pour la list tu peu ne pas mettre vsbar met juste size au cas que ça depase le scrool bar apparet automatiquement

puisque l'otion n'est pas "notheme" le bouton flat ça change rien (si je ne me trompe pas)

pourquoi "Echo -a Scan en cours.." puisque le dialog est ouvert donc le resultat est dans le dialog fait plutot un edit read c'est mieu a mon avis

pour le double click t'a pas besoin de condition je pense

Elseif ($devent == dclick) {
    if ($did(1).sel) whois  $did(1).seltext  

devient

Elseif ($devent == dclick) {
    whois $did(1).seltext  


pour raw

tu peu aussi faire

raw 352:*:if ($dialog(Ircop)) { did -a IRCOP 1 $6 | halt } | else { echo -a $6 | halt }

comme ça tu n'a plus besoin de "set %Scan-Ircop on" dans "init" ni de le unset en "close"

Commentaire de uaip le 08/08/2008 22:59:38

Je trouve cette source sans intérêt. Code vu et revu, sans difficulté majeure.
De plus, il ressemble beaucoup à celui (dont tu n'es pas l'auteur) que possède ton script.

Puis, si je voulais enfoncer le couteau dans la plaie, je dirais que ton dialog peut générer une erreur si on l'ouvre sans être connecté à un serveur.

Commentaire de uaip le 09/08/2008 21:08:38

<quote>
tu raged toujours autant "dapo"
</quote>
Désolé, je ne parle que le français...

Et flooder des serveurs, et flooder des pvs, et flooder des insultes en notices, etc.. ça sert à quelque chose, ça ? NON, et pourtant tu le fais.
Bref, pour en revenir à ton truc, peut importe s'il vient de toi ou pas, il ne sert à rien. Je le répète, ce n'est pas parce-que vous codez un nouvel addon de 10 lignes qu'il faut le poster sur des forums.
Sinon, voici ma nouvelle source à moi:
alias pouet dialog $iif($dialog(pouet),-v pouet ,-m pouet bouh)
dialog bouh {
title "Salut"
size -1 -1 50 50
option dbu
button "Hihi", 1, 10 10 20 20
}
on *:dialog:pouet:sclick:1:dialog -x pouet | echo -a Youpi !

Il est chouette mon super nouvel addon d'enfer qui permet de dire "Youpi" quand on appuie sur le bouton, hein ?
:)

Commentaire de WorldDMT le 10/08/2008 11:32:27

lol
cool UAIP :p
je met 9/10 pour ton code
parceque t'a codé quelque chose pour rien "dialog -x pouet" le bouton le fera :p

mais comme c'est un addon d'enfer tres important j'ai mis 9/10

bravo :D lool