Skip to content

Audit 04 — KISS (Keep It Simple, Stupid)

Objectif : verifier que le code reste simple et ne contient pas de sur-ingenierie.

Cible : document-parser/ (hors .venv/, __pycache__/), frontend/src/


Checklist

# Item Poids
4.1 Pas de design pattern complexe la ou un simple if ou une fonction suffit (factory, strategy, observer superflus) 2
4.2 Le code resout le probleme actuel, pas un probleme hypothetique futur (pas de genericite prematuree) 2
4.3 Pas de fonction wrapper qui ne fait qu'appeler une autre fonction sans valeur ajoutee 1
4.4 Utilisation des outils standard (Python stdlib, Vue composables natifs) avant de creer des solutions maison 1
4.5 Configuration simple — pas de systeme de config complexe la ou une variable d'env suffit 1
4.6 Pas d'indirection inutile — le chemin d'execution d'une requete ne traverse pas plus de couches que necessaire 2
4.7 Pas de meta-programmation ou de magie (decorateurs complexes, metaclasses) sauf necessite avere 2
4.8 Les structures de donnees utilisees sont les plus simples possibles (liste plutot que arbre si la liste suffit) 1

Commandes de verification

# 4.1 — Patterns potentiellement superflus
grep -rn "class.*Factory\|class.*Strategy\|class.*Observer\|class.*Builder\|class.*Singleton" document-parser --include="*.py" --exclude-dir=.venv

# 4.7 — Meta-programmation
grep -rn "__metaclass__\|type(.*,.*,.*)\|__init_subclass__\|__class_getitem__" document-parser --include="*.py" --exclude-dir=.venv

# 4.3 — Fonctions tres courtes (potentiels wrappers inutiles, < 3 lignes)
# Verification manuelle recommandee sur les fonctions identifiees

Regles de notation

  • Tout item de poids 3 non conforme = ecart [CRIT]
  • Tout item de poids 2 non conforme = ecart [MAJ]
  • Tout item de poids 1 non conforme = ecart [MIN]