mise au point solution

Readme.md

@@ -2,17 +2,12 @@
 
 ### Exercice 1
 
-Voir le fichier eval.sh
+le script eval.sh présente des redondances de code qui le rende difficile à maintenir.
 
-Essayez d'identifier sa fonction
+Dans l'idéal il faudra factoriser en créeant 2 fonctions
-
-Identifiez ses lacunes
-
-Proposez une correction
 
 ### Exercice 2
 
-
 Définir dans le fichier myfunctions.sh une fonction "myerror" prenant 2 arguments en entrée: un code et un message.
 
 La fonction affiche le message donné et termine le script en renvoyant le code

eval.sh

@@ -1,12 +1,30 @@
 #! /bin/bash
 IFS=$'\n\t'
+
+function fileMTime {
+        if [ ! -f $1 ] ; then echo -ne "0;" ; return 0 ; fi
+        stat --printf "%y;" $1
+}
+
+function fileMustContains {
+  file=$1
+  shift
+  string=$@
+  if [ ! -f $file ] ; then echo -ne "0;" ; return 0 ;fi
+  if ( grep -i -qE "${string}" $file ) ;
+    then echo -ne "1;"
+  else
+    echo -ne "0;"
+  fi
+}
+
 host=$1
 root=${host}/rootfs
 host=$(basename $host)
 hostname=$(grep HOSTNAME ${root}/etc/sysconfig/network 2>/dev/null| cut -f2 -d=)
-stat --printf "%y;" ${root}/root/.bash_history
+fileMTime  ${root}/root/.bash_history
-stat --printf "%y;" ${root}/home/formation/.bash_history
+fileMTime  ${root}/home/formation/.bash_history
-stat --printf "%y;" ${root}/etc/motd
+fileMTime  ${root}/etc/motd
-if( grep -i -qE "today"  ${root}/etc/profile.d/zalias.sh ) ; then echo -ne "1;" ; else echo -ne "0;"  fi
+fileMustContains  ${root}/etc/profile.d/zalias.sh "today"
-if( grep -i -qE "PS1"  ${root}/etc/profile.d/zconf.sh ) ; then echo -ne "1;" ; else echo -ne "0;"  fi
+fileMustContains  ${root}/etc/profile.d/zconf.sh "PS1"
-if( grep -i -qE "HISTSIZE"  ${root}/etc/profile.d/zconf.sh ) ; then echo -ne "1;" ; else echo -ne "0;"  fi
+fileMustContains ${root}/etc/profile.d/zconf.sh  "HISTSIZE"

myfunctions.sh

@@ -0,0 +1,7 @@
+function myerror {
+   code=$1
+   shift
+   msg="$*"
+   echo $msg
+   exit $code
+}

test.sh

@@ -0,0 +1,9 @@
+#! /bin/bash
+
+set -e
+set -u
+
+source myfunctions.sh
+
+
+myerror 42  "check \$? var"