7.8 Festkomma in String umwandeln

[michael (Gast)]

Hallo,
um eine vorzeichenbehaftete 7.8 Festkommazahl auf einem avr in einen 
String umzuwandeln, habe ich folgende Funktion gebastelt.
Ich würde mich über konstruktive Kritik an der Funktion freuen.

Gibt es noch eine einfachere Möglichkeit, die Umwandlung vorzunehmen?

Viele Grüße
Michael

#include <avr/io.h>
#include <string.h>
#include <stdlib.h>

void Q7p8toString(char * str, uint16_t number, uint8_t nachkommastellen) {

  str[0] = 0;
  
  if (!(number & 0x8000)) {
    // positiv    
  } else {
    // negativ
    number = (~number) + 1;
        
    // Minus einfügen
    strcat(str, "-");
  }
  
  // Vorokmmaanteil
  volatile uint16_t vorkomma = number >> 8;
  char hilf [4];
  itoa(vorkomma,hilf,10);
  strcat(str,hilf);
  
  // Vorkommaanteil löschen
  number = number & 0x00FF;
  
  // Komma
  uint8_t strpos = 0;
  strpos = strlen(str);
  str[strpos] = ',';
  strpos++;
  str[strpos] = 0;

  
  // Nachkommaanteil
  for (int i=0; i<nachkommastellen; i++) {
    number = number * 10;
    uint8_t stelle = number >> 8;
    
    // Stelle anhängen
    str[strpos] = 0x30+stelle;
    strpos++;
    str[strpos] = 0;
    
    number = number &0x00FF;
  }
}

[isnah (Gast)]

Diese Idee hatte vor 10 Jahren schon jemand anderes:
Beitrag "float to string"
isnah

[B. S. (bestucki)]

@isnah: Er will Fixkommazahlen umwandeln, nicht Fliesskommazahlen.

Schau mal hier:
Beitrag "Re: [C] Integer in String umwandeln"
Entweder meine Lösung, oder die im nachfolgenden Beitrag verlinkte. Evt. 
kannst du davon was weiterverwenden.

[Detlev T. (detlevt)]

Der Teil mit den Nachkommastellen ist vermutlich nicht das, was du 
erreichen willst. 0xFF sollte ja wohl auf ,996094... abbilden.

Wenn genug Flash-Speicher zur Verfügung steht, würde ich wohl auf 
(s)printf zurückgreifen. Ansonsten muss man den Wert mit so etwas wie 
9961 multiplizieren und durch 255 teilen und dann erst in Zeichen 
umwandeln. Mit 16-Bit kommt man da aber nicht mehr hin.

[Peter II (Gast)]

michael schrieb:
> if (!(number & 0x8000)) {
>     // positiv
>   } else {
>     // negativ
>     number = (~number) + 1;
>
>     // Minus einfügen
>     strcat(str, "-");
>   }

warum nicht lesbar schreiben?

if ( number < 0 ) ...


>   // Vorokmmaanteil
>   volatile uint16_t vorkomma = number >> 8;
was soll das volatile hier?

> str[strpos] = 0x30+stelle;
auch das kann man lesbar schreiben

str[strpos] = '0' + stelle;

[W.S. (Gast)]

michael schrieb:
> Gibt es noch eine einfachere Möglichkeit, die Umwandlung vorzunehmen?

Ja.
Vermutlich ist fast alles andere einfacher und effektiver als dein Code.
Ich hatte vor geraumer Zeit mal eine meiner Libs für e/a-Konvertierungen 
gepostet, mußt mal suchen (oder conv.c aus der Lernbetty nehmen). Für 
den ganzen Teil kannst du dort fündig werden.

Für den gebrochenen Teil geht das anders:
1.  zuerst den ganzen Teil wegschneiden und ausgeben
1a. nen Punkt ausgeben
2.  dann Zahl * 10 nehmen
3.  ganzen Teil als Ziffer ausgeben und dann wegschneiden
Punkt 3 so oft wiederholen, wie du magst.

W.S.