commentaire sur le fichier app.c

[PBYetml]

L80 => 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 (#L128), n'y aurait-il pas quelques choses que vous pouvez utiliser + que faut-il implémenter dans le sources pour que le type de structure soit correctement compris

[PBYetml]

L143 => dans cette déclaration bool appInitialized = true; rappelez que vous écrivez du C et non du C++, par convention/normalisation la déclaration de variables se fait en début de fct et non au début de bloc (ceci est du C++) + 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]

L144 => votre test if (appInitialized) ... est-ce que ce test a du sens ?

[PBYetml]

L146 à 150 => pour toutes les différentes fonctions liés à l'affichage LCD ( lcd_init(); + lcd_bl_on(); + printf_lcd(); + lcd_gotoxy();) qu'est-ce que vous n'avez pas implémenter dans le fichier source

[PBYetml]

L152 => 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]

L154 => que fait cet appel de fct : DRV_TMR0_Start(); ?

[PBYetml]

L156 => si vous regardez où est placé l'instruction appData.state = APP_STATE_SERVICE_TASKS;, est-ce que celle-ci est correctement placée ? est-ce que vous respectez le cdc ?

[PBYetml]

L167 => dans cette instruction ADCResults = BSP_ReadAllADC(); à quoi devez vous faire attention au niveau de la librairie ? -> le nom de la fct porte à confusion ! + voir commentaire en #L80

[PBYetml]

L190 => dans cette déclaration uint8_t FLAG_LED = FLAG_OFF; pour utiliser ce type uint8_t qu'avez-vous besoin de déclarer pour respecter la normalisation de l'écriture en C + si vous regardez en #L103 (app.h) quelle est la différence ? pourquoi ne pas utiliser votre énumération.

L218 => même remarque que ci-dessus

[PBYetml]

L191 => bien que cela va fonctionner pourquoi mélanger un type énum et un type entier, surtout avec des mêmes nom de variable

[PBYetml]

L192 - L203 => dans vos instructions de test if(FLAG_LED == FLAG_ON) & if(FLAG_LED == FLAG_OFF), pensez-vous que ce soit utile de chaque fois de tester 2fois la variable FLAG_LED

[PBYetml]

L219 à 262 => pensez-vous que votre algorithme avec l'implémentation d'un switch - case soit des plus adapté => imaginez que vous avez 50leds à gérer

[PBYetml]

L170 => pourquoi éteindre chaque fois vos leds avec l'appel de fct LED_ON_OFF(FLAG_OFF);