search

Boulboz / demoGit

creation projet
HomePage
0 stars 0 forks source link

Commentaires sur le fichier app.c #1

Open PBYetml opened 1 year ago

PBYetml commented 1 year ago

L139 => static bool First_LED_Off = true ; => est-ce que vous avez le droit réellement d'utiliser le typebool en C, ne manque-t-il pas quelque chose avec ce type, autrement ce n'est pas du C

PBYetml commented 1 year ago

L140 => static S_ADCResults Result; à quoi sert la notion de static pour cette variable, en avez-vous vraiment besoin + S_ADCResults que représente ce type, ou est-il déclaré ? que devez-vous associé à ce source ?

PBYetml commented 1 year ago

L150 => BSP_InitADC10(); que fait cet appel de fct et à quelle "librairie" est-il associé ? avez-vous tout implémenter ?

PBYetml commented 1 year ago

L157 à 164 => pensez-vous que votre algorithme pour allumer toutes les leds est bien adapté -> imaginer que ce n'est pas 8leds, mais plus d'une vingtaine, est-ce que votre code est efficient, que pourriez-vous changer ?!

L178 à 185 => même type de commentaire que ci-dessus, mais pour éteindre les leds

PBYetml commented 1 year ago

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

PBYetml commented 1 year ago

L198 à 233=>pensez-vous que votre algorithme est optimal, imaginez que vous avez 50leds à gérer, est-ce qu'unswitch casesera la meilleure manière de gérer des leds en mode chenillard ? + que représente pour vous les valeurs BSP_LED_X ?

PBYetml commented 1 year ago

L237 => que représente votre cst 7 ? imaginez que vous ayez plusieurs milliers de lignes et que cette cst se retrouve à plusieurs endroit, n'y aurait-il pas un moyen d'être plus efficient si vous deviez changer cette valeur par une autre