Konwerter binarny w c++ (błąd)

Question

Witam, czy jest ktoś w stanie powiedzieć mi, dlaczego mój kod nie działa poprawnie?

Jest to konwerter z języka dziesiętnego na język dwójkowy (binarny). Jednym ze sposobu konwersji liczby jest skorzystanie z reszty z dzielenia przez 2. Czyli dajmy na to mamy liczbę 23 i dzielimy ją na 2.

23/2 = 11 + r1

11/2=5 + r1

5/2=2 + r1

2/2=1 + r0

1/2=0 + r1

Następnie czytamy od tyłu- a więc 23(10) = 10111(2)

W ten sam sposób napisałem program- ilość bitów się zgadza, ale wszystkie z nich wyświetlają się jako zera.

#include <iostream>

using namespace std;
int tab[50],y,z,i;

int bin (int x)
{
    while (x!=0) 
    {
        y=x%2; //reszta z modulo (0 lub 1)
        tab[i]=y; //zapisanie wyniku dzielenia modulo do tablicy
        x/=2; //podzielenie x (czyli liczby podanej do programu w celu konwersji) przez 2
        z++; //licznik petli potrzebny do odczytania danych z tablicy
        i++; //inkrementacja i potrzebna do poprawnego zapisania (0/1) do tablicy
    }

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

int main()
{
    int num;
    cin>>num;
    bin(num);
    return 0;
}

Pozdrawiam.

Answer 1

Pętla wypisująca powinna być taka:

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

Powineneś zaczynać od elementu z - 1, a pętla powinna trwać dopóki j jest większe lub równe 0 (indeks pierwszego elemntu). Zmienna i chyba nie jest potrzebna.

Comments

Zmiennej i używałem przy zapisywaniu reszty do tablicy, ale teraz widzę, że można skorzystać z z.

Drugą pętlę najpierw robiłem pod i, potem przepisywałem pod j  i zapomniałem o cout'cie.

Dzięki.

Answer 2

chyba powineneś zastosować typ zmienno przecinkowy zamiast całkowitch

Comments

Int jest tutaj nieodzowny. Nie da się inaczej obliczyć reszty z dzielenia.

---

tak masz rację za szybko chciałem :)