Douchetimer (1)

algemene C code
Gebruikers-avatar
Berichten: 2655
Geregistreerd: 06 Aug 2016, 01:03

Re: Douchetimer (1)

Berichtdoor Koepel » 24 Nov 2017, 02:37

Mooi :D
Je hebt nu een aparte state om de timer te starten. Dat is een mooie oplossing.

Het tonen van de leds en het knipperen zou ik anders doen. Maar ik heb nog steeds geen heldere ingeving daarvoor.
Je gebruikt bijvoorbeeld 'pauzeStatus' en 'pulsStatus' om de millis() waarde te onthouden, maar ook als vlag door ze '0' en '1' te maken. Dat zou ik nooit doen, die twee dingen horen niet in dezelfde variabele aangezien millis() iedere waarde kan hebben.

Er is een iets in dat niet consequent is, maar dat is echt muggenzifterij. Dus alvast sorry.
Wanneer je "const byte MINUTENLED[]" doet, dan definieer je voor dat array geen vaste grootte.
Wanneer je "for (byte i = 0; i < 5; i++)" doet, dan gebruik je '5' als vast getal.
Je zou een #define kunnen maken met het getal 5, en dat bij beide gebruiken. Of je gebruikt voor de for-loop wat de compiler weet met sizeof(). De sizeof() is geen functie, maar daar vult de compiler in wat de grootte van iets is. Dat kan een variabele, een array, of een hele 'struct' zijn, en zelfs de grootte van een pointer.

Advertisement

Vorige

Terug naar C code

Wie is er online?

Gebruikers in dit forum: Geen geregistreerde gebruikers en 17 gasten