Question

我正在用C语言制作一个程序，我想我会遇到一些记忆问题。

所以我的问题是：我有2个返回结构的函数。当我一次只运行一个功能时，我没有任何问题。但是当我一个接一个地运行时，我总是在写入第二个结构时出错。

函数struct item* ReadFileBIN(char *name) - 读取二进制文件。 struct tables* getMesasInfo(char* Filename) - 读取文本文件。

我的代码是：

#include "stdafx.h"
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <time.h>

int numberOfTables=0;
int numberOfItems=0;

//struct tables* mesas;
//struct item* Menu;

typedef struct item{
    char nome[100];
    int id;
    float preco;
};
typedef struct tables{
    int id;
    int capacity;
    bool inUse;
};
struct tables* getMesasInfo(char* Filename){
    struct tables* mesas;
    char *c;
    int counter,numberOflines=0,temp=0;
    char *filename=Filename;
    FILE * G;
    G = fopen(filename,"r");
    if (G==NULL){
        printf("Cannot open file.\n");
    }
    else{
     while (!feof(G)){
     fscanf(G, "%s", &c);
     numberOflines++;
        }
    fclose(G);
    }  
    /* Memory allocate for input array */
    mesas = (struct tables *)malloc(numberOflines* sizeof(struct tables*));
    counter=0;
    G=fopen(filename,"r");
    while (!feof(G)){
        mesas[counter].id=counter;
        fscanf(G, "%d", &mesas[counter].capacity);
        mesas[counter].inUse= false;
        counter++;
    }
fclose(G);
numberOfTables = counter;
return mesas;
}

struct item* ReadFileBIN(char *name)
{
        int total=0;
        int counter;
        FILE *ptr_myfile;
        struct item my_record;
        struct item* Menu;
        ptr_myfile=fopen(name,"r");
        if (!ptr_myfile)
        {
            printf("Unable to open file!");
        }

        while (!feof(ptr_myfile)){
            fread(&my_record,sizeof(struct item),1,ptr_myfile);
            total=total+1;
        }
        numberOfItems=total-1;
        Menu = (struct item *)calloc(numberOfItems , sizeof(struct item));
        fseek(ptr_myfile, sizeof(struct item), SEEK_END);
        rewind(ptr_myfile);
        for ( counter=1; counter < total ; counter++)
        {
            fread(&my_record,sizeof(struct item),1,ptr_myfile);
            Menu[counter] = my_record;
            printf("Nome: %s\n",Menu[counter].nome);
            printf("ID: %d\n",Menu[counter].id);
            printf("Preco: %f\n",Menu[counter].preco);
        }
        fclose(ptr_myfile);
        return Menu;
}

int _tmain(int argc, _TCHAR* argv[])
{
    struct item* tt = ReadFileBIN("menu.dat");
    struct tables* t = getMesasInfo("Capacity.txt");
    getchar();
}**

我得到的错误是：

“test.exe中0x00411700处的未处理异常：0xC0000005：访问冲突写入位置0x00000000。”

在“菜单[计数器] = my_record;”

中

提前致谢。

Answer 1

您似乎在getMesasInfo()中分配了错误大小的内存块：sizeof(struct tables*)为您提供指针的大小，而不是它指向的结构的大小：

mesas = (struct tables *)malloc(numberOflines* sizeof(struct tables*));

因此您可以轻松覆盖未分配的内存。适当的分配应该是

mesas = (struct tables *)malloc(numberOflines* sizeof(struct tables));

或者，类似于在ReadFileBIN()中分配其他数组的方式：

mesas = (struct tables *)calloc(numberOflines, sizeof(struct tables));

此外，我不知道这是否有意，但在ReadFileBIN()中你分配（1）和阅读（2）的记录少于记录总数：

    numberOfItems=total-1;                                             // 1
    Menu = (struct item *)calloc(numberOfItems , sizeof(struct item)); // 1
    fseek(ptr_myfile, sizeof(struct item), SEEK_END);
    rewind(ptr_myfile);
    for ( counter=1; counter < total ; counter++)                      // 2
    ...

由于循环计数器从1开始（而不是C中正常的0），因此您可以有效地执行循环total-1次。也就是说，您读入了数组的元素1到total-1。但是，数组的实际元素从0索引到total-2，因此数组中的第一个元素保持未初始化，最后提交缓冲区溢出错误。

Answer 2

进一步说明彼得的观点：

struct tables {
  int id;
  int capacity;
  int inUse; /* bool is not a C type */
};

int main()
{
  printf("sizeof: %d\n",sizeof(struct tables*));
  printf("sizeof: %d\n",sizeof(struct tables));

}

将输出：

sizeof: 4
sizeof: 12

Answer 3

您遇到以下问题：

    numberOfItems=total-1;
    Menu = (struct item *)calloc(numberOfItems , sizeof(struct item));
    fseek(ptr_myfile, sizeof(struct item), SEEK_END);
    rewind(ptr_myfile);
    for ( counter=1; counter < total ; counter++)

首先，您将numberOfItems设置为少而不是总数。这是不正确的。你甚至不需要numberOfItems;由于total包含文件中的行数，因此下一行应该是Menu = (struct item*) calloc(total, sizeof(struct item));

其次，您尝试在for循环中使用Menu作为基于一的数组。 C数组是从零开始的。您应该使用for循环for (counter = 0; counter < total; counter++)。

最后，正如彼得指出的那样，在第一个函数中，你正在分配错误的对象大小。您需要malloc numberOfLines*(sizeof(struct tables)（不是sizeof(struct tables*)。

C - struct问题 - 写作

3 个答案: