ZitierenLeute......hier geht es um ein einfaches Blinkprogramm für den Arduino.
Hat den keiner der Arduino-Freunde oder anderen Programmieren hier eine Idee?
Eigenwilliger Blinker
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)
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...Code: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; __} }
Viele Grüße, Green
Geändert von Greensiver (18.11.2013 um 16:04 Uhr) Grund: quote zu code geändert :)
Weiß den keiner Rat?
Leute......hier geht es um ein einfaches Blinkprogramm für den Arduino.
Hat den keiner der Arduino-Freunde oder anderen Programmieren hier eine Idee?
mill is_rev_on=millis();
Ohne genaue Beschreibung deines Programms, weisst du in Kürze selbst nicht mehr was du gemacht hast.
JO, da war ein Leerzeichen da, wo es nicht hin sollte.mill is_rev_on=millis();
Ich hab mich jetzt mal bemüht eine gute Beschreibung hinzubekommen...Ich hoffe die is gut so, wenn nicht bitte sagen.Ohne genaue Beschreibung deines Programms
Auserdem waren noch 2 Variablen zu viel drin.
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.
Geändert von RoboHolIC (17.11.2013 um 23:25 Uhr)
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.
OK, ist für µC-Anwendungen in gewisser Weise auch sinnvoll.Zitat von Hubert.G
Wie siehst du das mit dem ungebremsten Durchrauschen durch alle drei IF-Blöcke? Das erscheint mit nach-wie-vor plausibel. Hab ich mich da auch geirrt? Dann würde ich mich eben mal leise pfeifend davonstehlen ...
Das Durchrauschen ist ansich auch OK, nur wenn einmal ein Takt verpasst wird oder sich innerhalb einer Abfrage was ändert, ist Schluss mit Lustig.
Ähm, tja, jetzt seh' ich es auch. Ich fang dann schon mal mit dem Pfeifen an ...
Danke für die Antworten!
... Ja, der komplette Blinkteil des Programmes, mit dem Inkrementalgeber bin ich ja fertig geworden.Ist dein gezeigter Code vollständig?
... in dem Falle bleib ich beim If.Die Phasen 2 und 3 sollten dann WHILE-Konstrukte sein mit der jeweils negierten Bedingung
... DANKE! Werde Ich mich in Zukunft ganz bestimmt drann halten.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.
... soweit, dass der µC mal was übersringen könnte habe Ich noch gar nicht gedacht, DANKE! Und JA, tatsächlich scheint dort der Wurm drinn gewesen zu sein.Arduino Code ist etwas problematisch. Es läuft einiges im Hintergrund ab, ähnlich wie bei BASCOM. ... 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.
Ich habe dem µC jetzt mehr Zeit zum erkennen gegeben. So ... genug gelabert! Hier mein derzeitiger Code.
Das ganze ist jetzt doch deutlich länger als dieses Beispiel geworden. Hat noch jemand Ideeen zum verkürtzen des Codes? Wenn sonst keiner mehr Anregungen oder Fragen hat, makier Ich den Thread als Erledigt.Code:int LED_1=5; int tmp=0; unsigned long timer=0; long time_on=100; //Anzeit der LED in Millisekunden long time_off=5000; //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=millis(); //erste Zeit nehmen ____tmp=1;_//dauerhaftes Zeitnehmen verhindern __} __if((millis() >= (timer + time_on)) && (millis() < (timer + time_on + time_off))) //wenn die vorher genommene Zeit PLUS der Anzeit mit der derzeitigen Zeit übereinstimmt UND kleiner als die Endzeit ist... __{ ____digitalWrite(LED_1, LOW); //...schalte LED aus __} __if(millis() >= (timer + time_on + time_off)) //Wenn Anzeit PLUS Auszeit abgelaufen ist, beginne von forn (jetzt mehr Sicherheit durch ">=") __{ ____tmp=0; __} }
Viele Grüße, Green
Berechtigungen
Lesezeichen