Multiplikation falsches Ergebnis

Ronny S. OP #1147357 
Hallo,

ich habe zum Umrechnen eines AD-Wertes eine Gleichung nach dem Dreisatz 
erstellt. Doch GCC spuckt das falsche Ergebnis aus.
1
unsigned int adc_wert;

2
unsigned int cache;

3
char buffer[10];

4


5
int main (void)

6
{

7
   adc_wert = 1000;

8
   cache = (adc_wert * 6006)/2048;

9
   itoa (cache, buffer, 10);

10
}


wenn ich aber mit 65536 als Divisor rechne stimmt das Ergbnis wieder.
1
unsigned int adc_wert;

2
unsigned int cache;

3
char buffer[10];

4


5
int main (void)

6
{

7
   adc_wert = 1000;

8
   cache = (adc_wert * 196608)/65536;

9
   itoa (cache, buffer, 10);

10
}

wo liegt bei mir hier der Fehler.
Danke
Ronny

Und ja ich habe die Forumssuche benutzt, aber nichts dazu gefunden. Und 
ich habe gerade mein C-Buch nicht zur Hand, ist in der Wäsche. :-)
Antwort Zitieren Bearbeiten Thread verschieben Thread sperren Langsamer Modus Anmeldepflicht aktivieren Anpinnen Thread löschen Thread mit anderem zusammenführen Melden Danke
Ronny S. OP #1147466 
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
Antwort Zitieren Bearbeiten Löschen Melden Danke
Jörg G. #1147481 
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.
Antwort Zitieren Bearbeiten Löschen Melden Danke
Ronny S. OP #1147501 
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
Antwort Zitieren Bearbeiten Löschen Melden Danke
Jörg G. #1147531 
>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
Antwort Zitieren Bearbeiten Löschen Melden Danke
yalu Gast #1147555 
Wenn Zeit- und Flash-Verbrauch eine Rolle spielen:

1
   cache = (adc_wert * 47)/16;


Das geht in 16 Bit und ist nur 0,17% falsch. Der ADC ist meist auch
nicht genauer.
Antwort Zitieren Bearbeiten Löschen Melden Danke
Thread beobachten

Antwort schreiben

Eine eigene Frage? Dieser Thread ist über 17 Jahre alt. Stell sie in einem eigenen Thread – dort lesen mehr Leute mit, und sie wird öfter und schneller beantwortet.

Neuen Thread starten

Eine Lösung, Ergänzung oder Rückmeldung zu diesem Thread? Dann antworte hier.

Formatierung
  • [c]C-Code[/c]
  • [code]Code in anderen Sprachen, ASCII-Zeichnungen[/code]
  • [math]Formel in LaTeX-Syntax[/math]
  • [[Titel]] – Link zu einem Artikel
  • Verweis auf einen anderen Beitrag: Rechtsklick auf dessen Beitrags-ID (#8078628), „Adresse kopieren“, in den Text einfügen

Mehr zur Formatierung

Längeren Sourcecode als Anhang, nicht in den Text. JPG nur für Fotos und Scans; Zeichnungen und Screenshots als PNG oder GIF. Siehe Bildformate.

Dein Name und deine E-Mail-Adresse

Schon registriert? Anmelden

Wird nicht angezeigt. Wir schicken dir einen Link; sobald du damit bestätigst, erscheint dein Beitrag.

Mit dem Abschicken legst du einen kostenlosen Account an und erkennst die Nutzungsbedingungen und die Datenschutzerklärung an.