Arduino Code klemmt / Anfänger

[Christian B. (summ)]

Hallo zusammen,

ich bin eigentlich Modellbauer und versuche nun neben dem Beruf, der 
Familie und dem Hobby Modellbau, einen Arduino zu programmieren.
Es fällt mir extrem schwer, aber ich bin immerhin soweit, einen Arduino 
mit einem TLC zu koppeln und den einiges machen zu lassen.
Aber in meinem momentanen Sketch klemmt es. Per Eingabe teste ich 
gerade, wie ich verschiedene PRogrammabläufe auslösen kann. Im diesem 
Fall sollte eigentlich nach Eingabe "c" das ledPins-Array blinken. Macht 
es aber nicht, sondern es leuchten alle Leds in dem Array stupide vor 
sich hin. Wo ist der Fehler und kann mir jemand einen Tipp geben?

//import TLC libraries
#include "Tlc5940.h"
#include "tlc_fades.h"
int ledPins[] = {0,1,2};
int pinCount = 4;
int max = 100;

void setup()
{
  // start communication with TLC chip
  Tlc.init();
  Serial.begin(9600);
}

TLC_CHANNEL_TYPE channel;


void loop(){

  Tlc.clear();

 if (Serial.available() > 0) {
     int input1 = Serial.read();
     switch (input1) {
     case 'a':
       Serial.println("a");
       Tlc.set(3,4000);
       Tlc.update();

       break;
     case 'b':
       Serial.println("b");
       Tlc.set(3,500);
       Tlc.update();
       break;
     case 'c':
      ledBlink();
      Serial.println("Blinker");

      break;
     }
  }
//delay(1);
 }

void ledBlink(void){
  Tlc.clear();

      for (int thisPin = 0; thisPin < pinCount; thisPin++) {
      Tlc.set (ledPins[thisPin], max);
      uint16_t duration = 150;
      uint32_t startMillis = millis()+ 2000;
      uint32_t endMillis = startMillis + duration;

      tlc_addFade(ledPins[thisPin], 4095, max, startMillis, endMillis);
      tlc_updateFades();
      Tlc.update();
}
}

[troll (Gast)]

Christian B. schrieb:
> int ledPins[] = {0,1,2};
> int pinCount = 4;
>      for (int thisPin = 0; thisPin < pinCount; thisPin++) {
>       Tlc.set (ledPins[thisPin], max);
ledPins[3] existiert nicht! out-of-bound-access

[Christian B. (summ)]

Ich hab das mal so korrigiert:

ledPins[]={2,3,4}
pincount = 3

aber dennoch passiert nach Eingabe von c nichts, ausser das die 3 Pins 
leuchten. Die Fades werden nicht beachtet.

[tono (Gast)]

Du definierst zwar die Fades nutzt sie aber nicht.

Versuche mal folgendes:

     in der Funktion ledBlink die Befehle
        tlc_updateFades();
        Tlc.update();
     entfernen

     in der Loop die Befehle um das Delay setzen
        tlc_updateFades();
        delay(25);
        Tlc.update();

Es scheint so als müsssen die gesetzten Fades per tlc_updateFades 
innerhalb der Loop aufgerufen werden.

[PittyJ (Gast)]

Dann ist wohl das "klassische" Debuggen angesagt. Hinter jder Zeile ein 
print(), um zu sehen, wo der Computer überhaupt ist. Und die wichtigsten 
Variablen mit in das print().

[tono (Gast)]

Bitte auch "Tlc.clear();" in der Loop entfernen.
Das schaltet alle LED aus.
Wird aber bisher nicht wirksam, weil das Tlc.update() fehlte ...

[Christian B. (summ)]

dann passiert: es blitzt jede led und dann ledPins einmal kurz,

[troll (Gast)]

Christian B. schrieb:
> ledPins[]={2,3,4}
> pincount = 3
Mach dir das Leben einfach...
pincount=sizeof(ledPins)/sizeof(int);

[Christian B. (summ)]

danke für den Tipp.

aber es ändert leider nicht die Tatsache, dass void ledBLink(void) nur 
einmal ausgeführt wird. Ich müsste wohl irgendwo sagen "mach ledBlink 
solange, bis Serial.read was anderes macht. Aber ich habe keine Ahnung, 
wie das genau geht

[tono (Gast)]

deine Antwort scheint plausibel.
Die Zeit zwischen max und min ist 150 ms.
Das wird man nur als Aufblitzen wahrnehmen.
Mit einem Wert von 5000 kann man das Verhalten sicher besser 
beobachten...

Wie läßt sich dein Wunsch umsetzen ...
relativ simpel ...  die Funktion muß in die Loop.

Zwei Variablen
      FadeAktiv    0/1     nicht aktiv  /  aktiv
      FadeArt      0/1     max zu min   /  min zu max

Innerhalb des "case 'c':" wird die Variable FadeAktiv invertiert.
Jede Eingabe von 'c' schaltet damit zwischen an/aus um.

Ausserhalb, aber innerhalb der Loop und nach
"if (Serial.available() > 0) {   }", wird abgefragt of FadeAktiv <> 0 
ist.
Innerhalb dieser Abfrage wird noch die Richtung abgefragt (FadeArt) und 
die min max Werte entsprechend gesetzt.
Ob der FadeVorgang abgeschlossen ist zeigt der Rückgabewert von 
"tlc_updateFades();" ...

ist nur ein grober Überblick ... aber so kann man es lösen

[Christian B. (summ)]

Ich habe jetzt lange herumgedoktort und warum es jetzt klapp, kann ich 
garnicht genau sagen. Woran es jetzt noch hängt ist, dass das Blinken 
beim Wechsel zwischen Mode 1 und 4 manchmal zufällig wird. Doppelblitzer 
oder nicht alle ledPins machen mit.
Hier mal der aktuelle Code:

#include "Tlc5940.h"
#include "tlc_fades.h" 
#define interval 500


int fadingMode = 0; //Beginn mit TLC Reset
int ledPins[] = {2,3,4};
int pinCount = sizeof(ledPins)/sizeof(int);
int max = 100;
unsigned char x;
unsigned long startTime = 0;
unsigned long waitUntil=0;

//TLC_CHANNEL_TYPE channel;

void setup()   {                

   Tlc.init();
  Serial.begin(115200);
}



void loop()
{    
  if(Serial.available()){
    if(Serial.peek() == 'l'){
      Serial.flush();
    }
    else{
      fadingMode = Serial.parseInt(); 
      Serial.print("Mode set to "); 
      Serial.print(fadingMode); 
      Serial.print(": ");
      startTime = millis();
      switch(fadingMode){
      case 0:
        Serial.println("allet uff Null");
        break;
      case 1:
        Serial.println("Flash");
        break;
      case 2:
        Serial.println("Schalte Led an Pin 5 ein");
        break;
      case 3:
        Serial.println("Dimme Led an Pin 5");
        break;
      case 4:
        Serial.println("Slow Blinker");
        break;
      }
    } 
  }
  unsigned char brightness;
  switch(fadingMode){
  case 0:
    // TLC Resert, alles aus
    for (int thisPin = 0; thisPin < pinCount; thisPin++) {
    Tlc.clear();
    tlc_removeFades(ledPins[thisPin]);
    Tlc.update();
    }
    break;
  case 1:
    Flash();
    break;
  case 2:
    ledset1();
    break;
  case 3:
    ledset2();
    break;
  case 4:
    slowBlink();
    break;
     
    break;   
  default:
    Serial.println("Unknown Mode!");
    delay(1000);
    break;
  }
}

void Flash(void){ // Flasher Variante
      for (int thisPin = 0; thisPin < pinCount; thisPin++) {
     // if (!tlc_isFading(ledPins[thisPin])) {
     // tlc_removeFades(ledPins[thisPin]);
      uint16_t duration = 150;
      uint32_t startMillis = millis()+ 2000;
      uint32_t endMillis = startMillis + duration;
      //Tlc.set (ledPins[thisPin], max);
      tlc_addFade(ledPins[thisPin], 4095, max, startMillis, endMillis);
      tlc_updateFades();
   //   Tlc.update(); 
}
    }
void ledset1(void){ // schaltet einfach nur die Led ein
      Tlc.set(5,4000);
      Tlc.update(); 
}

void ledset2(void){  // verändert einfach Led
     Tlc.set(5,500);
     Tlc.update(); 
}

void slowBlink(void){ //slower blinking
Tlc.clear();
    for (int thisPin = 0; thisPin < pinCount; thisPin++) {
    if (!tlc_isFading(ledPins[thisPin])) {
    Tlc.set(ledPins[thisPin],0); //Versuch, die Leds komplett auschalten 
        uint16_t duration = 1000;
        uint32_t startMillis = millis()+ 2000;
        uint32_t endMillis = startMillis + duration;
      //tlc_removeFades(ledPins[thisPin]);
      //Tlc.set (ledPins[thisPin], max);
        tlc_addFade(ledPins[thisPin], 0, 4095, startMillis, endMillis);
        tlc_addFade(ledPins[thisPin], 4095, 0, startMillis, endMillis);
  }
        tlc_updateFades();
    //  Tlc.update(); 
  }
}

[Ardu (Gast)]

Christian B. schrieb:
> Hier mal der aktuelle Code:

Danke!