Question

我正在尝试整理一个程序，这将总结非常大的数字。不幸的是，我被困住了 - 即使我评论出malloc＆amp; realloc（编译器似乎失败的地方）。有任何想法吗？我的代码：

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

int i,j,x;
char *actual = NULL;
char *sum = NULL;

void init () {
    sum = malloc(500);
    actual = malloc(500);
}

void calculate (char *argv[]) {
    int rest = 0;
    actual = *argv;
    actual = realloc(actual, strlen(*argv));
    if (strlen(actual) > strlen(sum)) {
    sum = realloc(sum, strlen(actual) + 1);
    } else sum = realloc(sum, strlen(sum) + 1);
    long b;
    for (b = 1; b < strlen(actual); b++) {
        rest = rest + atoi(&sum[strlen(sum) - b]) + atoi(&actual[strlen(actual) - b]);
        if (rest > 9) {
            sum[strlen(sum) - b] = rest - 10;
            rest = 1;   // carrying over 1
        } else {
            sum[strlen(sum) - b] = rest;
            rest = 0;
        }
    }
}

void writeResult () {
    printf("VYPIS - sum:");
    printf("strlen souctu je: %lu\n",strlen(sum));
    long c;
    for (c = 0; c <= strlen(sum); c++) {
        printf("%c",sum[c]);
    }
    printf("\n");
}

void emtpy () {
    free(actual);
    free(sum);
}

int main(int argc, char * argv[]) {
    init();
    for (i = 1; i < argc; i++) {
        calculate(&argv[i]);
    }
    writeResult();
    emtpy();
    return 0;
}

Answer 1

尝试realloc argv是未定义的行为。一般情况下，你不应该realloc你没有malloc - 从明确转移内存所有权的函数中获取或接收的内容。

另请注意，atoi期望以空字符结尾的C字符串，因此将长字符串的一部分传递给它是不正确的。如果您想获得char数字的数值，请减去'0'，如下所示：

int digit = actual[strlen(actual) - b] -'0';

要将单个十进制数字转换为char，请添加'0'：

res[b] = digit + '0';

Answer 2

您的代码太复杂，但有几个问题：

您无法使用atoi将字符转换为值，可以通过以下方式完成：int value = c - '0'。
您不应该修改argv数组中的字符串。特别是你无法重新分配它们。这会调用未定义的行为。
始终分配或重新分配比您在最终'\0'的结果数组中存储的字符串长度多1个字节，并记住设置此最后一个字节。
你应该从右到左计算加法，就像你手工做的那样，跟踪从一个数字到下一个数字的进位，可能会增加一个额外的前导数字。

以下是针对您的问题的简化版本，其中显示了如何处理基数10中的大数字：

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

static char *bigsum(char *res, const char *arg) {
    size_t pos1, pos2, pos3, len3;
    unsigned int carry;

    pos1 = strlen(res);
    pos2 = strlen(arg);
    len3 = pos3 = (pos1 < pos2) ? pos2 + 1 : pos1 + 1;
    /* reallocate the result array to one more than the larger operand */
    res = realloc(res, len3 + 1);
    /* set the terminating '\0' at the end of result */
    res[pos3] = '\0';
    for (carry = 0; pos3 > 0; carry /= 10) {
        if (pos1 > 0) carry += res[--pos1] - '0';
        if (pos2 > 0) carry += arg[--pos2] - '0';
        res[--pos3] = '0' + carry % 10;
    }
    while (res[0] == '0' && len3 > 1) {
        /* normalize the result: remove redundant initial zeroes */
        memmove(res, res + 1, len3--);
    }
    return res;
}

int main(int argc, const char **argv) {
    /* initialize the result to "0" as an allocated string */
    char *result = strcpy(malloc(2), "0");
    int i;

    for (i = 1; i < argc; i++) {
        result = bigsum(result, argv[i]);
    }
    printf("%s\n", result);
    return 0;
}

如何总结大数字

2 个答案: