[BASH] Condition sur valeur numérique

Bonjour,

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

Code:

---

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

Citation:

---

Envoyé par Newenda

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

---

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

Salut,

Code:

---

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

---

;)

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

Code:

---

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

---

il me renvoyait un "commande introuvable" :

Code:

---

1 2	$ $FileSize 13121743872: commande introuvable

---

?

merci encore

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 !

Citation:

---

Envoyé par Newenda

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:

---

echo $FileSize

---

Code:

---

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

---

:dehors::ptdr:

Salut, (ça mange pas de pain)

Citation:

---

Envoyé par 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".

---

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.

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

Citation:

---

Envoyé par zipe31

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 ?

Citation:

---

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. :mouarf:

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.

Citation:

---

Envoyé par N_BaH

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… ;)

Citation:

---

Envoyé par N_BaH

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.

Citation:

---

Envoyé par zipe31

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:

---

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:

---

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

---

:-o 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. :sm:

Citation:

---

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. ;)

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

Code:

---

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*»)

---

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

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

Code:

---

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.