Konwertowanie liczb

pytanie zadane 31 października 2018 w C i C++ przez BinaryMan Stary wyjadacz (12,620 p.)

Witam !
Zadanie polega na tym, że dopóki nie wpiszę -1, program ma wyświetlać liczbę. Jeśli wpiszę liczbę dziesiętną ma wyświetlić szesnastkową i odwrotnie.

Przykład:
Wejście:
4
7
44
0x80685
-1

Wyjście
0x4
0x7
0x2C
525957

Oto mój kod:

#include <iostream>
#include <string>
#include <stdlib.h>

using namespace std;

string liczba;
string przerwij = "-1";

void change_system_number(string liczba);

int main()
{
    cin >> liczba;
    while (liczba != przerwij)
    {
        change_system_number(liczba);
        liczba = "";
        cin >> liczba;
    }
    return 0;
}

void change_system_number(string liczba)
{

    if (liczba[1] == 'x') // hex to dec
    {

        int l_hex = stoi(liczba, 0, 16);
        cout << l_hex << endl;
        liczba = "";
        l_hex = 0;
    }
    else //dec to hex
    {
        int l_hex = atoi(liczba.c_str());
        cout << hex << "0x" << l_hex << endl;
        liczba = "";
        l_hex = 0;
    }
    liczba = "";
}

Problem:
Jeśli zamienię pierwszy raz liczbę 16 na 10 to działa dobrze, później wpisuję serię liczb dziesiętnych i też jest spoko. Problem jest wtedy, gdy chcę zamienić liczbę 16 na 10 raz jeszcze. Wyświetla mi się wtedy sama końcówka, po 0x...

2 odpowiedzi

odpowiedź 31 października 2018 przez niezalogowany
wybrane 31 października 2018 przez BinaryMan

Najlepsza

Jeżeli chcesz wypisywać liczby raz w systemie dziesiątkowym, a raz szesnastkowym musisz oprócz std::hex używać tez std::dec:

int l_hex = stoi(liczba, 0, 16);
cout << dec << l_hex << endl;

1	komentarz 31 października 2018 przez BinaryMan Stary wyjadacz (12,620 p.) Dzięki bardzo, działa !

komentarz 31 października 2018 przez niezalogowany

Mam jeszcze kilka uwag :) Możesz usunąć te wszystkie przypisania pustych łańcuchów i zer, bo są zbędne. Zmienne globalne wrzuć do main. Zamień atoi i c_str na std::stoi dla else.

komentarz 31 października 2018 przez BinaryMan Stary wyjadacz (12,620 p.)

Okay dzięki

odpowiedź 31 października 2018 przez criss Mędrzec (172,590 p.)

"0x" to umowny symbol stawiany przed liczbą który ma oznaczać, że jest zapisana w systemie szesnastkowym (mogłoby to nie być oczywiste gdyby nie było w niej żadnej cyfry większej od 9), ale w żaden sposób "0x" nie jest integralną częścią systemu szesnastkowego. Nawet zakładając tylko funkcje informacyjną tego "0x", to funkcja stoi() i tak tego nie potrzebuje bo system jest podawany z góry w 3 argumencie. Zatem wystarczy, że nie będziesz kazał przekonwertować na liczbe łącznie z "0x". Możesz użyć substr: stoi(liczba.substr(2), nullptr, 16);

komentarz 31 października 2018 przez BinaryMan Stary wyjadacz (12,620 p.)

Tylko, że ta część programu działa dobrze. To z górną częścią if-a jest problem

komentarz 31 października 2018 przez criss Mędrzec (172,590 p.)

Oh, wybacz. Więc widocznie stoi "0x" nie przeszkadza :D

Pozbądź się zmiennych globalnych. Tzn napisz to bez zmiennych globalnych. Założe się, że dziwnym trafem zacznie działać albo przynajmniej szybciej znajdziesz błąd