|
Anzeige
|
Problem mit Casts von DatentypenHallo! Ich möchte für meinen ATXmega eine Funktion schreiben, die den Wert des internen Temperatursensors in °C umwandelt:
Wenn ich nun das disassembler anschaue, sehe ich, dass vom Wert in Kelvin nur 0x11 und nicht 0x111 (=273) abgezogen werden:
Ich habe schon alle möglichen Varianten mit typen-cast ausprobiert, aber es hat alles nichts gebracht. Könnt ihr mir helfen? Vielen Dank! Erdbewohner
Gast
#2745962
Raphael F. schrieb: > Wenn ich nun das disassembler anschaue, sehe ich, dass vom Wert in > Kelvin nur 0x11 und nicht 0x111 (=273) abgezogen werden: 273-256 = 0x11 die 0x11 ist also richtig für das low-byte. Ich würde die rechenschritte mal einzeln machen. uint16_t tmp = (ADCA_CH1_RES * (uint32_t)tempCalcCoeff)>>13; tmp -= 273; return tmp; wie gross ist denn ADCA_CH1_RES? > int8_t calcTemperature(void)
******
Ich denke hier liegt das "Problem".
Ob du nur vom Lowbyte 0x11 abziehst, oder von einem unsigned 16 Bit
Ergebnis 0x0111 ist ziemlich irrelevant, wenn du dann sowieso vom
Ergebnis das High_byte verwirfst und das ganze danach als eine Zahl mit
Vorzeichen verkaufst.
Arrrrrggg!! Natürlich! Meine Überlegung war, dass der Wert in Kelvin ja noch grösser als 128 ist, und dass darum auch 273 und nicht nur 17 abgezogen werden sollten. Aber wenn man das High Byte weglässt und annimmt, dass es 1 ist (sonst geht es sowieso nicht), geht es natürlich. Vielen Dank! Ich würde hier
auch nicht nach unsigned casten, sondern auf int16_t. Das Ergebnis könnte sonst bei negativen Celsius Temperaturen 'interessant' werden, wenn du von 250K 273K abziehst. ja, das lasse ich lieber weg. Es war nur ein Versuch von vielen, die ich gemacht habe, um das vermeintliche Problem zu lösen.
Gast
#2747047
Raphael F. schrieb: > return (((uint16_t)((ADCA_CH1_RES * (uint32_t)tempCalcCoeff)>>13)) - 273); Bei solchen Zeilen wird mir schlecht. Schreib dein Zeugs doch besser einzeln und nicht in einem Klumpen. long temp; temp = ADCA_CH1_RES * tempCalcCoeff; if (temp>blabla) temp = blabla; // Sättigung ist besser als Abhacken temp = temp>>13; return temp; Bei einem einigermaßen vernünftigen Compiler kommt damit ein optimaler Code heraus, weil temp sowieso in einem Register gehalten wird. Es ist also überhaupt kein Grund vorhanden, mit Typecasts um sich zu schmeißen. Merke: Typecasts sollten die Ausnahme sein, nicht die Regel. Wenn jemand für jeden Furz einen Typecast braucht, dann macht er etwas gründlich falsch. W.S. Antwort schreiben |
Anzeige
|
