G-CONTACT MANAGER 1.0

Commentaire de davelister le 01/08/2004 13:35:23

Pas mal, juste un petit bug à la progress bar, je te le signale sur IRC :)

Commentaire de GFACTION le 01/08/2004 13:49:32

C'est étrange ce bug, j'ai testé avec un mIRC vierge, j'ai loadé... je me suis connecté, j'ai /who mon canal. Je tape /gcnt, et tout marche correctement... je ne comprends pas :/

Commentaire de tofu le 01/08/2004 14:40:58

le concept est bien, original, les explications sont claires, par contre je comprends pas pourquoi tu mets un on dialog par item pour ton menu ? oO

Commentaire de Kerrigan le 01/08/2004 15:24:26

c'est un code vraiment tres intéressant. Mais tu as fait une petite erreur d'optimisation :

var %nick $6
  var %host $4
  var %ident $3
  var %server $5
  var %realname $9-
  var %mainnick $mainnick(%nick)

tu peux mettre ça sur une seule ligne. Ca ne te ressemble pas de faire ce genre d'erreur ...

bref continuons :

while (1) {
      inc %o
      var %tmpnick $hget(%tnick,Host $+ %o)
      if (%tmpnick) { did -a gcnt 1 3 3 %tmpnick }
      else { break }
    }

Ce bout de code me laisse perplexe. C'est vraiment tres élégant certe, mais pourquoi ne pas la structurer comme une vraie while ?

je te propose ça :

var %o = 1
while ($hget(%tnick,Host $+ %o)) {
    did -a gcnt 1 3 3 $ifmatch
    inc %o
  }

c'est beacoup plus court et la seule condition est celle de la while, ça doit donc etre plus rapide. De surcroit tu utilises cette méthode assez souvent donc cette optimisation ne devrait pas faire de mal a ton code.

if (%n = 0) did -i %d %i 1 cb root
devient :
if (!%n) did -i %d %i 1 cb root

l'alias combine avec ces trois while imbriqué c un vrai tout de force :) (bravo)


dans l'alias "branch" les variable %d et %i ne servent a rien. A moins que leur existence soient justifiées, j'aimerais bien comprendre pourquoi tu les utilises. meme commentaire pour %b et %n

regarde ce que je te propose :

alias branch {
   if ($dialog($1)) {
       if (!%n) did -i $1 $2 1 cb root
    else { did -i $1 $2 1 cb root $gettok($gettok($did($1,$2,1),4-,32),1- $+ $ifmatch,32) }
    return $gettok($gettok($did($1,$2,$gettok($gettok($did($1,$2,1),4-,32),-1,32)),7-,32),1,9)
  }
}

c'est moins clair, mais ça doit etre plus rapide a mon avis. (cela reste a démontrer)

l'alias drop2 st une belle utilisation de $findfile (bravo)

Sinon je suis étonnée que Sorn_Drixer ne t'ait pas dit de regrouper les event de tes dialogues.

J'estime aussi que tu aurais pu donner le choix entre le français et l'anglais pour ton dialogue. Tu es francophone quoi... pense a tes compatriotes.

Voila c'est en gros tout ce que j'avais a dire. Le plus important est l'optimisation de la while que je te propose, le reste se sont des petits détails qui sont plus du style de chaque scripteurs que de la réelle optimisation.

Ca vaut 9/10, tu as perdu un point a cause des var et de la while.

Bon code.

Commentaire de Hades53 le 01/08/2004 18:12:53

Enfin un truc utile pour l'IRC.
L'idée est très bonne ;)

Maintenant en optimisant le tout, la source sera vraiment excellente.

Petite remarque:
"alias noext {
  unset %noext
  unset %pos
  var %i 1
  set %len $len($1)
  while (%i <= %len) {
    if ($mid($1,%i,1) == $chr(46)) {
      set %pos %i
    }
    inc %i
  }
  if (%pos) { set %noext $left($1,$calc(%pos - 1)) }
  else { set %noext $1 }
  return %noext
}
"
peut-être remplacé par ceci:
alias next {
  if ($numtok($1,46) == 1) return $1
  return $deltok($1,-1,46)
}

Commentaire de Kerrigan le 01/08/2004 20:21:39

Je reprend je n'ai pas été clair en fait je dit que

  var %nick $6
  var %host $4
  var %ident $3
  var %server $5
  var %realname $9-
  var %mainnick $mainnick(%nick)

peut etre remplacer par

var %nick  = $6,%host =  $4,%ident =  $3,%server =  $5,..

la prochaine fois je serais plus clair.

Commentaire de PaDa le 01/08/2004 22:08:37

j'aime beaucoup ta source , c'est très abouti ...
je regrette cependant (tu n'y es pour rien) que ce soit inutilisable sur de gros salons ... le /who qui me permet de rafraichir l'ial a l'entrée du salon me fait freezer correctement sur un salon de 450 users , j'ose pas imaginer a 800 je pars en ping timeout :( et puis avec le nombre de join part a la seconde j'me demande pourquoi le server m'a pas jété avec autant de /who ...
sinan rien a redire les ptites optimisations ont été dites je pense bwavo quand même :p

Commentaire de abhoryo le 05/08/2004 01:09:48

Hum
bien bien
comme c'est dommage : "En tout cas, ça convient sur des réseaux où seuls quelques canaux dépassent 100 users :)" ca existe des réseau si petit ?.
enfin bref passons. je ne reproche rien sur l'anglais étant bilingue mais je pense aux autres... pkoi ne pas faire un option qui traduise tout en francais et d'autres langues???

Bref étant donné que ce programme soit limité si vite, il n'est d'aucune utilité. Tu as perdu ton temps.

Commentaire de GFACTION le 05/08/2004 12:27:25

abhoryo, on voit que tu connais mal IRC, comme le dit tofu y'en a des milliers des réseaux comme ça, et même EpiKnet qui est le 3ème réseau francophone, alors chut :)

Programme limité ? Seulement car des gens ne sont même pas foutus de parler une langue essentielle ? -_-
D'aucune utilité ? Hum.. soit tu es limité psychologiquement, ou alors tu le fais exprès, car justement c'est un addon original, et très utile, quelque chose qu'on avait jamais vu.
C'est bizarre ça.. un seul mec contre tout le monde :o

Allez ciao abhorymachin.

Commentaire de tempusus le 06/08/2004 21:58:43 administrateur CS

Bonsoir.

un seul mot: Excellent.

Chez moi pas de bug visible et bien utile pour les netadmins :))

Quand aux reproches qui te sont fait... C'est triste mais malheureusement certains ne savent pas evaluer correctement une bonne source

8/10

Tempusus
Admin CS

Commentaire de PaDa le 09/08/2004 21:27:01

faire ami ami avec les ircops ? sur undernet ou quakenet , avec 200 000 de max users je pense que les ircops ont autre chose a faire lol
'fin bon :)
et puis mirc a toujours des bugs quelques soient les versions mais je vois pas vraiment ce que tu pointes du doigt par "buggé a l'origine" ... a chaque release des fonctions sont améliorées corrigées j'veux bien un exemple de véritable bug non signalé que tu aurais déniché :)

Commentaire de KiNdErMaXi le 27/09/2004 21:08:40

super joli, méga utile,
génial