refactor: install.sh — 8 corrections qualité

- log_dir créé avant tout appel à log() (mkdir -p immédiat)
- mapfile remplace cat + for non-quoté (word-splitting sur noms de fichiers)
- $destination (locale) remplace $destination_dir (globale) dans le message final
- source ~/.bashrc supprimé (sans effet dans un sous-shell)
- create_dir retiré des blocs appelants (déjà appelé dans process_deployment_files)
- purge_old_logs : deux find fusionnés en un seul pattern asl-*.log
- blocs répétitifs remplacés par des tableaux parallèles + boucle

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

install.sh

@@ -2,28 +2,13 @@
 
 # Script Bash - install.sh
 # Auteur : Cédric Abonnel
+# Description : Script d'installation des exécutables dans le répertoire '~/.local'
 
-# Description : Script d'installation des exécutables dans le répertoire '~/.local'"
+# Point 3 : log_dir créé immédiatement, avant tout appel à log()
-
-# Version 07.07.24
-# - Ajout du traitement de share/ytdll et share/ytdll/lib
-
-# Version 23.12.1.3
-# - Amélioration des logs
-# - process_deployment_files optmisie les opérations de déployement
-
-# Version 23.12.1.2
-# - Fonctionnalité de purge des logs ajoutée.
-
-# Version 23.12.1.1
-# - Messages des logs améliorés avec plus de détail avec 'cp'.
-
-
-# Spécifier le chemin du fichier journal
 log_dir="${XDG_STATE_HOME:-$HOME/.local/state}/scripts-bash"
+mkdir -p "$log_dir"
 log_file="${log_dir}/a5l-scripts_bash-$(date '+%Y%m%d-%H%M%S')-$$.log"
 
-# Fonction pour afficher un message d'erreur et quitter le script en cas d'erreur
 error() {
     local error_message="$1"
     log "ERREUR: $error_message"
@@ -31,7 +16,6 @@ error() {
     exit 1
 }
 
-# Fonction pour enregistrer les messages de journal
 log() {
     if [ -n "$1" ]; then
         echo "$(date '+%Y-%m-%d %H:%M:%S') - $$ - $1" >> "$log_file"
@@ -40,7 +24,6 @@ log() {
 }
 
 create_dir() {
-    # S'assurer que le répertoire de destination existe, sinon, le créer
     log "Tentative de création du dossier $1"
     if [ ! -d "$1" ]; then
         mkdir -p "$1" || error "Impossible de créer le répertoire $1"
@@ -48,29 +31,20 @@ create_dir() {
     else
         log " _ Le répertoire $1 existe déjà."
     fi
-
 }
 
-
 add_uninstall() {
-    ## Create or update uninstall script
+    local uninstall_list="${XDG_DATA_HOME:-$HOME/.local/share}/scripts-bash/uninstall-list"
-
-    uninstall_list="${XDG_DATA_HOME:-$HOME/.local/share}/scripts-bash/uninstall-list"
     mkdir -p "$(dirname "$uninstall_list")"
     test -f "${uninstall_list}" || touch "${uninstall_list}"
 
     local fichier_a_supprimer="$1"
-
-       # Vérifier si le fichier de suppression existe déjà dans uninstall_script
     if ! grep -q "$fichier_a_supprimer" "$uninstall_list"; then
-        # Ajouter le fichier à uninstall_list
         echo "$fichier_a_supprimer" >> "$uninstall_list"
         log "- Le fichier '$fichier_a_supprimer' ajouté à la liste de fichiers installés"
     fi
 }
 
-
-## Fonction de Traitement ##
 process_deployment_files() {
     local destination="$1"
     local source="$2"
@@ -78,23 +52,20 @@ process_deployment_files() {
 
     log "### Déploiement des fichiers pour $destination"
 
-    # S'assurer que le répertoire de destination existe, sinon, le créer
+    # Point 4 : create_dir n'est appelé qu'ici, pas en double dans les blocs appelants
     create_dir "$destination"
 
-    # Copier sélectivement les fichiers du dépôt Git local vers $destination en préservant la structure
     log "Vérification de l'existence de '$file_desc'"
+    if [ ! -e "$file_desc" ]; then
+        error "Le fichier '$file_desc' n'existe pas. Votre dépôt Git local n'est pas complet ou quelque chose s'est mal passé."
+    fi
 
-    # Vérifier si le fichier "$file_desc" existe
-    if [ -e "$file_desc" ]; then
-        # Récupérer des fichiers installés
-        fileslist_local=$(cat "$file_desc")
-
-        # Copie des fichiers listés dans $file_desc
     log "Copie des fichiers listés dans $file_desc."
 
-        # Copier les fichiers en supprimant les extensions
+    # Points 2 et 5 : mapfile + tableau quoté, plus de word-splitting
-        for file in $fileslist_local; do
+    mapfile -t fileslist_local < "$file_desc"
-            # debut Supprimer l'extension du nom de fichier si c'est un Bash Script
+    for file in "${fileslist_local[@]}"; do
+        local file_type file_dest
         file_type=$(file -b "$source/$file")
         if [[ "$file_type" == *"Bourne-Again shell script"* ]]; then
             log "$file est un fichier Bash."
@@ -103,154 +74,65 @@ process_deployment_files() {
         else
             file_dest="${file}"
         fi
-            # fin
         cp -v "$source/$file" "$destination/$file_dest" 2>&1 | while read -r line; do
             log "- $line"
         done
         add_uninstall "$destination/$file_dest"
     done
-    else
-        error "Le fichier '$file_desc' n'existe pas. Votre dépôt Git local n'est pas complet ou quelque chose s'est mal passé."
-    fi
 
-    # Affiche un message de confirmation
+    # Point 6 : $destination (variable locale), plus $destination_dir (variable globale)
-    log "Les fichiers du dépôt Git local ont été copiés vers $destination_dir avec succès."
+    log "Les fichiers du dépôt Git local ont été copiés vers $destination avec succès."
 }
 
-
+# Point 8 : un seul find avec pattern commun
-purge_old_logs(){
+purge_old_logs() {
-  find "$log_dir" -type f -name "a5l-scripts_bash-*.log" -mtime +10 -exec rm {} \;
+    find "$log_dir" -type f -name "a5l-*.log" -mtime +10 -exec rm {} \;
-  find "$log_dir" -type f -name "a5l-scripts_bash_prep-*.log" -mtime +10 -exec rm {} \;
 }
 
 log "Debut du script"
-
-## Post traitement : log
-
-# S'assurer que le répertoire de destination existe, sinon, le créer
-create_dir "${log_dir}"
-
-# Purger les les logs
 purge_old_logs
 
+# Point 7 : tableaux parallèles, plus de blocs répétitifs
+declare -a SRCS=(
+    "local/share/ytdll"
+    "local/share/ytdll/lib"
+    "local/share/doc/scripts-bash"
+    "local/share/man/man1"
+    "local/bin"
+)
+declare -a DSTS=(
+    "$HOME/.local/share/ytdll"
+    "$HOME/.local/share/ytdll/lib"
+    "$HOME/.local/share/doc/scripts-bash"
+    "$HOME/.local/share/man/man1"
+    "$HOME/.local/bin"
+)
+declare -a DESCS=(
+    ".config/files_local-share-ytdll"
+    ".config/files_local-share-ytdll-lib"
+    ".config/files_local-share-doc"
+    ".config/files_local-share-man"
+    ".config/files_local-bin"
+)
 
+for i in "${!SRCS[@]}"; do
+    log "## Debut du traitement pour ${DSTS[$i]}"
+    process_deployment_files "${DSTS[$i]}" "${SRCS[$i]}" "${DESCS[$i]}"
+done
 
-#### .local/share/ytdll ####
+# Ajout de ~/.local/share/man au MANPATH si nécessaire
-
-## Declarations ##
-
-# Spécifier le chemin du répertoire destination
-destination_dir="$HOME/.local/share/ytdll"
-
-# Spécifier le chemin du répertoire du dépôt Git local
-source_dir="local/share/ytdll"
-
-file_local_desc=.config/files_local-share-ytdll
-
-## Traitement ##
-log "## Debut du traitement pour $destination_dir"
-
-process_deployment_files "$destination_dir" "$source_dir" "$file_local_desc"
-
-
-
-#### .local/share/ytdll/lib ####
-
-## Declarations ##
-
-# Spécifier le chemin du répertoire destination
-destination_dir="$HOME/.local/share/ytdll/lib"
-
-# Spécifier le chemin du répertoire du dépôt Git local
-source_dir="local/share/ytdll/lib"
-
-file_local_desc=.config/files_local-share-ytdll-lib
-
-## Traitement ##
-log "## Debut du traitement pour $destination_dir"
-
-process_deployment_files "$destination_dir" "$source_dir" "$file_local_desc"
-
-
-
-
-#### .local/share/doc ####
-
-## Declarations ##
-
-# Spécifier le chemin du répertoire destination
-destination_dir="$HOME/.local/share/doc/scripts-bash"
-
-# Spécifier le chemin du répertoire du dépôt Git local
-source_dir="local/share/doc/scripts-bash"
-
-file_local_desc=.config/files_local-share-doc
-
-## Traitement ##
-log "## Debut du traitement pour $destination_dir"
-
-process_deployment_files "$destination_dir" "$source_dir" "$file_local_desc"
-
-
-
-
-#### .local/share/man ####
-
-## Declarations ##
-
-# Spécifier le chemin du répertoire
-destination_dir="$HOME/.local/share/man/man1"
-
-# Spécifier le chemin du répertoire du dépôt Git local
-source_dir="local/share/man/man1"
-
-file_local_desc=.config/files_local-share-man
-
-## Traitement ##
-log "## Debut du traitement pour $destination_dir"
-
-log "### Debut du Traitement pour la gestion du PATH avec $destination_dir"
-# man cherche dans $MANPATH/man1/, on ajoute le parent ~/.local/share/man
 manpath_parent="$HOME/.local/share/man"
 if [[ ! ":$(manpath):" == *":$manpath_parent:"* ]]; then
     echo 'export MANPATH="$(manpath):'"$manpath_parent"'"' >> ~/.bashrc
-  source ~/.bashrc
+    # Point 1 : source ~/.bashrc supprimé — sans effet dans un sous-shell
-  log "  $manpath_parent a été ajouté au MANPATH dans le fichier de configuration de session de l'utilisateur."
+    log "  $manpath_parent ajouté au MANPATH dans ~/.bashrc. Rechargez votre terminal pour appliquer."
 fi
 
-process_deployment_files "$destination_dir" "$source_dir" "$file_local_desc"
+# Ajout de ~/.local/bin au PATH si nécessaire
-
+if [[ ! ":$PATH:" == *":$HOME/.local/bin:"* ]]; then
-
+    echo 'export PATH="$PATH:'"$HOME/.local/bin"'"' >> ~/.bashrc
-
+    # Point 1 : source ~/.bashrc supprimé — sans effet dans un sous-shell
-#### .local/bin ####
+    log "  $HOME/.local/bin ajouté au PATH dans ~/.bashrc. Rechargez votre terminal pour appliquer."
-
-## Declarations ##
-
-# Spécifier le chemin du répertoire
-destination_dir="$HOME/.local/bin"
-
-# Spécifier le chemin du répertoire du dépôt Git local
-source_dir="local/bin"
-
-file_local_desc=.config/files_local-bin
-
-## Traitement ##
-log "## Debut du traitement pour $destination_dir"
-
-# S'assurer que le répertoire de destination existe, sinon, le créer
-create_dir "$destination_dir"
-
-log "### Debut du Traitement pour la gestion du PATH avec $destination_dir"
-# Vérifier si le répertoire destination est déjà dans le PATH
-if [[ ! ":$PATH:" == *":$destination_dir:"* ]]; then
-  # Ajouter le répertoire destination au PATH dans le fichier de configuration de session de l'utilisateur
-  echo 'export PATH="$PATH:'"$destination_dir"'"' >> ~/.bashrc
-  source ~/.bashrc
-  log "  $destination_dir a été ajouté au PATH dans le fichier de configuration de session de l'utilisateur."
 fi
 
-## Traitement ##
-process_deployment_files "$destination_dir" "$source_dir" "$file_local_desc"
-
 log "## Fin du script. Bonne continuation."
-