Problem bei Pointer-/Parameterübergabe und malloc

[Mark Huber (Gast)]

Hallo,

habe zwei kleine Funktionen geschrieben, die aus dem main aufgerufen 
werden:

int main(void)
{
    int *m, *n;
    double *arrayzeiger;
    einlesen (m, n, arrayzeiger);
    ausgeben (m, n, arrayzeiger);
    return (0);
}

void einlesen (int *m, int *n, double *arrayzeiger)
{
    int temp;
    char Dateiname[50];

    FILE *Datei;
    printf ("bitte Dateinamen eingeben\n");
    scanf ("%s", Dateiname);
    Datei = fopen (Dateiname, "r");
    if (Datei == NULL)
    {
    printf ("Datei nicht gefunden\n");
    exit (1);
    }
    fscanf (Datei, "%i", m);
    fscanf (Datei, "%i\n", n);
    ZV_c = (double*) malloc (n  sizeof (double*));
    for (temp= 0; temp< *n; temp++)
  {
    fscanf (Datei, "%lf\n", &arrayzeiger[temp]);
  }
    fclose (Datei);
}

void ausgeben (int *m, int *n, double *arrayzeiger)
{
  int temp;
  for (temp= 0; temp< *n; temp++)
  {
    printf ("nachher: = %lf\n", arrayzeiger[temp]);
  }
}

Die einlesen Funktion holt sich aus der Datei die Größe des Arrays und 
belegt anschließend mittels malloc Speicher fürs das Array.
Das Problem ist nur, dass mir die Funktion nicht den neuen Zeiger von 
malloc zurückgibt, der also nur lokal existiert :=(
Deshalb habe ich den Aufruf der Funktion auch schon mal zu 
&(*arrayzeiger) verändert, womit ich ja eigentlich die Adresse des 
Zeigers in die Funktion bekommen sollte. Aber das klappt auch nicht.
Wie macht man das richtig?

Gruß,

Mark

P.S. Wenn ich den Speicher mit malloc schon vorher im main reserviere 
geht alles gut (da der Zeiger ja auch nicht verändert wird).

[Stefan B. (stefan)]

Es ist sowieso Zufall, dass das Programm funktioniert. Du speicherst in 
der Zeile "fscanf (Datei, "%lf\n", &arrayzeiger[temp]);" wegen der 
uninitialisierten Variable arrayzeiger irgendwo im Speicher.

Einlesen könnte den allozierten Speicher als Pointer an Main 
zurückgeben. Und am Ende von main() (oder bei neuem Aufruf von 
einlesen()) kannst du den per free() auch wieder freigeben.

void * einlesen (int *m, int *n, double *arrayzeiger) // <==
{
    int temp;
    char Dateiname[50];
    FILE *Datei;

    printf ("bitte Dateinamen eingeben\n");
    scanf ("%s", Dateiname);
    Datei = fopen (Dateiname, "r");

    if (Datei == NULL)
    {
       printf ("Datei nicht gefunden\n");
       exit (1);
    }

    fscanf (Datei, "%i", m);
    fscanf (Datei, "%i\n", n);

    if (arrayzeiger != NULL) // <==
    {
       free(arrayzeiger);
    }

    arrayzeiger = (double *) malloc (*n * sizeof (double*)); // <==

    if (arrayzeiger == NULL) // <==
    {
       printf ("Speicherproblem\n");
       fclose (Datei);
       exit (1);
    }

    for (temp = 0; temp < *n; temp++)
    {
       fscanf (Datei, "%lf\n", &arrayzeiger[temp] /* oder arrayzeiger+temp */); // <==
    }

    fclose (Datei);

    return ((void *)arrayzeiger); // <==
}

int main(void)
{
    int *m, *n;
    double *arrayzeiger = NULL; // <==

    arrayzeiger = (double *) einlesen (m, n, arrayzeiger);
    ausgeben (m, n, arrayzeiger);
    free((void *)arrayzeiger); // <==

    return (0);
}

[Arc N. (arc)]

Eigentlich dürfte noch weniger funktionieren...
da die Zeiger alle uninitialisiert sind d.h. wenn einlesen in main 
aufgerufen wird, bekommt einlesen irgendetwas nur keine Zeiger auf 
benutzbare Variablen in denen die neuen Werte gespeichert werden 
könnten.
Soll arrayzeiger nach dem Aufruf auf den angeforderten Speicher zeigen, 
muss einlesen entweder den neuen Zeiger zurückliefern (double* 
einlesen(int* m, int* n) oder man übergibt einen Zeiger auf einen 
Zeiger.

// aus
    int *m, *n;
    double *arrayzeiger;
    einlesen(m, n, arrayzeiger)

// müsste entweder
    int m, n;
    double *arrayzeiger;
    einlesen(&m, &n, &arrayzeiger);
// oder 
    int mv, nv;
    int *m = &mv, *n = &nv;
    einlesen(m, n, &arrayzeiger)
// werden

[Stefan B. (stefan)]

Stimmt. Typ. Fall von Problemfixierung bei mir.

[Mark Huber (Gast)]

Hallo,

vielen Dank für die schnelle Rückmeldung. Nun bleibt noch ein Problem: 
später sollte die Funktion mehrere Arrays einlesen, d.h. ich müstte auch 
mehrere Zeiger zurückgeben. Aber wenn ich erst mal weiß wie ich die 
Zeiger korrekt in die Funktion ein/übergeben bekomme, dann probliere ich 
da erst mal weiter und melde mich evtl. noch mal.

Danke,

Mark

[Mark Huber (Gast)]

Hallo,

habe das ganze nun zu

void einlesen (int *m, int *n, double **arrayzeiger)
{
    int temp;
    char Dateiname[50];

    FILE *Datei;
    printf ("bitte Dateinamen eingeben\n");
    scanf ("%s", Dateiname);
    Datei = fopen (Dateiname, "r");
    if (Datei == NULL)
    {
    printf ("Datei nicht gefunden\n");
    exit (1);
    }
    fscanf (Datei, "%i", m);
    fscanf (Datei, "%i\n", n);
    if (*arrayzeiger != NULL)
    {
    free (*arrayzeiger);   // <== evtl. ist hier auch ein Fehler
    }
    *arrayzeiger = (double*) malloc (*n * sizeof (double*));
    if (*arrayzeiger == NULL)
    {
       printf ("Speicherproblem\n");
       fclose (Datei);
       exit (1);
    }
    for (temp = 0; temp < *n; temp++)
  {
    fscanf (Datei, "%lf\n", &(*arrayzeiger)[temp]);
  }
    fclose (Datei);
}

void ausgeben (int m, int n, double **arrayzeiger)
  int temp;
  for (temp= 0; temp< *n; temp++)
  {
    printf ("nachher: = %lf\n", arrayzeiger[temp]);
  }

int main(void)
{
    int m, n;
    double *arrayzeiger = NULL;
    einlesen (&m, &n, &arrayzeiger);
    ausgeben (m, n, &arrayzeiger);
    //free (*ZV_c);   // <== schlägt immer fehl
    return (0);
}

geändert. Nun läuft es (mal wieder), aber die free Aufrufe scheitern 
(bzw. der letzte auf jeden Fall, bei dem in der einlesen Funktion bin 
ich mir unsicher). Insofern nehme ich stark an, dass da mmer noch was 
nicht stimmt. Warum der letzte free Aufruf scheitert wundert mich, das 
das main ja jetzt den in arrayzeiger die Position/Adresse des 
eigentlichen Arrays hat.
Werde da wohl noch mal ne Nacht drüber schlafen müssen... Muss gestehen, 
so richtig klar ist mir das leider noch nicht.

Mark

[Arc N. (arc)]

Das free in einlesen ist soweit richtig, dafür das free in main nicht.
Die Funktion ausgeben dürfte auch nur falsche Werte ausgeben, ausser 
arrayzeiger[temp] wäre ein Tippfehler und sollte *arrayzeiger[temp] 
sein.
arrayzeiger[temp] wäre nur dann richtig, wenn auch nur ein einfacher 
Zeiger übergeben werden würde. So wie es jetzt da steht würde der tempte 
(nicht vorhandene Zeiger auf einen double Zeiger) ausgewertet.

Zur Verdeutlichung:

 
    double* array1 = (double *)malloc(sizeof(double) * 4);
    double* array2 = (double *)malloc(sizeof(double) * 4);
    double array3[4] = { 1.0, 2.0, 3.0, 4.0 };

    double* ptr = array1;
    double* ptr2[3] = { array1, array2, array3 };

    double** dptr1 = &ptr;
    double** dptr2 = &ptr2[0];

    array1[0] = 1.1; array1[1] = 2.1; array1[2] = 3.1; array1[3] = 4.1; 
    array2[0] = 1.2; array2[1] = 2.2; array2[2] = 3.2; array1[3] = 4.2; 

    // was geben die folgenden printfs aus?
    printf("%p", dptr[0]);
    printf("%f", *dptr[0]);
    printf("%f", **dptr);
    printf("%p", dptr2[1]);
    printf("%f", *dptr2[1]);
    printf("%f", **dptr2);

[Rolf Magnus (Gast)]

> double* array1 = (double *)malloc(sizeof(double) * 4);

Den Rückgabewert von malloc sollte man nicht casten. Der Cast ist 
unnötig und verdeckt potentielle Fehler.

[Arc N. (arc)]

Rolf Magnus wrote:
>> double* array1 = (double *)malloc(sizeof(double) * 4);
>
> Den Rückgabewert von malloc sollte man nicht casten. Der Cast ist
> unnötig und verdeckt potentielle Fehler.

In C++ ist dieser Cast zwingend nötig. In C nicht...

[Rolf Magnus (Gast)]

In C++ verwendet man eh gleich den Operator new. In C sollte man wie 
gesagt den Cast besser weglassen.

[Mark Huber (Gast)]

Hallo,

habe nun alles soweit ohne Fehler am laufen, nur das letzte free 
bereitet mir nach wie vor noch Kopfzerbrechen. Meiner Meinung (und dem 
C-Buch Beispiel) nach sollte ein

    [...]
    ausgeben (m, n, &arrayzeiger;
    if (ZV_c != NULL)
    {
    free (arrayzeiger);
    }
    return (0);
}

richtig sein. Das fliegt mir aber immer noch um die Ohren. Da ich 
gelesen habe, dass nach Terminierung des Programms der Speicher eh 
wieder freigegebn wird, hat das jetzt Zeit bis morgen. Werde da noch mal 
ein paar Dinge ausprobieren. Ist zwar mit Sicherheit kein sauberer 
Programmierstil, aber das zählt ja zum Glück auch nicht...
Bis hierhin: vielen Dank für eure Hilfe, habe wieder was dazugelernt und 
steige da (so langsam) durch.

Mark

[Peter M. (Firma: Sorry-No) (edison24)]

muss das nicht "free(*arryzeiger)" heißen....aslo mit "*"?

[Simon K. (simon)]

Nö.