Modulo macht - 1

[Christian M. (krusti)]

Hallo,
ich will mit Hilfe des Modulo-Operators eine Integer-Zahl in eine 
Dezimalzahl "wandeln" (Format xx.y) und über UART bzw LCD ausgeben. Mein 
Wert bevor ich ihn weiter verarbeite hat z.B. 316 nach dem Modulo ist 
der Wert 31.5 Woran liegt das?

Hier kommt noch der Code dazu, ist derzeit noch alles sehr experimentell 
gehalten.

        while (1) 
    {  
    adcValue = ADC_Read(ADC_CHANNEL);
    
    //////////////////////////////////////////////////////////////////////////
    //ADC-Wert ausgeben
    uart_puts("ADC-Wert: ");
    uart_puts(itoa (adcValue, s, 10));
    uart_putc('\0');    
    lcd_clear();
        
    lcd_setcursor(1,1);
    lcd_string("ADC-Wert: ");
    lcd_string(s);
    
    //////////////////////////////////////////////////////////////////////////
    //Druckwert ausgeben (noch nicht Dezimal)
    uart_puts(itoa (convert_ADC(adcValue), s, 10));
    uart_putc('\0');
    //Ergebnis => 316
    
    //////////////////////////////////////////////////////////////////////////
    //Dezimaler Druckwert
    uart_puts("Druck: ");
    uart_puts(itoa (convert_ADC(adcValue/10), s, 10));
    uart_puts(".");
    uart_puts(itoa (convert_ADC(adcValue%10), t, 10));
    uart_putc('\0');
    //Ergebnis => 31.5
    
    lcd_setcursor(2,1);
    lcd_string("Druck : ");
    lcd_string(s);
    lcd_string(".");
    lcd_string(t);
    lcd_string(" bar");
    //Ergebnis => 31.5
    
    uart_putc('\0');
    _delay_ms(1000);
  }

Ausgabe in hterm
ADC-Wert: 305
316
Druck: 31.5

Grüße

[Peter II (Gast)]

welchen datentype hat adcValue

[Christian M. (krusti)]

uint_16t

[Nop (Gast)]

Zeig doch bitte außerdem mal den Code von convert_ADC(). Denn das 
scheint ja irgendwie aus 305 eine 316 zu machen. Schlimmstenfalls ist 
das ja auch noch als Makro definiert, mit überraschenden Effekten.

[Christian M. (krusti)]

Nop schrieb:
> Zeig doch bitte außerdem mal den Code von convert_ADC(). Denn das
> scheint ja irgendwie aus 305 eine 316 zu machen. Schlimmstenfalls ist
> das ja auch noch als Makro definiert, mit überraschenden Effekten.

Die Wandlung von 305 zu 316 passt! Das ist nicht das Problem. Das ist in 
dem Bereich Zufall, dass es so nahe beieinander liegt.
Aber nicht die Wandlung von 316 in 31.5 => müsste 31.6 heißen eigentlich

[Nop (Gast)]

Christian M. schrieb:
> Die Wandlung von 305 zu 316 passt!

Darum ging es nicht. Es ist eine Funktion, die etwas tut, und wenn die 
als Makro implementiert ist, dann kann das schon überraschende Effekte 
geben.

[Peter II (Gast)]

ich glaube das Problem liegt bei convert_ADC(adcValue), du machst das 
Modulo vorher, das ist doch aber Unsinn.

Warum auch ständig convert_ADC aufrufen, das macht man einmal nach der 
Wandlung.

[Christian M. (krusti)]

Peter II schrieb:
> ich glaube das Problem liegt bei convert_ADC(adcValue), du machst das
> Modulo vorher, das ist doch aber Unsinn.
>
> Warum auch ständig convert_ADC aufrufen, das macht man einmal nach der
> Wandlung.

Ah klar, dass könnte es sein. Ein Rundungsfehler quasi in meiner 
convert_ADC. Das habe ich nicht bedacht, aber könnte passen. Probiere 
ich aus.

Wie gesagt ist im Moment noch experimentell der ganze Code. Ich konnte 
schnell testen ob das richtige raus kommt in dem ich einfach Copy&Paste 
gemacht habe. Da es grundsätzlich geht, werde ich ständige Konvertierung 
natürlich jetzt raus machen. Dann wird man auch sehen ob meine Zahl 
wieder stimmt.

Danke allen schon mal

[Bastian W. (jackfrost)]

Dein Faktor von ADC auf Druck sollte ja ca. 1,036 sein.
Beim ersten Modulo übergibst du ja nicht 305 sondern 30 was dann 31.088 
sind und das ist 31. Beim zweitenmal übergibst du 5 und das ist nach der 
Multiplikation 5.18 also 5.

Mach es wie weiter oben beschrieben einmal wandeln und dann davon den 
Modulo.

Gruß JackFrost

[Christian M. (krusti)]

Jetzt klappt es auch mit der Nachkommastelle. Vielen Dank