Artikel Festkommaarithmetik

[Rick M. (rick00)]

Hallo!

Hab mich gerade durch den Artikel durchgearbeitet und mir ist folgender 
kleine "Fehler" aufgefallen:

Unter der Fkt. "my_round" steht folgendes:

"Die Funktion ist auch auf einen String anwendbar, welcher mit my_itoa() 
erzeugt wurde (also vorzeichenbehaftete Zahlen). Allerdings muss der 
Funktion ein Pointer auf das zweite Byte (char) übergeben werden, da im 
ersten das Vorzeichen gespeichert ist. Das geschieht am einfachsten so. 
"

my_round(my_string+1, 5);

Hierbei wird aber nicht berücksichtigt, daß

1.) sich der Index für die Rundung mit verschiebt.
es müßte folgendermaßen heißen:

my_round(my_string+1, 5-1);

2.) in der for-Schleife der Index für vorzeichenbehaftete Zahlen bis 10 
und nicht nur bis 9 laufen muß, da für Vorzeichenbehaftete Zahlen ein 
12Byte großes Feld benötigt wird.

Ich hab das volgendermaßen gelöst:

void round_32bit(char* string , uint8_t index_digit)
{
  
  uint8_t i , index_end;    
  
  
  if( (string[0] == '-') || (string[0] == ' ') )
  {                // vorzeichenbehaftete Zahl? (12Byte String)
    string+1;          // 1. Byte = sign -> Pointer auf 2tes Byte stellen
    index_digit-1;        // Index würde sich mitverschieben, daher korrektur
    index_end = 10;        // index_end für 12 Byte string
  }
  else
    index_end = 9;          // index_end für 11 Byte String
  
  
  if (string[index_digit] >= '5')      // muß aufgerundet werden ?
  {
    
    for (i= index_digit-1 ; i>=0 ; i--)  // einzelne Ziffern ansprechen
    {
      
      string[i] += 1;          // Ziffer aufrunden
      if (string[i] <= '9')      // wenn kein Übertrag Schleife verlassen
        break;
      
      else
        string[i] = '0';      // Übertrag + Überlauf
    }
    
  } 
  
  for (i=index_digit ; i<=index_end ; i++)    // gerundete Stellen auf Null setzen
    string[i] = '0';  
  
}

Gruß Rick