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