RS232 P. Fleury Char und String Probleme mal wieder

[PCMan]

Hallo Leute,
ich möchte an meinen µC einen Befehl senden (na so wein Zufall ;)
Der Befehl soll im Prinzip so aussehen:
<befehl>
Das ganze müsste dann so interpretiert werden:

wenn '<' empfangen, dann
Command[i]=empfangenes_zeichen
i++

wenn '>' empfangen, dann keine weiteren Zeichen mehr aufnehmen.

Das habe ich bisher so versucht zu realisieren:


uint8_t c = uart_getc();
if (c=='<') RS232.Cmd_Start=1; //RS232.Cmd_Start ist ein Bitfeld

if (RS232.Cmd_Start==1) {
RS232.Cmd[RS232.i]=c; //RS232.Cmd ist ein Char[20]
RS232.i++; //ein uint8_t
}

if (c=='>') { //wenn fertig, '\0' anhängen und Index Resetten
RS232.Cmd_Start=0;
RS232.Cmd[RS232.i]='\0';
RS232.Cmd_Complete=1;
}

...
woanders
...

if (RS232.Cmd_Complete==1) {
lcd_gotoxy(0,1);
lcd_puts(RS232.Cmd);
uart_puts(RS232.Cmd);
RS232.Cmd_Complete=0;
}



Schön und gut, wenn ich nun mit uart_puts(RS232.Cmd) versuche den Befehl zurückzusenden bekomme ich im Terminal nur ein "<". Wieso?
Wenn ich folgendes sende:



RS232.Cmd[0]='a';
RS232.Cmd[1]='b';
RS232.Cmd[2]='c';
RS232.Cmd[3]='\0';

uart_puts(RS232.Cmd);


erhalte ich im Terminal "abc", egal ob ich (const char*) RS232.Cmd nehme oder nicht. Ich versteh' nicht, wo mein Denkfehler liegt.
Vieln Dank,
Simon

Hinweis: Es macht auch keinen Unterschied, wenn ich in der Auswertung (unsigned int)c=='>' anstelle von c=='>' verwende, obwohl's eigentlich korrekter wäre.

P.S.: ATMega32, XTAL=8MHz

[Andun]

if (c=='>') { //wenn fertig, '\0' anhängen und Index Resetten


Wo resetest du denn da i? Bzw. wie initialisiert du RS232.i?

Andun

[PCMan]

Hi Adun.
RS232.i wird in main initialisiert:


RS232.i=0;
for (;;) {
rs232_gets(); //funktion, wo zeichen zu string gepackt werden sollen...
lcd_gotoxy(0,0);
lcd_puts("Eingang:");

if (RS232.Cmd_Complete==1) {
lcd_gotoxy(0,1);
lcd_puts(RS232.Cmd);
uart_puts(RS232.Cmd);
RS232.Cmd_Complete=0;
}


} //ende for

RS232.i habe ich vergessen zu resetten. Sollte dort hin:
funktion rs232_gets()



if (c=='>') {
RS232.Cmd_Start=0;
RS232.Cmd[RS232.i]='\0';
RS232.Cmd_Complete=1;
RS232.i=0;
}


Das hatte ich da auch schonmal stehen gehabt. Es macht aber keinen Unterschied. Wo könnte das Problem sein?
VG Simon

[Andun]

Mh ... vielleicht hat noch jemand anderes ne idee?

Andun

[PCMan]

Hi Adun, ich glaube langsam den Fehler zu finden. Und zwar wird das char ab dem '<' immer angehängt. Egal ob danach gleich ein neues char kommt oder eben '0'. Das habe ich folgendermaßen rausgefunden:

Ich habe mir das eingelesene char-array stur ausgeben lassen:


for (int u=0; u<10; u++) {uart_putc(RS232.Cmd[u]); }


Wenn ich nun "<hello>" übertrage bekomme ich folgendes zurück:
"< e l l"
Wichtig: Die Leerzeichen haben den Decimal-Wert 000. Und dort liegt der Fehler, das muss ich noch rausfiltern.
Poste meine Ergebnisse hier dann...

[PCMan]

Okay, Problem behoben: eine Zeile Code hat's dann getan:


if (c!=0){
if(c=='<') {
RS232.Cmd_Start=1;
}
if(RS232.Cmd_Start==1) {
RS232.Cmd[RS232.i]=c;
RS232.i++;
}
if(c=='>') {
RS232.Cmd[RS232.i]='\0';
RS232.Cmd_Start=0;
RS232.Cmd_Complete=1;
RS232.i=0;
}
}

Das if (c!=0) war's!
Jetzt geht's so, wie ich mir das vorstelle.
vG Simon