Faire clignoter une LED

Version imprimable

bonsoir
Ecrire un programme servant à faire clignoter L1 si on appuis sur
BP1 et à arrêter le clignotement si on appuis une deuxième fois sur BP1.

VOICI comment j'ai procéder ,est ce que quelqu'un peut m'aider a améliorer mon programme :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 #define L1 P1_0 #define Led1On digitalWrite (L1,1); // Allume #define Led1Off digitalWrite (L1,0); // Eteint #define Led1bar digitalWrite (L1,!digitalRead(L1)); #define BP1 P1_3 #define BP1On (digitalRead (BP1)==0) // actif à 0V #define BP1Off (digitalRead (BP1)==1) // actif à 5V void setup() { pinMode (BP1,INPUT_PULLUP ); pinMode (L1,OUTPUT); } void loop () { if (BP1On) { Led1bar; delay (1000);} }

27/03/2016, 14h35
Kannagi
C'est du C pas de l'assembleur.
Bon j'imagine que la plupart des personnes qui font de l'assembleur connaisse le C mais bon ça reste pas dans la bonne section...

Je trouve que t'a un drôle de façon de coder en C (et n'oublie pas la balise code).

Je ne sait pas vraiment ce que fait exactement pinmode.
Par contre ça je trouve cela assez 'moche' pour du C :
Code:

1 2 if (BP1On) { Led1bar; delay (1000);
Mettre tout en define ça enlevé tout 'lisibilité'.
Autant en assembleur les macro sont un vrai plus autant en C je trouve les macro inutile.
Après oui on comprend facilement que si BP1On est activé alors ça allume Led1bar et en attend de 1seconde.
C'est un choix de conception si tu 'cache' le code ne t'attend pas que les autres le lise aisément pour l'amélioré.

Pour finir si le code marche tu attends quel retour , l'améliorer sur quel point ?
27/03/2016, 15h42
akybienvenu

bonsoir kannagi
je ne cache rien dans mon programme , normale si tu le trouve moche c'est par ce que je suis débutant dans cette programmation et je suis a la lettre
le cours que j’utilise . c'est pour cette raison j'ai demander que quelqu'un m'aide a améliorer ce programme en me donnant d’éventuelle suggestion

merci
27/03/2016, 17h31
f-leb

C'est de l'Arduino, non ?
28/03/2016, 00h09
akybienvenu

oui oui

bonjour,

enlève tes define. Tu y verras plus clair. Regarde l'exemple du blink().

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
int const LED=13; // sortie led
int const BTN = 12; // entrée bouton
setup()
{
   pinMode(LED, OUTPUT);
   pinMode(BTN , INPUT_PULLUP);  // résistance interne pullup
 
  digitalWrite(LED, LOW);
}
 
loop()
{
     if (digitalRead(BTN) == HIGH)
     {
          // suite du code
     }
    else
    {
         // suite du code
    }
}