c - 修改结构体数组中结构体的成员

Question

我正在尝试解决分配给我的作业中的一个问题 - 问题是我当前正在实现的功能之一不起作用。

首先，我被告知我的教授(如下)提供的代码不能以任何方式修改:

#include <crtdbg.h>
#include <stdio.h>
#include <time.h>
#include <string.h>

#define BLOCK_SIZE 64
    typedef enum { FALSE = 0, TRUE } BOOL;

typedef struct {
    char* fileName;         // Frame fileName
}Frame, *pFrame, **ppFrame;

typedef struct {
    unsigned int numFrames;     // number of Frame* 
    ppFrame frames;         // array of Frame*
}Animation, *pAnimation;

// Forward declarations
void initAnimation(pAnimation);
void insertFrame(pAnimation);
void deleteFrames(pAnimation);
void runFrames(pAnimation);

int main(void)
{
    char response;
    BOOL RUNNING = TRUE;
    Animation A;
    _CrtSetDbgFlag(_CRTDBG_ALLOC_MEM_DF | _CRTDBG_LEAK_CHECK_DF);
    initAnimation(&A);

    while (RUNNING)
    {
        printf("MENU\n 1. Insert a Frame\n 2. Delete all the Frames\n 3. Run the Animation\n 4. Quit\n");
        scanf("%c", &response);
        switch (response)
        {
        case '1':insertFrame(&A); break;
        case '2':deleteFrames(&A); break;
        case '3':runFrames(&A); break;
        case '4':RUNNING = FALSE; deleteFrames(&A); break;
            default:printf("Please enter a valid option\n");
        }
    printf("\n");
    while ((response = getchar()) != '\n' && response != EOF);// clear input buffer
}
return 0;
}

接下来，我需要为提供的代码中的前向声明语句创建函数。我遇到问题的函数在这里:

void insertFrame(Animation *pAnimation)
{
    char input[50];
    int count;
    int dupe = 0;
    int blockFrames = pAnimation->numFrames % BLOCK_SIZE;
    int blocks = (pAnimation->numFrames / BLOCK_SIZE) + 1;

    printf("Insert a frame into the Animation\n");
    scanf("%s", input);

    /* check the input to see if it matches any of the existing frames */
    for (count = 0; count < pAnimation->numFrames; count++)
    {
        printf("%s\n", pAnimation->frames[count]->fileName); /*test: show the file name to be compared against*/
        if (strcmp(input, pAnimation->frames[count]->fileName) != 0)
        {
            dupe = 1;
            printf("Dupe detected!");
            return;
        }
    }

    printf("dupe = %d\n", dupe);

    /* afterwards, actually add the frame if it's ok */
    if (dupe == 0)
    {
        pAnimation->numFrames++;
        strcpy(pAnimation->frames[pAnimation->numFrames - 1]->fileName, input); /* <-- This is where the error happens */

    }
}

每次我使用 printf 或 strcpy 显示或修改 pAnimation->frames[ ] 中的 fileName 结构成员时，都会显示读取访问冲突。经过仔细检查，似乎 pAnimation->frames[ ] 指向的地址中的文件名未知。

为什么会发生这种情况，如何解决？

<小时/>

好的，接受建议后，我的 initAnimation 看起来是这样的:

void initAnimation(Animation *pAnimation)
{
    pAnimation->numFrames = 0;
    pAnimation->frames = malloc(BLOCK_SIZE * sizeof(Frame));
}

我也联系了我的教授，他说我不需要分配任何东西，除非我“添加”一个新框架(通过 insertFrame)。我不太确定他的意思。

Answer 1

看着

void initAnimation(Animation *pAnimation)
{
    pAnimation->numFrames = 0;
    pAnimation->frames = malloc(BLOCK_SIZE * sizeof(Frame));
}

我假设您正在分配一个帧指针数组。因此每个单元格都是 sizeof(pFrame) 而不是 sizeof(Frame)。它足够大，可以容纳指向框架的指针，而不是框架本身。

其次，您需要在创建这些框架时对其进行分配。

类似于

pAnimation->frames[i] = malloc(sizeof(Frame));

最后一个Frame保存一个指向字符数组的指针。您还需要分配它

pAnimation->frames[i]->fileName = malloc(50); // Use your max size

您还应该检查 malloc 调用的结果。

我会将 2 个帧分配调用放入 insertFrame() 中，因为当您创建/添加新帧时。