Arduino: Arduino startet mitten im Code neu

[Jan-Niklas B. (janniklas_b)]

Hallo zusammen,

ich habe folgendes Problem und weiß leider nicht mehr weiter:
Ich habe einen Code für ein Meldersystem geschreiben, welches serielle 
Daten im Format ID;Typ;Text~ bekommt also z.B. 1;A;Text!~
Wenn ich diesen Befehl an den Arduino schicke, startet dieser jedoch 
beim Befehl atoi() neu.
Hier der Code:

//Libraries
#include <RFM12B.h>

//-----------------------

//Variables
uint8_t   key[] =           "1234567890123456";
int      serialSpeed =   19200;
int      ownId =         1;
int      group =         86;
char     destination[3];
char     type[1];
char     text[128];
char     sendText[128];
int      ackTime=        25;
bool     requestAck=     true;
int      dest=           0;
//-------------------------------

//Instances
RFM12B radio;

//------------------------------


void setup()
{
  Serial.begin(serialSpeed);
  Serial.println("0x00");
  radio.Initialize(ownId, RF12_433MHZ, group);
  radio.Encrypt(key);
}

void incoming()
{
  if (Serial.available())
  {
    delay(10);
    Serial.readBytesUntil(';',destination,3);
    Serial.println((String)destination);
    Serial.readBytesUntil(';',type,1);
    Serial.println((String)type);
    Serial.readBytesUntil('~',text,128);
    Serial.println((String)text);
    dest = atoi(destination);              //Hier steigt der Arduino aus
    Serial.println(dest);
    String stext = (String)type[0]+";"+text;
    stext.toCharArray(sendText, 128);
    Serial.println((String)sendText);
    radio.Send((uint8_t)dest, sendText, (uint8_t)sizeof(sendText), requestAck);
    if(waitForAck(dest))
    {
       Serial.println("Nachricht erfolgreich an Melder: "+(String)dest+" gesendet!");
    }
  }
}

static boolean waitForAck(int dest)
{
  long now = millis();
  while(millis() < (now+ackTime))
  {
    if(radio.ACKReceived(dest))
      return true;
  }
  return false;
}

void loop()
{
  incoming();
}

Es wäre super, wenn jemand einen Tipp hätte warum das nicht 
funktoniert...
Die Serielle ausgabe des Arduino sieht wie folgt aus:

0x00
1
A
;Te0x00


Ich freue mich auf ein paar Tipps.

Nette Grüße
Jan-Niklas

[Mark B. (markbrandis)]

char     destination[3];
dest = atoi(destination);              //Hier steigt der Arduino aus

destination muss nullterminiert sein. Probier mal vor dem Aufruf von 
atoi() das hier:

destination[2] = '\0';

[Gs S. (gs-scarnight)]

Hi,

nur eine Vermutung evtl. stört die unter Umständen fehlende 
Nullterminierung von destination. Alternativ sieh dir die Funktion 
strtol mal an, hier gibt es bei Fehlern bessere Möglichkeiten der 
Diagnose.
Als erster Test könntest du

char destination[4];

wählen und in der loop() alle Einträge von destination auf '\0' setzen.

[DirkZ (Gast)]

Jan-Niklas B. schrieb:
> char     destination[3];
> char     type[1];
> char     text[128];
> char     sendText[128];

Alle Arrays möchten nullterminiert sein, deswegen um eins vergrößern:

char     destination[4];
char     type[2];
char     text[129];
char     sendText[129];

Die Arrays werden eigentlich alle mit \0 initialisiert. Wenn man sich 
nicht darauf verlassen will, schreibt man:

char     destination[4] = { 0 };
char     type[2] = { 0 };
char     text[129] = { 0 };
char     sendText[129] = { 0 };

[Kirkenes (Gast)]

DirkZ schrieb:
> Die Arrays werden eigentlich alle mit \0 initialisiert. Wenn man sich
> nicht darauf verlassen will, schreibt man:char     destination[4] = { 0
> };
> char     type[2] = { 0 };
> char     text[129] = { 0 };
> char     sendText[129] = { 0 };

Damit setzt du aber nur das erste Array-Element auf 0!?

[DirkZ (Gast)]

Kirkenes schrieb:
> DirkZ schrieb:
>> Die Arrays werden eigentlich alle mit \0 initialisiert. Wenn man sich
>> nicht darauf verlassen will, schreibt man:char     destination[4] = { 0
>> };
>> char     type[2] = { 0 };
>> char     text[129] = { 0 };
>> char     sendText[129] = { 0 };
>
> Damit setzt du aber nur das erste Array-Element auf 0!?

Ja, das stimmt. Und nach C89 (3.5.7) oder C99 (6.7.8/21) auch den Rest 
des Arrays.

[Jan-Niklas B. (janniklas_b)]

Danke erstmal für die ganzen Rückmeldungen :)
Leider hat das noch nicht viel gebracht...
Ich hab den Code nun wie folgt abgeändert:

//Libraries
#include <RFM12B.h>

//-----------------------

//Variables
char     serialdata;
uint8_t   key[] =           "1234567890123456";
int      serialSpeed =   19200;
int      ownId =         1;
int      group =         86;
char     destination[4]= {0};
char     type[2]=        {0};
char     text[128]=      {0};
byte     sendType;
char     sendText[128]=  {0};
int      ackTime=        25;
bool     requestAck=     true;
int      dest=           0;
//-------------------------------

//Instances
RFM12B radio;

//------------------------------


void setup()
{
  Serial.begin(serialSpeed);
  Serial.println("0x00");
  radio.Initialize(ownId, RF12_433MHZ, group);
  radio.Encrypt(key);
}

void incoming()
{
  if (Serial.available())
  {
    delay(10);
    Serial.readBytesUntil(';',destination,3);
    Serial.readBytesUntil(';',type,1);
    Serial.readBytesUntil('~',text,128);
    destination[4]='\0';
    type[2]='\0';
    text[128]='\0';
    dest = atoi(destination);
    Serial.println(dest);
    String stext = (String)type[0]+";"+text;
    stext.toCharArray(sendText, 128);
    sendText[128]='\0';
    Serial.println("sendText nullterminiert");
    Serial.println((String)sendText);
    radio.Send((uint8_t)dest, sendText, (uint8_t)sizeof(sendText), requestAck);
    if(waitForAck(dest))
    {
       Serial.println("Nachricht erfolgreich an Melder: "+(String)destination+" gesendet!");
    }
  }
}

static boolean waitForAck(int dest)
{
  long now = millis();
  while(millis() < (now+ackTime))
  {
    if(radio.ACKReceived(dest))
      return true;
  }
  return false;
}

void loop()
{
  incoming();
}

Nun sieht die Ausgabe so aus:
0x00
1
sendTå0x00


Sonst irgendwelche ideen woran das liegen könnte?

[DirkZ (Gast)]

Jan-Niklas B. schrieb:

>     text[128]='\0';

>     sendText[128]='\0';

char     text[128]=      {0};

legt die Variable text mit 128 Feldern an, die - in C nullindiziert - 
von 0 bis 127 angesprochen werden. Oder hast Du vergessen, die Variablen 
text und sentText auf 129 zu erweitern?

Denn die Zuweisung

text[128]='\0';

schreibt irgendwo im Speicher rum. Mit bekanntem Resultat.

[Jan-Niklas B. (janniklas_b)]

Gut aufgepasst, aber das ist gewollt ;)
Das Funkmodul kann nur einen Datensatz von 128 Byte pro Sendung 
übertragen, desswegen lasse ich es dabei, somal der Text von meiner 
ansteuerungssoftware eh auf 100 Zeichen begrenzt wird.

[DirkZ (Gast)]

Jan-Niklas B. schrieb:
> Gut aufgepasst, aber das ist gewollt ;)

Das das Programm abstürzt? Tut es doch!


Jan-Niklas B. schrieb:
> Das Funkmodul kann nur einen Datensatz von 128 Byte pro Sendung

Und? Die Variable könnte auch 1000 Byte groß sein. Solange Du beim 
Senden nicht mehr als 128 Byte angibts, ist alles gut.
Entweder definierst Du

char     text[128];

und schreibst dazu passend

text[127]='\0';

[Bernhard F. (bernhard_fr)]

Exakt was DirkZ gesagt hat.
Und gleiches für folgendes:

destination[4]='\0';
type[2]='\0'

Du vereinbarst mit

char     type[2]=        {0};

eine Array aus 2 Chars.

Auf das erste Element greifst du mit dem Index 0 zu auf das zweite mit 1

Der Ausdruck

type[2]='\0'

würde auf ein drittes Element zugreifen. Das gibt es aber nicht... Der 
Speicher an dieser Stelle gehört nunmal zu jemand anderem. Dem Compiler 
ist das aber egal ... der meint du weißt was du tust, und lässt dich das 
da hinschreiben ... Resultat? undefiniertes Verhalten ...

Ich unterstell dir jetzt einfach mal, dass das nen Flüchtigkeitsfehler 
war. Ansonsten kann ich dir nur noch mal Grundlagenliteratur zum Thema 
Arrays empfehlen.

[Jan-Niklas B. (janniklas_b)]

Okay,dass war wohl wirklich eigene Dummheit -.-
Das Arrays 0 indexiert sind ist mir eigentlich bekannt, aber irgendwie 
hab ich da nicht dran gedacht...ich baus mal ein und mal schauen ob es 
dann läuft...
Danke für die HInweise

[Jan-Niklas B. (janniklas_b)]

Hmm... immernoch das selbe verhalten wie vorhin beschrieben...

[Jan-Niklas B. (janniklas_b)]

Kann man irgendwie alle Speicher des Arduino einmal löschen?
Er hat mir grade auf den Befehl hin einen Buchstabensalat mit Fragmenten 
von alten Codes rausgegeben...

[Mark B. (markbrandis)]

Du schreibst irgendwo über die Grenzen eines Arrays hinaus, jede Wette. 
Bzw. Du liest etwas was nicht (richtig) nullterminiert ist. Irgendwo im 
Speicher kommt mal wieder eine Null, und bis dahin kommt halt Grütze bei 
raus. ;-)

[Jan-Niklas B. (janniklas_b)]

Das mag sein, ich weiß nur leider nicht wo...ich hab inzwischen überall 
die null-terminierung eingebaut, und komischer weise stürzt er immer 
während des sendens eines Strings via Serial.println("xyz") ab... ein 
gedanke den ich momentan habe ist das ich entweder das RAM aufgebraucht 
habe oder ein Memmory-leak / segfault hab ...

[DirkZ (Gast)]

Jan-Niklas B. schrieb:
> Kann man irgendwie alle Speicher des Arduino einmal löschen?
> Er hat mir grade auf den Befehl hin einen Buchstabensalat mit Fragmenten
> von alten Codes rausgegeben...

schick mal den aktuellen Programmcode

[STS (Gast)]

Hallo Jan-Nicklas,

ich hatte auf meinem Arduino UNO unlängst möglicherweise das gleiche 
Problem. Mein Programm wurde (bei knappem SRAM Speicherplatz) durch 
Serial-Print()Befehle in einen unbeabsichtigten Reset-Zustand gebracht.

Nach längerem Studium der Beschreibungen im Internet kam ich zur 
Konklusion, dass diese Verhalten mit der Größe des Puffers im ARDUINO 
RAM zusammenhängt. Offensichtlich ist hier die Länge auf etwa 32 Zeichen 
limitiert.

Abhilfe brachte bei mir nur das "Verkürzen" der Ausgaben bzw. die 
Auslagerung einiger Texte in den Programm-Speicher.

Ich hoffe, dass dir dieser Hinweis bei Lösung deines Problems hilft.

[Christian J. (Gast)]

STS schrieb:
> Hallo Jan-Nicklas,
>
> ich hatte auf meinem Arduino UNO unlängst möglicherweise das gleiche
> Problem. Mein Programm wurde (bei knappem SRAM Speicherplatz) durch
> Serial-Print()Befehle in einen unbeabsichtigten Reset-Zustand gebracht.
>
> Nach längerem Studium der Beschreibungen im Internet kam ich zur
> Konklusion, dass diese Verhalten mit der Größe des Puffers im ARDUINO
> RAM zusammenhängt. Offensichtlich ist hier die Länge auf etwa 32 Zeichen
> limitiert.
>
> Abhilfe brachte bei mir nur das "Verkürzen" der Ausgaben bzw. die
> Auslagerung einiger Texte in den Programm-Speicher.
>
> Ich hoffe, dass dir dieser Hinweis bei Lösung deines Problems hilft.

Hallo,

kann ich so bestätigen, wie geschrieben. Grudsätzlich sollten Text mit 
dem
F(...) eingebunden werden damit sie ins Flash kommen.

Aus

Serial.println("Nachricht erfolgreich an Melder: "+(String)destination+" 
gesendet!");

wird

Serial.println(F("Nachricht erfolgreich an Melder: 
")+(String)destination+ F("gesendet!"));