c++ problem switch

Question

Mam pewien problem, pierwszy podstawowy znajduje się w moim kodzie. Dlaczego w tej mojej funkcji ten switch nie działa?

14 razy wyskakuje error : case label "(od 2 do 14)" not within a switch statement.

string from2to16(string binary)
{
    string hex="";
    int length = binary.size();
    int parts = ceil(length/4);


    for(int i=0;i<parts*4;i+=4)
    {
        int partvalue=0;
        for(int j=i;j<i+4;j++)
        {
            if(binary[0]==1)
            {
                partvalue += 1;
            }
            if(binary[1]==1)
            {
                partvalue += 2;
            }
            if(binary[2]==1)
            {
                partvalue += 4;
            }
            if(binary[3]==1)
            {
                partvalue += 8;
            }
            switch(partvalue)
                case 1: hex+="1"; break;
                case 2: hex+="2"; break;
                case 3: hex+="3"; break;
                case 4: hex+="4"; break;
                case 5: hex+="5"; break;
                case 6: hex+="6"; break;
                case 7: hex+="7"; break;
                case 8: hex+="8"; break;
                case 9: hex+="9"; break;
                case 10: hex+="A"; break;
                case 11: hex+="B"; break;
                case 12: hex+="C"; break;
                case 13: hex+="D"; break;
                case 14: hex+="E"; break;
                case 15: hex+="F"; break;
        }
    }
    return hex;
}

Poza tym mam jeszcze inne pytanie, czy isnieją w c++ gotowe fukcje do zamiany w rózne systemy liczbowe i jesli nie, jak mam np. zamienić liczbe z systemu 10 do 11 ? bo do 8 czy 16 to wiem.

Comments

//możesz to zrobić praktycznie dwoma/trzema linijkami, ale co tam

---

Zamiast switcha możesz zrobić tak:

 hex += "0123456789ABCDEF"[partvalue];

 

Answer 1

string from2to16(string binary)
{
    string hex="";
    int length = binary.size();
    int parts = ceil(length/4);
 
 
    for(int i=0;i<parts*4;i+=4)
    {A
        int partvalue=0;
        for(int j=i;j<i+4;j++)
        {
            if(binary[0]==1)
            {
                partvalue += 1;
            }
            if(binary[1]==1)
            {
                partvalue += 2;
            }
            if(binary[2]==1)
            {
                partvalue += 4;
            }
            if(binary[3]==1)
            {
                partvalue += 8;
            }
            switch(partvalue)
            { //Tutaj klamry ci brakowało
                case 1: hex+="1"; break;
                case 2: hex+="2"; break;
                case 3: hex+="3"; break;
                case 4: hex+="4"; break;
                case 5: hex+="5"; break;
                case 6: hex+="6"; break;
                case 7: hex+="7"; break;
                case 8: hex+="8"; break;
                case 9: hex+="9"; break;
                case 10: hex+="A"; break;
                case 11: hex+="B"; break;
                case 12: hex+="C"; break;
                case 13: hex+="D"; break;
                case 14: hex+="E"; break;
                case 15: hex+="F"; break;
        }//Tutaj chyba jest koniec switcha
    }
    return hex;
}

 

Answer 2

PS. Niemam pojęcia czy ten kod działa bo jeszcze go nie testowałem, narazie chodzi mi o ten switch a później będę się martwić o poprawnosc kodu ;)

Comments

Działać poprawnie nie będzie, bo liczy tylko pierwsze (albo ostatnie) cztery bity. Jeśli w binary trzymasz tekst, to te porównania znaków do wartości 1 są błędne.

---

string from10to2(int x)
{
    string binary="";
    while(true)
    {
        if(x%2==0)
        {
            binary+="1";
        }
        else
        {
            binary+="0";
        }
        if(x==0)
        {
            break;
        }
    }

    return binary;
}

A ta funkcja też jest zle napisana ? bo cos nie działa, a wszystko wygląda logicznie (wedlug mnie)

---

x % 2 nie powoduje zmiany wartości zmiennej x, dlatego nie działa.

---

A no tak, niby taka oczywistość.. :D

A mógłbyś mi jakoś obszerniej wytłumaczyć jak mam napisać tą funkcje from2to16? Bo nie wiem jak mam to napisać żeby to działało w takim razie. Wiem że już mi odpisałeś ale niezrozumiałem jakos ;/ I dzieki za dotychczasowa pomoc :)

---

std::string from10to2(int x)
{
    std::string binary;
    unsigned mask = 0x80000000;
    while(mask) { binary += x & mask ? '1' : '0'; mask >>= 1; }
    return binary;
}