zmiana systemu bin na dec w c++

Question

to testowy fragment kodu, w którym zmieniam rodzaj systemu 
 

#include <iostream>
#include <conio.h>
#include <cstdlib>
#include <cmath>

using namespace std;

int main() {
    long int coppyI = 0,coppy = 0; 
    int dlugosc = 0;
    string x;

    x = "100010111";

      coppyI = stoi(x);
      do {dlugosc++;} while(coppyI - pow(10, dlugosc) > 0);
      while (dlugosc > 0)
      {
        if(coppyI - pow(10, dlugosc) >= 0){
          coppyI -= pow(10, dlugosc);
          coppy += pow(2, dlugosc);
        }
          dlugosc --;
         cout << coppy << " | " << dlugosc << endl;
      }
    return 0;
}

z jakiegoś powodu nie wykrywa [1] za pierwszym powtórzeniem pętli  
więc zamiast 279 wychodzi 278

Comments

Strasznie skomplikowane.

W jakim celu zamieniasz postać binarną na liczbę dziesiętną funkcją stoi - przecież liczby binarne są znacznie dłuższe, więc dość szybko, dla niezbyt dużych wartości wyjdzie przepełnienie?

---

Jest lepszy sposób zmiany string na int ?
Korzystam ze stringów, bo jest system hex korzystający z A B C D E F więc znana wykorzystanego rodzaju zmiennej jest średnim pomysłem.
sprawdziłem w debageże i zmiana rodzaju zmiennej nie jest na razie problem.

---

Jest lepszy sposób zmiany string na int ?

std::stoi() opcjonalnie bierze jeszcze argument z systemem liczbowym :)

---

@Krloo, co konkretnie chcesz uzyskać? Bo jak tylko skonwertować bin na dec to użyj funkcji stoi (jak ci pisał adrian17) lub poszukaj rozwiązania w necie - zamiana systemów liczbowych to popularne ćwiczenie w C/C++ i innych językach.

---

Polecam stronę:

https://calculla.pl/systemy_liczbowe

---

@SzkolnyAdmin, ten kod został już zmieniony kilkukrotnie a pytanie zadane przez mnie jest niedokładne i wyjątkowo głupie za co przepraszam
poza tym zapomniałem go usunąć.
 

Życzę miłej nocy / dnia

---

@Krloo,  zamianę string na liczbę to masz właśnie napisać. Liczba jako liczba nie jest w żadnym systemie (bin, oct, dec, hex itp). System dotyczy konwersji liczby na postać tekstową i odwrotnie.

Zamiast skomplikowanej zabawy z liczbami, weź ten wejściowy string i przeleć po jego znakach. Jeśli znak jest 'cyfrą' w danym systemie, to dotychczasowy wynik mnożysz przez podstawę systemu i dodajesz wartość aktualnej cyfry.

Coś w pseudo-kodzie:

int bin2int(string binary)
{
  int value = 0;
  for  (char x : binary)
  {
     switch (x)
    {
        case '0':
              value *= 2;
              value += 0;
              break;
        case '1':
              value *= 2;
              value += 1;
              break;
        default:
              /// bład albo koniec konwersji
    }
  }
  retun value;
}

 

Answer 1

#include <iostream>
#include <conio.h>
#include <cstdlib>
#include <cmath>

using namespace std;

int main() {
    long int coppyI = 0,coppy = 0; 
    int dlugosc = 0;
    string x;

    x = "10010111";

      coppyI = stoi(x);
      while(coppyI - pow(10, dlugosc) > 0) {dlugosc++;}
      dlugosc --;
      while (dlugosc >= 0)
      {
        if(coppyI - pow(10, dlugosc) >= 0){
          coppyI -= pow(10, dlugosc);
          coppy += pow(2, dlugosc);
        }
          dlugosc --;
         cout << coppy << " | " << dlugosc << endl;
      }
    return 0;
}

czasami trzeba spać

Błędy w programie
- zły rodzaj pętli while

- brak dekrementacji dlugosc

-zły znak logiczny, który powodował to że ostatnia pętla się nie wykonała (10 do potęgi 0 )