malloc Frage

[Malte I. (maltomat)]

Hallo Forum

folgendes Problem: ich möchte ein dynamisches, lokales Array (ibuf) 
innerhalb einer anderen Funktion allozieren und das "gefüllte" Array 
dann zurückgeben. So wie im Code unten geht es nicht, denn innerhalb von 
main ist der Speicher wohl nicht alloziert. Aber wie macht man das?

Danke für die Antworten

int ReadData(char *inputfile, int *ibuf)
{
    FILE *stream_in;
    int n;

    stream_in = fopen(inputfile, "rb");
    fread(&n, sizeof(int), 1, stream_in);
    ibuf = (int *)malloc(n*sizeof(int));
    fread(ibuf, sizeof(int), n, stream_in);
    fclose(stream_in);

    return(n);
}

int main()
{
    int i, n, *ibuf;

    ibuf = 0;
    n = ReadData("test.bin", ibuf);
    printf("n=%d\n", n);          //das geht noch..
    for(i=0; i<n; i++) {
        printf("%d\n", ibuf[i]);  //das nicht
    }
    free(ibuf);
    return(0);
}

[asdfg (Gast)]

>int ReadData(char *inputfile, int *ibuf)

muss

int ReadData(char *inputfile, int **ibuf)


Wenn Du etwas zurückgeben willst, brauchst Du einen
Ptr.
Wenn Du einen Ptr zurückgene willst, deshalb eine
Ptr auf einen Ptr.

[Quack (Gast)]

Den Puffer hast du korrekt allokiert, aber du gibst den Pointer nicht 
zurueck an den Aufrufer. Die Zuweisung, die du in ReadData machst, ist 
nur lokal gueltig. Du musst entweder eine Referenz auf einen Zeiger 
uebergeben, oder einen Zeiger auf den Zeiger:

int ReadData(char *inputfile, int &*ibuf)
{
  ...
  ibuf = ...
  ...
}

int ReadData(char *inputfile, int **ibuf)
{
  ...
  *ibuf = ...
  ...
}

[Malte I. (maltomat)]

ok. Und wie rufe ich ReadData dann von main aus auf?

[asdfg (Gast)]

>ok. Und wie rufe ich ReadData dann von main aus auf?

    n = ReadData("test.bin", &ibuf);

[Malte I. (maltomat)]

geht leider noch nicht (stürzt bei Zugriff in main ab). So sieht es 
jetzt aus:

int ReadData(char *inputfile, int **ibuf)
{
    FILE *stream_in;
    int n;

    stream_in = fopen(inputfile, "rb");
    fread(&n, sizeof(int), 1, stream_in);
    *ibuf = (int *)malloc(n*sizeof(int));
    fread(ibuf, sizeof(int), n, stream_in);
    fclose(stream_in);

    return(n);
}


int main()
{
    int i, n, *ibuf;

    ibuf = 0;
    n = ReadData("test.bin", &ibuf);
    printf("n=%d\n", n);
    for(i=0; i<n; i++) {
        printf("%d\n", ibuf[i]);
    }
    free(ibuf);
    return(0);
}

[Malte I. (maltomat)]

ok, es geht jetzt. Muss heissen:

fread(*ibuf, sizeof(int), n, stream_in);


Vielen Dank für die Hilfe!

[Harper B. (harper)]

Malte Ibs schrieb:
>     *ibuf = (int *)malloc(n*sizeof(int));

Du solltest niemals den Rückgabewert von malloc casten.

- Der von malloc gelieferte type void* wird vom C-Compiler problemfrei 
implizit in jeden anderen Pointer-Type konvertert.
- Jeder Cast kann Fehler verdecken, die ohne Cast vom Compiler durch ein 
Diagnostic Message angezeigt worden wäre.

Beispiel:

#include <malloc.h>

#define TYPE int

TYPE var = (TYPE)malloc(sizeof(TYPE));

[us73 (Gast)]

Malte Ibs schrieb:
> *ibuf = (int *)malloc(n*sizeof(int));

Und vielleicht solltest Du auch noch den Rückgabewert von malloc prüfen, 
bevor Du ihn blindlings benutzt. Da kann schon auch mal NULL zurück 
gegeben werden.

[Malte I. (maltomat)]

Hallo,

noch mal vielen Dank für die Lösung meines Problems und die Tips.

Nun möchte ich das ganze mit einem zweidimensionalen Array machen (falls 
das ähnlich elegant geht). Ich kriegs aber nicht hin. Hier ist die 
"Arbeitsversion" des Codes mit Fragezeichen überall dort, wo ich nicht 
weiterkomme.

int ReadData(char *inputfile, int ???ibuf, int *ny)
{
    FILE *stream_in;
    int i, nx, ny1;

    stream_in = fopen(inputfile, "rb");
    fread(&nx, sizeof(int), 1, stream_in);
    fread(&ny1, sizeof(int), 1, stream_in);
    *ny = ny1;

    ?ibuf = malloc(nx * sizeof(int*));
    for (i=0; i<nx; i++){
          ?ibuf[i] = malloc(ny1 * sizeof(int));
    }

    for (i=0; i<nx; i++){
        fread(?ibuf[i], sizeof(int), ny1, stream_in);
    }
    fclose(stream_in);
    return(nx);
}


int main()
{
    int i, j, nx, ny, **ibuf;

    nx = ReadData("test.bin", ?ibuf, &ny);

    for (i=0; i<nx; i++){
        for (j=0; j<ny; j++){
            printf("%d ", ibuf[i][j]);
        }
        printf("\n");
    }

    for (i=0; i<nx; i++){
        free(ibuf[i]);
    }
    free(ibuf);

    return(0);
}

[Rolf Magnus (Gast)]

Malte Ibs schrieb:
> Hallo,
>
> noch mal vielen Dank für die Lösung meines Problems und die Tips.
>
> Nun möchte ich das ganze mit einem zweidimensionalen Array machen (falls
> das ähnlich elegant geht).

Geht es nicht. Es gibt da mehrere Möglichkeiten, so etwas zu erreichen, 
aber elegant ist das alles nicht. Du hast aber mit einem Array aus 
Pointern einen guten Ansatz gewählt.

  int ReadData(char *inputfile, int ***ibuf, int *ny)
> {
>     FILE *stream_in;
>     int i, nx, ny1;
>
>     stream_in = fopen(inputfile, "rb");
>     fread(&nx, sizeof(int), 1, stream_in);
>     fread(&ny1, sizeof(int), 1, stream_in);
>     *ny = ny1;
>
      *ibuf = malloc(nx * sizeof(int*));
>     for (i=0; i<nx; i++){
          (*ibuf)[i] = malloc(ny1 * sizeof(int));
>     }
>
>     for (i=0; i<nx; i++){
          fread((*ibuf)[i], sizeof(int), ny1, stream_in);
>     }
>     fclose(stream_in);
>     return(nx);

return ist übrigens keine Funktion. Da braucht man keine Klammern:

      return nx;

> }
>
> int main()
> {
>     int i, j, nx, ny, **ibuf;
>
      nx = ReadData("test.bin", &ibuf, &ny);
>
>     for (i=0; i<nx; i++){
>         for (j=0; j<ny; j++){
>             printf("%d ", ibuf[i][j]);
>         }
>         printf("\n");
>     }
>
>     for (i=0; i<nx; i++){
>         free(ibuf[i]);
>     }
>     free(ibuf);
>
>     return(0);
> }

[Malte I. (maltomat)]

Besten Dank, das klappt!

[Jörg W. (dl8dtl) (Moderator)]

Btw:

Harper Blues schrieb:

> #include <malloc.h>

Nein, seit 1989 nicht mehr so.  <stdlib.h> ist die zuständige
Headerdatei.