Problem mit einfacher Rechnung

[assi (Gast)]

Hallo,

ich habe eine Schaltung gebaut die mehrer KTY Temperaturfühler 
auswertet.
Um die ADC-Werte in °C umzurechnen verwende ich die im folgenden Code zu 
sehende Formel.
Ich habe nun Probleme wenn ich Werte unter 0°C habe das die Funktion 
einfach -22°C ausgibt ich kann mir aber leider nicht erklären warum sie 
das macht.

Von der Hardware Seite her sollte alles in Ordung sein das Habe ich 
schon mehrfach überprüft und die anderen Fühler mit Temperaturen über 
40°C werden auch richtig ausgegeben.

int8_t umrechnung (uint16_t adc)
{
int8_t y=0;


y = ((adc-387)*20)/37;

  return y;
}

Gruß
assi

[MWS (Gast)]

Teile dem Compiler per Cast auf das Zwischenergebnis mit, welchen 
Wertebereich er dafür verwenden soll, dann geht's.

[assi (Gast)]

Hallo,
danke erstmal für die schnelle Antwort, jedoch weis ich nicht so richtig 
wie ich dieses Casten anstellen soll. Ich habe gerade mal nachgelesen 
aber eigentlich sollte doch alles int8_t sin oder???

[MWS (Gast)]

Bzw. caste adc, das ist etwas sauberer.

[Karl H. (kbuchegg)]

assi schrieb:
> Hallo,
> danke erstmal für die schnelle Antwort, jedoch weis ich nicht so richtig
> wie ich dieses Casten anstellen soll. Ich habe gerade mal nachgelesen
> aber eigentlich sollte doch alles int8_t sin oder???


Wie kann das int8_t sein, wenn du hier

int8_t umrechnung (uint16_t adc)

adc als unsigned übergibst?

[MWS (Gast)]

assi schrieb:
> aber eigentlich sollte doch alles int8_t sin oder?

Nein, er rechnet zuerst auf Basis des Typen, der als Operand übergeben 
wird, deswegen geht ein negatives Ergebnis mit einem uint nicht. 
Außerdem würde auch ein 8Bit Integer falsch sein, da das 
Zwischenergebnis den Wertebereich eines int8 überschreiten kann.

[assi (Gast)]

Ja habs Verstanden entweder adc schon als int16_t übergeben oder mit 
Cast sollte es dann so aussehen:

y = (((int16_t)adc-387)*20)/37;

So funktioniert es wunderbar vielen Dank!!!!