mikroc

mikroc - C - Programmation

Marsh Posté le 25-05-2015 à 20:42:39    

bonsoir
je suis nouveau en programmation, et j'ai realise ce petit decompteur, je cherche a l'optimise et rajouter 2 ou 3 petit truck si quelqu'un a un instant pour moi merci d'avance voici le code:
 
#define plus PORTA.RA0
#define moin PORTA.RA1
#define star PORTA.RA2
char display;
  char i;
 
void main() {
      debut :;
      //display = 0;
      //i=0;
trisb = 0x00;
portb = 0x00;
//trisc = 0X03;
//portc = 0x00;
TRISA=0xff ;
PORTA=0x00;
display = 1 ;
 
while(1){
 
if(!plus)
{
display++;
while(!plus);
}
if(!moin)
{
display--;
while(!moin);
}
PORTB=Dec2Bcd(display);
delay_ms(20);
//if (i>90) display = 0;
//if (i<0) display = 99;
if(!star)
{
while(1)
{
if (i==0)
{
i=display;
}
PORTB=Dec2Bcd(i);
i-- ;
delay_ms(100);
if (i==0) goto debut;
}
while(!star);
}
}
}

Reply

Marsh Posté le 25-05-2015 à 20:42:39   

Reply

Marsh Posté le 25-05-2015 à 21:06:49    

quelque remarques:
-baliser le code pour une mise en page correcte!!
-décrire ton algorithme, personne à envie de décortiquer le code pour comprendre comment c'est censé fonctionner... Bon, pour cette fois ci ça va, c'est pas bien long.
-moins avec un s
-star c'est start? Alors autant l'écrire en entier pour que ce soit clair.
-PORTA.RA0 c'est très pratique et correct mais il faut savoir que c'est pas portable. méthode classique: bitshift et opérations binaires, soit if(!(PORTA&(1<<_RA0))) avec #define _RA0 0 (la position du pin sur le port, il y a certainement un fichier header avec les bonnes constantes quelque part)
-les variables globales c'est moche, à utiliser uniquement quand c'est absolument nécessaire ce qui n'est pas le cas içi
-le goto c'est très moche, à éviter! Si on en abuse on se retrouve avec du code "spaghetti" incompréhensible
-delay_ms: ouais... Si le processeur n'a rien d'autre à faire de toute façon on peut utiliser cette fonction, mais c'est pas très élégant. Si on veut faire autre chose alors que le compteur tourne faut passer par un timer et une interruption, mais là sur du PIC je sèche.
-soit cohérent, soit les noms des registres en majuscule, soit en minuscule mais pas les deux mélangés.
-on initialise une seule fois, donc le debut: est mal placé. (Faut mettre un point-virgule derrière? Je sais pas, jamais utilisé...)


Message édité par rat de combat le 25-05-2015 à 21:08:51
Reply

Marsh Posté le 25-05-2015 à 21:20:08    

la suite:
-utiliser des #define pour avoir les constantes en haut et à un seul endroit, ça facilite nettement les modifications éventuelles
-ne pas hésiter à utiliser des #define pour faire un code plus parlant (mais ça peut être un sujet de discussions, certains n'aiment pas le préprocesseur...)
 
non testé:

Code :
  1. #define bouton_plus PORTA.RA0
  2. #define bouton_moins PORTA.RA1
  3. #define bouton_start PORTA.RA2
  5. #define appui_bouton(b) (!(b))
  7. #define TEMPS_MAX 9
  8. #define TEMPS_MIN 0
  10. #define TICK_DELAY_MS 100
  12. void main()
  13. {
  14.     char temps;
  15.    
  16.     TRISB=0x00;
  17.     PORTB=0x00;
  18.     TRISA=0xff;
  19.     PORTA=0x00;
  21.     temps=1;
  23.     while(1)
  24.     {
  25.         if(appui_bouton(bouton_plus))
  26.         {
  27.             if(temps<TEMPS_MAX)
  28.                 temps++;
  29.             PORTB=Dec2Bcd(temps);
  30.             while(appui_bouton(bouton_plus));
  31.         }
  32.         else if(appui_bouton(bouton_moins))
  33.         {
  34.             if(temps>TEMPS_MIN)
  35.                 temps--;
  36.             PORTB=Dec2Bcd(temps);
  37.             while(appui_bouton(bouton_moins));
  38.         }
  39.         else if(appui_bouton(bouton_start))
  40.         {
  41.             while(temps>0)
  42.             {
  43.                 PORTB=Dec2Bcd(temps);
  44.                 delay_ms(TICK_DELAY_MS);
  45.                 temps--;
  46.             }
  47.             while(appui_bouton(bouton_start));
  48.         }
  49.     }
  50. }


 
edit: essayer de ne pas mélanger (trop) l'anglais et le français au niveau des noms de variables, moi-même j'ai du mal avec ça...


Message édité par rat de combat le 25-05-2015 à 21:23:13
Reply

Marsh Posté le 25-05-2015 à 21:48:36    

waw merci "rat de combat" pour ta réponse mais comme je l'ai dit c'est vraiment mon premier code, donc j'ai bien note toutes tes remarques et s'il en reste n’hésite pas, et aussi merci pour le Recodage.
je le test et je te dit quoi. merci encore pour l’intérêt.

Reply

Marsh Posté le 25-05-2015 à 22:02:40    

merci mon pote il super nikel propre ton code  .
dit , j'utilise 2X7447 es que le code change beaucoup si j'utilise 1 seul 74ls47 avec 2 digit 7 segments?

Reply

Marsh Posté le 30-05-2015 à 13:11:16    

Je viens juste de voir: Désolé, les 7447 et Co. j'y connais rien. C'est plutôt une question pour un forum d'électronique ça.

Reply

Marsh Posté le 01-06-2015 à 02:19:12    

y a pas de quoi "rat de combat";
voila j'y suis presque ,mais je cale sur la mémorisation de la dernière valeur et aussi la vitesse de l’exécution ( après avoir appuyer sur star il vas trop vite).
je vous joins le fichier complet (code +isis) si une bonne âmes aimerais bien m'aider.
 
le code: (en espérant qu'il est digne d'un codehttp://forum-images.hardware.fr/icones/message/icon7.gif)
 

Code :
  1. void displayDigit(int y)
  2. {
  3. switch(y)
  4. {
  5. case 0:PORTB        = 0x3F;break;
  6. case 1:PORTB        = 0x06;break;
  7. case 2:PORTB        = 0x5B;break;
  8. case 3:PORTB        = 0x4F;break;
  9. case 4:PORTB        = 0x66;break;
  10. case 5:PORTB        = 0x6D;break;
  11. case 6:PORTB        = 0x7D;break;
  12. case 7:PORTB        = 0x07;break;
  13. case 8:PORTB        = 0x7F;break;
  14. case 9:PORTB        = 0x67;break;
  15. }
  16. }
  18. #define bouton_plus PORTD.RD0
  19. #define bouton_moins PORTD.RD1
  20. #define bouton_start PORTD.RD2
  21. #define bouton_select PORTD.RD3
  22. #define appui_bouton(b) (!(b))
  23. #define TEMPS_MAX 9999
  24. #define TEMPS_MIN 0
  25. #define TICK_DELAY_MS 100
  27. unsigned int count = 1;
  28. unsigned int count1 = 0;
  29. unsigned int count2 = 0;
  30. unsigned int count3 = 0;
  31. unsigned int count4 = 0;
  32. unsigned int time = 0;
  33. unsigned int x = 0;
  35. void main(void)
  36. {
  38. TRISA = 0x00;
  39. TRISB = 0x00;
  40. PORTA = 0x00;        // RA3 RA2 RA1 RA0 (Common Cathode)
  41. PORTB = 0x00;        // dpgfedcba (Common Cathode)
  42. TRISD = 0xff;        // input button
  43. PORTD = 0x00;
  44. TRISC = 0x00;
  45. PORTC = 0x00;
  46.     //count=0;
  47.            while(1)
  48.     {
  49.                 count4= (count/1000)%10;
  50.                 count3= (count/100)%10;
  51.                 count2 = (count/10)%10;
  52.                 count1 = count%10;
  53.                 for (x = 0; x <=50; x--)
  54.                 PORTA        = 0x17; displayDigit(count1); Delay_ms(10);
  55.                 PORTA        = 0x1B; displayDigit(count2); Delay_ms(10);
  56.                 PORTA        = 0x1D; displayDigit(count3); Delay_ms(10);
  57.                 PORTA        = 0x16; displayDigit(count4); Delay_ms(10);
  60.         if(appui_bouton(bouton_plus))
  61.         {
  63.             if(count<TEMPS_MAX)
  64.                 count++;
  66.         }
  68.          if(appui_bouton(bouton_moins))
  69.         {
  70.             if(count>TEMPS_MIN)
  71.                 count--;
  72.            time=count;
  74.         }
  75.        
  77.           if(appui_bouton(bouton_start))
  78.           {
  79.                
  80.                do
  81.                {
  82.                 if (count > TEMPS_MIN)
  85.                 count4= (count/1000)%10;
  86.                 count3= (count/100)%10;
  87.                 count2 = (count/10)%10;
  88.                 count1 = count%10;
  89.                 for (x = 0; x <=50; x--)
  90.                 PORTA        = 0x17; displayDigit(count1); Delay_ms(10);
  91.                 PORTA        = 0x1B; displayDigit(count2); Delay_ms(10);
  92.                 PORTA        = 0x1D; displayDigit(count3); Delay_ms(10);
  93.                 PORTA        = 0x16; displayDigit(count4); Delay_ms(10);
  94.                }
  95.              
  96.                while(count--);
  97.                 PORTC=0xff;
  98.                 delay_ms(1000);
  99.                 count=1;
  100.                 PORTC=0;
  103.          
  104.          
  105.          
  106.          
  107.            }
  115.    }
  116.                
  117.                
  118. }


Message édité par nebukadnezare le 01-06-2015 à 02:24:10
Reply

Marsh Posté le 01-06-2015 à 02:23:13    

heu; je ne vois pas comment inséré une pièce jointe????

Reply

Marsh Posté le 01-06-2015 à 20:51:07    

Pour les PJ faut regarder l'aide: http://wiki.atlas.doctissimo.fr/di [...] d%27images (doctissimo? C'est lié à HW.fr ça?)

 

Pour ton code:
- mise en page encore améliorable, pas besoin de x espaces dans chaque ligne du switch(case) et attention à l'insertion(?? - le décalage à droite quoi) et au bon placement des accolades.
- Encore des variables globales, faut les mettre dans main().
- N'hésite pas à insérer des lignes blanches pour aérer un peu ton code! edit: Pardon, tu le fais déjà, ça doit être un problème de copier-coller (je mets le code dans un autre logiciel pour pouvoir l'éditer).

 

Sans regarder la logique du code on peut voir que tes boucles for() sont pas bonnes. for (x = 0; x <=50; x--) Tu vois le problème?

 

Ensuite pour ta première boucle tu veux éxecuter en boucle tout le bloc de code qui vient après je suppose? Alors il faut obligatoirement mettre des accolades, sinon seule la première instruction (soit PORTA = 0x17; ) est exécutée en boucle!

Code :
  1. for (x = 0; x <=50; x--)
  2. {
  3.     PORTA = 0x17; displayDigit(count1); Delay_ms(10);
  4.     PORTA = 0x1B; displayDigit(count2); Delay_ms(10);
  5.     PORTA = 0x1D; displayDigit(count3); Delay_ms(10);
  6.     PORTA = 0x16; displayDigit(count4); Delay_ms(10);
  7. }
 

Dans le bloc  if(appui_bouton(bouton_moins)) on trouve ceci time=count; sauf que time est utilisé null part.

 

Pour la logique tu code faut que je refléchisse un moment. Apparemment tu as un affichage LED multiplexé, le mieux serait de gérer ça avec un timer et une interruption mais là tu risque de galérer et moi je connais pas les PIC...

 

(à suivre)


Message édité par rat de combat le 01-06-2015 à 20:52:44
Reply

Marsh Posté le 01-06-2015 à 21:11:39    

Touts est dans la code, en attente de vos lire merci
 
 
 

Code :
  1. // Projet décompteur sur LSD 4 digit 7 segments
  2. // pour l'instant comme sa il marche comme je veut;
  3. // saut que le delays de la dernière phase "START " il descend trop vite
  4. // il n'ais pas calibre en temps réel (seconde)
  6. void displayDigit(int y)
  7. {
  8. switch(y)
  9. {
  10. case 0:PORTB        = 0x3F;break;
  11. case 1:PORTB        = 0x06;break;
  12. case 2:PORTB        = 0x5B;break;
  13. case 3:PORTB        = 0x4F;break;
  14. case 4:PORTB        = 0x66;break;
  15. case 5:PORTB        = 0x6D;break;
  16. case 6:PORTB        = 0x7D;break;
  17. case 7:PORTB        = 0x07;break;
  18. case 8:PORTB        = 0x7F;break;
  19. case 9:PORTB        = 0x67;break;
  20. }
  21. }
  23. #define bouton_plus PORTD.RD0
  24. #define bouton_moins PORTD.RD1
  25. #define bouton_start PORTD.RD2
  26. #define bouton_select PORTD.RD3
  27. #define appui_bouton(b) (!(b))
  28. #define TEMPS_MAX 9999
  29. #define TEMPS_MIN 0
  30. #define TICK_DELAY_MS 10
  32. unsigned int count = 1;
  33. unsigned int count1 = 0;
  34. unsigned int count2 = 0;
  35. unsigned int count3 = 0;
  36. unsigned int count4 = 0;
  37. unsigned int x = 0;
  39. void main(void)
  40. {
  41. TRISA = 0x00;
  42. TRISB = 0x00;
  43. PORTA = 0x00;        // RA3 RA2 RA1 RA0 (Common Cathode)
  44. PORTB = 0x00;        // dpgfedcba (Common Cathode)
  45. TRISD = 0xff;        // input button
  46. PORTD = 0x00;
  47. TRISC = 0x00;        // OUTPUT LED
  48. PORTC = 0x00;
  50.            while(1)
  51.     {
  52.                 count4= (count/1000)%10;             // AFFICHAGE 0000
  53.                 count3= (count/100)%10;
  54.                 count2 = (count/10)%10;
  55.                 count1 = count%10;
  56.                 PORTA = 0x17; displayDigit(count1); Delay_ms(TICK_DELAY_MS);
  57.                 PORTA = 0x1B; displayDigit(count2); Delay_ms(TICK_DELAY_MS);
  58.                 PORTA = 0x1D; displayDigit(count3); Delay_ms(TICK_DELAY_MS);
  59.                 PORTA = 0x16; displayDigit(count4); Delay_ms(TICK_DELAY_MS);
  62.         if(appui_bouton(bouton_plus))    // BP PLUS TEMPS
  63.         {
  64.             if(count<TEMPS_MAX)
  65.                 count++;
  67.         }
  69.          if(appui_bouton(bouton_moins))    //BP MOINS TEMPS
  70.         {
  71.             if(count>TEMPS_MIN)
  72.                 count--;
  74.         }
  75.     // avant de passer à START je voudrais que "count" soie mémorisée pour qu'a la fin
  76.     // du décomptage il m'affiche ce dernier en début.
  77.          
  78.          
  79.           if(appui_bouton(bouton_start))     // BP DEBUT DECOMPTAGE
  80.           {
  81.               do
  82.                {
  83.                 if (count > TEMPS_MIN)               // AFFICHAGE DECOMPTE
  84.                 count4= (count/1000)%10;
  85.                 count3= (count/100)%10;
  86.                 count2 = (count/10)%10;
  87.                 count1 = count%10;
  88.                 PORTA = 0x17; displayDigit(count1); Delay_ms(TICK_DELAY_MS);
  89.                 PORTA = 0x1B; displayDigit(count2); Delay_ms(TICK_DELAY_MS);
  90.                 PORTA = 0x1D; displayDigit(count3); Delay_ms(TICK_DELAY_MS);
  91.                 PORTA = 0x16; displayDigit(count4); Delay_ms(TICK_DELAY_MS);
  92.                }
  93.              
  94.                while(count--);                // CONDITION
  95.                 PORTC=0xff;                   // ACTIVATION PORTC...
  96.                 delay_ms(1000);               // ...1S
  97.                 count=1;
  98.                 PORTC=0;
  99.          
  100.            }
  101.    }
  102. }

Reply

Marsh Posté le 01-06-2015 à 21:11:39   

Reply

Marsh Posté le 01-06-2015 à 21:19:07    

Reply

Marsh Posté le 01-06-2015 à 21:19:54    

EDIT: Ce message est la suite de mon dernier, je n'ai pas vu le message 21:11:39 avant de poster!

 

Ici il manque encore des accolades: if (count > TEMPS_MIN) (dans la boucle do-while). En plus cette condition on peut s'en passer. Pour appui bouton moins il y a déjà un test pour voir si on peut décrémenter. Par contre si le temps est écoulé il ne faut pas remettre 1 dans count comme tu le fais ici count=1; mais il faut mettre TEMPS_MIN.

 

Pour le programme entier:

 

Lorsqu'on a appuyé sur start il faut faire s'écouler le temps mais en même temps maintenir l'affichage. Entre parenthèses, pourquoi une boucle do while() plutôt que while(). C'est un truc en rapport avec l'affichage du zéro ou pas non? Je manque de pratique, j'ai un doute là, à voir...

 

Bon, si on prends la méthode naive

Code :
  1. if(appui_bouton(bouton_start))
  2. {
  4.     do
  5.     {
  6.         count4= (count/1000)%10;
  7.         count3= (count/100)%10;
  8.         count2 = (count/10)%10;
  9.         count1 = count%10;
  10.         PORTA = 0x17; displayDigit(count1);
  11.         PORTA = 0x1B; displayDigit(count2);
  12.         PORTA = 0x1D; displayDigit(count3);
  13.         PORTA = 0x16; displayDigit(count4);
  15.         delay_ms(1000);
  16.        
  17.     }while(count--);
  18.    
  19.     PORTC=0xff;
  20.     count=TEMPS_MIN;
  21.     PORTC=0;
  22. }


on va bloquer l'affichage. (Entre parenthèses les instructions d'affichage prennent du tems aussi qu'il faudrait enlèver sur le delay. Le soucis c'est que ce temps de calcul n'est pas constant mais dépends de count. Le chrono ne sera pas très précis. Pour régler ce problème faut utiliser un timer (le PIC en a je suppose).)

 

Pour maintenir l'affichage on "triche": Au lieu d'attendre une fois 1000ms on attends p.ex. 200 fois 5ms et entre chaque attente on s'occupe de l'affichage. La première approche

Code :
  1. if(appui_bouton(bouton_start))
  2. {
  4.     do
  5.     {
  6.         count4= (count/1000)%10;
  7.         count3= (count/100)%10;
  8.         count2 = (count/10)%10;
  9.         count1 = count%10;
  10.        
  11.         unsigned int nombre;
  12.        
  13.         for(nombre=0; nombre <200; nombre++)
  14.         {
  15.             PORTA = 0x17; displayDigit(count1);
  16.             PORTA = 0x1B; displayDigit(count2);
  17.             PORTA = 0x1D; displayDigit(count3);
  18.             PORTA = 0x16; displayDigit(count4);
  20.             delay_ms(5);
  21.         }
  22.        
  23.     }while(count--);
  24.    
  25.     PORTC=0xff;
  26.     count=TEMPS_MIN;
  27.     PORTC=0;
  28. }


devrait faire apparaître un défaut: Le dernier nombre est bien plus lumineux que les autres. C'est normal parce que c'est lui qui reste affiché pendant le delay de 5ms (Entre parenthèses même remarque qu'en haut: La boucle for prends aussi un peu de temps --> imprécis). La solution tu y as déjà pensé: Diviser le temps d'attente de nouveau pour attendre le même temps entre chaque digit. Bon, voyons, il faut un multiple de 4 et c'est pas le cas du cinq. Bon, alors on dit qu'on attends 250 fois 4x1ms, ça devrait être bon. Faut choisir des temps raisonnables, là on est encore plutôt haut avec 250Hz de fréquence de multiplexage.
Bref, voici l'idée en C:

Code :
  1. if(appui_bouton(bouton_start))
  2. {
  4.     do
  5.     {
  6.         count4= (count/1000)%10;
  7.         count3= (count/100)%10;
  8.         count2 = (count/10)%10;
  9.         count1 = count%10;
  10.        
  11.         unsigned int nombre;
  12.        
  13.         for(nombre=0; nombre <250; nombre++)
  14.         {
  15.             PORTA = 0x17; displayDigit(count1); delay_ms(1);
  16.             PORTA = 0x1B; displayDigit(count2); delay_ms(1);
  17.             PORTA = 0x1D; displayDigit(count3); delay_ms(1);
  18.             PORTA = 0x16; displayDigit(count4); delay_ms(1);
  19.         }
  20.        
  21.     }while(count--);
  22.    
  23.     PORTC=0xff;
  24.     count=TEMPS_MIN;
  25.     PORTC=0;
  26. }
 

À toi de reposter un nouveau code complet, je fais exprès de ne pas le faire. Tout mes dires sont sans garantie, comme je disais je manque de pratique (et l'erreur est humaine...) et je ne peux pas tester le code faute du support hardware!


Message édité par rat de combat le 01-06-2015 à 21:55:46
Reply

Marsh Posté le 01-06-2015 à 21:50:09    

Plusieurs messages en parallèle... Regarde mon dernier message pour "calibrer" ton truc et aussi pour toutes les autres remarques.  
 
Mémoriser START? Tu veux dire le temps programmé? Comment pourrait-on faire...? Peut-être en sauvegardant le temps dans une variable de sauvegarde avant de démarrer le compteur et réinitialiser count avec cette valeur sauvegardée à la fin?
 

Citation :

LSD 4 digit 7 segments


:lol:
 
Et heu pardon mais un minimum d'application svp...

Citation :


    // pour l'instant comme ça il marche comme je veux;
    // sauf que le délai de la dernière phase "START " il descend trop vite
    // il n'est pas calibré en temps réel (seconde)


Reply

Marsh Posté le 01-06-2015 à 22:06:08    

ok ok  ;)
 pour le temps c'est super merci ( il y a juste de petit étincellement au nivaux de l'affichage mais je crois que c'est ISIS).
il ne me reste plus que la mémorisation du temps choisie avant le START , et bingo ;).
petit remarque quand j'utilise le " unsigned int nombre; " il me fait une erreur donc je l'ai mis au début et sa marche.

Message cité 1 fois
Reply

Marsh Posté le 01-06-2015 à 22:09:33    

Citation :

il y a juste de petit étincellement au nivaux de l'affichage


C'est à dire? Qu'entends tu par étincellement?

 
Citation :

petit remarque quand j'utilise le " unsigned int nombre; " il me fait une erreur donc je l'ai mis au début et sa marche.


Oui c'est possible, faut déclarer au début de la fonction.

 

edit: "au début"... mais dans la fonction main()? Ou t'as toujours tes variables globales? Poste donc une version actuelle du code après avoir pris en compte tout mes messages (au cas où tu en aurait raté un avec nos messages qui se sont recoupés).


Message édité par rat de combat le 01-06-2015 à 22:12:21
Reply

Marsh Posté le 01-06-2015 à 22:19:24    

étincellement, c'est que les chiffres ne défile pas normalement, il sont saccadé, je pence que c'est le simulateur.  
Es que ta essayé  la simulation?

Reply

Marsh Posté le 01-06-2015 à 22:20:50    

J'ai rien simulé du tout faute d'avoir le bon logiciel. Fais donc un test sur le hardware réel...

Reply

Marsh Posté le 01-06-2015 à 22:24:40    

Ok ;il me faut juste la fameuse ligne de mémorisation

Reply

Marsh Posté le 01-06-2015 à 22:28:27    

A désolé, je ne vais pas faire le travail à ta place. Je t'ai donné une indication comment faire, à toi de voir...

Reply

Marsh Posté le 02-06-2015 à 00:01:33    

ok je vais essaye de moi même on verras ce que sa va donner et je te dit quoi.

Reply

Marsh Posté le 02-06-2015 à 02:51:54    

voila je crois que c'est bon ,et vraiment vraiment merci "rat de combat" du soutient; voici le code finale, s'il y a des choses en trop n’hésite pas.
 

Code :
  1. void displayDigit(int y)
  2. {
  3. switch(y)
  4. {
  5. case 0:PORTB        = 0x3F;break;
  6. case 1:PORTB        = 0x06;break;
  7. case 2:PORTB        = 0x5B;break;
  8. case 3:PORTB        = 0x4F;break;
  9. case 4:PORTB        = 0x66;break;
  10. case 5:PORTB        = 0x6D;break;
  11. case 6:PORTB        = 0x7D;break;
  12. case 7:PORTB        = 0x07;break;
  13. case 8:PORTB        = 0x7F;break;
  14. case 9:PORTB        = 0x67;break;
  15. }
  16. }
  18. #define bouton_plus PORTD.RD0
  19. #define bouton_moins PORTD.RD1
  20. #define bouton_start PORTD.RD2
  21. #define bouton_select PORTD.RD3
  22. #define appui_bouton(b) (!(b))
  23. #define TEMPS_MAX 9999
  24. #define TEMPS_MIN 0
  25. #define TICK_DELAY_MS 10
  27. unsigned int count = 0;
  28. unsigned int count1 = 0;
  29. unsigned int count2 = 0;
  30. unsigned int count3 = 0;
  31. unsigned int count4 = 0;
  32. unsigned int x = 0;
  33. unsigned int nombre;
  34. unsigned short mem;
  35. void main(void)
  36. {
  37. count=EEPROM_Read(0x32);
  38. TRISA = 0x00;
  39. TRISB = 0x00;
  40. PORTA = 0x00;        // RA3 RA2 RA1 RA0 (Common Cathode)
  41. PORTB = 0x00;        // dpgfedcba (Common Cathode)
  42. TRISD = 0xff;        // input button
  43. PORTD = 0x00;
  44. TRISC = 0x00;        // OUTPUT LED
  45. PORTC = 0x00;
  47.            while(1)
  48.     {
  49.                 count4= (count/1000)%10;             // AFFICHAGE 0000
  50.                 count3= (count/100)%10;
  51.                 count2 = (count/10)%10;
  52.                 count1 = count%10;
  53.                 PORTA = 0x17; displayDigit(count1); Delay_ms(TICK_DELAY_MS);
  54.                 PORTA = 0x1B; displayDigit(count2); Delay_ms(TICK_DELAY_MS);
  55.                 PORTA = 0x1D; displayDigit(count3); Delay_ms(TICK_DELAY_MS);
  56.                 PORTA = 0x16; displayDigit(count4); Delay_ms(TICK_DELAY_MS);
  59.         if(appui_bouton(bouton_plus))    // BP PLUS TEMPS
  60.         {
  61.             if(count<TEMPS_MAX)
  62.                 count++;
  64.         }
  66.          if(appui_bouton(bouton_moins))    //BP MOINS TEMPS
  67.         {
  68.             if(count>TEMPS_MIN)
  69.                 count--;
  71.         }
  72.           if(appui_bouton(bouton_start))     // BP DEBUT DECOMPTAGE
  73.           {
  74.           EEPROM_Write(0x32, count);         // Mémorisation de la valeur
  76.              
  77.               do
  78.                {
  79.                    if (count > TEMPS_MIN)               // AFFICHAGE DECOMPTE
  80.                    {
  81.                      count4= (count/1000)%10;
  82.                      count3= (count/100)%10;
  83.                      count2 = (count/10)%10;
  84.                      count1 = count%10;
  85.                    
  86.                      //unsigned int nombre;
  88.                       for(nombre=0; nombre <150; nombre++)
  89.                       {
  90.                       PORTA = 0x17; displayDigit(count1); delay_ms(1);
  91.                       PORTA = 0x1B; displayDigit(count2); delay_ms(1);
  92.                       PORTA = 0x1D; displayDigit(count3); delay_ms(1);
  93.                       PORTA = 0x16; displayDigit(count4); delay_ms(1);
  94.                       }
  96.                 }
  97.                }
  98.              
  99.                while(count--);                // CONDITION
  100.                 PORTC=0xff;                   // ACTIVATION PORTC...
  101.                 delay_ms(1000);               // ...1S
  102.                 //count=TEMPS_MIN;
  103.                 PORTC=0;
  104.                 count = EEPROM_Read(0x32);
  106.          
  107.            }
  108.    }
  109. }

Reply

Marsh Posté le 02-06-2015 à 13:56:23    

Il y a toujours les variables globales, si ce terme ne te dit rien faut regarder dans ton cours, un livre ou sur le net! Il y a aussi deux variables inutilisées. Quand tu en auras fait des variables locales ton compilateur devrait d'avertir, sinon il est mal configuré (faut toujours demander le maximum de warnings!).
 
Pour la mémorisation on peut faire comme ça.
 
Ton chrono n'est pas calibré, voir mon message 01-06-2015 à 21:19:54.
 
Des problèmes de mise en page qui sont peut-être réels ou proviennent d'un problème au niveau du forum.
 
Il y a deux version du code d'affichage, une avec Delay_ms(TICK_DELAY_MS); et l'autre. 4x10ms=25Hz=peu, ça risque de se voir. (C'est peut-être ça cet "étincellement" dont tu parlais?)
 
De manière générale: Attention de ne pas mettre des commentaires "bidons" genre

Code :
  1. int i=4; //on stocke la valeur 4 dans i


D'accord, c'est vrai, mais le code le dit clairement, inutile de répéter... Exemple concret:

Code :
  1. if(appui_bouton(bouton_plus))    // BP PLUS TEMPS

Le code est (à mon avis) suffisamment clair (mais je connais les profs qui veulent qu'on commente chaque ligne...).

Reply

Marsh Posté le    

Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed