Petite boucle difficile en sh

Résolu
marine -  
zipe31 Messages postés 36402 Date d'inscription   Statut Contributeur Dernière intervention   -
Bonjour,

observez cette séquence svp: X a b c d Y 1 2 3 4

Je souhaite écrire une boucle en sh qui fait ceci : si l'argument lu est X, on crée les répertoires a b c d, si l'argument lu est Y on crée les fichiers 1 2 3 4. Je travaille depuis 2 jours dessus et j'ai essayé pleins de méthodes. Une me parait prometteuse mais elle bloque, je vous décris l'algorithme et vous montre mon code:

Pour chaque élément dans l'ensemble des arguments
si l'élément lu est X
on incrémente de 1 la variable indice
on affecte à une variable z la valeur X
sinon si l'argument lu est différent de X, que z vaut X et que l'argument est différent de Y
on crée le répertoire de l'argument indice

Voici mon début de code:

for element in "$@"
do
if [ "$element" = "-d" ]
then
indice='expr $indice + 1'
"$y"="$element"
elif [ "$element" != "-d" ] && [ "$y" = "-d" ] && [ "$element" != "-f" ]
then
mkdir rep/$indice
fi
done

Mais dans mon shell il me sort =-d : commande introuvable.
DOnc je suis un peu désespérée, quelqu'un aurait il une idée svp?
merci bcp.

12 réponses

Réponse 1 / 12
zipe31 Messages postés 36402 Date d'inscription   Statut Contributeur Dernière intervention   6 430
 
Salut,

Je te propose une approche différente ;-)

On découpe la liste des arguments en 2 (avec "sed" en ajoutant un saut de ligne devant les lettres X ou Y lorsqu'elles sont précédées par un espace), puis on teste si la ligne commence par X ou par Y et en fonction on crée des répertoires ou des fichiers avec les arguments restants...

Ce qui donne : 

$ cat marine.sh 
#! /bin/bash

#set -xv

while read line
do
if [ "${line%% *}" = "X" ]
        then mkdir $(echo ${line#* })
        else touch $(echo ${line#* })
fi
done < <(echo "$@" | sed 's/ X\|Y/\n&/g')

$ ./marine.sh X a b c d Y 1 2 3 4

$ ls
1  2  3  4  a/  b/  c/  d/  marine.sh*

$ ls -l
total 4
-rw-r--r-- 1 jp jp   0 2010-11-25 18:00 1
-rw-r--r-- 1 jp jp   0 2010-11-25 18:00 2
-rw-r--r-- 1 jp jp   0 2010-11-25 18:00 3
-rw-r--r-- 1 jp jp   0 2010-11-25 18:00 4
drwxr-xr-x 2 jp jp  40 2010-11-25 18:00 a/
drwxr-xr-x 2 jp jp  40 2010-11-25 18:00 b/
drwxr-xr-x 2 jp jp  40 2010-11-25 18:00 c/
drwxr-xr-x 2 jp jp  40 2010-11-25 18:00 d/
-rwxrw-r-- 1 jp jp 177 2010-11-25 17:59 marine.sh*

;-))
0
Réponse 2 / 12
marine
 
bonjour zipe et merci, mais je n'ai pas le droit d'utiliser sed, je souhaite rester sur mon approche qui me semble bien partie. tu as utilisé un read ça je peux mais j'ai essayé de l'utiliser dans mon script, sans succès.

comment attribuer à a variable y la valeur -d dans mon script?
0
zipe31 Messages postés 36402 Date d'inscription   Statut Contributeur Dernière intervention   6 430
 
Il faudrait voir ton script en entier parce que là il y a des incohérences, genre la variable "$indice" n'est pas déclarée, d'où sort ce fameux "-d" ? etc. ;-(

Et ça tu y a droit : 

$ echo "$@"
X a b c d Y 1 2 3 4

$ echo -e "${@/Y/\nY}"
X a b c d
Y 1 2 3 4
0
Réponse 3 / 12
marine
 
non tu utilises des notions que je n'ai pas vu zipe, en fait je dois écrire un script tel que si je le lance comme ça:

./script.sh repertoire X 1 2 3 4 Y a b c d

il me crée les répertoires 1 2 3 4 si il lit X et crée les fichiers a b c d si il lit Y. MOn algo me semble pas mal parti, voici le script que j'ai commencé:

if [ $# -lt 1 ]
then
echo Il faut au moins un paramètre.
exit 1
fi

indice=1
y=""

mkdir $1
shift [1]

for i in "$@"
do
if [ "$i" = "X" ]
then
indice='expr $indice + 1'
$y=$i
elif [ "$i" != "X" ] && [ "$y" = "X" ] && [ "$i" != "Y" ]
then
mkdir repertoire/$indice
fi
done

qu'en dis tu?
0
zipe31 Messages postés 36402 Date d'inscription   Statut Contributeur Dernière intervention   6 430
 
Ok, on va essayer de faire avec ce que tu connais et qui est autorisé... 

$ cat marine2.sh                
#! /bin/bash                                        

#set -xv
if [ $# -lt 1 ]
then           
echo Il faut au moins un paramètre.
exit 1                             
fi                                 

# Si le 1er param est "X"
if [ "$1" = X ]          
# On décale
then shift
fi

# Tant que le 1er paramètre est différent de "Y"
while [ "$1" != "Y" ]
do
# On crée un répertoire au nom du paramètre
mkdir "$1"
# On décale
shift
done

# Si le 1er param est "Y"
if [ "$1" = Y ]
# On décale
then shift
fi

# Pour chaque paramètre restant, on crée un fichier à son nom
for i in "$@"
do
# On crée un fichier à son nom
touch "$i"
done

$ ls
marine2.sh*

$ ./marine2.sh a b c d Y 1 2 3 4

$ ls
1  2  3  4  a/  b/  c/  d/  marine2.sh*  marine.sh*

$

;-))
0
Réponse 4 / 12
marine
 
ok je vois zipe j'ai progressé dans ma compréhension grâce à toi mais je me suis très mal expliquée car lorsque j'appelle le script comme ça:

./script.sh repertoire X 1 2 3 4 Y a b c d

je dois tout créer dans le 1er paramètre qui est repertoire, ça je sais le corriger .

Mais mon script doit etre plus général, par exemple je peux l'appeler comme ça:

./script.sh repertoire X 1 2 3 4 Y a b c d X toto titi tata Y z e r t

Donc en gros il me faut une boucle générale qui prenne en compte tous les cas d'appels que je souhaite. Ma seule question est : est ce possible de faire ça dans une boucle générale?
Je veux juste que tu me répondes oui ou non, si c'est oui je tente d'écrire un code et je te montre le résultat ( car je veux bosser et j'ai horreur des codes tout faits si je n'y comprends rien ).
merci
0
zipe31 Messages postés 36402 Date d'inscription   Statut Contributeur Dernière intervention   6 430
 
Ma seule question est : est ce possible de faire ça dans une boucle générale?
Surement que oui ;-))

Peut être voir du côté d'une boucle "while true", avec des instructions "beak" et "continue" et le tout dans un "case ... esac" ;-\

Le tout sous réserve que ces commandes et instructions soient autorisées ;-)
0
zipe31 Messages postés 36402 Date d'inscription   Statut Contributeur Dernière intervention   6 430
 
Bon je confirme c'est possible juste avec une boucle "while true" et des "breaks" ;-))
0

Vous n’avez pas trouvé la réponse que vous recherchez ?

Posez votre question
Réponse 5 / 12
marine
 
non je n'ai pas vu true ni breaks mais il me vient une autre idée: le script que tu as écrit, si on le modifie un peu pour le rendre récursif en rajoutant le cas de base "si il n'y a plus d'arguments on arrete l'exécution", que penses tu de cette idée? sans donner de code bien sur :-).
0
zipe31 Messages postés 36402 Date d'inscription   Statut Contributeur Dernière intervention   6 430
 
Re-

en rajoutant le cas de base "si il n'y a plus d'arguments on arrete l'exécution"
Ben c'est ce que fait le script que j'ai testé, mais avec un "while true" et des "break" ;-((

Sans ces conditions je ne vois pas trop comment faire autrement ;-\

A suivre... ;-))
0
Réponse 6 / 12
Char Snipeur Messages postés 9813 Date d'inscription   Statut Contributeur Dernière intervention   1 299
 
Salut.
Moi, je verrai bien une boucle en for in $@
ensuite, un case, si tu lis 'X', alors tu mets une variable à 1, si tu lis 'Y' tu met la variable à 2. Si tu lis autre chose, tu crée un répertoire si ta variable vaut 1 et n fichier si elle vaut 2.
0
zipe31 Messages postés 36402 Date d'inscription   Statut Contributeur Dernière intervention   6 430
 
Salut,

Pas mieux et surtout très efficace ;-))
0
Réponse 7 / 12
marine
 
salut, alors ok avec des case ça me va bien mais il n'est rien marqué dans mon cours dessus et j'ai cherché sur le net rien de convaincant non plus dessus, j'ai néanmoins testé un code ( qui ne marche pas , probleme de syntaxe ):

for i in "$@"
do
case $i in "-X"
$var=1;;
case $i in "Y"
$var=2;;
esac
if [ $i != "X" ] && [ $var = 1 ]
then
mkdir rep/$i
elif [ $i != "Y" ] && [ $var = 2 ]
then
touch rep/$i
fi
done

comment marche le case en fait, on fait case variable dans "valeur" ?
0
zipe31 Messages postés 36402 Date d'inscription   Statut Contributeur Dernière intervention   6 430
 
Re-

Petit exemple : 

$ cat case_marine.sh 
#! /bin/bash                            

for i in "$@"
do           

case "$i" in
[0-9])  echo "$i est un chiffre."
        echo                     
        ;;                       
[a-zA-Z])       echo "$i est une lettre."
        echo                             
        ;;                               
*)      echo "$i n'est ni un chiffre ni une lettre."
        echo                                        
        ;;                                          
esac                                                
done        
                                        
$ ./case_marine.sh 1 a 2 3 A U N J , . p 5 $ F
1 est un chiffre.                                                

a est une lettre.

2 est un chiffre.

3 est un chiffre.

A est une lettre.

U est une lettre.

N est une lettre.

J est une lettre.

, n'est ni un chiffre ni une lettre.

. n'est ni un chiffre ni une lettre.

p est une lettre.

5 est un chiffre.

$ n'est ni un chiffre ni une lettre.

F est une lettre.

$

;-))
0
Réponse 8 / 12
marine
 
et bien j'avais bien compris le case alors pq mon code ne marche pas, j'ai ce message:


Erreur de syntaxe près du symbole inattendu « newline »
case $element in "X"'
0
zipe31 Messages postés 36402 Date d'inscription   Statut Contributeur Dernière intervention   6 430
 
Non ta syntaxe n'est pas bonne ;-((

Mauvais :
for i in "$@"
do
case $i in "-X"
$var=1;;
case $i in "Y"
$var=2;;
esac

Non : 

for i in "$@"
do
case $i in 
X) var=1;;
Y) var=2;;
esac
0
Réponse 9 / 12
marine
 
d'accord, mais je deviens folle car dans cette expression vois tu une erreur zipe:

if [ "$i"!="-d" ] && [ $var -eq 1 ]

j'ai essayé tous les changements possibles et inimaginables dans var - eq... et c'est tjs une erreur de syntaxe, meme avec = , des guillements...
0
zipe31 Messages postés 36402 Date d'inscription   Statut Contributeur Dernière intervention   6 430
 
Il me faut le script en entier, sans quoi pour moi "-d" et "$var" ne veulent rien dire ;-((

Au cas ou, comme ça (attention aux espaces) : 

if [ "$i" != "-d" -a "$var" -eq 1 ]
0
Réponse 10 / 12
marine
 
voilà le script entier:

if [ $# -lt 1 ]
then
echo Il faut au moins un paramètre.
exit 1
fi


mkdir $1
shift [1]
var=""

for i in "$@"
do
case $i in
-d) $var=1;;
-f) $var=2;;
esac
if [ "$i" !="-d" ] && [ "$var" -eq 1 ]
then
mkdir rep/$i
elif [ "$i" !="-f" ] && [ "$var" -eq 2 ]
then
touch rep/$i
fi
done
0
zipe31 Messages postés 36402 Date d'inscription   Statut Contributeur Dernière intervention   6 430
 
Difficile de te répondre dans la mesure où il faudrait poster la solution pour te faire comprendre comment utiliser le "case" avec les "$var".

Ce que je peux te dire c'est que le test de '"$var"=1' doit se trouver dans le "case" dans la partie que tu as oublié le fameux "*)" (le reste en dehors des conditions "-d" et "-f") et le seul truc que tu auras à tester c'est le contenu de la variable "$var" ;-\ 

do
case "$i" in
-d) $var=1;;
-f) $var=2;;
*) if ...
   then ...
   else ...
   fi
   ;;
esac
0
Réponse 11 / 12
marine
 
je ne comprends plus rien je sens que je vais abandonner, mon probleme c'est juste un probleme de syntaxe, impossible de le corriger? J'ai testé ton dernier message et rien n'y fait c'est tjs le meme probleme de syntaxe...
DOnc ok j'accepte la solution car ça fait 3 jours que je suis dessus et j'estime avoir assez cherché...
0
zipe31 Messages postés 36402 Date d'inscription   Statut Contributeur Dernière intervention   6 430
  
$ cat foo.sh                                                                 
#! /bin/bash                                                                                    

#set -xv
if [ $# -lt 1 ]
then           
echo Il faut au moins un paramètre.
exit 1
fi

echo "mkdir $1"

shift

for i in "$@"
do
case "$i" in
X)      VAR=1
        ;;
Y)      VAR=2
        ;;
*)      if [ "$VAR" = 1 ]
        then
                echo "mkdir $i"
        else
                echo "touch $i"
        fi
        ;;
esac
done

$ ./foo.sh rep X a b c Y 1 2 3 X Q S D Y 7 8 9
mkdir rep
mkdir a
mkdir b
mkdir c
touch 1
touch 2
touch 3
mkdir Q
mkdir S
mkdir D
touch 7
touch 8
touch 9
0
Char Snipeur Messages postés 9813 Date d'inscription   Statut Contributeur Dernière intervention   1 299
 
manque le cas où l'utilisateur est con : 
#!/bin/bash                                                                                    

#set -xv
if [ $# -lt 1 ]
then           
echo Il faut au moins un paramètre.
exit 1
fi
VAR=0
echo "mkdir $1"

shift

for i in "$@"
do
case "$i" in
X)      VAR=1
        ;;
Y)      VAR=2
        ;;
*)      case $VAR in
        1)
                echo "mkdir $i"
                ;;
        2)
                echo "touch $i" 
                ;;
        *)
                echo "mettre X ou Y avant toute chose"&&exit 1
                ;;
        esac
        ;;
esac
done
juste à vérifier qu'on peu imbriquer deux case.
0
zipe31 Messages postés 36402 Date d'inscription   Statut Contributeur Dernière intervention   6 430
 
Salut,

Tout d'abord merci pour l'algo, sans quoi j'étais parti dans des méandres digne des meilleurs labyrinthes ;-))

Si si des "case ... esac" peuvent parfaitement s'imbriquer.

Par contre on peut dans ce cas là insérer le cas oublié dans le "if" : 

*)	if [ "$VAR" = 1 ]
	then
		echo "mkdir $i"
	elif [ "$VAR" = 2 ]
	then
		echo "touch $i"
	else
		echo "mettre X ou Y avant toute chose" && exit 1
	fi
	;;
esac

;-))
0
Réponse 12 / 12
marine
 
BIen, merci bcp pour ton aide ça marche, au moins j'aurais appris pleins de choses tant mieux, juste une question sur le case: le *) veut dire en fait: pour tous les autres valeurs différentes de X ou Y?
0
zipe31 Messages postés 36402 Date d'inscription   Statut Contributeur Dernière intervention   6 430
 
le *) veut dire en fait: pour tous les autres valeurs différentes de X ou Y?
Oui.
0

Discussions similaires

lancer un script sh
user -
Un utilisateur novice -

7 réponses
Éxécuter un fichier *.sh
Dredd14 -
mlp_Epilepsy -

4 réponses
A quoi correspond cette croix sur snapchat
Sbnet -
Doucesensible -

8 réponses
Appeler un script .sh depuis un autre script .sh
mum767 -
mamiemando -

6 réponses
Mon pc s'allume et s'éteint en boucle
Jack -
Daniel -

16 réponses