Discussion :

[eric-bsm]

bonsoir,

Je suis novice en la matière et j'essaye de faire de petits scripts. En voici un que je vous soumet.
A votre avis, comment puis-je le perfectionner ?

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
22
23
24
25
26
27
----------------------------------
#!/bin/bash
cd /
clear
:choix
echo "***************************"
echo "*** Verification ARKEIA ***"
echo "***************************"
echo "1- lister le contenu du dossier NAS"
echo "2- remonter le lien iSCSI"
echo "Q : quitter"
echo ""
read -p 'tapez votre choix  :' rep
if $rep ="1" then goto liste
if $rep ="2" then iscsi
if $rep ="Q" or "q" then quit
else end
 
:liste
cd /mnt/nas
ls
goto choix
 
:iscsi
iscsiadm --mode node --targetname iqn.2004-04.com.qnap:ts-269l:iscsi.naschm.d7c5ba --portal @IPNAS --login
goto choix
-----------------------------------------------------------------

merci encore pour votre aide et votre patience.

eric

[BufferBob]

salut,

comment puis-je le perfectionner ?

ton code ne fonctionne pas, l'instruction goto n'existe pas en bash, il ne s'agit donc pas de le "perfectionner" mais déjà au minimum de le tester avant de poster...

[gangsoleil]

Bonjour,

Utilise des fonctions
Fais une boucle
Utilise un switch au lieu des if

Et bien sûr, enlève le goto, et teste