Discussion :

[trealais]

Bonjour,
J'ai un script a écrire pour un travail de session. J'obtiens l'erreur suivante quand j'execute le script:

line 6: syntax error near unexpected token `do

Voici le code:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
#!/bin/sh
 
while [ -n $1 ]
 
do
 
list= ls $1
 
for i in $list
 
do
 
cat $i >> basename $i
 
rm $i
 
done
 
shift 1
 
done

Je suis certain que c'est une erreur de syntaxe mais j'arrive pas a trouver.

Merci

[N_BaH]

Ah, oui, mais là, non !

Allez, je vais pas être plus méchant, prend le temps de lire ce tutoriel.

[Sve@r]

Je suis certain que c'est une erreur de syntaxe mais j'arrive pas a trouver.Merci

Comment veux-tu que le shell fasse la différence entre

Code bash :

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

couleur=bleu

et

Code bash :

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

list=ls $1

???

Accessoirement, tu peux remplacer

Code bash :

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

1
2
3
4
5
while [ -n $1 ]
do
 .... $1
shift
done

par

Code bash :

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

1
2
3
4
for file in $*
do
 .... $file
done

[trealais]

Je viens de commencer a programmer en shell. En fait ca fait 1 semaine.
Faut me laisser une chance...

Comment veux-tu que le shell fasse la différence entre

Code bash :

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

couleur=bleu

et

Code bash :

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

list=ls $1

???

Comment écrirais-tu list=ls $1 alors?

Je cherches encore une solution

Merci

[trealais]

J'obtiens toujours l'erreur même si j'écris avec des apostrophes.

comme suit:

Code :

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

list='ls $1'

[Sve@r]

Je viens de commencer a programmer en shell. En fait ca fait 1 semaine.
Faut me laisser une chance...

ok

J'obtiens toujours l'erreur même si j'écris avec des apostrophes.

comme suit:
list='ls $1'

Ce que tu cherches à faire est le mécanisme le plus important du shell et ce qui fait sa force. Ca permet au shell de déléguer à d'autres programmes (comme "ls") ce qu'il ne sait pas faire lui-même (comme lister le contenu du répertoire).

Il faut encadrer la commande d'accent graves (ou backquottes). A ce moment là, le shell exécute la commande mais récupère pour son propre usage le texte qui aura été affiché. Et donc ici, ce texte sera stocké dans la variable "list"

Code bash :

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

list=`ls $1`

Dans les shells récents, on peut remplacer les backquottes par $(...) plus lisible

Code bash :

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

list=$(ls $1)

Ainsi, quel que soit ton besoin, t'auras toujours un programme qui te donnera ce que t'attends (au pire tu l'écris toi-même en C ou autre) puis tu demandes au shell d'exécuter ledit programme et grâce à ce mécanisme, tu récupères dans ton script le résultat dudit programme. Bref tu fais tout faire par ce qui existe au lieu de le réinventer à chaque fois.

S'il y a une seule chose que tu dois apprendre à faire en shell et que tu dois étudier hyper attentivement, c'est ça parce que grâce à ça, tous tes besoins en programmation shell sont à 90% déjà résolus. T'as simplement qu'à appeler le bon programme à chaque fois que t'as un truc à faire.

[trealais]

Dans les shells récents, on peut remplacer les backquottes par $(...) plus lisible

Code bash :

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

list=$(ls $1)

Merci pour les infos.
C'est bizzare j'obtiens toujours la même erreur et même ce après avoir remplacé:

Code :

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

list=`ls $1`

par

Code :

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

list=$(ls $1)

Le problème semble être avec:

Code :

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

1
2
3
for i in $list
 
do

Merci

[Zartan]

Il y a un problème avec le while que j'arrive à isoler mais je ne sais pas d'où cela vient, ça fait trop longtemps que je n'ai pas fait de bash pour ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
#!/bin/bash
while [ -n $1 ]
do
if [[ -n $1 ]]
then
	echo X$1X not null ;
else
	echo X$1X null ;
fi
shift 1
done

Il rentre dans le while alors que $1 est vide...

en utilisant for file comme indiqué ça fonctionne mais cela m'intrigue quand même.

[Sve@r]

Il y a un problème avec le while que j'arrive à isoler mais je ne sais pas d'où cela vient, ça fait trop longtemps que je n'ai pas fait de bash pour ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
#!/bin/bash
while [ -n $1 ]
do
if [[ -n $1 ]]
then
	echo X$1X not null ;
else
	echo X$1X null ;
fi
shift 1
done

Il rentre dans le while alors que $1 est vide...

en utilisant for file comme indiqué ça fonctionne mais cela m'intrigue quand même.

Remplace while [ -n $1 ] par while [ -n "$1" ] et t'auras plus ce soucis.

Dans les shells récents, on peut remplacer les backquottes par $(...) plus lisible

Code bash :

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

list=$(ls $1)

Merci pour les infos.
C'est bizzare j'obtiens toujours la même erreur et même ce après avoir remplacé:
list=`ls $1`
par
list=$(ls $1)

Le problème semble être avec:
for i in $list

do

Merci

Le meilleur débugging c'est le "echo". Rajoute un echo "[$list]" juste avant le for histoire de voir avec exactitude ce que contient ta variable...

[trealais]

Merci pour votre aide.
J'obtiens toujours l'erreur même avec: while [ -n "$1" ]
Je me demande maintenant comment implanter le: for file in $*

Est-ce comme ceci?:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
#!/bin/sh
 
for file in $*
 
do
 
for i in $file
do
 
cat $i >> `basename $i`
 
rm $i
 
done
 
shift 1
 
done

Si c'est le cas j'obtiens toujours la même erreur.
Il y a un mec sur un autre forum qui dit que mon code semble fonctionner sur Mac OS X.
Bizarre...

Merci pour votre aide.

[Sve@r]

Ok, avant de continuer, j'aimerais être sûr de ce que tu veux faire. Il me semble en lisant ton script que tu cherches à lister le contenu de répertoires et à recopier les fichiers desdits répertoires dans ton dossier courant.

Si c'est ça, alors ton script a une erreur dans le "for i in $file". Il te faut mettre à la place "for i in $file/*" afin d'avoir le contenu de "$file" et non son simple nom.

Accessoirement le shift 1 devient inutile. De plus, il peut être judicieux de choisir ses noms de variable en rapport avec leur rôle. $file semble être destiné à recevoir un nom de dossier donc autant l'appeler "$dir".

De plus, l'erreur de ton premier script devient claire. Quand tu fais "list=`ls $1`", la variable "list" ne contient que les noms des fichiers situés dans "$1" et non leur chemin d'accès. Exemple: si t'as un répertoire "aaa" qui contient "toto", ta variable "list" contiendra "toto" et non "aaa/toto".
Il s'ensuit que pour que ton premier script fonctionne, il suffisait de remplacer

Code bash :

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

1
2
cat $i >> basename $i
rm $i

par

Code bash :

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

1
2
cat $1/$i >> $i
rm $1/$i

A titre d'exemple, voici ton premier script que j'ai corrigé dans le but de le faire correspondre à ce que je crois que tu cherches à faire

Code bash :

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

1
2
3
4
5
6
7
8
9
10
11
12
#!/bin/sh
while test -n "$1"
do
    test ! -d "$1" && shift 1 && continue
 
    for file in `ls "$1"`
    do
        cat "$1/$file" >> "$file"
        rm "$1/$file"
   done
shift 1
done

Et le même avec un for à la place d'un while

Code bash :

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

1
2
3
4
5
6
7
8
9
10
11
#!/bin/sh
for dir in $*
do
    test ! -d "$dir" && continue
 
    for file in $dir/*
    do
        cat "$file" >> `basename "$file"`
        rm "$file"
   done
done

Tu remarqueras qu'il y a une différence notable entre "ls <nom_de_dossier>" et "<nom_de_dossier>/*"

[troumad]

trealais, il serait important que tu dises exactement ce que doit faire ton programme ! Et ce dès le début, ça éviterait de se poser trop de questions et ça permettrait de trouver plus rapidement la réponse...
Ainsi que les retours demandés par Sve@r, il serait bien de les donner ! Ce qu'il dit me semble cohérent bien que je ne sois pas un expert shell.

[trealais]

trealais, il serait important que tu dises exactement ce que doit faire ton programme ! Et ce dès le début, ça éviterait de se poser trop de questions et ça permettrait de trouver plus rapidement la réponse...
Ainsi que les retours demandés par Sve@r, il serait bien de les donner ! Ce qu'il dit me semble cohérent bien que je ne sois pas un expert shell.

Pas de problèmes voici l'énoncé de la question à laquelle je dois répondre:

Faire un script en Shell avec un nombre variable de paramètres qui sont des
directories existants dans le directory courant. Pour chacun des directories nommés, le script doit déplacer tous ses fichiers au directory courant. Si les noms des fichiers les mêmes, le script doit concaténer les fichiers sous un même nom dans le directory courant.


Merci pour votre aide!

[Sve@r]

Pas de problèmes voici l'énoncé de la question à laquelle je dois répondre:

Faire un script en Shell avec un nombre variable de paramètres qui sont des
directories existants dans le directory courant. Pour chacun des directories nommés, le script doit déplacer tous ses fichiers au directory courant. Si les noms des fichiers les mêmes, le script doit concaténer les fichiers sous un même nom dans le directory courant, SAUF s'il s'agit d'un fichier
de type PDF où le programme élimine simplement le fichier.

Quand les directories nommés en paramètre sont vidés, ils doivent être détruits.

N.B. On suppose que les directories nommés ne contiennent que des fichiers (pas de sous-directories), et que lorsque votre programme trouve un nom identique au directory courant, ce dernier est un fichier valide à concaténer.

Merci pour votre aide!

Ouais ok. Mes exemples correspondent quasiment à ce que tu dois faire. Faut juste rajouter la commande d'effacement du dossier devenu vide (rmdir) et le filtre du pdf => if echo $file |grep -y "\.pdf$" 1>/dev/null renverra vrai si ton nom de fichier porte l'extension ".pdf" en minuscules ou majuscules.

Toutefois tu triches un peu car tu fais une concaténation systématique alors que ton énoncé suppose à un moment donné un examen du fichier trouvé pour voir s'il n'existe pas déjà. Ca marche mais si c'est un TP, alors peut-être que ton prof attend de toi que tu écrives ce test afin que tu lui montres que t'es bien à l'aise à la fois dans ta tête (ça ça semble ok) et à la fois dans ton shell (un peu d'expérience et ça viendra aussi).

Appuies-toi sur mes scripts pour comprendre leur façon d'agir et sur mes remarques à propos de la façon dont tes variables sont remplies (le coup des chemins d'accès présents ou pas) et complète ton tp.