Int Berechnungen

[Markus P. (sebastianwurst)]

Hallo,
warum wird diese Berechnung falsch gemacht?

int iStep_Puls_Pause=0;
uint16_t iZeit_Puls_Pause=0;
uint16_t iZeit_Puls=0; 
uint16_t iZeit_Pause=0;     
uint16_t iSollwert=0;

int main(void)
{

while(1)
{
    

float fmemmSollwert=0;
    
fmemmSollwert= (100-((215/250)*100))+0,5; 
iSollwert=fmemmSollwert;
    
switch (iStep_Puls_Pause)
{

//Puls Pausenzeit berechnen
case(1):
iZeit_Puls_Pause = 100;
iZeit_Puls = ((iZeit_Puls_Pause/100)*iSollwert);
iZeit_Pause = iZeit_Puls_Pause - iZeit_Puls;
          
usart_write_int0(iZeit_Puls_Pause);
usart_write_int0(iZeit_Puls);
usart_write_int0(iSollwert);
usart_write_int0(iZeit_Pause);

iStep_Puls_Pause = 2;

Ergebnis auf USART:

100
100
100
0


iSollwert müsste doch 14 sein...

(100-((215/250)*100))+0,5 = 14.5

Nach int dann -> abrunden/schneiden = 14.

[goldeneyes1987 (Gast)]

Markus P. schrieb:
> Hallo,
> warum wird diese Berechnung falsch gemacht?

Ich vermute mal  das der Fehler hier liegt:

  fmemmSollwert= (100-((215/250)*100))+0,5; 

 //Probier mal; 
  fmemmSollwert= (100.0f-((215.0f/250.0f)*100.0f))+0.5f;

Gruß
GoldenEyes

[Krapao (Gast)]

215/250 = 0

[Markus P. (sebastianwurst)]

Krapao schrieb:
> 215/250 = 0

Eigentlich doch 0,85 deswegen ja auch erst float...

fmemmSollwert= (100.0f-((215.0f/250.0f)*100.0f))+0.5f;

das hat aber geholfen, weiss jetzt nur noch nicht ob es vielleicht schon 
b , an , und . in 0.5 gelgen haben kann, das probier ich grad nochmal 
aus...

[Krapao (Gast)]

> float fmemmSollwert=0;
> fmemmSollwert= (100-((215/250)*100))+0,5;

Das wird - so lange nix Gegenteiliges angegeben (=> f Suffix bei 
funktionierender Lösung) - hauptsächlich in int  berechnet:

215/250 = 0
0*100 = 0
100-0 = 100
(float) 100 = 100.0
100.0 + 0.5 = 100.5
fmemmSollwert = 100.5

[Krapao (Gast)]

Rechnen mit Ganzzahlen:

Solche Ausdrücke (215/250)*100 = 85 rechnet man besser so (215*100)/250 
= 85. Wobei man im Einzelfall aufpassen muss, dass durch die 
Multiplikation kein Überlauf passiert.

[goldeneyes1987 (Gast)]

Markus P. schrieb:
> weiss jetzt nur noch nicht ob es vielleicht schon
> b , an , und . in 0.5 gelgen haben kann, das probier ich grad nochmal
> aus...

also du kannst davon ausgehen dass es allein durch "215.0f/250.0f" 
funktionieren müsste.
Ob da 0.5 oder 0,5 steht , sollte hier keinen unterschied machen!
Noch nen Frage:

iZeit_Puls_Pause = 100;
iZeit_Puls = ((iZeit_Puls_Pause/100)*iSollwert);

Was soll das bringen?
das ergebnis ist doch immer:

iZeit_Puls = iSollwert;

[Karl H. (kbuchegg)]

Markus P. schrieb:
> Krapao schrieb:
>> 215/250 = 0
>
> Eigentlich doch 0,85 deswegen ja auch erst float...

Nö.

Eigentlich 215/250 -> 0

215 ist ein int
250 ist ein int

Also wird int durch int dividiert.

Dass deine Variable auf der linken Seite der Zuweisung ein float ist, 
interessiert keinen.

http://www.mikrocontroller.net/articles/FAQ#Datentypen_in_Operationen

> fmemmSollwert= (100.0f-((215.0f/250.0f)*100.0f))+0.5f;
>
> das hat aber geholfen,

Ist aber Unsinn. Float Rechnen ist um mindestens den Faktor 30 langsamer 
als int rechnen. Formel ein wenig umstellen und schon gibts keine 
Probleme mehr.

[Stefan E. (sternst)]

goldeneyes1987 schrieb:
> Ob da 0.5 oder 0,5 steht , sollte hier keinen unterschied machen!

Das macht natürlich einen Riesenunterschied.
C-Buch raus holen, und Komma-Operator nachschlagen.

[Stefan E. (sternst)]

Nachtrag: Wobei ich mich aber frage, wieso im Original-Code 100 raus 
kommt, und nicht 5. Ist das wieder eine Frage der Tokenaufteilung wegen 
dem nicht vorhandenen Leerzeichen zwischen dem + und der 0 (hatten wir 
da kürzlich nicht etwas ähnliches)?

[apr (Gast)]

Na = bindet halt stärker :)
Ergo wird Ausgewertet (und ich hoffe mich hier nicht zu vertun):
1) 215/250
2) 0 * 100
3) 100 - 0
4) 100 + 0
5) fmemmSollwert = 100 (+ Cast)
6) 5
Der Ausdruck ergibt also 5 -- aber die Zuweisung 100.0.

[Stefan E. (sternst)]

apr schrieb:
> Na = bindet halt stärker :)

Sch..., stimmt natürlich (peinlich) ;-)