Discussion :

[Newenda]

Bonjour,

J'aimerais faire une condition sur la valeur de la taille d'un fichier

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
#! /bin/bash
 
FileSize='stat -c "%s" <file.ext>'
 
if [ $FileSize -gt 100000 ] 
then echo "ok"
else echo "non"
fi

FileSize est une chaine de caractère.. comment la passer en valeur numérique pour servir dans la condition?

Merci d'avance

[al1_24]

FileSize est une chaîne de caractères..

C'est vague...
Quand tu l'exécutes, que reçoit exactement FileSize ?

[zipe31]

Salut,

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
#! /bin/bash
 
FileSize=$(stat -c "%s" <file.ext>)
 
if [ "${FileSize}" -gt 100000 ] 
then echo "ok"
else echo "non"
fi

[Newenda]

Merci, cela semble marcher.
Cependant je me demandais pourquoi en définissant FileSize de cette manière:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$ FileSize=$(stat -c "%s" <fichier.ext>)

il me renvoyait un "commande introuvable" :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
$ $FileSize
13121743872: commande introuvable

?

merci encore

[N_BaH]

les accolades , quand il n'y a rien d'accolé, sont inutiles.
ainsi que les guillemets autour de nombres, parce qu'ils sont "insécables".

comment sait-on que c'est un nombre ?
parce qu'on utilise un opérateur de comparaison arithmétique !

[Flodelarab]

pourquoi en définissant FileSize de cette manière, il me renvoyait un "commande introuvable" ?

As-tu réellement un logiciel, un script, un alias, ou un lien qui s'appelle 13121743872 ? Non ? Ben voilà. Tu as la réponse à ta question. On n'exécute pas 13121743872. La substitution de commande $( ) a rempli la variable FileSize avec le résultat de la commande. Si tu voulais l'afficher, il faudrait faire :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

echo $FileSize

[disedorgue]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
$ $(stat --format "test %s -gt 100000" fichier2.txt) && echo ok || echo non
non
$ $(stat --format "test %s -gt 100" fichier2.txt) && echo ok || echo non
ok

[zipe31]

Salut, (ça mange pas de pain)

les accolades , quand il n'y a rien d'accolé, sont inutiles.
ainsi que les guillemets autour de nombres, parce qu'ils sont "insécables".

T'es tout le temps en train de rouspéter après tous les zigues qui ne respectent pas les fondamentaux en shell, donc, même si c'est inutile, autant prendre de bonnes habitudes, non ?

Bonne journée.

[Newenda]

Merci à tous ! chaque réponse m'a été utile.

[balkany]

même si c'est inutile, autant prendre de bonnes habitudes, non ?

La bonne habitude ne serait-elle pas plutôt de savoir pourquoi on met des guillemets et des accolades, et de ne les mettre dès lors que quand il faut ?

[N_BaH]

T'es tout le temps en train de rouspéter après tous les zigues qui ne respectent pas les fondamentaux en shell, donc, même si c'est inutile, autant prendre de bonnes habitudes, non ?

non, là, c'est comme étaler du beurre sur une tranche de quatre-quart, ou de saupoudrer du sucre sur une part de kouign amann.

ici, le fondamentaux , je le redis, et je ne sais pas comment le dire autrement tellement ça me semble évident : si il n'y a rien d'accolé à la suite de la variable, les accolades ne sont pas nécessaires.
or, si ce n'est pas indispensable, le faire est inutile.

[zipe31]

si il n'y a rien d'accolé à la suite de la variable, les accolades ne sont pas nécessaires.
or, si ce n'est pas indispensable, le faire est inutile.

Avec des SI on mettrait Paris en bouteille…

comment sait-on que c'est un nombre ?
parce qu'on utilise un opérateur de comparaison arithmétique !

Encore faut-il être sûr de ce que contient la variable.

[Flodelarab]

Encore faut-il être sûr de ce que contient la variable.

Je pense pareil.

Soit un test anodin (sans guillemets puisque les nombres sont insécables dit-il).

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

if test $zozo -eq 0;then echo OK;else echo bouh;fi

Évidemment, dans un monde idéal, ça marche bien. Mais imaginons qu'à l'endroit où cette valeur est récupérée, un hacker a placé un code malicieux arbitraire; au sens anglais du terme, c'est-à-dire n'importe quel code que l'on souhaite.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
$ zozo='-n "$(ls)" -o 0'
$ if test $zozo -eq 0;then echo OK;else echo bouh;fi
OK
$ if test "$zozo" -eq 0;then echo OK;else echo bouh;fi
-bash: test: -n "$(ls)" -o 0 : nombre entier attendu comme expression
bouh

Ohhhhh ! Quelle bonne idée d'avoir mis la variable entre guillemets ! Sinon, non seulement on peut exécuter du code arbitraire (ici "ls"), mais en plus, on peut forcer le branchement conditionnel en branchement inconditionnel (0 == 0). C'est le client qui ne va pas être content.

[N_BaH]

si quand il n'y a rien d'accolé...

on part quand même du postulat qu'on va comparer des nombres puisqu'on utilise un opérateur de comparaison ARITHMÉTIQUE !!!

si quand on ne sait pas ce que va contenir la variable, mais qu'on sait que ce doit être un nombre, on fait un teste en amont pour interrompre le script parce que le type de données ne correspond pas à ce qui est attendu, et que sinon le client ne sera pas content.

[Flodelarab]

Ou alors, on en revient au C/C++. Il faut explicitement typer les variables dès le début.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
$ declare -i zozo
$ zozo='-n "$(ls)" -o 0'
-bash: -n "$(ls)" -o 0 : erreur de syntaxe : opérateur arithmétique non valable (le symbole erroné est «*"$(ls)" -o 0*»)

[N_BaH]

pas de problèmes; puisqu'on veut que la variable soit un nombre.

[disedorgue]

Dans ce cas précis, le mieux serait de passer, comme suggérer par Flodelarab, par un declare -i et de passer par:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
declare -i XX
XX=<un nombre ou une commande qui envoi un nombre>
# Et la syntaxe pour le test numérique correcte serait:
[ ${XX:=0} -gt 1000 ]

Dans le principe, on ne mettrait pas de guillemet car cela symbolise une chaine de caractère alors qu'ici on traite un nombre.