app.c - remarques / améliorations / bugs

[PBYetml]

• [x] => bugs / amélioration / question => libraire => quelle librairie est associé au type S_pwmSettings ? https://github.com/CyrilFeliciano/TP2_PWM-RS232/blob/756bb8862e794ac2d4ada40023fd43727109f20e/firmware/src/app.c#L50C1-L58C20
• [x] => bugs / amélioration / question => variables => Local VS globale (un seul fichier) VS globale (plusieurs fichier) => quelle différence ? comment implémentez-vous cela ?
• [x] => bugs / amélioration / question => variables => pourquoi 3 variables globales différentes de meme type ?
• [x] => bugs / amélioration / question => variables => relation avec le fichier headerfile : gestPWM.h https://github.com/CyrilFeliciano/TP2_PWM-RS232/blob/756bb8862e794ac2d4ada40023fd43727109f20e/firmware/src/gestPWM.h#L33C1-L35C29

FCT : callback_timer1

• [x] => amélioration / question => constante numérique => hardcodage => maintenabilité ? https://github.com/CyrilFeliciano/TP2_PWM-RS232/blob/826fa09ddc1fa81be11922409560baff8ba975a0/firmware/src/app.c#L116

APP_Tasks

• [x] =>bugs / amélioration / question => déclaration => norme du langage C => quel compilateur utilisez-vous ? C VS C++ ? où devez-vous placer des déclarations ? https://github.com/CyrilFeliciano/TP2_PWM-RS232/blob/756bb8862e794ac2d4ada40023fd43727109f20e/firmware/src/app.c#L248 https://github.com/CyrilFeliciano/TP2_PWM-RS232/blob/756bb8862e794ac2d4ada40023fd43727109f20e/firmware/src/app.c#L285C1-L286C42

APP_Tasks -> ETAT : APP_STATE_SERVICE_TASKS

• [x] => bugs / amélioration / question => norme en C => règle => écriture => indentation => test/condition => if => gestion des blocs ? quelle symbolique pour les blocs de code ? https://github.com/CyrilFeliciano/TP2_PWM-RS232/blob/756bb8862e794ac2d4ada40023fd43727109f20e/firmware/src/app.c#L290 https://github.com/CyrilFeliciano/TP2_PWM-RS232/blob/756bb8862e794ac2d4ada40023fd43727109f20e/firmware/src/app.c#L308
• [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 => i => 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/CyrilFeliciano/TP2_PWM-RS232/blob/756bb8862e794ac2d4ada40023fd43727109f20e/firmware/src/app.c#L306C1-L321C14

AUTRES voir issue TP1 - app.c

[CyrilFeliciano]

1. La librairie suivante : gestPWM.h
2. Mettre une variable global dans un fichier donne accès à cette variable depuis n'importe ou dans le fichier concerné. Une variable local pourrait être utiliser seulement dans la fonction ou elle est déclarée. Pour mettre mettre un variable en global pour plusieurs fichier, il faut déclarer la variable dans le point h du fichier avec le préfixe "extern" avant celle-ci. La variable sera initialisé dans le fichier .c.
3. La structure "pData" ne sert a rien, car j'utilise deux structure une pour le mode local et l'autre pour le remote. C'est structure ne doivent pas être déclaré dans ce fichier, car elles sont déjà declarer dans le fichier gestPWM.h.
4. J'ai déplacé ces déclarations de structure dans le fichier app.c
5. Cette valeur me permet de rester dans l'état d'init pendant trois secondes, puis de passer dans l'état service task. J'ai changé la valeur par un define.
6. Les variables doivent être déclaré au début de la fonction "APP_Tasks". J'utilise le compilateur C.
7. Je m'excuse d'avance M. Bovey mais je n'ai pas réussi à mettre le tout en un bloc "if" tout en respectant de recevoir continuellement et de n'envoyer qu'une fois sur 5. Je vous pris de ne pas me remettre cette issues malgré que cela vous ferait plaisir à vous et à moi aussi!
8. Avec un compteur qui compterait jusqu'à 4. Voir corrections.
9. La variable "comptSend" est un compteur qui permet d'envoyer les données une fois sur 5. A chaque fois que le service task sera appelé, la valeur du compteur va changer. Le message sera envoyé toutes les 100ms.