Multitasking

[sketchbook_andrea] / multitasking / BlinkWithoutDelay_3_funzione / BlinkWithoutDelay_3_funzione.ino

diff --git a/multitasking/BlinkWithoutDelay_3_funzione/BlinkWithoutDelay_3_funzione.ino b/multitasking/BlinkWithoutDelay_3_funzione/BlinkWithoutDelay_3_funzione.ino
index e27cf55d8c4e1de6ff5e67fa5022d7826c6b6198..db0ed89e2c2ba043b3e395a795bc836cc52e742a 100644 (file)
--- a/multitasking/BlinkWithoutDelay_3_funzione/BlinkWithoutDelay_3_funzione.ino
+++ b/multitasking/BlinkWithoutDelay_3_funzione/BlinkWithoutDelay_3_funzione.ino
@@ -1,4 +1,4 @@
-/* Blink without Delay
+/* Blink without Delay: Refactoring
  
  Blink con funzione
  
@@ -7,27 +7,29 @@
     - Quali variabili determinano il comportamento del LED?
     - Come cambiano durante il corso dello script?
     - Sono globali o locali? 
- 
+
+Variabili: http://www.maffucci.it/2011/12/15/appunti-di-programmazione-su-arduino-variabili/ 
+
  */
 
 /////////////
 // First LED
-const int ledA =  13;      // the number of the LED pin
+const int ledA =  13;       // the number of the LED pin
 // Variables will change:
-int ledStateA = LOW;             // ledState used to set the LED
-long previousMillisA = 0;        // will store last time LED was updated
+int ledStateA = LOW;        // ledState used to set the LED
+long previousMillisA = 0;   // will store last time LED was updated
 // the follow variables is a long because the time, measured in miliseconds,
 // will quickly become a bigger number than can be stored in an int.
-long intervalA = 1000;           // interval at which to blink (milliseconds)
+long intervalA = 1000;      // interval at which to blink (milliseconds)
 void lightLedA () ;
 
 //////////////
 //  Second LED
 // Now with less global variables thanks to static (see function body)
 const int ledB = 12; //Secondo LED
-           // ledState used to set the LED
-long previousMillisB = 0;        // will store last time LED was updated
-           // interval at which to blink (milliseconds)
+                     // ledState used to set the LED
+long previousMillisB = 0;    // will store last time LED was updated
+                             // interval at which to blink (milliseconds)
 void lightLedB () ;
 
 
@@ -47,15 +49,12 @@ void loop()
 // Funzioni:
 
 void lightLedA () {
-  if (millis() > previousMillisA + intervalA) {
+  if (millis() - previousMillisA >=  intervalA) {
     // save the last time you blinked the LED 
-    previousMillisA = millis();   
+    previousMillisA += intervalA;
 
     // if the LED is off turn it on and vice-versa:
-    if (ledStateA == LOW)
-      ledStateA = HIGH;
-    else
-      ledStateA = LOW;
+      ledStateA = !ledStateA ;
     // set the LED with the ledState of the variable:
     digitalWrite(ledA, ledStateA);
   }
@@ -65,15 +64,12 @@ void lightLedA () {
 void lightLedB () {
   long intervalB = 500;
    static int ledStateB ;  // https://www.arduino.cc/en/Reference/Static
-  if (millis() > previousMillisB + intervalB) {
+  if (millis() - previousMillisB >= intervalB) {
     // save the last time you blinked the LED 
-    previousMillisB = millis();   
+    previousMillisB += intervalB ;
 
     // if the LED is off turn it on and vice-versa:
-    if (ledStateB == LOW)
-      ledStateB = HIGH;
-    else
-      ledStateB = LOW;
+      ledStateB = !ledStateB;
     // set the LED with the ledState of the variable:
     digitalWrite(ledB, ledStateB);
   }