Question

我正在尝试在数组中打印整数摘要。例如，如果用户键入：“1 4 5 8” 结果应为：“1 5 10 18”（0 + 1 = 1,1 + 4 = 5,5 + 5 = 10,10 + 8 = 18）当我使用输入运行此代码时：“1 1 1” 第一次结果是：“1 2 3 32767 256” 和任何其他时间按预期运行：\ 这样做的算法很简单，它可以工作，但我已经习惯了OOP和刚刚开始学习C.这里的东西似乎不对..用指针或malloc的东西？任何帮助？实现这个的“正确”方式是什么？干杯..！

#include <stdio.h>
#include <stdlib.h>
#define MAX 50

/*function prototype*/
int* summary(int a[], int length);
void printArray(int array[], int i);

int main(){
    int k;
    int length=0;
    int userArray[MAX];
    printf("Enter some numbers:\n");

    while((scanf("%d", &k) ==1)){
        userArray[length] = k;
        length++;

        /*print the array after 'Enter' key pressed*/
        if(getchar() == '\n'){
            summary(userArray, length);
            length = 0;
            printf("\n");
        }/*if*/ 
    }/*while*/

    summary(userArray, length);
    printf("\n");

    return 0;
}/*main*/

int* summary(int a[], int length){
    int i;
    int counter = 0;
    int *p;
    p = (int *) malloc(length * sizeof(int));

    if(!p){
        printf("memory allocation failed!");
        exit(0);
    }/*if null*/


    for(i=0 ; i<length ; i++){
        counter+=a[i];
        p[i] = counter;
    }/*for*/

    a[length] = '\0';
    printArray(p, 0);
    return p;
    free(p);
}/*summary*/

void printArray(int array[], int i){
    if(array[i]=='\0'){
        printf("finished printing");
        return;
    }
    else{
        printf(" %d\n", array[i]);
        printArray(array, i+1);
    }
}/*printArray*/

Answer 1

问题出在

行

a[length] = '\0';
printArray(p, 0);

printArray迭代，直到找到值为0的元素。您已尝试在a的末尾写入零值，然后将数组p传递到printArray。 p没有任何零元素，因此printArray会在不可预测的时间内继续读取它，直到它崩溃或找到值为0的内存。

要解决此问题，您应该在length+1中为p元素分配空间，然后在调用p[length] = 0之前设置printArray。或者，您可以将第二个参数更改为printArray作为数组大小并将其重新实现为

void printArray(int* array, int size){
    for (int i=0; i<size; i++) {
        printf(" %d\n", array[i]);
    }
    printf("finished printing");
}/*prinArray*/

您还需要决定如何处理p中为summary分配的内存。您当前从函数返回此函数，然后尝试free它。由于前面的free(p)语句将控制权返回给调用函数，因此永远不会执行return行。由于summary的调用者没有使用返回值，因此最简单的解决方法是将summary更改为返回void并删除return p行。

请注意，如果启用警告，编译器应标记此最后一个问题。（将-Wall添加到gcc的命令行或MSVC的/W4

如何在C中打印数组元素的摘要

1 个答案: