Cast macht nicht was er soll

probier mal +1 ausserhalb der Klammer, da erst danach der Datentyp 
gecastet ist. Innerhalb der Klammer ist er noch 32 Bit.

Nur ein Versuch ...

>Ich möchte einen 32 bit Wert in 2 16 bit Teile Zerlegen und getrennt
>voneinander abspeichern.

uint16_t temph, templ;

temph = (uint16_t)(freeCluster >> 16);
templ = (uint16_t)freeCluster;


Mach was draus;)

Omega schrieb:
> probier mal +1 ausserhalb der Klammer, da erst danach der Datentyp
> gecastet ist. Innerhalb der Klammer ist er noch 32 Bit.
>
> Nur ein Versuch ...

1

*(uint16_t*)&Sektor.Array[Adresse.Offset] = *(((uint16_t*)(&freeCluster)) + 1);

Du hast Recht Omega. Funktioniert. Hätte ich auch selbst drauf kommen 
können.


Vielen Dank!

>*(uint16_t*)&Sektor.Array[Adresse.Offset] = *(((uint16_t*)(&freeCluster)) + 1);

>Du hast Recht Omega. Funktioniert. Hätte ich auch selbst drauf kommen
>können.

Und diesen komplizierten fehleranfälligen Mist willst du jetzt
in deinem Sourcecode behalten? Da weisst du morgen doch schon nicht
mehr was du da gemacht hast;)

holger schrieb:
>>Ich möchte einen 32 bit Wert in 2 16 bit Teile Zerlegen und
> getrennt
>>voneinander abspeichern.
>
> uint16_t temph, templ;
>
> temph = (uint16_t)(freeCluster >> 16);
> templ = (uint16_t)freeCluster;
>
> Mach was draus;)


Ja, das funktioniert auch, ich finde aber die Cast-Codezeile 
übersichtlicher.

Bin zwar kein Compiler Experte, aber ich würde sagen, das Cast spart 
auch Prozessorzeit, da keine Variablen erstellt werden und und geshiftet 
werden müssen.

Markus schrieb:
>
> Bin zwar kein Compiler Experte, aber ich würde sagen, das Cast
> spart auch Prozessorzeit, da keine Variablen erstellt werden und
> geshiftet werden müssen.

Das kann alles wegoptimiert werden.
Schau doch mal den generierten Assembler-Code für beide Varianten an.

>Ja, das funktioniert auch, ich finde aber die Cast-Codezeile
>übersichtlicher.

Du findest das hier (der Fehler ist dort noch drin):

*(uint16_t*)&Sektor.Array[Adresse.Offset] = *(uint16_t*)&freeCluster;
*(uint16_t*)&Sektor.Array[Adresse.Offset] = (uint16_t*)(&freeCluster + 
1);

übersichtlicher als das hier?

Sektor.Array[Adresse.Offset++] = (uint16_t)freeCluster;
Sektor.Array[Adresse.Offset] = (uint16_t)(freeCluster >> 16);

Du hast doch nen Sockenschuss. Deinem Müll da oben sieht man
an das er von einem grottigen Anfänger kommt.

...und warum muß es gleich wieder auf die persönliche Ebene gehoben 
werden, oder ging es nicht sachlich ?

Leider immer wieder im Forum zu finden, was ich persönlich sehr schade 
finde.

Gruß Gordon Shumway aka TOKABLN

holger schrieb:
> Sektor.Array[Adresse.Offset++] = (uint16_t)freeCluster;
> Sektor.Array[Adresse.Offset] = (uint16_t)(freeCluster >> 16);

Hier sind man doch direkt was gemacht wird. Da kann dich jemand aus dem 
Schlaf holen und man sieht es sofot.
Und wenn du ein Timingproblem hast werden dich diese 2 Zeilen nicht 
retten. Da stinkts dann wohl an einer oder vielen anderen Stellen.
Eine der wichtigsten Regeln bei uns in der Firma ist die Lesbarkeit und 
Verständlichkeit vom Code. Das solltest du auch beachten