Problem z przelicznikiem z binarnego na dziesiętny

Question

#include <iostream>
#include <cmath>

using namespace std;

void bin_to_dec(int dlugosc, string liczba) ;

int main()
{
    string liczba ;
    cout << "Podaj liczbe w binarnym: " ;
    cin >> liczba ;

    int dlugosc ;
    dlugosc = liczba.length() ;
    cout << "Dlugosc: " << dlugosc << endl ;

    bin_to_dec(dlugosc, liczba) ;
    return 0;
}

void bin_to_dec(int dlugosc,string liczba)
{
    int wynik = 0, j = 0;
    for (int i = (dlugosc-1) ; i >= 0 ; i-- , j++)
    {
        wynik += pow (liczba[i] , j) ;
    }

    cout << "Twoja liczba to: " << wynik ;
}

 

Oto co stworzyłem, nie bardzo wiem co jest źle :/

Answer 1

Pierwsza sprawa:
Jeśli chcesz robić jakiś użytek (obliczenia) z liczb wprowadzanych z klawiatury i zapisywanych do zmiennej typu string musisz najpierw dokonać ich konwersji na typ do przechowywania liczb, np int. Przykładowo wprowadzona w Twoim programie liczba z klawiatury, np. 1111 to w pamięci komputera znaki ASCII liczby 1, czyli 31h. Używając tego "na żywca" do obliczeń, działa się na liczbie 31h a 1.  Trzeba o tym pamiętać.
Druga sprawa:
Przy konwersji BIN na DEC, do kolejnych potęg podnosisz liczbę 2 (w końcu to system binarny), a nie 1 lub 0.
http://www.edu.godula.com/?zo=sl_bintodec

Answer 2

Poniżej poprawiony kod.

void bin_to_dec(int dlugosc, string liczba)
{
	int wynik = 0, j = 0;
	for (int i = (dlugosc - 1); i >= 0; i--, j++)
	{
		wynik += liczba[i] == '1' ? pow(2, j) : 0;
	}

	cout << "Twoja liczba to: " << wynik;
}

 

Comments

Dziękuję bardzo ^^