Variable in einem String

[Daniel P. (daniel90)]

Hallo,

ich habe ein kleines Problem an dem ich mir zurzeit die Zähne ausbeisse, 
und zwar möchte ich eine Variable in einen String einfügen und diesen 
dann mit Festkommaarithmetik in eine "Komma-Zahl" umwandeln, aber leider 
weis ich nicht wie ich das machen soll. Ich habe mir schon die Seite 
"http://www.mikrocontroller.net/articles/Festkommaarithmetik"; durch 
gelesen und es funktioniert auch soweit alles nur das mit der Variable 
geht nicht?!

Evtl habt Ihr eine Idee oder einen Denkanstoß für mich ?

void my_print_LCD(char* string, uint8_t start, uint8_t komma, uint8_t frac)
{
  uint8_t i;
  uint8_t flag=0;
  if (string[0]=='-')  lcd_data('-');  else lcd_data(' ');
  for (i=start; i<komma; i++)
  {
    if (flag==1 || string[1]!= '0')
    {
      lcd_data(string[i]);
      flag = 1;
    }
    else lcd_data(' ');
  }
  lcd_data('.');
  for (;i<(komma+frac); i++) lcd_data(string[i]);
}

char my_test[12]="1234\0";
  my_print_LCD(my_test, 0, 2, 2);

soweit funktioniert alles auf dem Display steht 12.34

nur möchte ich jetzt anstatt "1234" eine variable einsetzen. Habt ihr 
eine idee ?

[Georg G. (df2au)]

Sieh dir mal itoa() und Co an. Da du nichts näheres zu deiner Skalierung 
sagst, kann man schlecht ein Beispiel schreiben.

[Daniel P. (daniel90)]

Hey, danke für die Antwort mein Ziel ist es mit dem Adc einen Wert 
einlesen den dann umzurechnen und dann mit Komma zu versehen.  z.b 
Adc-wert 1023 dann umrechnen 1023*375/128 wäre 2997 möchte aber dann auf 
dem Display 29.97 stehen haben. Oder gibt es  da einen anderen Weg? Ich 
muss auch zugeben das ich noch am Anfang stehe aber bis jetzt  doch gut 
durch gekommen sind bin:-) Zur Zeit ändere ich noch die variable mit 
tasten da ich den adc noch erstmal aussen vor lasse.

[Rene H. (Gast)]

Hallo Daniel,

ich weiss nicht, ob ich Deine Frage richtig verstanden habe, aber so wie 
ich das sehe, reicht ja das umrechnen ganz normal. Dann müsstest Du 
einen Float bekommen. Diesen musst Du dann noch in einen String pappen, 
da bietet sich sprintf an.

Grüsse,
René

[Rene H. (Gast)]

Also so ähnlich:

    float wert;
    char wertStr[16];

    sprintf (wertStr, "%2.2f", wert);

Grüsse,
René

[Daniel P. (daniel90)]

Hallo René

leider klappt das mit dem float und dem sprintf nicht so richtig, auf 
dem Display steht dann leider nichts, ich habe mal meinen ganzen code 
angehangen ich würde schon gerne die Festkomma-Funktion nutzen.

#define F_CPU 16000000UL
#include <avr/io.h>
#include "lcd-routines.h"
#include "lcd-routines.c"
#include <util/delay.h>
#include <stdlib.h>
#include <inttypes.h>
#include <stdint.h>

uint32_t i=0;
uint32_t t=0;
uint32_t d;

void initial (void)
{
  DDRD=0xE0;
}

void my_uitoa(uint32_t zahl, char* string)
{
  int8_t i;
  
  string[10]='\0';
  for(i=9; i>=0; i--)
  {
    string[i]=(zahl % 10) +'0';
    zahl /= 10;
  }
}

void my_round(char* string, uint8_t digit)
{
  uint8_t i;
  
  if (string[digit]>='5')
  {
    for (i=(digit-1); i>=0; i--)
    {
      string[i] +=1;
      if (string[i]<='9')
        break;
         else  
           string[i]='0';
      
    }
  }
  for (i=digit; i<10; i++) string[i] ='0';
}
  

void my_print_LCD(char* string, uint8_t start, uint8_t komma, uint8_t frac)
{
  uint8_t i;
  uint8_t flag=0;
  if (string[0]=='-')  lcd_data('-');  else lcd_data(' ');
  for (i=start; i<komma; i++)
  {
    if (flag==1 || string[1]!= '0')
    {
      lcd_data(string[i]);
      flag = 1;
    }
    else lcd_data(' ');
  }
  lcd_data('.');
  for (;i<(komma+frac); i++) lcd_data(string[i]);
}

int main(void)
{
          lcd_init ();
          initial();
      
  while(1)
  {


  if (PIND & (1<<2))
  {
    i++;
    PORTD = (1<<5);
    _delay_ms(1000);
  }
  else
  {
    PORTD = 0x00;
  }
  
  if (PIND & (1<<3))
  {
    i= i+10;
    
    PORTD = (1<<6);
    _delay_ms(200);
  } 
  else
  {
    PORTD = 0x00;
  }
  if (PIND & (1<<4))
  {
    u++;
    PORTD = (1<<5);
    _delay_ms(100);
  } 
  else
  {
    PORTD = 0x00;
  }
  
  t = i * 375 / 128;
  d = t + 1000;

    
    lcd_setcursor( 0, 1 );
    
    int test = d;
    {
      char Buffer[20];
      itoa( test, Buffer, 10 );
      lcd_string ( Buffer );
    }
        
    lcd_setcursor ( 0, 2 ); 
    
    int variable = i;
    {
      char Buffer[20];
      itoa( variable, Buffer, 10 );
      lcd_string ( Buffer );
    }
    lcd_data ( 'a');
    lcd_data ( 'd');
    lcd_data ( 'd');
  
  lcd_setcursor(0,3);

char my_test[12]="2997\0";
my_print_LCD(my_test, 0, 2, 2);
  
  }
  return 0;
    
}

[Karl H. (kbuchegg)]

> Evtl habt Ihr eine Idee oder einen Denkanstoß für mich ?
>
> void my_print_LCD(char* string, uint8_t start, uint8_t komma, uint8_t frac)

Die Idee hinter diesem my_print_LCD ist es, so benutzt zu werden

char buf[8];
int  value = 1234;

  itoa( value, buf, 10 );
  my_print_LCD( buf, 0, 2, 2 );

d.h. das ist in erster Linie eine String-Ausgabe Funktion, die 
allerdings davon ausgeht, dass in diesem String Zahlendarstellungen 
enthalten sind. Ergo musst du dir aus deiner Zahl erst mal einen String 
machen. Und genau da kommt itoa ins Spiel.

Im übrigen hat diese Funktion keinerlei Vorkehrungen dafür, wenn die 
Formatierung nicht mit dem Text übereinstimmt. Um universell benutzbar 
zu sein, müsste man da also noch nachlegen.

   char buf[8];
   int value = 1234;

   sprintf( buf, "%2d.%02d", value / 100, value % 100 );
   lcd_string( buf );

ist einfacher, wenn du dir das bischen Mehrcode durch sprintf leisten 
kannst.



Edit:
Ich seh grad, du hast dir vom Original auch 2 Funktionen

void my_uitoa(uint32_t zahl, char* string)
void my_round(char* string, uint8_t digit)

mitkopiert.
Mit denen geht das. my_uitoa ist als Ersatz für itoa gedacht. Die sind 
zum Zusammenspiel mit deiner my_print_LCD gedacht.

[Lukas P. (lks)]

Daniel Pfeiffer schrieb:
> Hey, danke für die Antwort mein Ziel ist es mit dem Adc einen Wert
> einlesen den dann umzurechnen und dann mit Komma zu versehen.  z.b
> Adc-wert 1023 dann umrechnen 1023*375/128 wäre 2997 möchte aber dann auf
> dem Display 29.97 stehen haben. Oder gibt es  da einen anderen Weg? Ich
> muss auch zugeben das ich noch am Anfang stehe aber bis jetzt  doch gut
> durch gekommen sind bin:-) Zur Zeit ändere ich noch die variable mit
> tasten da ich den adc noch erstmal aussen vor lasse.

Hi,

also hast du den Wert als int/uint vorliegen und möchtest den quasi 
durch 100 Teilen?!

int16_t upper = adc_value/100; //links vom komma
uint16_t lower = adc_value%100; //rechts vom komma

also bei 2997 wäre upper = 29 und lower = 97. Dann brauchst du das nur 
noch ausgeben.

Gruß

[Karl H. (kbuchegg)]

Karl Heinz schrieb:

> Edit:
> Ich seh grad, du hast dir vom Original auch 2 Funktionen
>

> void my_uitoa(uint32_t zahl, char* string)
> void my_round(char* string, uint8_t digit)
>

>
> mitkopiert.
> Mit denen geht das. my_uitoa ist als Ersatz für itoa gedacht. Die sind
> zum Zusammenspiel mit deiner my_print_LCD gedacht.

Aber Achtung

void my_uitoa(uint32_t zahl, char* string)
{
  int8_t i;
  
  string[10]='\0';
  for(i=9; i>=0; i--)
  {
    string[i]=(zahl % 10) +'0';
    zahl /= 10;
  }
}

will ganz offensichtlich ein char Array der Länge 11 haben, um dort den 
String reinzuschreiben.

[Daniel P. (daniel90)]

Hallo wollte mich für eure Hilfe bedanken mit "sprint" hat alles super 
geklappt :) danke.

Gruß Daniel