Pointer als Parameter und Rückgabewert

[Bumbum]

Hallo,

ich möchte eine kleine Funktion schreiben die mir in einem Char-Array ein Zeichen sucht. Die Funktion soll mit einem Pointer auf das erste Zeichen, bei dem die Suche begonnen werden soll aufgerufen werden. Dieser Pointer als Parameter soll auch als Rückgabewert dienen. Hierbei happert es noch ein bisschen. Hier mal meine Funktion:

bool MoveBuffer2Char (volatile char **Buffer, char Char, U8 MaxChars)
{
 volatile char *CheckBuffer = *Buffer;
 
 while ((*CheckBuffer++ != Char) & (--MaxChars > 0));
 
 if (MaxChars > 0)
 {
  Buffer = &CheckBuffer;
  return (TRUE);
 }
 else
  return (FALSE);
}

Mein Aufruf sieht so aus:

volatile char RS232Buffer[0x50];
volatile char *Buffer = &RS232Buffer[0];

// ...

if (MoveBuffer2Char (&Buffer, ',', 6))
{
// ...
}

Leider steht die Variable Buffer nach dem Aufruf der Funktion wieder auf dem Startwert. Was mache ich falsch, bzw. wie mache ich es richtig?

Viele Grüße
Andreas

[DanielSan]

Bin mir nicht ganz sicher aber ich glaub wenn du sie als "static" machst müsste sich das erledigt haben.

[Bumbum]

Nein, ich habe es mittlerweile durch rumprobieren herausgefunden. So funktioniert es:

 if (MaxChars > 0)
 {
  *Buffer = CheckBuffer;
  return (TRUE);

Viele Grüße
Andreas

[Wsk8]

Wieso so kompliziert?
Geht doch auch etwas einfacher:

char *func(char *buffer, char ch, size_t size) {
    int i = 0;
    for(i = 0; i < size; i++) {
        if(*buffer == ch) {
            return buffer;
            break;
        }
        buffer++;
    }


    return NULL;
}

char buf[] = "Hallo";
char *c = 0;


c = func(buf, 'a', sizeof(buf));


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

mfg

[Bumbum]

Stimmt, so gehts auch (nicht getestet, aber es fehlt zumindest ein & im Funktionsaufruf). Bei dieser Lösung ist die Fuktion zwar kürzer, aber dafür die Auswertung um eine Zeile länger. Quasi Geschmackssache.
Ich habe die Funktion so implementiert, damit die Auswertung übersichtlicher ist und bleibe somit bei meiner Lösung.

Viele Grüße
Andreas

[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