Question

所以我回来参加我的学校项目。老师给了我们更多的代码。十六进制计算器。乍看之下，它看起来已经完成了，但是...数学是错误的。有人可以看一下这段代码，然后告诉我什么地方出错了，或者我可以做些什么来使它正确？我需要将大部分代码保持原样。所以我不能改变原则。我必须坚持这一点。当我运行此代码时，我确实得到了结果，但这不是我需要的结果。我需要执行类似“ 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';
}

Answer 1

add2将r（似乎是一个进位位）作为参数，但实际上并没有使用它。

无论如何，计算r的方式是没有意义的。恰好在一个地方将其设置为1，但随后又无条件地将其重置为0。

最重要的是，您要放弃所有进位。

十六进制计算器问题。有人可以再次检查我的代码吗？

1 个答案: