String leeren

[Isak S. (Firma: Youzone) (pirat)]

Hallo !

Ich lese Zeichen von der seriellen Schnittstelle.

Das einkommende Zeichen habe ich in der Variablen Temp. Alles was vor 
einem ";" reinkommt soll in posteingang:

// an Posteingang anhängen
 
while(*posteingang) posteingang++; // stringende suchen
*posteingang++ = Temp;     // zeichen anhängen
*posteingang = 0;       // neues stringende

Aber wie kann ich meinen Posteingang jetzt nach einem ';' wieder leeren 
?

ich abes es mal mit

posteingang[0]='\0';

probiert, allerdings funktioniert das nicht.

Muss man vielleicht das Stringende anders suchen ?

Danke

Gruß

pirat

[Jörg G. (joergderxte)]

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

[Isak S. (Firma: Youzone) (pirat)]

Ich leere den String jetzt nach dem ; so:

while(*posteingang) {
*posteingang='\0';
posteingang++;

}

Geht das besser ?

pirat

[Gast (Gast)]

Warum nicht einfach

memset(posteingang, 0, iPOSTEINGANG_SIZE);

? Leert zwar ggf. mehr als nötig, funktioniert dafür aber extrem 
zuverlässig:-)

Und wegen der Performanz: Wenn man sich einen potenziellen 
Pufferüberlaufkandidaten einhandeln will, dann nullt man nur

memset(posteingang, 0, posteingang_last_used -posteingang);

aus.

HTH

[Rufus Τ. F. (rufus)]

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

> while(*posteingang) {
> *posteingang='\0';
> posteingang++;
> 
> }
>

>
> 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?).

[Isak S. (Firma: Youzone) (pirat)]

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

[Isak S. (Firma: Youzone) (pirat)]

Ich werde mir erstmal Pointer in C anschauen.

pirat

[Rufus Τ. F. (rufus)]

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".

[Isak S. (Firma: Youzone) (pirat)]

Hallo!

Ginge das so:

http://pastebin.ca/1609600

?

piratus

[Karl H. (kbuchegg)]

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

[Isak S. (Firma: Youzone) (pirat)]

Wie könnte man das einfacher schreiben ?

piratus

[Karl H. (kbuchegg)]

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.

char * canfuegen(char * string, size_t stringLen, char einfg )
{
  char * start = string;
  while( *string )
    string++;

  if( (size_t)( string - start ) > stringLen - 2 )
    return NULL;

  *string++ = einfg;
  *string = '\0';
  
  return start;
}

int main()
{
  char string[3] = "";

  canfuegen( string, sizeof( string ), 'a' );
  canfuegen( string, sizeof( string ), 'b' );
  canfuegen( string, sizeof( string ), 'c' );
}