Co jest nie tak w tym kodzie ?

Question

Witam !
Pisząc ten program chciałem, aby zamieniał on liczby dziesiętne na szesnastkowe (tak, wiem, że jest prostszy sposób). Niestety nie mogę zlokalizować co jest z nim nie tak :( Program normalnie się kompiluje, ale konsola zwraca błąd.

/* ------------------------------------------------------------------------------------------------ */
 

#include <cstdlib>
#include <iostream>
using namespace std;

void dec_to_hex(int liczba)
{
  int i=0,tab[31];
  char *tab1;
  while(liczba) //dopóki liczba bêdzie ró¿na od zera
  {
    if(liczba%16<10) tab1[i++]=48+(liczba%16);
    else if(liczba%16==10) tab1[i++]='A';
    else if(liczba%16==11) tab1[i++]='B';
    else if(liczba%16==12) tab1[i++]='C';
    else if(liczba%16==13) tab1[i++]='D';
    else if(liczba%16==14) tab1[i++]='E';
    else if(liczba%16==15) tab1[i++]='F';
    liczba/=16;
  }

  for(int j=i-1;j>=0;j--)
    cout<<tab[j];
}

int main()
{
  int liczba;

  cout<<"Podaj liczbe: ";
  cin>>liczba;

  cout<<liczba<<" po zamianie na postac szesnastkowe: ";
  dec_to_hex(liczba);
  cout<<endl;

  return 0;
}

 

Answer 1

tab1 -> tab przy warunkach = powinno śmigać.

Comments

Niestety nie działa. Nie mogę zamienić tab1 na tab w void przy ifie, ponieważ nie zapiszę do tablicy ( tab[] ) znaku.

---

No to zmień typ tablicy tab[] z int na char, u mnie takie coś działa:

#include <cstdlib>
#include <iostream>
using namespace std;

void dec_to_hex(int liczba)
{
  int i=0;
  char tab[31];
  while(liczba) //dopóki liczba bêdzie ró¿na od zera
  {
    if(liczba%16<10) tab[i++]=48+(liczba%16);
    else if(liczba%16==10) tab[i++]='A';
    else if(liczba%16==11) tab[i++]='B';
    else if(liczba%16==12) tab[i++]='C';
    else if(liczba%16==13) tab[i++]='D';
    else if(liczba%16==14) tab[i++]='E';
    else if(liczba%16==15) tab[i++]='F';
    liczba/=16;
  }

  for(int j=i-1;j>=0;j--)
    cout<<tab[j];
}

int main()
{
  int liczba;

  cout<<"Podaj liczbe: ";
  cin>>liczba;

  cout<<liczba<<" po zamianie na postac szesnastkowe: ";
  dec_to_hex(liczba);
  cout<<endl;

  return 0;
}

 

Answer 2

#include <iostream>

using namespace std;

int main()
{
    cout << "Podaj liczbe" << endl;
    int x;
    cin >> x;
    cout<<endl<<(int*)x;
    return 0;
}

Tak jest trochę prościej

Comments

Faktycznie, ale znam już tą metodę.  Chciałem po prostu napisać ten kod tak od zera, ale dzięki za odp 

---

Ok, tylko pamiętaj, że to co dzieje się w komputerze po uruchomieniu mojego kodu i twojego to dwie różne rzeczy.
Rzutowanie typu jest procesem mniej złożonym niż sprawdzanie tylu if - else if - else.

Oczywiście moje rozwiązanie nie można użyć zawsze, ale do takich prostych celów jak sprawdzenie jak ta liczba jest w systemie hexadec. jest łatwiejsze

---

Zgadzam się z tym co napisałeś, do tak prostej rzeczy jak zamiana liczby i zobaczenie jak ona wygląda w szesnastkowym Twój jest lepszy