Index
Documentation Générale Qualité
Documentation Classes Metier
Documentation Liste Qualité Code Review
Documentation Liste Qualité Guideline Development
Documentation Bugs Defaults
Documentation Tests Unitaires
Documentation i18n
Documentation SQL Indexation
Documentation Timeframe
| A | B | C | D | E | F | G | H | I | J | K | L | M | N | O | P | Q | R | S | T | U | V | W | X | Y | Z | AA | |
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
1
|
GENERAL | ||||||||||||||||||||||||||
|
2
|
La fonctionnalité implémentée : est-ce qu’elle respecte tout le cahier des charges et le besoin auquel elle répond ? | ||||||||||||||||||||||||||
|
3
|
Est-ce que le code fonctionne? Est-ce qu’il effectue son fonctionnement attendu, la logique est correcte, etc. | ||||||||||||||||||||||||||
|
4
|
Est-ce que tout le code se comprend facilement ? | ||||||||||||||||||||||||||
|
5
|
Est-ce que les noms utilisés sont parlant et informent du sens de la variable ou fonction ?
|
||||||||||||||||||||||||||
|
6
|
Est-ce qu’il correspond à nos conventions de code ? Cela comprend généralement la position des accolades, les noms de fonction et de variable, la taille des lignes, l’indentation, le format et les commentaires.
|
||||||||||||||||||||||||||
|
7
|
Est-ce qu’il y a du code redondant ou dupliqué ? | ||||||||||||||||||||||||||
|
8
|
Le code est-il aussi modulaire que possible ? | ||||||||||||||||||||||||||
|
9
|
Une variable globale peut-elle être remplacée ? | ||||||||||||||||||||||||||
|
10
|
Y-a-t-il des lignes de codes commentées ? | ||||||||||||||||||||||||||
|
11
|
Les boucles ont-elles une taille définie et une condition de fin correcte ? | ||||||||||||||||||||||||||
|
12
|
|||||||||||||||||||||||||||
|
13
|
PERFORMANCES ET Notice & Warning | ||||||||||||||||||||||||||
|
14
|
Y-a-t-il des optimisations évidentes pour améliorer la performance ? | ||||||||||||||||||||||||||
|
15
|
Est-ce que le code tient compte de la montée en charge potentielle dans le contexte ? | ||||||||||||||||||||||||||
|
16
|
Est-ce qu’une partie du code peut être remplacé par une librairie ou une fonction du langage ?
|
||||||||||||||||||||||||||
|
17
|
1 testeur spécial notice et warning qui fait ça sur la version pull request >> Reste-t-il du code concernant les logs ou le debug à supprimer ?
|
||||||||||||||||||||||||||
|
18
|
|||||||||||||||||||||||||||
|
19
|
SECURITE (in progress) | ||||||||||||||||||||||||||
|
20
|
Est-ce que toutes les données d’entrées sont vérifiées et nettoyées? Au niveau de leur type, taille, format et portée | ||||||||||||||||||||||||||
|
21
|
Est-ce que les liens envoyés dans des emails sont protégés par un système de token. |
@see include/controller/consultants/redirections.inc.php
|
|||||||||||||||||||||||||
|
22
|
|||||||||||||||||||||||||||
|
23
|
DOCUMENTATION | ||||||||||||||||||||||||||
|
24
|
Y-a-t-il des commentaires et décrivent-ils les objectifs du code ? | ||||||||||||||||||||||||||
|
25
|
Toutes les fonctions sont-elles commentées ? | ||||||||||||||||||||||||||
|
26
|
Est-ce que les commentaires vous semblent cohérents et à jour par rapport au code ? | ||||||||||||||||||||||||||
|
27
|
Est-ce que l’usage et le fonctionnement des librairies tierces sont documentés ? | ||||||||||||||||||||||||||
|
28
|
Y-a-t-il du code incomplet ? Si oui, doit-il être supprimé ou marqué avec un TODO ou un autre moyen ?
|
||||||||||||||||||||||||||