Arduino-Code: fehlende Definition, benötige Hilfe

[Rabenauge]

Ja..Franzis nimmt leider so ziemlich alles...

Und es stimmt: der Code ist ziemlicher Pfusch. Ich weiss zwar nicht, was du alles raus gelöscht hast, aber wenn es wirklich _nur_ Kommentare waren, kommt der nicht mal in die Nähe dessen, was er tun sollte *grusel

[Andi#87]

Hier mal der ursprüngliche Code:

// +++ QUELLCODE FÜR DEN DRUCKTASTER MIT PERMA-LED-LAUFLICHT +++ //

// Definition von Variablen für den Taster + LEDs
const int LED_PINS[] = {2, 3, 4, 5};
const int PUSH_BUTTON = 6;
int buttonStateOld = 0;
bool buttonFlag = false;

void setup() {
  // Initialisiere digitale Pins als Ausgang in einer For-Schleife
  for (int a = 0; a < 4; a++) {
    pinMode(LED_PINS[a], OUTPUT);
  }

  // Initialisiere den Drucktaster als Eingang
  pinMode(PUSH_BUTTON, INPUT);
}

void loop() {
  if (buttonState != buttonStateOld) {
    // Ändere das Flag nur, wenn der Taster gedrück wird
    if (buttonState == HIGH) {
      if (buttonFlag == false) {
        buttonFlag = true;
      } else {
        buttonFlag = false;
      }
    }

    // Speichere den aktuellen Wert des Tasters
    buttonStateOld = buttonState;
  }

  if (buttonFlag == true) {
    for (int a = 0; a < 4; a++) {
      // Schalte eine LED an
      digitalWrite(LED_PINS[a], HIGH);
      // Sende eine Nachricht an den seriellen Monitor
      Serial.print("LED Status: LED ");
      Serial.print(LED_PINS[a]);
      Serial.println(" is ON");
      // Warte 50 Millisekunden
      delay(50);
    }
    
    for (int a = 0; a < 4; a++) {
      // Schalte eine LED aus
      digitalWrite(LED_PINS[a], LOW);
      // Sende eine Nachricht an den seriellen Monitor
      Serial.print("LED Status: LED ");
      Serial.print(LED_PINS[a]);
      Serial.println(" is OFF");
      // Warte 50 Millisekunden
      delay(50);
    }
  }
}

In Thread #1 hatte ich die Kommentare gelöscht. Ebenfalls den seriellen Monitor. Und die Delays von 50 auf 200 geändert.

Ob der Code "...nicht mal in die Nähe dessen (kommt), was er tun sollte...", kann ich nicht beurteilen. Möglicherweise verfolgt der Autor ein späteres Ziel und man könnte speziell diesen Code zwar anders schreiben, aber evt. damit das spätere Ziel nicht schon im Blick haben. Wie gesagt, das kann ich mit meinem Kenntnisstand nicht einschätzen, hoffe aber darauf, dass (trotz der enthaltenen Fehler im aktuellen Code) der Autor den Leser an einem "roten Faden" konsequent durchs Buch führt.


Andi

[Rabenauge]

Bau da mal die von mir vorgeschlagene Zeile ein.
Ich bin mir relativ sicher: die LED's werden nur funktionieren, wenn man den Button gedrückt _hält_.

Zum Entprellen wird hier wohl das delay() mit genutzt, das ist zwar ziemlich hemdsärmelig, müsste aber halbwegs funktionieren.

Für buttonStateOld brauchts auch keineswegs eine int- da nur zwei Möglichkeiten bestehen, ist eine bool völlig ausreichend.
Geht in dem Testprogramm natürlich auch, aber irgendwann kommst du an den Punkt, wo du mal den Speicher voll kriegst, daher: gewöhn dir besser an, für Variablen nicht mehr Speicher zu reservieren (genau das passiert am Anfang nämlich) als nötig.

Was den Aufbau des Buches angeht: ich weiss ja nicht, wie du es durcharbeitest, aber möglicherweise sind da bewusst Fehle im Code, die auf den nächsten Seiten dann untersucht werden?

[Andi#87]

Was den Aufbau des Buches angeht: ich weiss ja nicht, wie du es durcharbeitest, aber möglicherweise sind da bewusst Fehle im Code, die auf den nächsten Seiten dann untersucht werden?

Das war auch eine Option, die ich erhoffte, aber da habe ich keine Hinweise oder Lösungen/Korrekturen gefunden. Ansonsten wäre das ja durchaus eine gute Möglichkeit, etwas zu lernen. Die Codes werden für die Käufer des Buches auch im Download angeboten. Auch dort ist keine Korrektur zu finden.

Danke für Deine Tips, werde ich in den kommenden Tagen ausprobieren. Heute komme ich nicht mehr dazu.

Andi