Script shell erreur débutant

Bonjour à tous j'ai un petit soucis des que j'essai d'executer mon script shell qui a pour but d afficher la taille totale de tout les fichier pg d'un dossier , car j ai l erreur "unexpected end of file" je suppose que c'est une erreur de point ou de détail mais je debute dans le milieu je n arrive pas a demasquer cette erreur merci d'avances à tout ceux qui me mettrons sur la piste :)
le voici

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

#!/bin/bash if(($#==0)) then echo "il faut au minimun au moin 1 agument" exit 1 fi for $arg do if (test ! -d $arg) then echo "$arg : est innexistant" else if (test ! -r $arg) if find $arg/*jpg 1>/dev/null 2>/dev/null then $P>(du -c $arg/*.jpg | grep total) echo "$arg pèse $P octés" ; fi

---

Bonjour,

Citation:

---

j ai l erreur "unexpected end of file"

---

bah, oui ! c'est du grand n'importe quoi ton script !

on va te le dire, à toi aussi :
avant d'écrire des commandes dans un script, on les teste dans le terminal !

ligne 1 : il manque une espace
ligne 7 : les tests n'ont pas à être dans un sous-shell.
ligne 9 : tous les if doivent être fermés par un fi.
ligne 11 : $P n'existe pas.

Bonsoir merci de votre réponse rapide,effectivement je n avais pas vu pour l espace :oops: , les test à l interieur m'ont été conseillés par mes professeurs , pour le fi si je les met j ai des erreurs c 'est pour cela que j ai laisser que le dernier et pour le $p je voulais faire une affectation pour afficher dans la phrase finale mais apparemment il ne faut pas faire comme cela . merci de votre aide

je ne dirai pas ce que je pense des profs :massacre:, parce qu'il y en a de très bons ici :ange:, mais ailleurs... :cfou:

poste à nouveau ton script avec les modifications, qu'on voit où tu en es.

je suis obliger de laisser les test car ils le veulent en revanche pour la fin sans affecter de variables pour le total je ne vois pas car ce que j ai écrit dans ce code est impossible :weird:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

#! /bin/bash if(($#==0)) then echo "il faut au minimun au moin 1 agument" exit 1 fi for $arg do if (test ! -d $arg) then echo "$arg : est innexistant" else if (test ! -r $arg) if find arg/*jpg 1>/dev/null 2>/dev/null then (du -c $arg/*.jpg | grep total)= $p echo "$arg pèse $p octets "; fi

---

c'est pas les test le problème, c'est qu'ils soient inutilement entre parenthèses (dans un sous-shell).

quant aux if, il doit y avoir autant de fi !

Code:

---

if test ! -r "$arg"

---

il manque une partie du code : if...;then...(;elif...);else...;fi !

Citation:

---

Envoyé par N_BaH

quant aux if, il doit y avoir autant de fi !

---

Oui... et, si possible, autant de do que de done... sinon on risque d'avoir quelque chose comme "unexpected end of file".

Attention! Si tu utilises else if, tu as besoin d'un fi de plus que si tu utilises elif.

De plus, pour éviter ce genre d'erreur, il est assez commode d'utiliser un éditeur de texte qui fasse l'indentation à ta place (j'utilise emacs, mais il y en a plein d'autres).

Citation:

---

Envoyé par Aurelioux

pour le $p je voulais faire une affectation pour afficher dans la phrase finale mais apparemment il ne faut pas faire comme cela

---

Pour affecter une variable, il faut... commencer par lire un peu de doc ou de tutoriels, plutôt que d'essayer des trucs comme:

Code:

---

$P>(du -c $arg/*.jpg | grep total)

---

Code:

---

(du -c $arg/*.jpg | grep total)= $p

---

Ou alors, carrément essayer un truc comme:

Code:

---

p=$(du -c $arg/*.jpg | grep total)

---

Bonjour,

Après avoir corrigé toutes les petites erreurs qui ont été citées, il faudra revoir le fonctionnement de la commande find (=>man find), car son utilisation dans un if est improbable: la commande find retournera vrai même si elle ne trouve pas de(s) fichier(s)...

Citation:

---

Envoyé par jack-ft

Oui... et, si possible, autant dedo que de done

---

pas nécessairement, dans ce cas précis : for i do :peu importe; done est POSIX, et est équivalent à for i in "$@"; do :peu importe; done:)
est-ce qu'Aurelioux le savait ???

Citation:

---

Envoyé par N_BaH

pas nécessairement,

---

Euh... ben si... enfin, je crois... Je persiffle et singe: autant de do que de done (que le do vienne de for, while ou until (cf. syntaxe)).

Citation:

---

dans ce cas précis : for i do :peu importe; done est POSIX, et est équivalent à for i in "$@"; do :peu importe; done:)

---

mais pas nécessairement autant de in que de for. C'est ça que tu voulais dire, N_BaH? :koi:

(aurais-tu besoin de vacances? ;))

Dans les autres langages que je pratique ne necessite pas de done dans une boucle for donc je n'en ai pas mis et personne de mon entourage ne m'avait dit d 'en mettre
En effet j ai rajouter les fi et elif mais je pense que le elif n'est pas a sa place et je pense avoir réussi a donner une valeur a $p
pour le find je vais revoir cela

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

#! /bin/bash if(($#==0)) then echo "il faut au minimun au moin 1 agument" exit 1 fi for $arg do if (test ! -d $arg) then echo "$arg : est innexistant";fi elif (test ! -r $arg) else if find $arg/*jpg 1>/dev/null 2>/dev/null then $p=(du -c $arg/*.jpg | grep total) echo "$p" ; fi

---

Citation:

---

Envoyé par Aurelioux

Dans les autres langages que je pratique ne necessite pas de done dans une boucle for

---

Hum... Quels autres langages pratiques-tu?

Dans la plupart des langages, soit la boucle for accepte de n'avoir qu'une seule instruction à répéter, soit elle impose un bloc délimitant le groupe d'instructions à répéter, par exemple, avec une accolade { (équivalent du do) et une fermante } (équivalent du done).

Citation:

---

donc je n'en ai pas mis et personne de mon entourage ne m'avait dit d 'en mettre

---

Et ça marche sans done ? :koi:

Citation:

---

je pense avoir réussi a donner une valeur a $p

---

Euh... au risque de me répéter: et ça marche? :koi:

Citation:

---

Envoyé par jack-ft

C'est ça que tu voulais dire, N_BaH?

---

<mauvaise foi>oui, oui, c'est ça.</mauvaise foi>
ou c'est juste que contrairement au shell, moi, je ne sais plus compter. :oops:

Citation:

---

Envoyé par Aurelioux

j ai rajouter les fi et elif mais je pense que le elif n'est pas a sa place

---

le elif n'est surtout pas utile.

en des termes quotidiens :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17

pour arg dans la liste des arguments du script faire si $arg n'est pas un dossier alors afficher_erreur autrement si $arg n'est pas accessible en lecture alors afficher_erreur autrement si il existe un/plusieurs fichier(s) *.jpg alors afficher la somme de la taille des fichiers *.jpg finsi finsi finsi finpour

---

les parenthèses autour des tests ne sont toujours pas nécessaires, et ralentissent l'exécution du script.
il faut un signe dollar devant la substitution de commande : variable=$(commande | commande); variable, qui ne sert, d'ailleurs, à rien. :(

dans les langages que tu pratiques, l'indentation n'est pas requise ?
à défaut d'être obligatoire, comme en python, elle aide à la lecture d'un script, et donc son débogage. :)

Citation:

---

personne de mon entourage ne m'avait dit d 'en mettre

---

elle est énorme celle-là !
et t'as pas lu un petit tuto, histoire de voir comment ça marche ?

le shell n'est pas un sous-langage.
c'est un langage simple pour exécuter des tâches simples.
il ne mérite pas moins de respect en le pratiquant, qu'un autre.
non, mais !

tu me copieras 10000 fois : « je dois poser un pied devant l'autre pour avancer »
quoi ? personne ne te l'avait dit, non plus ?

merci à vous j ai finalement réussi il fallait bien mettre les fi et le done ;)
mais il me reste une question quand j utilise du dans le terminal cela me donne la taille du fichier mais ce qui est intrigant c est que celle ci est deux fois superieur à la taille du fichier une idée?:cfou::cfou: :mrgreen:

Normal,

Ton BLOCKSIZE doit être à 512 au lieu de 1024, utilise l'option '-k' pour un BLOCKSIZE à 1024.

je ne connaissais pas merci de l'aiguillage mais quel commande dois-je utiliser pour ilitialiser BLOCKSIZE -K ?

Par l'exemple:

Code:

---

1 2 3 4 5 6

$ du -c script.zip 520 script.zip 520 total $ du -kc script.zip 260 script.zip 260 total

---