[bash] Problème de boucleRésolu/Fermé

Question

Bonjour,

etant debutant en linux, je dois creer un petit script.
juste une invite prompt qui demande si l'utilisateur veut faire ceci ou cela.
et une boucle condition agissant en consequence.

j'ai donc fait sa :

#!/bin/bash
demande2=non

for liste in `ls /home/mimic/Bureau/packages/`
do
	echo $liste
done

echo Taper 1 Pour installer le premier item, tapez 2 pour installer le deuxieme...
read choix

if [ $choix=="1" ]; then
	#until [  $demande2="non"]; do
	echo "Vous avez donc choisit Direct News en version pro ? [o/n]"
	read conf
	
	if [ $conf=='o' ]; then
		echo "L'installation de unzip va demarrer pour verifier si le paquet est installé"
		sudo apt-get install unzip
		cd /home/mimic/Bureau/packages
		unzip directnews-4.10.1-pro.zip -d /home/mimic/Bureau/packages/DirectNewsPro/
		echo "Archive decompressé"
		#$demande = 'oui'
		$demande2= oui
		
	#elif [ $conf='n' ]; then
		#$demande = 'non'
		#$demande2=oui
		
	else
		echo "Reponse indisponible"
		$demande2 = non
	fi
	#done
fi


La j'ai commenté la boucle until pour pas qu'elle ne soit active.
mais j'ai quand meme mes if
seulement quand je repond 'n' pour 'non' dans mon terminal quand je lance mon script
il me fis quand meme ce qui figure dans le if [ $conf == 'o']

Comment faire pour que sa marche?

jipicy · Answer

Salut,

Il faut des espaces autour du signe "égale" !

if [ "${conf]" == 'o' ]; then

aymbob · Answer

merci jipicy

j'essai tout de suite

aymbob · Answer

Sa ne marche pas, je met mon code en entier

#!/bin/bash
demande2=non

for liste in `ls /home/mimic/Bureau/packages/`
do
	echo $liste
done

echo Taper 1 Pour installer le premier item, tapez 2 pour installer le deuxieme...
read choix

if [  "$choix" == 'o' ]; then
	#until [  $demande2="non"]; do
	echo "Vous avez donc choisit Direct News en version pro ? [o/n]"
	read conf
	
	if [ "$conf" == 'o' ]; then
		echo "L'installation de unzip va demarrer pour verifier si le paquet est installé"
		sudo apt-get install unzip
		cd /home/mimic/Bureau/packages
		unzip directnews-4.10.1-pro.zip -d /home/mimic/Bureau/packages/DirectNewsPro/
		echo "Archive decompressé"
		#$demande = 'oui'
		$demande2= oui
		
	#elif [ $conf='n' ]; then
		#$demande = 'non'
		#$demande2=oui
		
	else
		echo "Reponse indisponible"
		$demande2 = non
	fi
	#done
fi

if [ $choix=="2" ]; then
	$demande2 = "non"
	#until [  $demande2="non"]; do
	echo "Vous avez donc choisit Direct News en version Open ? [o/n]"
	read conf
	
	if [ $conf=="o" ]; then
		echo "L'installation de unzip va demarrer pour verifier si le paquet est installé"
		sudo apt-get install unzip
		cd /home/mimic/Bureau/packages
		unzip directnews-4.10-open-20070618.zip -d /home/mimic/Bureau/packages/DirectNewsOpen/
		echo "Archive decompressé"
		$demande = "oui"
		$demande2= "oui"
		
	elif [ $conf=n ]; then
		$demande = "non"
		$demande2= "oui"
		
	else
		echo "Reponse indisponible"
		$demande2 = "non"
	fi
	#done
fi

jipicy · Answer

echo Taper 1 Pour installer le premier item, tapez 2 pour installer le deuxieme...
read choix

if [ "$choix" == 'o' ]; then 

Forcément la condition est toujours fausse ;-((

aymbob · Answer

la honte j'ai mal recopier le code que tu m'avais donné (j'evite le copier coller quand j'apprend)
j'essai et je dit le resultat que j'obtient

aymbob · Answer

sa marche beaucoup mieux :)
pourrait tu me dire ce que je pourrai faire pour optimiser mon script : 
j'utilise deux grandes boucles if avec les memes choses dedans
je pourrai pas faire un for qui choisirait de faire tel instructions en fonction de tels choix
par exemple :

#!/bin/bash
demande2=non
i=0

for liste in `ls /home/mimic/Bureau/packages/`
do
	echo "$liste"
done

echo Taper 1 Pour installer le premier item, tapez 2 pour installer le deuxieme...
read choix

for liste in `ls /home/mimic/Bureau/packages/`
do
	///INSTRUCTION
done

exit;

garion28 · Answer

relis ton code, tu met une valeur alphanumérique dans une variable, mais cette valeur n'est pas entre ""

aymbob · Answer

merci du conseil

jipicy · Answer

Il y a quoi dans "/home/mimic/Bureau/packages/" ?
S'il n'y a que 2 objets (apparemment connus d'avance), pourquoi faire un listing depuis une boucle ?

aymbob · Answer

j'ai fait ce code...


#!/bin/bash
demande2=non
i=1

for liste in `ls /home/mimic/Bureau/packages/`
do
	echo "$((i++)) $liste"
done

echo Taper 1 Pour installer le premier item, tapez 2 pour installer le deuxieme...
read choix

i=0
for liste in `ls /home/mimic/Bureau/packages/`
do	
	$((i++))
	
	if [ $choix == $i ]; then
		echo $liste
	else
		continue
	fi
done

exit;


Mais sa me retourne un probleme :

./script.sh: line 16: 0 : commande introuvable
directnews-4.10.1-pro.zip
./script.sh: line 16: 1 : commande introuvable

Que faire ?

aymbob · Answer

fait ;)

jipicy · Answer

jp@MDK:~/tmpfs ssh$ ls rep/
paquet1  paquet2  paquet3  paquet4

jp@MDK:~/tmpfs ssh$ cat foo.sh
#! /bin/sh

#set -xv

echo "Liste des paquets : "
ls -1 rep/ | nl
read -p "Choisissez le paquet à installer (entrez son numéro) : " choix
nom="$(ls -1 rep/ | sed -n "${choix}p")"
echo "Installation de ${nom} en cours..."

jp@MDK:~/tmpfs ssh$ ./foo.sh
Liste des paquets :
     1  paquet1
     2  paquet2
     3  paquet3
     4  paquet4
Choisissez le paquet à installer (entrez son numéro) : 3
Installation de paquet3 en cours...

jp@MDK:~/tmpfs ssh$
;-))

aymbob · Answer

merci beaucoup c'est une bonne methode 
Pourrait tu juste m'expliquer ces lignes :

ls -1 rep/ | nl
et
nom="$(ls -1 rep/ | sed -n "${choix}p")"

jipicy · Answer

ls -1 rep/ 
man ls pour plus d'explications. 
-1 Présenter un fichier  par ligne


 | 
Pipe. On envoie le résultat de "ls -1" à travers un "pipe" (filtre) à la prochaine commande


nl
man nl. Afficher un fichier en numérotant ses lignes.


nom="$(ls -1 rep/ | sed -n "${choix}p")"
On récupère dans une variable le fichier correspondant au numéro choisi à l'aide de la commande "sed"

aymbob · Answer

Je ne te remercierai jamais assez pour toutes ces informations precieuses qui me font progresser
juste pour sa : 

nom="$(ls -1 rep/ | sed -n "${choix}p")"

je ne comprend pas a quoi sert le p a la fin de la commande

merci encore

[bash] Problème de boucle

15 réponses

Discussions similaires

Newsletters