Casten funktioniert nicht

dost0011 schrieb:
> In meinem Programm ist das natürlich etwas komplizierter und die 78
> können berechnete Werte von 0-255 sein.

Dann wird in dem etwas komplizierteren der Fehler sein.

Was passiert wenn du es genau so wie oben eingibst?

Bitte Netiquette beachten. Es nützt nix, wenn du ein ähnliches 
Programm hier zeigst, das hat deinen Fehler nicht!

versuch mal

unsigned short temp_us = (unsigned short)78;
unsigned char temp_uc = (unsigned char)(unsigned int)temp_us;

dost0011 schrieb:
> unsigned short temp_us = (unsigned short)78;
> unsigned char temp_uc = (unsigned char)temp_us;

Natürlich funktioniert das. Da das aber nichts weiter macht, als mit 
sich selbst beschäftigt zu sein, wird es wegoptimiert. Daran ändert auch 
das unsinnige Casten nichts.

dost0011 schrieb:
> In meinem Programm ist das natürlich etwas komplizierter und die 78
> können berechnete Werte von 0-255 sein.
Das ist vollkommen egal. Entscheidend ist, was danach damit passiert.
Mach die erste Variable global und volatile, dsnn siehst du auch, ob 
deine Funktion das richtige zurück gibt. Die zweite Variable ist ja 
ohnehin nur zum Testen der ersten.

Simulator schrieb:
> Wenn er simuliert, muss er die Optimierung ausschalten. Das ist
> Grundvoraussetzung.
Damit geht man zwar auf Nummer sicher. Aber ein Muss ist das nicht.

mfg.

dost0011 schrieb:

> In meinem Programm ist das natürlich etwas komplizierter und die 78
> können berechnete Werte von 0-255 sein.

Ich würde mich sogar soweit aus dem Fenster lehnen, um zu behaupten das 
das Problem in der Berechnung liegt.

> Was mache ich falsch?
Du postest irgendeinen Code, der nichts mit deinem Problem zu tun hat, 
anstelle deines richtigen Codes.

Peter II schrieb:
> wie kommst du auf 222?
>
>> unsigned short temp_us = (unsigned short)((21UL * (unsigned long)v_soll)
>> / (unsigned short)v_max)  + 1300U;
>
> (21 * 1000 / 85 ) + 1300 = 1547

v_soll ist 10000 nicht 1000. Du hast eine 0 zu wenig.

Seine Zwischenergbenisse decken sich mit dem was er schreibt. Ich kann 
auch keinen Overflow in den Berechnungen sehen. Ist m.A. nach alles OK.

Ich hab keinen AVR-gcc auf meinem jetzigen Computer, sonst würd ich das 
mal laufen lassen und mir den Assembler Code ansehenm, wo die 0 als 
Ergebnis her kommt. Meiner Meinung nach ist der Code in Ordnung.

Die Erklärung des Problems ist gaz einfach: es gibt nämlich zwei 
Variable namens temp_us. Eine besitzt als Gültigkeitsbereich das Innere 
des else-Zweiges (Zeile 40-52) und die andere Variable im Rest von 
Calc_Ua() gültig.

Dies kann man auch sehr schön beim Betrachten von main.lss erkennen.

Andreas Schweigstill schrieb:
> Die Erklärung des Problems ist gaz einfach: es gibt nämlich zwei
> Variable namens temp_us. Eine besitzt als Gültigkeitsbereich das Innere
> des else-Zweiges (Zeile 40-52) und die andere Variable im Rest von
> Calc_Ua() gültig.

OMG.
Jep. Das waren Tomaten auf den Augen.

Und wirf die Casts wieder raus. Du verschleierst mit den ganzen Casts 
viel zu viel.

zb
  21UL * (unsigned long)v_soll

hier braucht es keinen Cast. 21 ist schon ein unsigned long. Damit ist 
klar, dass v_soll automatisch zumindest auf unsigned long 'angehoben' 
wird, wenn es nicht selbst schon unsigned long ist.

(Ich kann mir vorstellen, dass du die Casts auf der Suche nach dem 
Fehler eingebaut hast, widersteh aber der Versuchung sie drinnen zu 
lassen. Bei Casts sollte man noch der Devise vorgehen: Casts sind 
Waffen! Sie werden nur dann benutzt, wenn es nicht anders geht.