TP1 - app.c -> bug / amélioration

[PBYetml]

• => bug => librairie => utilité => règle sur le librairies dans un fichier source : voir les articles suivants article 1 article 2
• => bug => librairie => association => variable => type => uint8_t => quelle librairie est associée à cette variable
  https://github.com/NicolasBessson/TP1_TimerPWM/blob/f652cdede10ab833ab73bbc1231be5fe7df48c46/TP1_TimerPWM/firmware/src/app.c#L185C12-L185C20
• => bug => librairie => association => appel de fct => BSP_InitADC10 => quelle librairie est associée à cette fct
  

  TP1_TimerPWM/TP1_TimerPWM/firmware/src/app.c

  Line 243 in f652cde

  BSP_InitADC10(); //initialisae l'adc

• => amélioration / question => variable => globales => utilité => PWMData
  

  TP1_TimerPWM/TP1_TimerPWM/firmware/src/app.c

  Line 81 in f652cde

  S_pwmSettings PWMData;

• => question => variable => globales VS local => comment pourriez-vous remplacer une variable globale ?
• => question => variable => globales => quand est préconisé d'utiliser une variable globale
  

  TP1_TimerPWM/TP1_TimerPWM/firmware/src/app.c

  Line 82 in f652cde

  APP_DATA appData;

• => amélioration => documentation => fct => implémentation cartouche pour décrire la fct => pas implémenté partout
  

  TP1_TimerPWM/TP1_TimerPWM/firmware/src/app.c

  Line 203 in f652cde

  void APP_Timer2CallBack(void)

  
  

  TP1_TimerPWM/TP1_TimerPWM/firmware/src/app.c

  Line 208 in f652cde

  void APP_Timer3CallBack(void)

  
  

  TP1_TimerPWM/TP1_TimerPWM/firmware/src/app.c

  Line 213 in f652cde

  void APP_Timer4CallBack(void)

  
  

  TP1_TimerPWM/TP1_TimerPWM/firmware/src/app.c

  Line 219 in f652cde

  void APP_UpdateState(APP_STATES newState) {

• => amélioration / question => déclaration fct : APP_Timer2CallBack & APP_Timer3CallBack => utilité ?!
  

  TP1_TimerPWM/TP1_TimerPWM/firmware/src/app.c

  Line 203 in f652cde

  void APP_Timer2CallBack(void)

  
  

  TP1_TimerPWM/TP1_TimerPWM/firmware/src/app.c

  Line 208 in f652cde

  void APP_Timer3CallBack(void)

• => amélioration => algorithme => fct => FullLedOff => appel de fct => BSP_LEDOff => n'y a pas la possibilité de faire mieux => itération => incrémentation d'énumération ? => au lieu d'écrire une ligne par led => imaginer si vous devez gérer 100Leds ? voir issues #4_NBN et #2_MBR
  https://github.com/NicolasBessson/TP1_TimerPWM/blob/f652cdede10ab833ab73bbc1231be5fe7df48c46/TP1_TimerPWM/firmware/src/app.c#L258C1-L269C1
  =>FCT : APP_Timer1CallBack<=
• => bug / amélioration => voir CDC p5 => interprétation => appel de fct => APP_UpdateState => combien de fois devez-vous appel cette fct ?
  https://github.com/NicolasBessson/TP1_TimerPWM/blob/f652cdede10ab833ab73bbc1231be5fe7df48c46/TP1_TimerPWM/firmware/src/app.c#L197C9-L197C24
• => amélioration => instruction => est-ce que cette instruction est utile ?
• => amélioration => instruction => hardcodage => cst numérique
  

  TP1_TimerPWM/TP1_TimerPWM/firmware/src/app.c

  Line 196 in f652cde

  CntInit = 150;

• [ ]

[NicolasBessson]

amélioration / question => variable => globales => utilité => PWMData :
Amélioration possible : Nous pourrions la rendre static pour qu’elle ne soit accessible qu’à app.c,

=> question => variable => globales VS local => comment pourriez-vous remplacer une variable globale ?:
Au lieu de définir la variable au niveau global, on la déclare en local dans la fonction on la transmet par adresse avec pointeur.

=> amélioration / question => déclaration fct : APP_Timer2CallBack & APP_Timer3CallBack => utilité ?!
Supprimer

=> bug / amélioration => voir CDC p5 => interprétation => appel de fct => APP_UpdateState => combien de fois devez-vous appel cette fct ?
Vue le 08.04.2025 avec vous

[NicolasBessson]

=> amélioration => instruction => est-ce que cette instruction est utile ?
vue