Question

对于我正在进行的程序，我需要执行＆＃34; String Math＆＃34;其中，给定两个字符串，我需要沿字符串中的所有字符执行加法或减法以获得最终结果。

我在解释指针时遇到问题，并且在理解如何在循环中使用指针时感觉自己真的陷入困境。

目前我有：

int main() {
    char * input = (char *)malloc(sizeof(char)*1024);
    char * stringA = (char *)malloc(sizeof(char)*64);
    char * stringB = (char *)malloc(sizeof(char)*64);
    char * result;
    char operand;
    int matched = 0;

    while(fgets(input, 1024, stdin) != NULL) {
        matched = sscanf(input, "%s %s %c", stringA, stringB, &operand);
        if (matched == 3) {
            if (strlen(stringA) < strlen(stringB)) {
                stringA = balanceLengths(stringA, stringB);
            } else if (strlen(stringA) > strlen(stringB)) {
                stringB = balanceLengths(stringB, stringA);
            }

            result = (char *)calloc(strlen(stringA)+2, sizeof(char));
            result = performStringMath(result, stringA, stringB, operand);

            printf("%s\n", result);

        }
    }

    //printf("%s\n%s\n", stringA, stringB);

    free(input);
    free(stringA);
    free(stringB);
    return 0;
}

我真的很难理解如何在循环的每次迭代中释放结果，因为如果我在printf语句之后添加一个空闲（结果），我会从valgrind中获得几个无效的读写错误如果我在循环之前为calloc分配内存，我似乎也会遇到类似的错误。

其他方法：

    char * performStringMath(char *result, char *stringA, char *stringB, char operand) {
    if (operand == '+') {
        return performAddition(result, stringA, stringB);
    } else if (operand == '-') {
        return performSubtraction(stringA, stringB);
    } else {
        return 0;
    }
}

char * balanceLengths(char *shorter, char *longer) {
    int toAdd, i;
    toAdd = (int)(strlen(longer) - strlen(shorter));
    char *retString = (char *) malloc(sizeof(char) * strlen(longer)+ toAdd);

    for (i = 0; i < toAdd; i++) {
        retString[i] = '0';
        retString[toAdd+i] = shorter[i];
    }
    return retString;
}

char * performSubtraction(char *stringA, char *stringB) {
    return "";
}

char * performAddition(char *result, char *stringA, char *stringB) {
    int carry, sum, i;
    sum = 0;
    carry = 0;
    for (i = (int)strlen(stringA)-1; i >= 0; i--) {
        sum = (stringA[i]-48) + (stringB[i]-48) + carry;
        carry = 0;
        if (sum >= 10) {
            sum = sum-10;
            carry = 1;
        }
        stringA[i] = (char)(sum+48);
    }

    if (carry != 0) {
        result[0] = '1';
        result = strcat(result, stringA);
    } else {
        result = stringA;
    }

    return result;
}

我实际上已经通过替换

解决了这个问题

result = stringA;

要 result = strcpy（result，stringA）;

在功能执行添加。

Answer 1

您的代码看起来并不是很困惑。它看起来大多正确。我看到free()只有一个丢失的电话。在循环中，您调用calloc()，但从不在该内存上调用free()。如果您在打印出来之后无法使用result，那么您可以在打印后将其释放：

        result = (char *)calloc(strlen(stringA)+2, sizeof(char));
        result = performStringMath(result, stringA, stringB, operand);

        printf("%s\n", result);
        free(result);

可以在循环的每次迭代中分配和释放其他字符串，但这会使程序运行得更慢而且无论如何都没有帮助。

如果他们使用像valgrind这样的工具给你错误，那么performStringMath()可能会出现问题。但是，如果没有该功能的代码，我们无法真正说出它可能是什么。

C - 释放循环中分配的内存

1 个答案: