Dotaz: [C] program na prevod dec -> hex

[martin()]

Dobry den, snazim sa napisat program, ktory ako vstup zoberie integer a na obrazovku vypise hex. Zatial som vyprodukoval toto:

#include <stdio.h>

char *string_hexnum(int number)
{
	char mask[] = "0123456789ABCDEF";
	char str[12];
	int i = 10;

	str[11] = '\0';

	if (number < 0){
		puts("ERROR: number is negative\n");
		return 0;
	}

	if (number == 0){
		str[i--] = '0';
	} else {
		while (number != 0){
			str[i--] = mask[number % 16];
			number /= 16;
		}
	}
	str[i--] = 'x';
	str[i] = '0';

	return str+i;
}


void print_hexnum(int number)
{
	char* pnt;
	puts("test");
	pnt = string_hexnum(number);
	puts(pnt);
	puts("test");
}


int main(void)
{
	print_hexnum(888);

	return 0;
}

Problemom je, ze sa to cislo nevypise, zostane prazdny riadok. Nechcem na to pouzit ziadnu funkciu z kniznice. Ked som to debuggoval, tak som videl, ze do pamete sa vsetko uklada ako ma. pnt ukazuje na prvy znak retazca - '0'.

Dakujem.

---

---

Odpovědi

[martin()]

Chyba je v tom, ze vracite odkaz na lokalni promennou str. Bud pouzijte malloc/free, anebo zmente deklaraci char str[12] na static char str[12]. Pak to ovsem nebude fungovat v multithread programech, ale to, tusim, ani nechcete :)

[martin()]

return str+i;

Nemůžete vracet ukazatel na lokální proměnnou - jakékoliv další volání funkce zásobník přepíše. V lepším případě to nebude fungovat, v horším případě to upadne.

[martin()]

Dakujem za reakcie, zas som sa naucil nieco nove

Eclipse mi vyhodil warning, ze vraciam adresu lokalnej premennej, ale uspesne som to ignoroval.

Este raz dakujem.

[Pavel Stárek]

To byla nějaká školní úloha se zákazem používat knihovní funkce a funkce pro formátování výstupu? Mě z toho zadání vyšlo toto:

#include <stdio.h>
#include <string.h>

int main(int argc, char *argv[]) {
  unsigned long iCislo;
  
  if (argc > 1) {
    iCislo = atoi(argv[1]);
    printf("Cislo %s je hexa: 0x%X\n", argv[1], iCislo);
    return(0);
  }
  printf("Malo parametru!\n");
  return(1);
} 

[martin()]

Nie. Povedal som si, ze naprogramujem funkciu, ktora bude vypisovat cisla v hex. So skolou to nemalo nic spolocne.

[l4m4]

Rozhodl-li ses takovou funkci naprogramovat, tak ji naprogramuj a nenech ji za sebe programovat jiné.

Pokud takovou funkci pouze potřebuješ, tak bych vzhledem k tomu, že vstup decimálních čísel a výpis hexadecimálních jsou operace, u kterých lze čekat, že už je někdo naprogramoval před dvaceti lety a dal do standardní knihovny, čekal spíš otázku na to, kterou knihovní funkci k tomu použít než jak to naprogramovat.

[dark]

Sak ji naprogramoval a nefungovala mu, tak se zeptal, nic spatneho na tom nevidim.

[Ash]

Však l4m4 jen shrnul a potvrdil co bylo už v zásadě řečeno.

[dark]

Nevim, jestli je 2x rozumne pouzit %X na 'unsigned long'.

[l4m4]

Když už používáš ten long jako typ dat, tak používej odpovídající funkce...

printf("%lx\n", atol(argv[1]));
         ^         ^

A return není funkce.

[Pavel Stárek]

Nojo, nojo, to byl jen takový rychlý pokus. Spíš pro ukázání, že takovéto funkce není třeba programovat, neboli netřeba vynalézat kolo. Já céčko moc nepoužívám, já raději Python.