Ich bin mir nicht sicher, ob dies Ihr Problem ist, aber setPinValuesfür mich sieht es komisch aus:

void setPinValues(String message) {
  for(int i = 1; i <= sizeof(pins) / sizeof(pins[0]); i++ ) {
    String pinNumber = String(i);

    if( message == pinNumber + "H\n" ) {
      pinValues[i] = HIGH;
    }
    if( message == pinNumber + "L\n" ) {
      pinValues[i] = LOW;
    }
  }
}

Außerhalb der Grenzen gehen

Jedes Mal, wenn Sie anrufen setPinValues, ireicht von 1bis length(pins), wenn also pinsals deklariert wurde pins[5], iwerden 1, 2, und . Angenommen , das ist ein Problem für sich, weil es gar nicht existieren würde!345length(pins) == length(pinValues)pinValues[5]

pinValues[5]erstellt nur pinValues[0]zu pinValues[4](ja, ich weiß, es ist seltsam, Sie deklarieren 5 Elemente, aber da C 0-indiziert ist, sind sie 0, 1, 2, 3, und 4), also isollten Sie NIEMALS 5 erreichen, sonst sind Sie außerhalb der Grenzen, was ist undefiniertes Verhalten (und könnte durchaus die Ursache Ihrer Probleme sein), weil Sie an die falsche Speicherstelle schreiben.

Der richtige Weg, dies zu tun, ist eine Schleife von 0bis i < sizeof(pins) / sizeof(pins[0])(beachten Sie den strengen Kleiner-als- <Operator), die von 0bis length(pins) - 1und dann nur reicht String pinNumber = String(i + 1);. Auf diese Weise 1Hwird der Wert von geändert pinValues[0], indem Sie 1-indizierte serielle Nachrichten auf 0-indizierte C-Arrays abbilden.

Schleifen ist unnötig

Ich bin mir nicht sicher, warum Sie überschleifen pins. StringSie machen unnötige Arbeit und belasten wahrscheinlich das uC (zumal Sie, wie Dave Tweed in der anderen Antwort betont, teure Funktionen wie und Zeichenfolgenvergleich mit verwenden ==).

messagehat z. B. eine einzelne Instanz, 1H\nsodass keine Schleife benötigt wird. char pinNumber = message[0] - '0';speichert den ganzzahligen Wert des ersten Zeichens in pinNumber, mit dem Sie dann das Array indizieren können.

Wie soll das gehen?

• message[0]wäre ein charTyp (zB '1')
• '0'ist auch ein char, was dem ASCII-Wert 0 (48 dezimal) entspricht.
• Wenn message[0]das Zeichen enthalten ist '0', dann ist das Ergebnis'0' - '0' == 48 - 48 == 0
• Da Zahlen im ASCII-Satz zusammenhängend und aufsteigend sind , ist das Ergebnis der Subtraktion die tatsächliche Zahl als einzelne Byte-Ganzzahl.
• '1' - '0' == 49 - 48 == 1und weiter und weiter.

Möglicherweise möchten Sie auch falsche Eingaben ablehnen:

if (pinNumber >= sizeof(pins) / sizeof(pins[0]))
  return;

Mehr Scheinarbeit

Bei jeder loop()Iteration tun Sie dies:

// Write to pins        
for (int i = 1; i <= sizeof(pins) / sizeof(pins[0]); i++) {
  digitalWrite(pins[i], pinValues[i]);
}

Die hat das gleiche Problem mit den Ausgrenzen wie zuvor.

Es macht auch unnötige Arbeit, da Sie ALLE Pin-Werte schreiben, egal ob sie sich geändert haben oder nicht! Warum steigst du nicht digitalWrite(pins[pinNumber], HIGH or LOW);direkt ein setPinValuesund vermeidest diese Schleife? Sie können das Array sogar entfernen, pinValueswenn Sie es nicht wirklich speichern müssen (oder Sie können es behalten, wenn Sie es aus irgendeinem Grund benötigen).

Meine Alternative setPinValues

void setPinValue(char pin, char value) {
  char pinNumber = pin - '0';  // Get the integer value of the ASCII char

  if (pinNumber >= sizeof(pins) / sizeof(pins[0]))  // Reject out-of-bounds input
    return;

  switch (value) {
    case 'H':
      pinValues[pinNumber] = HIGH;  // Not necessary if you don't want to store the pin values
      digitalWrite(pins[pinNumber], HIGH);
      break;
    case 'L':
      pinValues[pinNumber] = LOW;  // Not necessary if you don't want to store the pin values
      digitalWrite(pins[pinNumber], LOW);
  }
}

Und dann nennen Sie es einfach loop()alssetPinValue(message[0], message[1]);

Lassen Sie mich wissen, ob dies Ihr Problem löst.

Es ist ein Leistungsproblem. Die erste Version von setPinValues()führt bei String()jedem Aufruf viele Aufrufe durch, und dies ist eine nicht triviale Funktion. Dies führt dazu setPinValues(), dass die Fertigstellung viel länger dauert als Sie denken, daher ist es nicht allzu überraschend, dass der Arduino schließlich nicht mit den eingehenden Nachrichten mit 57600 bps mithalten kann.

In der zweiten Version von sind setPinValues()die Argumente für die String()Aufrufe alle Konstanten, sodass sie zur Kompilierzeit ausgewertet werden. Diese Version läuft viel schneller und kann mit 57600 mithalten.