penso che dovresti riscrivere meglio alcuni pezzi di codice, per esempio ecco come dovresti riscrivere il controllo sulle directory ~/backup e ~/script e come potresti migliorare il punto 5:
- Codice: Seleziona tutto
--- /home/bash/Desktop/sirm2.old 2008-03-24 18:00:31.000000000 +0100
+++ /home/bash/Desktop/sirm2 2008-03-25 14:52:37.000000000 +0100
@@ -7,8 +7,19 @@
###############################################
#Crea le cartelle necessarie allo script se non esistono
-[ -d "$HOME/backup/" ] || mkdir $HOME/backup/ | chmod 777 $HOME/backup/ | echo "Creata la cartella $HOME/backup/"
-[ -d "$HOME/script/" ] || mkdir $HOME/script/ | chmod 777 $HOME/script/ | echo "Creata la cartella $HOME/script/"
+if [[ ! -d "$HOME/backup/" ]]
+then
+ mkdir $HOME/backup/
+ chmod 777 $HOME/backup/
+ echo "Creata la cartella $HOME/backup/"
+fi
+
+if [[ ! -d "$HOME/script/" ]]
+then
+ mkdir $HOME/script/
+ chmod 777 $HOME/script/
+ echo "Creata la cartella $HOME/script/"
+fi
# Verifica che l'utente abbia permessi di root
@@ -121,15 +132,20 @@ case $scelta in
read script
echo "Ora digita il nome del tuo editor di testo preferito (per esempio nano, gedit, kwrite)"
read editor
- if [[ $editor == "" ]]
- then
- nano $HOME/script/$script
- exit
- else
- $editor $HOME/script/$script &
- fi
-exit
-;;
+ if [[ -z "$editor" ]]
+ then
+ default=nano
+ esiste_def=$(whatis "$default" | grep /)
+ if [[ -z "$esiste_def" ]]
+ then
+ $default $HOME/script/$script &
+ exit
+ else
+ $editor $HOME/script/$script &
+ fi
+ fi
+ exit
+ ;;
6)
echo "Ciao"
exit
PS. e indenta un pò meglio
