diff --git a/audits/2024-07-14 b/audits/2024-07-14 new file mode 100644 index 0000000..a513581 --- /dev/null +++ b/audits/2024-07-14 @@ -0,0 +1,804 @@ + +
+ + + + +version 0.0
commit
+8ce7191df3652ddd6284c4ac6d8d42c3cf32d2d5
Voici l’arborescence de la version actuelle du projet.
+├── data
+│ ├── dimers random walk.xml
+│ ├── image
+│ │ ├── [...]
+│ └── text
+│ ├── [...]
+├── include
+│ ├── automat.h
+│ ├── base.h
+│ ├── calls.h
+│ ├── graph.h
+│ ├── parse.h
+│ └── widgets.h
+├── Makefile
+├── manifest.scm
+├── [...]
+└── src
+ ├── automat.c
+ ├── calls.c
+ ├── graph.c
+ │ ├── area.c
+ │ ├── draw.c
+ │ ├── grid.c
+ │ ├── init.c
+ │ ├── shader.frag
+ │ ├── shader.vert
+ │ └── stack.c
+ ├── main.c
+ ├── parse.c
+ └── widget.c
+ ├── heads.c
+ ├── labo.c
+ ├── rules.c
+ ├── state.c
+ ├── stock.c
+ └── tree.c
+
+8 dossiers, 46 fichiers
+Un certain nombre d’élements doivent être respectés :
+src
, excepté
+base.h
src
constitue un module clairement définiErreurs critiques :
+src
sont nommés comme des fichiers
+(*.c
)widget
est mal nommé car il est pointé par le
+header widgets.h
Généralités :
+Problèmes critiques relevés :
+pragma once
base.h
indûment[critique]
+[à retravailler]
pragma once
base.h
[critique]
[à retravailler]
pragma once
base.h
xor
est totalement incompréhensible et inutileautomat_
[critique]
+[à retravailler]
pragma once
base.h
on_
, courant pour un
+module d’évènements[critique]
+[à retravailler]
pragma once
base.h
(ok car types utilisés)graph_
[critique]
+[à retravailler]
pragma once
base.h
parse_
[critique]
[à retravailler]
pragma once
base.h
widget_
Problèmes critiques relevés :
+[critique]
+[à retravailler]
le nom est clair +et correspond à une fonctionnalité
contient un en-tête de +licence valide selon la norme SPDX
style de +commentaire minimal et systématique
style de +code/déclaration du projet respecté
clarté et utilité +des commentaires
@:340
l’id de l’application est incohérent
[critique]
[à retravailler]
@:101:get_str_STATE_RULES_DATA()
très sous-optimal
+[critique]
+[à retravailler]
@:249:switch_STATE_RULES_DATA()
+[critique]
+[à retravailler]
[critique]
[à retravailler]
[critique]
+[à retravailler]
[critique]
+[à retravailler]
[critique]
+[à retravailler]
[critique]
+[à retravailler]
[critique]
[à retravailler]
[critique]
[à retravailler]
[critique]
+[à retravailler]
[critique]
[à retravailler]
[critique]
+[à retravailler]
[critique]
+[à retravailler]
You who enter here, leave all hope of an understandable +code
+Ok, je saute.
+[critique]
+[à retravailler]
@:32
En GTK4, il ne faut jamais utiliser la taille de
+fenêtre comme comme référence ou aide au layout. Ce n’est pas dans
+l’esprit de l’API. On utilisera plutôt les méthodes de layout (GtkGrid
+par exemple).get
+devraient être des des getters. C’est à dire renvoyer des valeurs et non
+pas des widgets ! C’est un nommage trompeur et problématique.[critique]
+[à retravailler]
@:34
En GTK4, il ne faut jamais utiliser la taille de
+fenêtre comme comme référence ou aide au layout. Ce n’est pas dans
+l’esprit de l’API. On utilisera plutôt les méthodes de layout (GtkGrid
+par exemple).get
+devraient être des des getters. C’est à dire renvoyer des valeurs et non
+pas des widgets ! C’est un nommage trompeur et problématique.[critique]
+[à retravailler]
[critique]
+[à retravailler]