Welche Möglichkeiten der Fehlersuche hat man beim Arduino?

[fredyxx]

Danke für die ausführliche Info. Was ich UP nenne, nennst du offensichtlich Funktion. Soll mir recht sein.

Zum Thema goto hier mal ein Beispiel. mir ist klar, dass man das auch anders machen kann, aber ist es hier nicht sinnvoll?

  if (dir == 0 && (micros() - M5_microsalt)  > Schrittdauer)  {                            //  Drehrichtung


    switch (M5_i) {             //   es soll nur ein Schritt ausgeführt werden.

      case 0:
        M5_i = 1;
        goto M5_0;
        break;
      case 1:
        M5_i = 2;
        goto M5_1;
        break;
      case 2:
        M5_i = 3;
        goto M5_2;
        break;
      case 3:
        M5_i = 4;
        goto M5_3;
        break;
      case 4:
        M5_i = 5;
        goto M5_4;
        break;
      case 5:
        M5_i = 6;
        goto M5_5;
        break;
      case 6:
        M5_i = 7;
        goto M5_6;
        break;
      case 7:
        M5_i = 0;
        goto M5_7;
        break;

    }

  }   //  ************ ENDE if (dir == 0 && (micros() - M5_microsalt)  > Schrittdauer)


  else  if (dir == 1 && (micros() - M5_microsalt)  > Schrittdauer)  {      //  Drehrichtung

    switch (M5_i) {             //   es soll nur ein Schritt ausgeführt werden.

      case 0:
        M5_i = 1;
        goto M5_7;
        break;
      case 1:
        M5_i = 2;
        goto M5_6;
        break;
      case 2:
        M5_i = 3;
        goto M5_5;
        break;
      case 3:
        M5_i = 4;
        goto M5_4;
        break;
      case 4:
        M5_i = 5;
        goto M5_3;
        break;
      case 5:
        M5_i = 6;
        goto M5_2;
        break;
      case 6:
        M5_i = 7;
        goto M5_1;
        break;
      case 7:
        M5_i = 0;
        goto M5_0;
        break;

    }





M5_0:

    digitalWrite(p1, HIGH);
    digitalWrite(p2, LOW);
    digitalWrite(p3, LOW);
    digitalWrite(p4, LOW);
    goto M5_Schritt;

M5_1:

    digitalWrite(p1, HIGH);
    digitalWrite(p2, HIGH);
    digitalWrite(p3, LOW);
    digitalWrite(p4, LOW);
    goto M5_Schritt;

M5_2:

    digitalWrite(p1, LOW);
    digitalWrite(p2, HIGH);
    digitalWrite(p3, LOW);
    digitalWrite(p4, LOW);
    goto M5_Schritt;

M5_3:

    digitalWrite(p1, LOW);
    digitalWrite(p2, HIGH);
    digitalWrite(p3, HIGH);
    digitalWrite(p4, LOW);
    goto M5_Schritt;

M5_4:

    digitalWrite(p1, LOW);
    digitalWrite(p2, LOW);
    digitalWrite(p3, HIGH);
    digitalWrite(p4, LOW);
    goto M5_Schritt;

M5_5:

    digitalWrite(p1, LOW);
    digitalWrite(p2, LOW);
    digitalWrite(p3, HIGH);
    digitalWrite(p4, HIGH);
    goto M5_Schritt;

M5_6:

    digitalWrite(p1, LOW);
    digitalWrite(p2, LOW);
    digitalWrite(p3, LOW);
    digitalWrite(p4, HIGH);
    goto M5_Schritt;

M5_7:

    digitalWrite(p1, HIGH);
    digitalWrite(p2, LOW);
    digitalWrite(p3, LOW);
    digitalWrite(p4, HIGH);
    goto M5_Schritt;

[Mxt]

Das ganze wiederholt sich viel zu oft unnötigerweise.

Statt viele Male

    digitalWrite(p1, wert);
    digitalWrite(p2, wert);
    digitalWrite(p3, wert);
    digitalWrite(p4, wert);

würde einmal

void SetBits(int v1, int v2, int v3, int v4)
{
    digitalWrite(p1, v1);
    digitalWrite(p2, v2);
    digitalWrite(p3, v3);
    digitalWrite(p4, v4);
}

reichen was man dann z.B. als SetBits(LOW, HIGH, HIGH, LOW); aufruft. Dann hat das Programm schon viele Zeilen verloren.

So müsste man mehrmals da drüber gehen. Am Ende hätte das Programm vielleicht noch 10 % der jetzigen Länge. Dann hätte man schon viel für die Fehlersuche gewonnen.

- - - Aktualisiert - - -

Als Ergänzung, selbst wenn im obigen Falle p1 bis p4 Variablen sind, könnte man sowas machen

struct MotorPin
{
  int p1;
  int p2;
  int p3;
  int p4;
};

const MotorPin pins[] = { {1, 2, 3, 4}, {5, 6, 7, 8} , /* mehr Pins */}; // nur Beispielwerte

void SetBits(int motor, int v1, int v2, int v3, int v4)  // Motornummer beginnt hier bei 0
{
    digitalWrite(pins[motor].p1, v1);
    digitalWrite(pins[motor].p2, v2);
    digitalWrite(pins[motor].p3, v3);
    digitalWrite(pins[motor].p4, v4);
}