Eigenwilliger Blinker

[Greensiver]

Hi Leute,

ich habe mir neulich einen Inkrementalgeber zugelegt. Für den Anfang wollte ich ein kleines Programm schreiben, womit ich mit dem Inkrementalgeber die Blinkfrequenz variieren kann. Nunja den Inkrementalgeber kann ich auslesen, aber beim Blinken wird es Interessant! Die LED blinkt zwar, hört aber nach kurzer Zeit damit auf. Ich kann mir nicht ganz erklären warum. Aber vil. ist es ja nur ein ganz einfacher Logikfehler von mir.

Naja hier der Code: (geht bestimmt noch eleganter zu lösen)

int LED_1=5;
int tmp=0;
unsigned long timer_1=0;
unsigned long timer_2=0;
unsigned long time_on=50; //Anzeit der LED in Millisekunden
unsigned long time_off=50; //Auszeit der LED in Millisekunden 


void setup()
{
__pinMode(LED_1, OUTPUT);
}

void loop()
{


__if(tmp==0) //Wenn Blinkvorgang beendet, von forn bginnen
__{
____digitalWrite(LED_1, HIGH); //LED anschalten
____timer_1=millis(); //erste Zeit nehmen
____tmp=1;//dauerhaftes Zeitnehmen verhindern
__}
__if(millis() == (timer_1+time_on)) //wenn die vorher genommene Zeit PLUS der Anzeit mit der derzeitigen Zeit übereinstimmt...
__{
____digitalWrite(LED_1, LOW); //...schalte LED aus... 
____timer_2=millis(); //...und nehme Zeit 2
__}
__if(millis() == (timer_2+time_off)) //Wenn Auszeit abgelaufen ist, beginne von forn
__{
____tmp=0;
__}


}

Habt Ihr einen Lösungsansatz für mich? millis() braucht ja glaube so an die 70Tage bis es auf Null zurückfällt, daran kann es also nicht liegen...


Viele Grüße, Green

[Greensiver]

Leute......hier geht es um ein einfaches Blinkprogramm für den Arduino.

Hat den keiner der Arduino-Freunde oder anderen Programmieren hier eine Idee?

[Hubert.G]

mill is_rev_on=millis();

Ohne genaue Beschreibung deines Programms, weisst du in Kürze selbst nicht mehr was du gemacht hast.

[Greensiver]

mill is_rev_on=millis();

JO, da war ein Leerzeichen da, wo es nicht hin sollte.

Ohne genaue Beschreibung deines Programms

Ich hab mich jetzt mal bemüht eine gute Beschreibung hinzubekommen...Ich hoffe die is gut so, wenn nicht bitte sagen.

Auserdem waren noch 2 Variablen zu viel drin.

[RoboHolIC]

Hi Geensiver.

Ist dein gezeigter Code vollständig? Ich bin nicht so firm mit C, aber ich kann nicht erkennen, wo die beiden Funktionen setup() und loop() aufgerufen werden, weil ich kein main() finde.
Ich vermute, du willst eine Weiterschaltung von Phase zu Phase realisieren:
1: Zeitnehmen und einschalten
2: Warten auf Ausschaltbedingung
3: Warten auf Auszeit-Ende

Die Phasen 2 und 3 sollten dann WHILE-Konstrukte sein mit der jeweils negierten Bedingung ("solange Abbruch-Kriterium NICHT erreicht").
Dann fehlt m.E. noch der direkte Sprung nach oben zu Phase 1, oder eben: main() {loop();}

Dass es bisher überhaupt zu einem Blinken kommt, erscheint mir eher zufällig (wenngleich offenbar reproduzierbar).

Was ich noch sagen wollte:
Ich fände es besser, die veröffentlichte ursprüngliche Programmversion nicht nachträglich zu verändern, weil dann die innere Stimmigkeit des Threads verloren geht.
Lieber durch jeweils aktuelle Versionen die sukkzessive Verbesserung des Programms erkennbar machen.

[Hubert.G]

Arduino Code ist etwas problematisch. Es läuft einiges im Hintergrund ab, ähnlich wie bei BASCOM.
Vom Syntax her ist das geschriebene schon richtig, auch das loop()
Problematisch betrachte ich allerdings dieses unsigned long.
Wenn das hier: if(millis() == (timer_2+time_off)) nicht genau erreicht wird, ist es vorbei und es gibt keinen Sicherheitsausstieg. Das unsigned long ist eine 8Byte Zahl. Wenn es da während der Addition und Vergleich eine Verschiebung gibt, ist es vorbei.
Ich bin mir sicher das hier irgendwo der Wurm steckt, auch wenn ich ihn nicht erkenne.