[C] flexibles Array Memeber in struct initialisieren / memcpy

[Peter L (Gast)]

Hallo,

ich habe ein struct welches ein float Array flexibler länge beinhalten 
soll. Dieses struct möchte ich anschließend mit Werten befüllen.

Kann mir jemand sagen, was ich falsch mache? Das float-Array wird 
offensichtlich nicht korrekt mit Werten initialisiert.

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

struct data{ 
   int data_id; // irgendeine Variable
   int a_size;
   char vals[]; // FAM
}; 

struct data *createData(struct data *s, int id, int a_size, float a[]) { 
    s = malloc( sizeof(*s) + sizeof(float) * a_size); 
    s->data_id = id;
    s->a_size = a_size;
    memcpy(s->vals, a, sizeof(float) * a_size); 
    return s; 
} 

void printData(struct data *s){ 
    printf("Data_id : %d\n", s->data_id);
    int i;
    for ( i=0 ; i<s->a_size ; i++){
      printf("%f, ",s->vals[i]);
    }
    printf("\n");
} 
int main(void) {
  float A[] = {1.0, 7.5, 9.3};
  float B[] = {2.0, 2.8, 3.0, 5.5};
  struct data *s1 = createData(s1, 523, 3, A);
  struct data *s2 = createData(s2, 206, 4, B);
  printData(s1);
  printData(s2);
  return 0;
}

Ergibt

Data_id : 523
0.000000, 0.000000, 0.000000, 
Data_id : 206
0.000000, 0.000000, 0.000000, 0.000000,

https://ideone.com/rsxhaP

Besten Dank,

Peter

[Thomas W. (goaty)]

struct data *s1 = createData(s1, 523, 3, A);

Du übergibst s1 gleichzeitig während du es erzeugst ? Das macht doch 
keinen Sinn, oder ?

[mh (Gast)]

Überleg dir mal genau was der Typ von vals ist.

[Test (Gast)]

Thomas W. schrieb:
> Du übergibst s1 gleichzeitig während du es erzeugst ? Das macht doch
> keinen Sinn, oder ?

Ich initialisiere die Variable, allokiere Speicherplatz in dieser 
Funktion und gebe einen Zeiger darauf zurück. Spricht für mich nichts 
dagegen. Die ganze Struktur habe ich mir übrigens nicht ausgedacht, 
sondern von hier adaptiert (nur eben mit float anstatt strings) 
https://www.geeksforgeeks.org/flexible-array-members-structure-c/

[Test (oder auch Peter) (Gast)]

mh schrieb:
> Überleg dir mal genau was der Typ von vals ist.

Oh Mann -- tausend dank ;)
Muss natürlich float und nicht char heißen -> es funktioniert wie 
gewünscht.

Danke

Peter

[Holger (Gast)]

Hi Peter!

Schau Dir nochmal die die Übergabeparameter dieser Methode an - 
unabhängig vom Typproblem, dass ja bereits gelöst ist :)

struct data *createData(struct data *s, int id, int a_size, float a[]) { 

    s = malloc( sizeof(*s) + sizeof(float) * a_size); 
...
    return s; 

}

Den Pointer auf "struct data" zu übergeben ist hier ohne Wirkung.

Auf folgende Art hast Du wesentlich sinnvolleren Code und man überlegt 
nicht erst, was Du machen wolltest ;)

struct data *createData(int id, int a_size, float a[]) { 
    struct data *s, 
    s = malloc( sizeof(*s) + sizeof(float) * a_size); 
...
    return s;
} 
...
int main(void) {
  float A[] = {1.0, 7.5, 9.3};
  struct data *s1 = createData(523, 3, A);
...
  return 0;
}

Du hast praktisch einen Übergabeparameter als Stackvariable missbraucht 
;) sollte man nicht tun.

Gruß Holger

[Dirk B. (dirkb2)]

Peter L schrieb:
> Das float-Array wird
welches float-Array?

Test schrieb:
> Ich initialisiere die Variable, allokiere Speicherplatz in dieser
> Funktion und gebe einen Zeiger darauf zurück. Spricht für mich nichts
> dagegen.

Warum dann aber als Parameter der Funktion und nicht lokal in der 
Funktion?