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

[PBYetml]

ETAT : APP_STATE_SERVICE_TASKS

• [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/JoaoManuelMarquesChaves/TP2_MINF_2024/blob/58e2dc8c02d2c3f120136e658d78dc7e51cdcfbf/TP2_UART/TP1_PWM%202/firmware/src/app.c#L297 https://github.com/JoaoManuelMarquesChaves/TP2_MINF_2024/blob/58e2dc8c02d2c3f120136e658d78dc7e51cdcfbf/TP2_UART/TP1_PWM%202/firmware/src/app.c#L316
• [x] => bugs / amélioration / question => algorithme => test/condition => comment pourriez gérer cette notion de test de cycle ?
• [x] - => bugs / amélioration / question => algorithme => variable => IcycleTx => 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/JoaoManuelMarquesChaves/TP2_MINF_2024/blob/58e2dc8c02d2c3f120136e658d78dc7e51cdcfbf/TP2_UART/TP1_PWM%202/firmware/src/app.c#L313C1-L324C14

AUTRE voir issue 5 - app.c voir issue 2 - app.c

[JoaoManuelMarquesChaves]

1. Explication de else et simplification du test CommStatus L'utilisation de else dans votre code signifie qu'une action doit être effectuée lorsque la condition précédente (dans le if) n'est pas remplie. Pour améliorer le code, nous pouvons initialiser PwmData directement dans le cas else (mode remote), ce qui nous permet d'éliminer le besoin de vérifier CommStatus une seconde fois.

2. Gestion de IcycleTx Dans le if, lorsqu'on atteint le nombre de cycles désiré (IcycleTx >= 5), on effectue l'envoi et réinitialise IcycleTx à 0. Dans le else, on incrémente simplement IcycleTx.

3. IcycleTx et la notion de cycle IcycleTx est un compteur qui mesure le nombre de fois qu'on entre dans l'état task. Dans votre cas, un cycle est égal à 100 ms.