SYSTEM D'AWAY BASIQUE ...
cs_Asthar
Messages postés
208
Date d'inscription
dimanche 18 avril 2004
Statut
Membre
Dernière intervention
6 septembre 2005
-
18 avril 2004 à 17:35
GFACTION Messages postés 199 Date d'inscription lundi 19 mai 2003 Statut Membre Dernière intervention 7 avril 2005 - 19 avril 2004 à 21:53
GFACTION Messages postés 199 Date d'inscription lundi 19 mai 2003 Statut Membre Dernière intervention 7 avril 2005 - 19 avril 2004 à 21:53
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.
https://codes-sources.commentcamarche.net/source/22055-system-d-away-basique
https://codes-sources.commentcamarche.net/source/22055-system-d-away-basique
19 avril 2004 à 21:53
Y'en a des tonnes et des tonnes, je vais faire une indigestion :x
En suite, je vais pas relancer la polémique, mais ça sert pas à grand chose de toute façon, si je veux me marquer comme away, je fais /Away, ça suffit amplement.
19 avril 2004 à 11:16
Pour ce qui concerne la source en elle meme, je ne prefere pas la modifié maintenant et vous proposer une 2e version plus complete ...
Bonne semaine.
19 avril 2004 à 01:15
18 avril 2004 à 22:32
mais ca :
Looser ! - System d'away - Raison :
t'été pas obliger de le métre...
18 avril 2004 à 21:55
En revanche tu peux amélioré ton addon pour qu'il devienne intéressant en y ajoutant :
-auto away
-ajout, supression et modification de raison d'away
-msg configurable
-etc...
Je ne met pas de note pour l'instant j'attend de voir si tu fais mieux :)
18 avril 2004 à 19:33
En +, gros point noir sur ta source : ton systeme d'away ne gère pas le multiserveur.
>> unset %awayon
>> unset %raison
>> unset %particule
Prends des variables commencant par la même chose, exemple :
%aw.awayon, %aw.raison, %aw.particule
Puis il te suffira de faire unset %aw.* pour qu'elles disparaissent toutes d'un coup !
Voila, essaie d'améliorer ta source en tenant compte des commentaires de chacun, et la prochaine fois, fais quelque chose d'original, par pitié ! lol
18 avril 2004 à 18:56
Perso jte dirai pas de tout grouper dans un event pake je n'en vois pa l'utilité et pour moi séparer en differents events rend le code bcp plus clairs
Sinon ben comme les 3/4 d posts on a deja vu koi, o pire si kkun porpose ce genre de systeme qu'il essaye d'innover pake la ca a un gout de réchauffé.
18 avril 2004 à 18:56
if (%raison == $null) se remplace par if (!%raison) { }
if ($did(away,6).state 1) par if ($did(8).state 1) pas besoin de specifie le nom du dialog car il est deja specifie dans l'evenement.
Avec ton add-on, si on est deja away et qu on le reouvre, on peut partir doublement away, si on met rien dans les edit quand on veut partir en away ca merde :/, et "particule d'away c'est pas tres clair.
Sinon l'idée de la picwin est pas mal :), malheureusement ce type d'add-on est deja vu et revu,...
Je note pas.
18 avril 2004 à 18:15
Le problème, c'est que tu marques trés basique alors forcement ...
Et puis en effet un système d'away, c'es déjà vu et revu.
Passons aux commentaires du codage :
Alors déjà, tu peux rassembler les événements avec $devent, exemple :
on *:dialog:away:*:*: {
if $devent == init { ... }
if $devent == sclick {
if $did == 1 { ... }
}
}
Voilà.
Ensuite, les / sont inutiles.
Voilà, rien d'autre à dire, j'espère que mes remarques sont constructives, je ne note pas.
Scripteusement vôtre !
18 avril 2004 à 17:35
l'alias le voici, mais vous auriez deviner :)
Alias dia { dialog -m $$1 $$1 }