search

MaelleClerc / MINF-TP0_LedAd

TP0 MINF
HomePage
0 stars 0 forks source link

commentaire sur le fichier app.c #1

Open PBYetml opened 1 year ago

PBYetml commented 1 year ago

L139 => static bool Service_Task_First_Cycle = true; =>est-ce que vous avez le droit réellement utiliser le type bool dans un code C, ne manque-t-il pas quelques choses => respect de la norme C

PBYetml commented 1 year ago

L141=> que représente pour vous la notion de static ? => y a-t-il vraiment besoin de déclarer votre tableau static uint8_t Leds_Address[8] en static => représente pour vous les valeurs BSP_LED_X

PBYetml commented 1 year ago

L140 => dans l'instruction static uint8_t Led_Counter; pour pouvoir utiliser correctement le type uint8_t qu'avez-vous besoin d'implémenter ?

PBYetml commented 1 year ago

L156 => bool appInitialized = 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++)

PBYetml commented 1 year ago

L159 à L165 => pour les différents appels de fonctions soient bien compris lors de la compilation, que devez-vous contrôler / implémenter ?

PBYetml commented 1 year ago

L171, L198, L209 => attention à vos cst numériques, pensez à la définir... il est fort probable que si votre code est grand, plus vous allez engendrer des erreurs si vous devez changer une cst numérique par ci par là.

PBYetml commented 1 year ago

L179 => pensez-vous que ce ce test a du sens ?! if (appInitialized)...

PBYetml commented 1 year ago

L207 => après le test de la ligne positif de la ligne 196, quel sera la valeur dans votre variableLed_Counter + auriez-vous pu améliorer votre algo dans le gestion du if au lieu uniquement du compteur de leds