Commentaire sur le fichier app.c

[PBYetml]

L82 & #L179 => S_ADCResults AdcRes; dans cette déclaration pourquoi en faire une variable globale, est-ce que vous l'utiliser plusieurs fois dans ce fichier ? ou ailleurs + regardez la déclaration de structure APP_DATA dans le fichier app.h (#L118), n'y aurait-il pas quelques choses que vous pouvez utiliser

[PBYetml]

L150 => BSP_InitADC10();cet appel de fct est lié à quelle librairie => bien que MPLABX vos "prémache" du code, controlez toujours l'inclusion vos "include" dans le fichier source

[PBYetml]

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]

L200 et #L212 => vos déclarations de fonctions : void LED_On (void){...} & void LED_Off (void){...}, imaginer que vous avez une 50taine de diodes à piloter, pensez-vous qu'il est judicieux de déclarer chaque instruction ?

[PBYetml]

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]

L224 =>void CheniLed (void) {}dans cette fct pensez-vous que votre algorithme est optimale => imaginer que vous avez 50leds à gérer, est-ce qu'un switch case sera la meilleure manière gérer les leds en mode chenillard

[PBYetml]

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 etre difficile ;)