funkcja wyswietlajaca liczby na lcd /gcc/ rozwiazane

Question

write_text(123); elegancko mi wyswietla te chary... chcialem zrobic funkcje wyswietlają dowolne liczby calkowite int: cos takiego dziala mi przez pewien czas,po czy pojawiaja sie krzaki... nie moge dopatrzec sie bledu....

piti___ · Answer

Witam

Te linie są błedne:
znaki=znaki+ile+1;
*znaki=0;

w pierwszej ustalasz wskaznik na adres rowny "znaki+ile+1" czyli nie wiadomo co. W następnej lini zerujesz w pamieci komórke o tym adresie...


void write_int(int liczba){

char znaki[10],i; 

i=0;

    while(1){ 
             
             znaki[i++]==(liczba%10) +48; 
             liczba=liczba/10; 
             if (liczba==0)
             {
              znaki[i] = 0;
              break; 
             }
    } 
write_text(znaki); 
}

Jesli chciales to zrobic na wskaznikach to musialbys zaalokowac pamiec (malloc nie wszystkie kompilatory uC 8bitowych posiadaj) o jakiejs dlugosci i ustawic na ten obszar wskaznik. Lub wskaznik ustawiac na jakas znana Ci, zadeklarowana wczesniej tablice.

Ps nie wiem czy kod jest poprawnie napisany jezeli chodzi o konwersje... dlaczego nie skorzystac z itoa (stdlib) ?

dturczak · Answer

0 na koncu daje ,bo tym sie charakteryzuje koncowka kazdego stringu.
nie korzystam z wbudowanej funkcji bo jest pamieciozerna

znaki=znaki+ile+1; czy obszar w pamieci+dlugosc cyfry + miejsce na NULL
no to w takim razie :


//wyświetlanie liczby na lcd
void write_int(int liczba,int uzupelnienie){
int ile;
int pom=liczba;
char cyfra;
for(ile=1;(liczba=liczba/10);ile++);
char wyraz;
char *znaki=&wyraz;
znaki=znaki+ile+1;
*znaki=0;
    while(1){
             znaki--;
             cyfra=pom%10;
             *znaki=cyfra+48;
             pom=pom/10;
             if (pom==0) break;
    }

int zera,l;
zera=uzupelnienie-ile;
if(zera>0){
	for(l=1;l<=zera;l++) write_char(48);
	  }
write_text(znaki);
}

zalatwia sprawe?

chyba tak,bo dziala

dodalem zaokraglanie pozycji zerami...
czyli np. write_int(liczba,2); w przypadki liczby <10 da nam na lcd 01,02...itd...
pozdrawiam

piti___ · Answer

Musisz zarezerwowac miejsce na kilka znakow nie na jeden.

char wyraz;
char *znaki=&wyraz;
znaki=znaki+ile+1;

Kompilator wg kodu rezerwuje miejsce na jeden znak (char wyraz;). Ustawiles wskaznik na to miejsce i go przesunales. Obok tego miejsca mozesz miec inne zadeklarowane przez Ciebie zmienne ktore nadpiszesz. Musisz zadeklarowac tablice znakow. Przyjmij sobie ze bedziesz wyswietlal liczby z przedzialu od -2^16 do 2^16 (lub innego) i zadeklaruj tablice znakow
char wyraz[6];

Mad Bekon · Answer

możesz też skorzystać z gotowej funkcji itoa()

dturczak · Answer

no to moze zamiast char wyraz; char *znaki=&wyraz; to: char *znaki=new char ;

pgp · Answer

a czemu nie wykorzystasz printf w libc stdio.h ?? az taki pamięcio żerny?

markosik20 · Answer

Oj, jest pamięciożerny . Zresztą sam autor napisał

funkcja wyswietlajaca liczby na lcd /gcc/ rozwiazane

Post #1

Post #2

Post #3

Post #4

Post #5

Post #6

Post #7

Post #8

Podsumowanie tematu