String leeren

Du wirst dir schon den Startpunkt (=den Wert von 'posteingang' vor der 
Schleife) merken müssen.

Ich leere den String jetzt nach dem ; so:

1

while(*posteingang) {

2

*posteingang='\0';

3

posteingang++;

4

5

}

Geht das besser ?

pirat

> Ich leere den String jetzt nach dem ; so:
>
>

1

> while(*posteingang) {

2

> *posteingang='\0';

3

> posteingang++;

4

> 

5

> }

6

>

>
> Geht das besser ?

Nein. Solange Du nur einen veränderlichen Pointer auf den String benutzt 
("posteingang"), kannst Du den String nicht löschen/leeren.

Das geht nur, wenn Du Dir einen Pointer auf den Anfang des Puffers 
aufhebst, auf den zu Beginn Deiner Aktivitäten posteingang zeigt (bzw. 
zeigen sollte; initialisierst Du posteingang überhaupt?).

Ja.

Ich versuche Befehle vom Schema <befehl>; über die Serielle 
Schnittstelle zu empfangen. Hier ist die ganze Funktion:

http://pastebin.ca/1607673

Sie wird in einer Endlosschleife immer wieder aufgerufen.

Der Vergleich des Strings liefert immer ist immer FALSE.

??

Ich weiß nicht, wie das mit den Pointern geht.

pirat

Ich werde mir erstmal Pointer in C anschauen.

pirat

Das solltest Du wirklich tun.

Du initialisierst Deinen "posteingang" so:

        char *posteingang="";

Wieviele Bytes Platz bietet der Speicher, auf den "posteingang" zeigt?

Genau eines.

Das bedeutet, daß so gut wie jeder Schreibzugriff, den Du auf diesen 
Pointer durchführst, auf nicht dafür vorgesehenen Speicher durchgeführt 
wird.

Das überschreibt also munter irgendetwas und kann die interessantesten 
Nebeneffekte haben - Stichwort "buffer overflow".

Hallo!

Ginge das so:

http://pastebin.ca/1609600

?

piratus

Grundsätzlich ja, auch wenn es sehr kompliziert geschrieben ist.

Wie könnte man das einfacher schreiben ?

piratus

Keine globalen Werte für die maximale Stringlänge. Und dafür aber einen 
vernünftigen Returnwert mit dem der Aufrufer gleich weiterarbeiten kann 
bzw. der eine Aussage über Fehler geben kann. Sieh dir zb. die ganzen 
str... bzw. strn... Funktionen an, wie die das machen.

1

char * canfuegen(char * string, size_t stringLen, char einfg )

2

{

3

  char * start = string;

4

  while( *string )

5

    string++;

6

7

  if( (size_t)( string - start ) > stringLen - 2 )

8

    return NULL;

9

10

  *string++ = einfg;

11

  *string = '\0';

12

13

  return start;

14

}

15

16

int main()

17

{

18

  char string[3] = "";

19

20

  canfuegen( string, sizeof( string ), 'a' );

21

  canfuegen( string, sizeof( string ), 'b' );

22

  canfuegen( string, sizeof( string ), 'c' );

23

}