Проблема гекса калькулятора.Может кто-нибудь еще раз проверить мой код? - PullRequest
0 голосов
/ 15 июня 2019

Итак, я вернулся с моим школьным проектом. Учитель дал нам больше кода для работы. Шестнадцатеричный калькулятор. На первый взгляд это выглядит законченным, но ... математика неверна. Может кто-нибудь просмотреть этот код и сказать мне, что не так или что я могу сделать, чтобы исправить это? Мне нужно сохранить большую часть этого кода, как есть. Поэтому я не могу изменить принцип. Я должен придерживаться этого. Когда я запускаю этот код, я получаю результат, но это не тот результат, который мне нужен. Мне нужно сделать что-то вроде "1cec + bec = 28d8". Опять же, я не получаю ошибок, но я не получаю правильный результат.

#include <iostream>
#include <cstring>

using namespace std;

int test(char[], int);
void decal(char[], int n);
int add2(char a, char b, int r);
void add(char a[], char b[], char c[]);

int main()
{
    char a[10], b[10], c[10];
    int valid;

    do {
        cout << "Insert first number (maximum 5 char hexa):" << endl;
        cin >> a;
        valid = test(a, strlen(a));
        if(!valid)
            cout << "Error." << endl;
    } while (!valid);
    cout << "First number: " << a << endl;
    decal(a, strlen(a));
    cout << "First number after completing: " << a << endl;

    do {
        cout << "Insert 2nd number (maximum 5 char hexa):" << endl;
        cin >> b;
        valid = test(b, strlen(b));
        if(!valid)
            cout << "Error." << endl;
    } while (!valid);

    decal(b, strlen(b));
    cout << "2nd number: " << b << endl;

    add(a, b, c);  
    cout << "Rezultat: " << c << endl;
    return 0;
}

void add(char a[], char b[], char c[])
{
    int i, r=0, sab;
    c[6] = '\0';
    for(i = 4; i>= 0; i--)
    {
        sab = add2(a[i], b[i], r);  
        if(sab < 10)
        {
            r = 0;
            c[i+1] = sab + 48; // '0' ... '9'
        }

        else
            if(sab < 16)
            {
                r = 0;
                c[i] = 97 + (sab - 10); // 'a' ... 'f'
            }

            else
            {
                r = 1;
                sab = sab - 16;
                if(sab < 10)
                {
                    r = 0;
                    c[i+1] = sab + 48; // '0' ... '9'
                }
                else
                {
                    r = 0;
                    c[i] = 97 + (sab - 10); // 'a' ... 'f'

                }
            }
    }
    c[0] = 48 + r;
}

int add2(char a, char b, int r)
{
    int ai, bi, ci, ai1, bi1;
    ai = tolower(a);
    if(ai <= 57)
        ai1 = ai - 48;
    else
        ai1 = ai - 97 + 10;
    bi = tolower(b);
    if(bi <= 57)
        bi1 = bi - 48;
    else
        bi1 = bi - 97 + 10;
    ci = ai1 + bi1;
    return ci;
}

int test(char x[], int n)
{
    if(n > 5)
        return 0;  

    for(int i = 0; i < n; i++)
    {
        if(x[i] <48 || (x[i] > 57 && x[i] < 65) || (x[i] > 70 && x[i] < 97) || x[i] > 102)
            return 0;  
    }
    return 1;
}

void decal(char x[], int n)
{

    int i, nz;
    x[5] = '\0';
    nz = 5 - strlen(x); 
    if(nz > 0) {
        for(i = 0; i < n; i++)
            x[5 - i-1] = x[n-i-1];
    }

    for(i = 0; i < nz; i++)
            x[i] = '0';
}

1 Ответ

1 голос
/ 16 июня 2019

add2 принимает r (который представляется как бит переноса) в качестве параметра, но фактически не использует его.

В любом случае, способ вычисления rне имеет смысла.Есть ровно одно место, где он установлен на 1, но затем он безоговорочно сбрасывается обратно на 0 пару строк спустя.

Суть в том, что вы отбрасываете все керри.

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...