Farben nach R,G,B umwandeln in 4-stell. hex-code?

[Moppi]

Ich hatte das hier schon geschrieben (Beirag #20): https://www.roboternetz.de/community...l=1#post646541

Ich denke, er führt die Berechnung mit 8 Bit aus.

so:  (uint8_t)(color16 / 2048 )*8;

Du brauchst aber 16 Bit während der Berechnung, weil zumindest Dein Farbwert color16 ja 16 Bit umfasst.

Du müsstest hinterher eine Umwandlung vornehmen

(uint8_t)R = (color16 / 2048 )*8;

Aber das funktioniert nicht!

Deshalb habe ich jetzt das probiert, da gibt es kein Gemecker:

void Color16bit2colorRGB(uint16_t color16, uint16_t &R, uint16_t &G, uint16_t &B) {

   R = (color16 / 2048)*8; ;
   G = (color16 & 2016)/8;;
   B = (color16 & 31)*8;
   R = (uint8_t)R;
   G = (uint8_t)G;
   B = (uint8_t)B;
   
   Serial.println("");
   Serial.println("Unterprogramm color16 zu RGB:");
   Serial.println( (String)"color16="+(String)color16);
   Serial.println( (String)"R="+(String)R);
   Serial.println( (String)"G="+(String)G);
   Serial.println( (String)"B="+(String)B); 
}

[HaWe]

bittebittebitte mal selber testen!

- - - Aktualisiert - - -

...ach so, jetzt habe ich deinen Einwand verstanden!
Klammer ums ganze, dann erst casten!

- - - Aktualisiert - - -

nein, stimmt auch nicht dann.

[Moppi]

Schau noch mal im letzten Beitrag vor Deinem. Habe es geändert.

Allerdings glaube ich auch, dass das

R = (uint8_t)R;

überflüssig ist. Weil der kleinste Typ, den der Arduino kennt, ist: integer
Er würde den von Dir geglaubten 8-Bit-Wert vermutlich als Integer ablegen/speichern. So, wie er nach der Berechnung schon vorliegt.

[HaWe]

das habe ich verstanden und auch so gemacht.
aber nein, der kleinste Wert ist uint16_t , nicht int, denn int ist bei mir int32_t (ARM Cortex M0 = Arduino M0/Zero), deshalb ist es immer explizit auf uint16_t oder uint8_t gecastet.

Tipps sind wirklich sehr willkomen, aber bitte immer vorher selber testen!

[Moppi]

Na ja, ich hatte geschrieben: "vermutlich" und vorher schon, dass ich mich mit den Typen nicht so genau bei den Teilen auskenne.

Aber um das aufzuklären: Gibt es noch ein Problem?
Und welchen Tipp sollte ich testen - was soll ich da testen? - k.A.

[HaWe]

Der Code soll mit dem expliziten Casting damit eben auch auf 8bit Arduinos lauffähig sein.

Teste einfach meinen Testcode, in den du deine Algorithmen einsetzt, du bekommst dann ja über Serial die Debug-Informationen zur Kontrolle.

[Moppi]

Da habe ich noch byte gefunden:

   R = byte((color16 / 2048)*8);
   G = byte((color16 & 2016)/8);
   B = byte((color16 & 31)*8);

[HaWe]

das ist doch wurscht, byte == uint8_t

bitte testen und Serial Meldungen posten!