Zeiger auf Struktur

Rudi schrieb:

>

1

>   my_struct_ptr->data[0] = ...;

2

>

>
> Meckert mirs der Compiler an.
> "expression must be a pointer to a complete object type"

der SChlüssel ist an dieser Stelle die Phrase "complete object type". An 
der Stelle an der du das machst, weiß der Compiler nicht, wie ein 
my_struct_t aussieht.
Wahrscheinlich hast du nur vergessen, das Header File zu inkludieren, in 
dem die struct definiert ist.

Rudi schrieb:
> Dieser ist vom Typ void* anstatt uint8*.
>
> Aber das sollte doch keine Rolle spielen?

O doch, void-Pointer können nicht dereferenziert werden.

Rudi schrieb:
> Hm garned bedacht. Macht aber Sinn, immerhin kennt er ja dann die Größe
> des Elements nicht, sodass er spätestens beim Zugriff mitells [0],
> [1]... überfordert wäre, richtig?


richtig.
In die Indexberechnung geht ja logischerweise auch immer die sizeof des 
Typs ein um zu Berechnen, wieviele Bytes zwischen Anfang des Arrays und 
dem interessierenden Arrayelement liegen.

Rudi schrieb:
> Kann man das dann irgendwie umcasten?

natürlich kann man umcasten.
Allerdings solltest du dich fragen, ob da prinzipiell der void* 
überhaupt eine gute Idee war. Wenn du sowieso den Datentyp kennst auf 
den du casten willst, warum steht dann der nicht gleich da drinnen?

Es gibt natürlich Fälle, an denen man bewusst einen void* einsetzt. Zb 
wenn ein Modul eine Strukturobjekt rausgeben muss und nicht haben will, 
dass sich derjenige der das Objekt benutzt, an den Daten vergreift. Der 
void* ist dann mehr oder weniger ein 'Finger weg! An diesen Daten hast 
du nichts zu suchen'.

Rudi schrieb:
> Weil der Typ von einer Lib, genauer gesagt dem LwIP Stack, vorgegeben
> ist.
>
> Dort gibt es die Struktur pbuf, die als Buffer für ein- und ausgehende
> Datenströme dient. Und die zeigt leider auf void.

Eigenartig.
Sicher, dass das so gedacht ist, dass du auf diese Werte zugreifen 
darfst?

(Vielleicht wollten sie aber auch den Aufrufer auch nur dazu zwingen, 
dass er ein wenig darüber nachdenkt, was das eigentlich für Daten sind, 
die er da bekommt: int, float, double, bytes, ...
üblich ist es allerdings nicht. Solche Buffer werden normalerweise als 
Bytes angesehen - also 'unsigned char' Arrays.


>
> Wie müsste denn der Cast aussehen?

Na ja, eben den Pointer entsprechend umcasten

    ((uint8_t*)(my_struct_ptr->data))[0]

sieht ein wenig seltsam aus, daher wäre wohl der Weg über eine 
Zwischenvariable der syntaktisch lesbarere. Der Compiler schmeisst die 
Variable ja sowieso wieder raus.

Rudi schrieb:
> Karl Heinz Buchegger schrieb:
>> Eigenartig.
>> Sicher, dass das so gedacht ist, dass du auf diese Werte zugreifen
>> darfst?
>
> Laut der (leider sehr dürftigen) Doku im Internet und den Beispielen die
> ich gefunden habe wird das durchaus so gemacht.
>
> Die allokierte Buffergröße wird auch in der Struktur abgelegt, somit ist
> es wohl dem Benutzer überlassen den Zugriff ordentlich zu regeln.

:-)
Womit sich dann allerdings die Frage erhebt, wie denn die Buffergröße 
angegeben ist. Sind das Bytes, Floats, andere Objektgrößen, Äpfel, 
Birnen ...
:-)

(Ist jetzt keine Kritik an dir. Soll dir nur zeigen, dass an dieser 
Stelle der void* nicht sehr logisch ist. WEnn ich schon die Buffergröße 
(in Bytes) in der Struktur habe, dann wäre es logisch auch den Pointer 
als Byte-Pointer anzulegen. Dann passen beide in den Einheiten zusammen.

Ändert nichts daran, dass man des öfteren sowieso umcasten muss. Aber 
dann muss man ja auch die Längenangabe umrechnen und kann das so gesehen 
ohnehin nicht verpassen.