NOUVEL ALIAS POUR HASH TABLE /HREP

Commentaire de SornDrixer le 06/07/2004 14:36:47 administrateur CS

Tes 3 if de vérification d'erreur, tu peux en faire qu'un seul :
if !$3 { echo -a *** /hrep Erreur Paramètre manquant | halt }
Si $3 n'existe pas, c'est que $1 et $2 non plus.
Si cela te gêne cette méthode de raisonner, tu peux toujours faire :
if (!$1 || !$2 || !$3) { }
Si tu veux absolument afficher à quoi correspond le paramètre manquant, utilises $iif()

Voila, sinon c'est une très bonne chose de vouloir compléter les commandes pour hashtable, mais je trouve le code un peu maigre.
une idée :
En spécifiant le paramètre -n à ton alias, si jamais la table n'existe pas, alors ton alias la crée.

Commentaire de ReMi34 le 06/07/2004 16:36:15

Wapsnest avait réaliser la même chose :/

Commentaire de Seregon le 06/07/2004 19:52:29

$1 $2 devien $1-2
$1 $2 $3 devien $1-

Commentaire de Melnofil le 23/11/2004 03:32:28

Bravo félicitations je t'ai mis 1 (Bon maintenant je me justifie avant de me faire casser la g***le)

Dans un alias appelé sous la forme /hrep il est impossible que les paramêtres $1 et $2 soient nuls si $3 ne l'est pas. Et plus simplement tu peux tester si "$0 > 2" ($0 contient le nombre de paramètres).
Note : Dans un identifiant il faudrait d'abords utiliser "/tokenize 32 $1-" pour parvenir au même résultat.

De plus tu utilise l'option "-m", je cite l'aide :
"L'option -m fait que /hadd crée la table si elle n'existe pas déjà."
Et ben alors... pkoi diable est-ce qu'on viens de faire un "if (!$hget($1))" ??

De plus il est toujours plus judicieux (pas obligatoire !) de traiter en premier les cas ou tout ce passe bien, puis traiter les erreurs à la fin ; surtout quand on se trouve dans une structure if/then/else facile à lire.

Et enfin une remarque -juste pour faire plus classe- on peut remplacer tous les 5 "halt" par seulement 1 "halt" pour le prix d'un "return" en plus.

Voilà ce que ca donne :

alias hrep {
if $0 > 2 {
if $hget($1,$2) != $null { .hadd -m $1- | return }
elseif !$hget($1) { echo -a *** /hrep Erreur table $1 Inexistante }
else echo -a *** /hrep Erreur Item $2 de la table $1 Inexitant
}
else echo -a *** /hrep Erreur Il manque $calc(3 - $0) paramètre(s).
halt
}

Commentaire de tofu le 23/11/2004 07:44:00

j'ai rajouté le point pour qu'on voye dans mon commentaire que ct une commande, comme si je connaissais pas les hash tables lais mdr quoi ..