search

Gajamukan / TP0_REGL_GIT

0 stars 0 forks source link

commentaire sur le fichier app.c #1

Open PBYetml opened 1 year ago

PBYetml commented 1 year ago

L82 => S_ADCResults AdcRes; => pourquoi faire une variable globale de cette structure, est-ce que vous l'utilisez plusieurs fois dans ce fichier (fct différentes) ? ou ailleurs

PBYetml commented 1 year ago

L157 => printf_lcd("TP0 Led+AD 2023 - 2y"); => dans votre texte pourquoi la chaine de caractère suivante : TP0 Led+AD 2023 - 2y et surtout pourquoi 2Y ??

Gajamukan commented 1 year ago

2 ème année en anglais

Envoyé à partir de Outlook pour Androidhttps://aka.ms/AAb9ysg

From: Philippe Bovey ETML ES @.> Sent: Sunday, January 22, 2023 2:24:25 PM To: Gajamukan/TP0_REGL_GIT @.> Cc: Subscribed @.***> Subject: Re: [Gajamukan/TP0_REGL_GIT] commentaire sur le fichier app.c (Issue #1)

@.*** semble vous avoir déjà envoyé du courrier, mais peut-être s’agit-il d’une autre personne. Découvrez pourquoi cela pourrait constituer un risquehttps://aka.ms/LearnAboutSenderIdentification

L157 => printf_lcd("TP0 Led+AD 2023 - 2y"); => dans votre texte pourquoi la chaine de caractère suivante : TP0 Led+AD 2023 - 2y et surtout pourquoi 2Y ??

— Reply to this email directly, view it on GitHubhttps://che01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FGajamukan%2FTP0_REGL_GIT%2Fissues%2F1%23issuecomment-1399483320&data=05%7C01%7CN-Gajamukan.Subramaniyam%40eduvaud.ch%7C30f3827666a843154af908dafc7bfc76%7C906ab90804f94a80ba9c875a36e77bc1%7C0%7C0%7C638099906688379795%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=RhRyBguign4Xr5x7tj5KDSg9T%2FVAdwAujeTrzm6PPZY%3D&reserved=0, or unsubscribehttps://che01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FA4GO43QOIOF4AMOVZ2YWVBLWTUYITANCNFSM6AAAAAAUC7PZ4U&data=05%7C01%7CN-Gajamukan.Subramaniyam%40eduvaud.ch%7C30f3827666a843154af908dafc7bfc76%7C906ab90804f94a80ba9c875a36e77bc1%7C0%7C0%7C638099906688379795%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=ncXKS8yp8LXrH%2Bjc0n9w9M7MKI6eRIhBsW%2BElmsspgg%3D&reserved=0. You are receiving this because you are subscribed to this thread.Message ID: @.***>

Gajamukan commented 1 year ago

Non, il n'y a pas de raison spéciale

Envoyé à partir de Outlook pour Androidhttps://aka.ms/AAb9ysg

From: Philippe Bovey ETML ES @.> Sent: Sunday, January 22, 2023 2:21:29 PM To: Gajamukan/TP0_REGL_GIT @.> Cc: Subscribed @.***> Subject: [Gajamukan/TP0_REGL_GIT] commentaire sur le fichier app.c (Issue #1)

@.*** semble vous avoir déjà envoyé du courrier, mais peut-être s’agit-il d’une autre personne. Découvrez pourquoi cela pourrait constituer un risquehttps://aka.ms/LearnAboutSenderIdentification

L82 => S_ADCResults AdcRes; => pourquoi faire une variable globale de cette structure, est-ce que vous l'utilisez plusieurs fois dans ce fichier (fct différentes) ? ou ailleurs

— Reply to this email directly, view it on GitHubhttps://che01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FGajamukan%2FTP0_REGL_GIT%2Fissues%2F1&data=05%7C01%7CN-Gajamukan.Subramaniyam%40eduvaud.ch%7C2d289059729b45e758e808dafc7b93a3%7C906ab90804f94a80ba9c875a36e77bc1%7C0%7C0%7C638099904934911188%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=Zcc%2FIVslod%2Fhg%2FDCH5Ds7AyoEgKtcos8h4CdgMIfWaU%3D&reserved=0, or unsubscribehttps://che01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FA4GO43VQYZCP75GDGIKAE6DWTUX5TANCNFSM6AAAAAAUC7PZ4U&data=05%7C01%7CN-Gajamukan.Subramaniyam%40eduvaud.ch%7C2d289059729b45e758e808dafc7b93a3%7C906ab90804f94a80ba9c875a36e77bc1%7C0%7C0%7C638099904934911188%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=YNoAMmRkEPVQjk0rfCpBIsWMszH33u7dwyj%2Bc31s9B0%3D&reserved=0. You are receiving this because you are subscribed to this thread.Message ID: @.***>

PBYetml commented 1 year ago

L158 =>lcd_bl_on(); => que fait cet appel de fct ? attention à avoir un petit commentaire surtout si les noms des fct sont abrégées

PBYetml commented 1 year ago

L206 à 216=> void LED_ON(void){..} => dans votre déclaration de fct, pensez-vous que votre algorithme pour allumer toutes les leds est bien adapté -> imaginer que ce n'est pas 8leds, mais plus d'une vingtaine, est-ce que votre code est efficient, que pourriez-vous changer ?!

L219 à 229 => void LED_OFF(void) => pour cette déclaration voir commentaire ci-dessus

PBYetml commented 1 year ago

l234 => static uint32_t i = 0; => dans cette déclaration quelle est la variation de votre variablei, quelle est la plage de valeur pour un type uint32_t et pour pouvoir utiliser correctement ce type, qu'avez-vous besoin d'implémenter ?

L236 => switch (i) => est-ce que le nom de la variable ici est bien adapté ?!

PBYetml commented 1 year ago

L236 à 279 => pensez-vous que votre algorithme est optimal, imaginez que vous avez 50 leds à gérer, est-ce qu'un switch case sera la meilleure manière de gérer des leds en mode chenillard ? + pour utiliser convenable les appels de fonction BSP_LEDOn(x);ou BSP_LEDOff(x);qu'est-ce que vous devez veiller à ce qu'y soit implémenter dans ce fichier source ? bien qu'MPLABX, ici le fasse pour vous !?

PBYetml commented 1 year ago

L282 => if (i < 7 ) dans cette instruction de test, que représente la cst numérique 7 + imaginez que vous ayez plusieurs milliers de lignes et que vous devez remplacer cette cst à différent endroit où elles se retrouve, quelle sera le taux d'erreur possible selon vous ?, y aurait-il pas un moyen d'être plus efficient pour gérer cette / ces cst numériques ?