app.c -> bugs / amélioration / remarque

[PBYetml]

ETAT INIT

• [x] => bug / amélioration / question => variable => écriture => emplacement => en C, ou doit-on placer les déclarations de variables ?
• [x] => bug / amélioration / question => type => uint8_t => à quel librairie est associé ce type ? ou devez-vous placer cette lib ? https://github.com/CyrilFeliciano/MINF_TP1/blob/a66637f99f8c948ba6626841b47fb1093a76d55e/TP1_PWM%202/firmware/src/app.c#L234
• [x] => amélioration / question => condition => est-ce que vraiment du sens ? est-ce dans votre machine d'état vous allez revenir dans l'état INIT, à quel moment cet état est-il appelé https://github.com/CyrilFeliciano/MINF_TP1/blob/a66637f99f8c948ba6626841b47fb1093a76d55e/TP1_PWM%202/firmware/src/app.c#L236
• [x] => amélioration / question / bug => librairie => appel de fonction => à quelle "librairie - headerfile" l'appel des fonctions lcd_init, lcd_bl_on, lcd_gotoxy, etc (toues les appels de fonctions concernant le LCD) est liée ? qu'est-ce qui manque dans votre code ? https://github.com/CyrilFeliciano/MINF_TP1/blob/a66637f99f8c948ba6626841b47fb1093a76d55e/TP1_PWM%202/firmware/src/app.c#L239C1-L240C29 https://github.com/CyrilFeliciano/MINF_TP1/blob/a66637f99f8c948ba6626841b47fb1093a76d55e/TP1_PWM%202/firmware/src/app.c#L249C1-L254C47
• [x] => amélioration / question / bug => librairie => appel de fonction => à quelle "librairie - headerfile" l'appel de fonction BSP_InitADC10Alt est liée ? qu'est-ce qui manque dans votre code ? https://github.com/CyrilFeliciano/MINF_TP1/blob/a66637f99f8c948ba6626841b47fb1093a76d55e/TP1_PWM%202/firmware/src/app.c#L243
• [x] => amélioration / question / bug => librairie => appel de fonction => à quelle "librairie - headerfile" l'appel de fonction GPWM_Initialize est liée ? qu'est-ce qui manque dans votre code ? https://github.com/CyrilFeliciano/MINF_TP1/blob/a66637f99f8c948ba6626841b47fb1093a76d55e/TP1_PWM%202/firmware/src/app.c#L257C17-L257C32

ETAT WAIT

ETAT EXECUTE

AUTRES

• [x] => amélioration / question => algorithme => si vous avez 100 leds à gérer, vous allez écrire 100 appel de fonction, est-ce qu'il y aurait une meilleure solution ? conception d'itération ? voir => issue 4 ou issue 2 https://github.com/CyrilFeliciano/MINF_TP1/blob/a66637f99f8c948ba6626841b47fb1093a76d55e/TP1_PWM%202/firmware/src/app.c#L178C1-L188C2

[CyrilFeliciano]

1. Les déclarations de variables doivent se trouver au début de la déclaration de fonction.
2. Ce type est liée a la librairie "stdint.h" et il faut l'inclure dans le fichier ou l'on l'utilise.
3. Non, cela n'a pas de sens car notre programme ne doit pas retourner dans l'init.
4. Ces fonctions sont liée à la librairie "Mc32DriverLcd.h". Nous aurions dû inclure la librairie dans le fichier app.c.
5. Ces fonctions sont liée à la librairie "Mc32DriverAdcAlt.h". Nous aurions dû inclure la librairie dans le fichier app.c.
6. Ces fonctions sont liée à la librairie "gestPWM.h". Nous aurions dû inclure la librairie dans le fichier app.c.
7. Oui il y a une meilleur solution. Voici comment l'on aurait pu faire : void EteindreLEDS(void) { for (int i = 0; i <= 8; i++) { BSP_LEDOff(i); } }

Nous avons apporter les modifications au fichier app.c.