Asymmetrischer Blinkzyklus ohne delay()

[widarr]

Hallo!
Erster Beitrag mit komischem Threadtitel, aber ich komme bei diesem Problemchen einfach nicht weiter.

Ich würde gerne eine LED für eine gewisse Zeit an und eine gewisse Zeit ausschalten, aber ohne delay zu verwenden, der untenstehedne Code macht noch nicht ganz was ich will.

Der Code sieht so aus:

Code:

#define ULONG unsigned long

int LED = 13;
boolean LEDON = true;
boolean NEXT_LED_ACTION = false;
ULONG previousMillis = 0;
ULONG ONTIME = 4000;
ULONG OFFTIME = 1000;

void ledon() {
    digitalWrite(LED, HIGH);
    LEDON = true;
    NEXT_LED_ACTION = false; // next turn LEDs off
}
void ledoff() {
    digitalWrite(LED, LOW);
    LEDON = false;
    NEXT_LED_ACTION = true; // next turn LEDs on
}

void LEDcycle() {
    ULONG currentMillis = millis();
    
    if(NEXT_LED_ACTION) {
        if((currentMillis - previousMillis) > OFFTIME) { // is the off time already over?
            ledon();
        }
    } else {
        if((currentMillis - previousMillis) > ONTIME) { // is the on time already over?
            ledoff();
        }
    }
    previousMillis = currentMillis;
}


void setup()
{
    pinMode(LED, OUTPUT);
    digitalWrite(LED, HIGH);
}

void loop()
{
    LEDcycle();
}

Danke schonmal für die Hilfe

[widarr]

Ok hat sich erledigt, bin selber dahintergestiegen, wens interessiert:

Die aktualisierung der previousMillis war am falschen platz, so gehört es:

Code:

#define ULONG unsigned long

int LED = 13;
boolean LEDON = true;
boolean NEXT_LED_ACTION = false;
ULONG previousMillis = 0;
ULONG ONTIME = 4000;
ULONG OFFTIME = 1000;

void ledon() {
    digitalWrite(LED, HIGH);
    LEDON = true;
    NEXT_LED_ACTION = false; // next turn LEDs off
}
void ledoff() {
    digitalWrite(LED, LOW);
    LEDON = false;
    NEXT_LED_ACTION = true; // next turn LEDs on
}

void LEDcycle() {
    ULONG currentMillis = millis();
    
    if(NEXT_LED_ACTION) {
        if((currentMillis - previousMillis) > OFFTIME) { // is the off time already over?
            previousMillis = currentMillis;
            ledon();
        }
    } else {
        if((currentMillis - previousMillis) > ONTIME) { // is the on time already over?
            previousMillis = currentMillis;
            ledoff();
        }
    }
}


void setup()
{
    pinMode(LED, OUTPUT);
    digitalWrite(LED, HIGH);
}

void loop()
{
    LEDcycle();
}