Multiplikation falsches Ergebnis

Rechne mit 32 Bit und nicht mit 16. Deine Zehlen passen nämlich nicht in 
16 Bits rein.

Johann

das Ergebnis der ersten Berechnung ist 0 und das 2. Ergebnis ist 3000 
(richtig). Ich habe auch schon die Variable cache auf unsigned long 
geändert und dann wieder auf int16 gecastet (für ITOA). Keine Änderung. 
Wenn es daran liegen soll, würde die 2. Gleichung auch nicht 
funktionieren. Das Ergebnis stimmt aber.

Ronny

Hast du es mal so probiert (was gjlayde sagte):

1

int main (void)

2

{

3

   adc_wert = 1000;

4

   cache = (adc_wert * 6006L)/2048;

5

//                         ^ so wird mit 32 Bit gerechnet

6

/* oder auch:

7

   cache = (adc_wert * 192192)/65536;

8

                       ^^^^^^ (ist NICHT die Zahl von dir)

9

*/

10

   itoa (cache, buffer, 10);

11

}

Der C Standard schreibt vor, dass im größten beteiligten Datentyp 
gerechnet wird - und nicht mit dem, für das Ergebnis, nötigen.

hth, Jörg
ps.: Gewöhne dir solche 'Magic Numbers' (unerklärte Zahlen) ab und 
schreib entweder die Gleichung hin oder benutz Macros.

Jörg G. wrote:
> Hast du es mal so probiert (was gjlayde sagte):
>

1

> int main (void)

2

> {

3

>    adc_wert = 1000;

4

>    cache = (adc_wert * 6006L)/2048;

5

> //                         ^ so wird mit 32 Bit gerechnet

6

> /* oder auch:

7

>    cache = (adc_wert * 192192)/65536;

8

>                        ^^^^^^ (ist NICHT die Zahl von dir)

9

> */

10

>    itoa (cache, buffer, 10);

11

> }

12

>

> Der C Standard schreibt vor, dass im größten /beteiligten Datentyp/
> gerechnet wird - und nicht mit dem, für das Ergebnis, nötigen.
>
> hth, Jörg
> ps.: Gewöhne dir solche 'Magic Numbers' (unerklärte Zahlen) ab und
> schreib entweder die Gleichung hin oder benutz Macros.

Vielen Dank, das war der richtige Hinweis. Was mich aber wundert ist die 
Tatsache das (adc_wert * 196608)/65536 auch funktioniert ohne dem "L"

Ronny

>Tatsache das (adc_wert * 196608)/65536 auch funktioniert ohne dem "L"
                          ^^^^^^
                          Das kann der Compiler nur als Long darstellen

deshalb rechnet er dann den rest in long

Gruss Helmi

>Was mich aber wundert ist die Tatsache das
>(adc_wert * 196608)/65536 auch funktioniert ohne dem "L"
Weil "196608" nicht in ein int passt und deshalb gleich als long 
gerechnet wird.
Evtl. erzeugt ein cast der Variablen besseren (kürzeren) Maschinen-Code, 
ich hab
aber keine Lust das jetzt auszuprobieren ;)

1

   adc_wert = 1000;

2

   cache = ((long)adc_wert * 6006)/2048;

Falls du dir also den Asm-Code anschaust (.lss/.lst-Datei), poste das 
mal bitte.

hth, Jörg

Danke Anwort reicht mir schon, war nur interessehalber warum die 2. 
Berechnung funktionert hat.

Das beweist wieder den Satz: Der "Dumme" sitzt immer VOR dem Rechner. 
;-)

Ronny