Problem mit malloc

[Georg (Gast)]

Hab hier n problem mit folgendem code:

double workarray[WIDTH][HEIGHT]   = (double *)malloc((WIDTH*HEIGHT*4));

gcc meckert was von "invalid initializer" . . . ich hab keine Ahnung 
warum. . .

Sieht einer von euch, was da faul ist?

[Andreas S. (andreas) (Admin)]

Wenn du ein Array definierst brauchst du kein malloc mehr, der Speicher 
wird automatisch reserviert.

[I_ H. (i_h)]

(double *)malloc((WIDTH*HEIGHT*4));

Und das gewöhn dir mal ganz schnell ab, das ist fehlerträchtig und wie 
man sieht hast du den Fehler auch gleich gemacht.

So schaut's gut aus:

(double *)malloc(WIDTH*HEIGHT*sizeof(double));

Ein double hat übrigens 8 Byte...

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

I_ H. wrote:

> Ein double hat übrigens 8 Byte...

Er kann auch 12 oder 16 haben.  Dafür gibt's ja sizeof.  Beim AVR-GCC
hat er derzeit übrigens wirklich 4 Byte.

[yalu (Gast)]

Wenn du das zweidimensionale Array tatsächlich dynamisch anlegen
willst, geht das so:

  double (*workarray)[HEIGHT] =
    (double (*)[HEIGHT])malloc(WIDTH*HEIGHT*sizeof(double));

Hinterher kannst du dann in gewohnter Weise mit

  workarray[x][y]

darauf zugreifen.

[yalu (Gast)]

... oder sauberer:

  double (*workarray)[HEIGHT] =
    (double (*)[HEIGHT])malloc(WIDTH*sizeof(double [HEIGHT]));

[Georg (Gast)]

ok, danke, hat jetz geklappt. . .

[Simon K. (simon)]

Und warum nicht einfach .. ?

double workarray[WIDTH][HEIGHT];

Sofern WIDTH und HEIGHT konstant sind.