结构数组长度错误

Question

我是 C 的初学者，经过长时间的努力，我完成了我的 1500 行 C 代码。它运行完美并产生了预期的结果，但是当我尝试添加比测试更多的输入时，我遇到了故障.

更具体地说，我设法在结构数组中找到了问题的根源。

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

#define ROWS 5000
#define MAXLOGGERS 26

struct data
    {
        int day;
        int month;
        int year;
        int hour;
        int minute;
        int second;
        float value;
        float temp;
        float hum;
    };  

int main()
{
    printf("Enter the number of loggers that you want to import: ");
    int n = 10;

    while (n > MAXLOGGERS)
    {
        printf("You can input data from maximum %d loggers: ", MAXLOGGERS);
        scanf("%d", &n);
    }

    struct data mydata[n+1][ROWS];

    printf("\n\nSuccess!!\n");
}

现在，当我使用 n = 10 或更少时，程序按预期完成。当我将 n 更改为 11 或更多时，它会崩溃。

我怀疑结构数组的声明有问题，但我真的想不通。

我们非常欢迎任何帮助！ 谢谢（：

Answer 1

struct data mydata[n+1][ROWS];

您正在堆栈上分配一个非常大的（二维）数组，该数组完全被 n >= 11 消耗（在我的系统上也是如此，导致堆栈溢出，显示为分段错误）。

对于如此大的数据，您应该在堆上分配内存。这很好用（至少在我的机器上）：

//struct data mydata[n+1][ROWS];
struct data* mydata = (struct data*)malloc(sizeof(struct data) * (n+1) * ROWS);
printf("\n\nSuccess!!\n");
free(mydata);

只是，对数组的访问不再那么好，因为您不能使用双索引运算符。您必须自己计算偏移量：

// mydata[x][y]; gets: 
mydata[x*ROWS + y];

不要监督对free 的调用 - 每个数据malloced 都应该是freed。从一开始就习惯这个原则（在输入malloc 的同时想到free）将有助于防止内存泄漏。

正如 cmets 中所暗示的（以及引用的答案），堆栈大小通常是有限的，而对堆施加的唯一限制是您机器的（但未使用的）物理内存的大小，这（通常）很远较大。如果您碰巧用完了系统内存，malloc 不会崩溃，而是返回一个空指针。所以你实际上应该检查：

struct data* mydata = (struct data*)malloc(sizeof(struct data) * (n+1) * ROWS);
if(mydata)
{
    printf("\n\nSuccess!!\n");
    free(mydata);
}
else
{
    printf("\n\nOut of memory!!\n");
}