Suppression du Code Mort - Guide technique pour développeurs – Tous langages

📚 Méthodologie de Nettoyage

1. Détection

Outils par langage

Recherche manuelle

# Fichiers non modifiés depuis >1 an (hors tests/config)
git log --since="1 year ago" --pretty=format: --name-only | \
  grep -vE '(\.test\.|__tests__|config|vendor|node_modules)' | \
  sort | uniq -u

# Recherche de fonctions suspectes (exemple PHP)
grep -r "function " src/ | grep -v "test" | sort

2. Validation

Avant toute suppression :

1. Recherche globale :

   grep -r "nom_de_la_fonction" src/

2. Vérification des appels dynamiques :
   • PHP : $this->$methodName()
   • JavaScript : window["someFunction"]()
3. Consultation des logs :
   • Vérifier si la fonction apparaît dans les logs d'erreur ou d'accès.
4. Test de suppression :
   • Supprimez le code localement et exécutez la suite de tests.
   • Vérifiez les fonctionnalités critiques manuellement.

3. Suppression

Bonnes pratiques

• Ne pas commenter le code : Supprimez-le. L'historique Git permet de le retrouver si nécessaire.
• Commits clairs :

  git rm fichier_inutile.php
  git commit -m "Suppression de old_auth_system.php (code mort, non utilisé depuis #123)"

• Mise à jour de la documentation :
  • Supprimer les références dans les README, CHANGELOG, ou la documentation API.
  • Mettre à jour les schémas de base de données si des tables associées sont supprimées.

Cas particuliers

4. Prévention

Intégration dans le workflow

Exemple de Git Hook (Husky)

// package.json
{
  "husky": {
    "hooks": {
      "pre-commit": "lint-staged --concurrent false",
      "pre-push": "npm run audit:deadcode"
    }
  },
  "lint-staged": {
    "*.{js,ts}": [
      "eslint --rule 'no-unused-vars: error'",
      "git add"
    ]
  }
}

Template de Pull Request

## Description
[Description des changements]

## Checklist Code Mort
- [ ] Aucune fonction/variable inutilisée ajoutée.
- [ ] Les imports dépendances sont tous nécessaires.
- [ ] Les fichiers modifiés/supprimés sont référencés dans le projet.

🔍 Études de Cas Concrets

1. Endpoint API Oublié (Sécurité)

Contexte :

• Une route /debug/sql avait été ajoutée pour le développement.
• Oubliée en production pendant plusieurs années.

Risque :

• Exposition des requêtes SQL en clair (injection SQL possible).
• Découverte via un scan de routine par un attaquant.

Solution :

1. Audit des routes avec :

   # Symfony
   php bin/console debug:router
   
   # Laravel
   php artisan route:list

2. Suppression physique de la route et des contrôleurs associés.
3. Vérification des logs pour détecter d'éventuelles exploitations passées.

2. Bibliothèque Inutilisée (Performance)

Contexte :

• Un projet Node.js incluait lodash et moment alors qu'ils n'étaient plus utilisés.
• Impact : augmentation inutile de la taille du bundle.

Actions :

1. Détection avec depcheck :

   npx depcheck

2. Suppression des dépendances :

   npm uninstall lodash moment

3. Vérification du bundle avec webpack-bundle-analyzer.

3. Fonction de Hachage Obsolète (Bug)

Contexte :

• Une fonction oldHashPassword() utilisant MD5 était toujours présente.
• Un développeur l'a réutilisée par erreur, introduisant une faille de sécurité.

Solution :

1. Suppression de la fonction.
2. Ajout d'un test de régression pour vérifier que seul le nouveau système de hachage (bcrypt) est utilisé :

   public function testPasswordHashingUsesBcrypt()
   {
      $hashed = hash_password('test123');
      $this->assertTrue(password_verify('test123', $hashed));
      $this->assertStringStartsWith('$2y$', $hashed); // Préfixe bcrypt
   }

⚠️ Pièges Courants

📋 Checklist de Nettoyage

Détection

• [ ] Exécuter un audit statique (ESLint/PHPStan).
• [ ] Vérifier la couverture de test pour identifier les branches non exécutées.
• [ ] Lister les fichiers non modifiés depuis >1 an.
• [ ] Scanner les dépendances inutilisées (depcheck, composer unused).

Validation

• [ ] Rechercher globalement les noms de fonctions/variables (grep -r).
• [ ] Vérifier les appels dynamiques (ex : $this->$method()).
• [ ] Confirmer avec l'équipe pour les cas ambigus.

Suppression

• [ ] Supprimer physiquement le code (pas de commentaires).
• [ ] Mettre à jour la documentation et les schémas.
• [ ] Ajouter un test de non-régression si nécessaire.

Prévention

• [ ] Configurer des hooks Git (Husky, pre-commit).
• [ ] Intégrer l'audit dans la CI/CD (SonarQube, GitHub Actions).
• [ ] Organiser un audit trimestriel du code mort.

🔧 Outils Recommandés

📌 Ressources Complémentaires

• Refactoring Guru : Code Smells
• Google : Tree-Shaking in JavaScript