Stringverarbeitung mit AVR

[shpank (Gast)]

Guten Abend zusammen.

Seit einigen Tagen versuche ich, eine RS232-Schnittstelle für eine 
Kaffeemaschine zu basteln. Das ganze ist Teil eines Schulprojektes, wo 
es darum geht, den Kaffeekonsum in einer Firma zu erfassen und eine 
bargeldlose Bezahlmöglichkeit zu realisieren. Dies ist mein erstes 
Projekt mit AVRs

Aber bereits beim Befehle auswerten hapert es. Das Ding macht nicht das 
was es sollte/ich will.

Der UART selbst geht, das hab ich durch ein lokales Echo-Testprogramm 
auf dem AVR getestet.

Nur das strcmp-verzweigungs-konstrukt treibt mich in den Wahnsinn. 
Irgendwie blick ich da noch nicht so ganz durch.

Könnte mir da bitte jemand sagen welch groben Fehler ich da begangen 
habe?

Kompiliert wird mit AVR-GCC.

Danke im Voraus,

shpank

[P. S. (Gast)]

Schon recvstr() ist ziemlicher Unsinn. Mal in ein C-Buch geguckt?

[Mano W. (Firma: ---) (manow)]

ich hab's nur kurz überflogen, aber in recvstr() machst Du einen groben 
Fehler!

void recvstr(unsigned char * text)
{
  unsigned int next;
    unsigned char * tmp;
  text = tmp;                             
    do
  {
    next = USART_Receive();
      *tmp++ = next;
    }while( next != '\r');

    *tmp = '\0';
}

Zeiger angelegt, aber keinen Speicher => Du schreibst deine Sachen 
irgendwo hin, kann gut gehen, aber idR macht das nur Probleme.

C-Buch Stichwörter: Zeiger, Array, malloc

[shpank (Gast)]

Wäre es für den Anfang vielleicht besser, das ganze mit einem großen 
char-Array mit fixer größe zu machen, wo ich einfach alles reinschreibe?

Tut mir leid wenn ich euch auf den Wecker gehe. So wirklich viel Ahnung 
hab ich von dem Zeugs hier noch nicht, da ich aus der schönen bunten 
Welt der objektorientierten Programmierung (C#, Java) komme.

[Marius W. (mw1987)]

Ja, mach dir nen kleinen Zeilenpuffer (so klein wie möglich, aber so 
groß wie nötig) und schreib da rein.

Irgendwie sowas in der Art:

char zeile[32];

void recvstr()
{
  int i;
  char c;

  for(i=0; i<(32 - 1); i++)
  {
    c = usart_getc();

    // Zeilenende erreicht?
    if(c == '\r')
    {
      // ja: dann String terminieren
      zeile[i] = '\0';
      break;
    }
    else
    {
      // nein: empfangenes Zeichen anhängen
      zeile[i] = c;
  }

  zeile[31] = '\0';   // zwingende Terminierung
}

MfG
Marius

[shpank (Gast)]

Danke Marius!

Ich werd das heute mal ausprobieren.

gibts sonst noch grobe fehler? jaja... ich weiß, meine main-methode 
sieht grauenhaft aus. das kommt vom rumfrickeln. das werd ich heute noch 
bereinigen.

schönen sonntag noch.