jchafla97 / TP0_MINF_GIT

Creation projet Tp0 MINF
HomePage
0 stars 0 forks source link

commentaire sur le fichier system_init.c #3

Open PBYetml opened 1 year ago

PBYetml commented 1 year ago

L76 => static uint32_t i = 0; => même commentaire que la ligne #L224 dans le fichier app.c

app.c => #L226 => static uint32_t i = 0; quelle est la variation de votre variable i, quelle est la plage de valeur pour un type uint32_t

PBYetml commented 1 year ago

L77 = static bool flag = 0; => même commentaire qu'en ligne #L170

app.c => #L170 => static bool LEDs_OFF = true; => rappelez que vous écrivez du C et non du C++, par convention/normalisation la déclaration de variable se fait en début de fct et non au début de bloc (ceci est un norme du C++) + + pourquoi utiliser la notion de static ici ? est-ce que vous avez le droit réellement utiliser le type bool, ne manque-t-il pas quelques choses => respect de la norme C

PBYetml commented 1 year ago

L83 & L87 => if (i < 30) => même commentaire que dans le fichier app.c aux lignes L273 & L279 + que représente votre constante numérique => pourquoi 30 ??

app.c => #L273 & L279 => attention à vos cst numériques, pensez à la définir... imaginez que vous devez changer vos limites, est-ce que vous aller "parser" tout votre code... et que vous plus 3000 lignes... la Maj va être difficile ;)