app.c -> remarques / améliorations / bugs

[PBYetml]

• [x] => bugs / amélioration / question => libraire => quelle librairie est associé au type uint8_t ? https://github.com/Melissaperret/TP2_MINF/blob/fa99e17c153a6489e48d0ae328a2941fd4b3edaa/TP2_PWM_RS232/TP2_PWM_RS232/firmware/src/app.c#L143C1-L144C81

ETAT : APP_STATE_SERVICE_TASKS

• [x] => bugs / amélioration / question => variable => globale VS local => PWMData => pourquoi en faire une variable globale ? https://github.com/Melissaperret/TP2_MINF/blob/fa99e17c153a6489e48d0ae328a2941fd4b3edaa/TP2_PWM_RS232/TP2_PWM_RS232/firmware/src/app.c#L181C1-L198C14 https://github.com/Melissaperret/TP2_MINF/blob/fa99e17c153a6489e48d0ae328a2941fd4b3edaa/TP2_PWM_RS232/TP2_PWM_RS232/firmware/src/gestPWM.h#L32 https://github.com/Melissaperret/TP2_MINF/blob/fa99e17c153a6489e48d0ae328a2941fd4b3edaa/TP2_PWM_RS232/TP2_PWM_RS232/firmware/src/gestPWM.c#L20
• [x] => bugs / amélioration / question => algorithme => appel de fonction => est-ce que les fonction GPWM_DispSettings & GPWM_ExecPWM ne peuvent être appelée qu'une seule fois ?
  https://github.com/Melissaperret/TP2_MINF/blob/fa99e17c153a6489e48d0ae328a2941fd4b3edaa/TP2_PWM_RS232/TP2_PWM_RS232/firmware/src/app.c#L189C1-L190C40 https://github.com/Melissaperret/TP2_MINF/blob/fa99e17c153a6489e48d0ae328a2941fd4b3edaa/TP2_PWM_RS232/TP2_PWM_RS232/firmware/src/app.c#L196C1-L197C40
• [x] => bugs / amélioration / question => algorithme => variable => cntCycles => que représente-elle pour vous ? comment définissez-vous la notion de cycle ? pouvez-vous y définir une notion de temps ? https://github.com/Melissaperret/TP2_MINF/blob/fa99e17c153a6489e48d0ae328a2941fd4b3edaa/TP2_PWM_RS232/TP2_PWM_RS232/firmware/src/app.c#L144 https://github.com/Melissaperret/TP2_MINF/blob/fa99e17c153a6489e48d0ae328a2941fd4b3edaa/TP2_PWM_RS232/TP2_PWM_RS232/firmware/src/app.c#L200C1-L225C14
• [x] => bugs / amélioration / question => algorithme => test/condition => etat de commStatus => pourquoi en faire deux tests séparés ? que représente la notion else pour vous ? https://github.com/Melissaperret/TP2_MINF/blob/fa99e17c153a6489e48d0ae328a2941fd4b3edaa/TP2_PWM_RS232/TP2_PWM_RS232/firmware/src/app.c#L184 https://github.com/Melissaperret/TP2_MINF/blob/fa99e17c153a6489e48d0ae328a2941fd4b3edaa/TP2_PWM_RS232/TP2_PWM_RS232/firmware/src/app.c#L206
• [ ]

AUTRE voir issue 2 - app.c - TP1 voir issue 5 - app.c - TP1 voir issue 4 - system_interrupt.c - TP1 voir issue 7 - system_interrupt.c - TP1

[Melissaperret]

1) C'est la librairie : stdint.h, nous l'avons dans le "gestPWM.h", mais nous avons oublié de l'inclure dans notre fichier.

ETAT : APP_STATE_SERVICE_TASKS

1) On l'a mis en globale car on l'utilise dans plusieurs fichiers

2) Non, car en fonction de l'état de connexion il envoi certains paramètre ainsi que chaque 5 cycles il doit envoyer également.

3) Chaque itération du programme le compteur de cycles s'incrémente pour dire que c'est un cycle du programme.

4) La notion de "else" dans un test (if/else) veut dire que si le résultat du test est vrai il fait un programme sinon (else) fait un autre programme.