TIT HIGHLIGHT BIEN SYMPAS
cs_SornDrixer
Messages postés
2084
Date d'inscription
jeudi 12 décembre 2002
Statut
Membre
Dernière intervention
30 janvier 2011
-
2 juil. 2006 à 10:47
hisoka2501 Messages postés 264 Date d'inscription lundi 19 mai 2003 Statut Membre Dernière intervention 7 juillet 2009 - 4 août 2006 à 13:30
hisoka2501 Messages postés 264 Date d'inscription lundi 19 mai 2003 Statut Membre Dernière intervention 7 juillet 2009 - 4 août 2006 à 13:30
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/38400-tit-highlight-bien-sympas
https://codes-sources.commentcamarche.net/source/38400-tit-highlight-bien-sympas
4 août 2006 à 13:30
comme l'a dis sorn niveau redondance c crade !
je vais par contre me permetre une petie chose sur:
alias -l bleh {
if !$2- { var %msg.hl 1,0et un 4,8 HIGHLIGHT 1,0 pour 4,8 $2 $+ 8,8-1,0, un ! :p }
else var %msg.hl $3-
var %comchans_ [ $+ [ $2 ] ] $comchan($2,0)
while (%comchans_ [ $+ [ $2 ] ]) {
$iif($1,describe,msg) $comchan($2,%comchans_ [ $+ [ $2 ] ]) %msg.hl
dec %comchans_ [ $+ [ $2 ] ]
}
echo 7 $$1 s'est fait HL sur $comchan($2,0) chans
}
alias bme bleh 0 $1 $2-
alias bmsg bleh 1 $1 $2-
voila, pourqoi faire des /Set si c'est pour les /unset apres ? si ce n'est risquer un conflit de nom de variables avec un autre addon
j'ai préféré utiliser 0 et 1 plutot que 1 et 2 pour les utilier comme des boléen et optimiser un poil le code :)
3 juil. 2006 à 15:06
Tu pars avec l'envie d'améliorer ton code ? Sois imaginatif ! Regarde ce qui a été fait dans le domaine, trouve des fonctionnalités sympas, passes y quelques semaines et tu pourras sortir quelque chose de potable ;o)
Là c'est du déjà vu 1000 fois, y'a rien à dire d'autre :/
2 juil. 2006 à 10:47
T'es sûr d'avoir testé ton code avant de le poster ? car la commande "descrcribe" n'existe pas, c'est describe.
Ensuite, niveau redondance, on retrouve 3 fois la même portion de code, autant regrouper ca vu que le code est quasi-identique (ya juste la commande qui change, et éventuellement le message à envoyer)
alias -l bleh {
if !$2- { set %msg.hl 1,0et un 4,8 HIGHLIGHT 1,0 pour 4,8 $2 $+ 8,8-1,0, un ! :p }
else set %msg.hl $3-
set %comchans_ [ $+ [ $2 ] ] $comchan($2,0)
while (%comchans_ [ $+ [ $2 ] ] > 0) {
$iif($1 == 1,describe,msg) $comchan($2,%comchans_ [ $+ [ $2 ] ]) %msg.hl
dec %comchans_ [ $+ [ $2 ] ]
}
echo 7 $$1 s'est fait HL sur $comchan($2,0) chans
unset %comchans_ [ $+ [ $2 ] ]
}
alias bme bleh 1 $1 $2-
alias bmsg bleh 2 $1 $2-
C'est un peu plus court non ? :o
"c'est pas un addon genre hl paramétrable ou autre. C'est juste un alias tout con pour hl un pseudo sur plusieurs chan en même temps"
Ca c'est clair que ton addon risque pas de se démarquer des autres, car il est tout simplement moins complet que la pluspart des autres sources du genre.
http://www.ircfr.com/recherche.aspx?r=highlight&tr=source
Pourquoi poster moins bien ? Ca n'aurait pas été plus intéréssant d'en poster une plus complète que celle déjà existance ? Tu n'as plus qu'à améliorer ta source :)
PS : pense "qualité" et non "quantité", paufine tes sources, rends-les le plus complet possible, et essaie de passer plus de 10min par code ;) (N'hésite pas à passer plusieurs semaines sur un code, du moment que tu pars d'une bonne idée, il faut l'exploiter. Là, ton highlight c'est pas la meilleure idée, c'est vu et revu malheureusement)
Bonne continuation