Pointer als Parameter und Rückgabewert

[Klebwax]

Stimmt, so gehts auch (nicht getestet, aber es fehlt zumindest ein & im Funktionsaufruf).

Nein, das & fehlt nicht, das gehört da nicht hin.

Es geht aber noch einfacher:

#include <string.h>
char buf[] = "Hallo";
char *c = 0;

c = strchr(buf, 'a');

if(!c)
    printf("Fehler");
else
    printf("%c", *c);

Diese Version hat den Vorteil, daß sie nicht durch "rumprobieren" entstanden ist, sondern der Programmierer wußte, was er tat. Zusätzlich ist sie millionenfach getestet worden und hat eine sehr schöne Doku, die man leicht in einem C-Buch oder im Internet nachlesen kann. Und wenn man schon etwas eigenes schreibt, daß eine Funktion der libc nachbildet, sollte man Übergabeparameter und Returnwerte wie die Libraryfunktion verwenden. Das macht anderen und einem selbst nach einigen Monaten das Leben und das Verstehen leichter.

MfG Klebwax

[Bumbum]

Hallo Klebwax,

ich habe mir die Beschreibung der Funktion nicht genau angesehen, aber mein String ist nicht Null-Terminiert und kann es aus technischen Gründen ohne vorher umkopiert zu werden auch nicht werden. Deshalb bin ich überhaupt auf die Idee gekommen eine eigene Funktion dafür zu schreiben, und deshalb auch der Parameter MaxChars.

Viele Grüße
Andreas

[Klebwax]

... aber mein String ist nicht Null-Terminiert ...

Das ist natürlich unschön. Dann kann man die Stringfunktionen der libc garnicht verwenden.

MfG Klebwax

[Wsk8]

Hallo Klebwax,
aber mein String ist nicht Null-Terminiert

Das ist aber ganz schlechter Programmierstil!!! Den Fehler wissen ihn aber dann nicht ausbessern geht ja mal gar nicht. Wenn dein String zu kurz ist, musste ihn halt ein Zeichen länger machen oder wenn er lang genug ist, setzt du halt bei maxchar+1 einfach direkt ein '\0'.

mfg

[Bumbum]

Hallo Wsk8,

es ist immer leicht zu schimpfen, wenn man nicht alle Umstände kennt. Zu meiner Verteidigung: Der String "wächst" während er ausgewertet ist. Es handelt sich hierbei um einen RS232-Ring-Buffer, der ständig via IRQ mit Daten gefüllt wird. Ich muss also während des Empfangs auswerten. Eine bessere, performantere und ressourcenschonendere Lösung ist mir nicht eingefallen.

Viele Grüße
Bumbum

[ahabakukuk]

Hallo Bumbum

Zum Thema Ring- Buffer:

Ist das Ziel eines Ring- Buffers nicht, das man die Daten in einem Rutsch einlesen , danach auswerten und wieder löschen kann und so dem Ring- Buffer auch eine fixe Grösse zuweisen kann?

Ich sehe das jedenfalls so und mache es auch so.

Gruss

[Bumbum]

Hallo ahabakukuk,

ich glaube genau das ist nicht der Sinn eines Ringbuffers. Wenn du eh weißt wieviele Daten kommen kannst du dann, wie von dir schon herausgefunden, einen Buffer fester Größe definieren, alles einlesen und dann auswerten. Beim nächsten Datensatz fängst du einfach wieder vorne am Buffer an.

Einen Ringbuffer nimmt ma bei anderen Gegenheiten wie z.B. bei meinem Fall. Da kommen ca. 64kB (64000 Bytes) Daten oder mehr in ein paar Sekunden über die RS232. Von dieser "Datenflut" brauche ich dann ein paar Bytes zwischendrin. Die Daten werden per Interrupt in den Ringbuffer geschrieben und zwar ringförmig. Wenn ich das letzte Byte in den Buffer geschrieben habe kommt das nächste automatisch wieder an den Anfang des Buffers.
Ich muss nur schneller auswerten, wie Daten gesendet sind. In meinem Fall komme ich z.B. mit ca. 50 Bytes an Ringbuffer zurecht und kann die Kilobytes an Daten mit einem ATMega162 mit gerade mal 1kB RAM problemlos auswerten.

Viele Grüße
Andreas