Commentaires sur une source : STATISTIQUE DE VOS DISQUES DUR.

cs_PaTaTe Messages postés 2126 Date d'inscription mercredi 21 août 2002 Statut Contributeur Dernière intervention 19 février 2021 2
21 févr. 2007 à 16:24

Y a t-il moyen de detecter tous disque durs en vigueur sur la machine et d'additionner le tout ?

RCA ArKanis Messages postés 1287 Date d'inscription mercredi 21 avril 2004 Statut Membre Dernière intervention 21 février 2009
4 nov. 2006 à 21:21

encore un truc qui choque ...

if ($chr(58) !isin $$2) { .timer 1 2 msg $chan 1[ Analyse ] Nom du volume invalide ! Veuillez respecter cette syntaxe :1 .state c: | halt }
tu /halt, donc tu ne verras pas le .state, mais ton timer va tout de même faire envoyer un message sur le salon. Tu ne ferais pas mieux de mettre un echo ?

sinon, vu que tu as mis if (!$2-) { ... halt }, tu n'as pas besoin de doubler le $ à la ligne suivante, et le $chr j'pense pas qu'il soit utile. Ca devient :
if (: !isin $2)

en fait, ce qu'il serait bien, c'est d'avoir :
if (!$2-) .timer 1 1 msg $chan ...
elseif (: !isin $2) .timer 1 1 msg $chan ...
else {
etc ....
}

sinon le else n'a aucun intérêt vu que tu /halt

Nort0n Messages postés 41 Date d'inscription jeudi 27 novembre 2003 Statut Membre Dernière intervention 18 octobre 2010
4 nov. 2006 à 17:25

Bon aller, la source dort depuis quelques temps, et vu les commentaires, j'vais me lancer dans une p'tite optimisation/Mise à Jour du code.

Merci pour vos comments, j'espère avoir quelques nouvelles quand la remote sera up ^^

GenerationNoNNoN Messages postés 10 Date d'inscription dimanche 18 juin 2006 Statut Membre Dernière intervention 23 juin 2006
19 juin 2006 à 02:19

Tu set beaucoup trop de variables je pense, sinon sa pourrait être optimisé .

cs_wims Messages postés 2466 Date d'inscription vendredi 23 juillet 2004 Statut Membre Dernière intervention 1 août 2010 1
21 avril 2006 à 12:50

Hum le site a buguer dsl du double post :)

cs_wims Messages postés 2466 Date d'inscription vendredi 23 juillet 2004 Statut Membre Dernière intervention 1 août 2010 1
21 avril 2006 à 12:50

Tout a ete dis par Arkanis sauf deux trucs :)

if ( $disk($$2).size == $null ) { .timer 1 2 msg $chan 1[14Analyse1]14 Ce volume n'est pas supporté par votre ordinateur, ou n'est pas présent ! | halt }
if ( $disk($$2).size == 0 ) { .timer 1 2 msg $chan 1[14Analyse1]14 Ce volume n'est pas supporté par votre ordinateur, ou n'est pas présent ! | halt }

Tu peux regrouper ca en une seul condition :

if (!$disk($$2)) { .timer 1 2 msg $chan 1[14Analyse1]14 Ce volume n'est pas supporté par votre ordinateur, ou n'est pas présent ! | halt }

Et :
if ( %type.disk == fixed ) { set %trans.type Interne }
if ( %type.disk != fixed ) { set %trans.type Externe }

par

set %trans.type $iif(%type.disk == fixed,Interne,Externe)

Sinon :

Pour les locales, j't'avoue que j'les utilises TRES rarement (c'est peut-être un tort -_-)

oui :)

cs_wims Messages postés 2466 Date d'inscription vendredi 23 juillet 2004 Statut Membre Dernière intervention 1 août 2010 1
21 avril 2006 à 12:49

Tout a ete dis par Arkanis sauf deux trucs :)

if ( $disk($$2).size == $null ) { .timer 1 2 msg $chan 1[14Analyse1]14 Ce volume n'est pas supporté par votre ordinateur, ou n'est pas présent ! | halt }
if ( $disk($$2).size == 0 ) { .timer 1 2 msg $chan 1[14Analyse1]14 Ce volume n'est pas supporté par votre ordinateur, ou n'est pas présent ! | halt }

Tu peux regrouper ca en une seul condition :

if (!$disk($$2)) { .timer 1 2 msg $chan 1[14Analyse1]14 Ce volume n'est pas supporté par votre ordinateur, ou n'est pas présent ! | halt }

Et :
if ( %type.disk == fixed ) { set %trans.type Interne }
if ( %type.disk != fixed ) { set %trans.type Externe }

par

set %trans.type $iif(%type.disk == fixed,Interne,Externe)

Sinon :

Pour les locales, j't'avoue que j'les utilises TRES rarement (c'est peut-être un tort -_-)

oui :)

Nort0n Messages postés 41 Date d'inscription jeudi 27 novembre 2003 Statut Membre Dernière intervention 18 octobre 2010
21 avril 2006 à 01:51

Merci de ta remarque. Tu as parfaitement raison, j'aurais pu faire plus court. Quant aux variables, j'les avais surtout utiliser pour eviter de poser les calcules directement dans les msg (c'est inutile, j'te l'accorde, c'est pour mieux decortiquer la remote).
Pour les locales, j't'avoue que j'les utilises TRES rarement (c'est peut-être un tort -_-)
Niveau interet, j'en vois pas reelement non plus, mais c'était pour un ami, et avoir des stats fiables assez rapidement.
Et pour conclure, pour le if ( $2- == $null ) a la place du (!$2-), c'est une vieille habitude que j'ai prise, ça m'fait tapoter plus de code, mais j'ai tjrs fais comme ça. (Qui a dit que j'etais feignant ? ;p)

RCA ArKanis Messages postés 1287 Date d'inscription mercredi 21 avril 2004 Statut Membre Dernière intervention 21 février 2009
21 avril 2006 à 00:14

tu utilises beaucoup trop de variables à mon goût :x

pense aux variables locales ! (/var)

tu n'en as même pas besoin, remplace dans tous tes timers la valeur d'origine des variables
ex :
.timer 1 3 msg $chan 1Type de disque :14 $iif($disk($$2).type == fixed,Interne,Externe) $+ 1
...
.timer 1 8 msg $chan 1Le volume est donc rempli à14 $round($calc(100 - %per.free.s),2) $+ % $+ 1.
(tu peux utiliser %per.free.s vu que tu l'utilises 2x, pareil avec %free.space)

if ( $2- == $null )
ici, tu peux utiliser if (!$2-), car de toute façon, volume $false ou 0, ça te retournera une erreur à un moment ...

T'aurais pu faire dire des lignes plus longues à mon avis. Comme tu l'as fait, ça flood un peu trop.

Point de vue intérêt : ça dépend des personnes. Moi, ça me branche pas trop mais bon, ...

STATISTIQUE DE VOS DISQUES DUR.

Votre réponse

Discussions similaires