compiler fehler setzen daten 2d-array

[Chandler B. (chandler)]

Hallo,
ich hab ein Problem, wo ich nicht weiter komme.

typedef struct
{
    uint32_t rows_u32;
    uint32_t cols_u32;
    uint8_t data_au8[60][60];
} graphicData_ts;

das data_au8 soll später noch variable werden (also die Anzahl der 
Reihen und Spalten).

in der .C habe ich dann

static graphicData_ts graphicData_s =
{
    .rows_u32 = 0,
    .cols_u32 = 0,
    .data_au8 = {{0}},
};
... ... ...
... ... ...
... ... ...
... ... ...
void graphicInit_v(void)
{
    graphicData_s.cols_u32 = 60;
    graphicData_s.rows_u32 = 60;
    graphicData_s.data_au8 = {
        {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0},
        {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0},
        {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0

... ... ...
... ... ...
... ... ...

Jetzt bekomme ich auf der Zeile
    graphicData_s.data_au8 = {
den Fehler:

error: expected expression before '{' token
   59 |     graphicData_s.data_au8 = {

aber was ist daran verkehrt?
ich hbae genau 60x60 daten (auch != 0)

[Jim M. (turboj)]

Du versuchst hier einem Array einen neuen Wert zuzuweisen, das geht in C 
so nicht. Man kann nur einzelnen Elementen einen neuen Wert zuweisen - 
für das ganze Array bräuchte es also eine Schleife.

Ich würde hier allerdings überlegen ob man nicht mit Pointern arbeiten 
möchte.

Hinweis: Einige C-Compiler betrachten Dateien mit Endung .C (anstatt .c) 
als C++ Source Code - diese Sprache hat aber signifikante Unterschiede 
zu C.

[Clemens L. (c_l)]

Arrays kann man nicht zuweisen, aber Strukturen:

void graphicInit_v(void)
{
    graphicData_s = (graphicData_ts){
        .rows_u32 = 60,
        .cols_u32 = 60,
        .data_au8 = {{0, 0}, {0}}
    };
}

[Chandler B. (chandler)]

Clemens L. schrieb:
> Arrays kann man nicht zuweisen, aber Strukturen:
>
>

void graphicInit_v(void)
> {
>     graphicData_s = (graphicData_ts){
>         .rows_u32 = 60,
>         .cols_u32 = 60,
>         .data_au8 = {{0, 0}, {0}}
>     };
> }

Jup, das funktioniert

Jim M. schrieb:
> Ich würde hier allerdings überlegen ob man nicht mit Pointern arbeiten
> möchte.

Ja, das möchte ich später auch machen, zumal es nicht immer die selbe 
Größe ist.

Jim M. schrieb:
> Hinweis: Einige C-Compiler betrachten Dateien mit Endung .C (anstatt .c)
> als C++ Source Code - diese Sprache hat aber signifikante Unterschiede
> zu C.

Das ist ein Rechtschreibefehler hier. Die Datei ist .c . Dennoch ein 
guter Hinweis. Danke :)

[Daniel A. (daniel-a)]

Du könntest "memset(graphicData_s.data_au8, 0, 
sizeof(uint8_t[graphicData_s.rows_u32][graphicData_s.cols_u32]));" 
verwenden.

Wenn es sowieso Variabel werden soll, würde ich da erst mal aus dem 
data_au8 einen Pointer machen.

Unten noch ein Beispiel mit VMTs, die sind bei sowas echt praktisch. 
(Werden aber vom MSVC nicht unterstützt, aber wer nutzt das schon).

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

#define graphic_data_vmt(S,N) \
  typedef uint8_t N##_row_t[(S).rows_u32]; \
  typedef N##_row_t N##_t[(S).cols_u32]; \

typedef struct {
    uint32_t rows_u32;
    uint32_t cols_u32;
    void* data_au8;
} graphicData_ts;

// Objekte mit static storage duration, also alle globalen, werden per default mit 0 initialisiert
// Man kann auch einzelnen Member per designated initializer initialisieren, alle anderen werden dann immer mit 0 initialisiert
static graphicData_ts graphicData_s;

void graphicInit_v(void){
  graphicData_s.cols_u32 = 60;
  graphicData_s.rows_u32 = 60;

  { // Scope des VMT einschränken. Nachträgliche Änderungen an graphicData_s.cols_u32 / graphicData_s.rows_u32 werden nicht im Typ übernommen.
    graphic_data_vmt(graphicData_s, gd_data); // Definiert VMT Typen, support ist in einigen C Versionsstandards optional!
    void* mem = realloc(graphicData_s.data_au8, sizeof(gd_data_t));
    // TODO: Prüfen, ob realloc fehlgeschlagen ist
    graphicData_s.data_au8 = mem;
    memset(graphicData_s.data_au8, 0, sizeof(gd_data_t));

    // Als Beispiel, ein Rechteck in der Mitte zeichnen, halbe Bildgrösse
    gd_data_row_t* gd_data = graphicData_s.data_au8;
    for(uint32_t y=graphicData_s.rows_u32*1/4+1; y<graphicData_s.rows_u32*3/4; y++)
      for(uint32_t x=graphicData_s.cols_u32*1/4+1; x<graphicData_s.cols_u32*3/4; x++)
        gd_data[y][x] = 0xFF;
  }
}

[Wastl (hartundweichware)]

Chandler B. schrieb:
> compiler fehler setzen daten 2d-array

Erst mal die Schuld auf den Compiler schieben, der ist fehlerhaft.

"einen Fehler in meiner Programmierung kann ich ausschliessen"

[Anton (antang)]

Wozu überhaupt initialisieren? Ich zitiere aus 6.7.8 von ISO 8899:1999:

If an object that has static storage duration is not initialized explicitly,
then:
— if it has pointer type, it is initialized to a null pointer;
— if it has arithmetic type, it is initialized to (positive or unsigned) zero;
— if it is an aggregate, every member is initialized (recursively) according to these rules;
— if it is a union, the first named member is initialized (recursively) according to these rules.

[Rolf M. (rmagnus)]

Wastl schrieb:
> Chandler B. schrieb:
>> compiler fehler setzen daten 2d-array
>
> Erst mal die Schuld auf den Compiler schieben, der ist fehlerhaft.

Ich habe das interpretiert als "der Compiler meldet einen Fehler", nicht 
als "der Compiler hat einen Fehler".

Anton schrieb:
> Wozu überhaupt initialisieren?

Ja, die explizite 0-Initialisierung ist nicht nötig, schadet aber auch 
nicht. Die ist hier aber auch gar nicht das Problem.