Discussion :

[Olivier Regnier]

Bonjour,

Je viens de retrouver un script suite à une perte de données

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
get_base() {
 check_usergroup
 if [ $? -eq 0 ]; then
    while true; do
      echo -n "Get base system? (y/n) [y] "
      read answer
      case $answer in
      ""|y|Y)
         if (cd /usr && cvs checkout -rOPENBSD_6_6 -P src); then
             echo ">> Base system fetched."
         else
                 echo ">> Unable to fetch the base system!"
             fi
         return 0
         ;;
         n|N)
         echo "GoodBye!"
         break
         ;;
           *)
         echo "$answer: is not allowed!"
         ;;
       esac
   done
 fi
}

Je me demande si le test en bleue est toujours utile ?

En vous remerciant de votre réponse.

[Flodelarab]

Bonjour

check_usergroup
if [ $? -eq 0 ]; then

Moi, je m'demande surtout pourquoi faire ça en 2 lignes.

Code :

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

if check_usergroup; then

[jack-ft]

Je viens de retrouver un script suite à une perte de données

Code :

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

1
2
3
4
5
6
7
8
9
get_base() {
 check_usergroup
 if [ $? -eq 0 ]; then
    while true; do
      echo -n "Get base system? (y/n) [y] "
[snip]
   done
 fi
}

Je me demande si le test en bleue est toujours utile ?

Je suis d'accord avec la remarque de Flodelarab.

Et, manifestement, l'auteur de la fonction get_base a voulu que la récupération interactive de l'openBsd ne soit proposée qu'après vérification d'un retour sans anomalie de l'appel à check_usergroup...
C'est du moins ce que dit le code (plus que les commentaires du code).

[ctac_]

Bonjour,
Je suis d'accord avec Flodelarab.
Il serait de plus intéressant de voir la fonction check_usergroup.
Cordialement.