c-使用指针的结构会出错

Question

我制作了bob的代码，它是一个结构，每个bob都有名称和两个以上的整数（其实并不重要）。有三个功能

1. 初始化结构（使用"bob"和0和3）

2. 第二个函数得到两个结构，它需要在这些结构之间复制

3. 第三个函数是释放每个bob的名称（char*）。

首先，第二个函数（copy）在调试中出错，因为它没有复制名称（需要您的帮助来分析发生这种情况的原因），其次，代码在免费函数中崩溃。有人能告诉我如何释放结构的名称（char*）吗？

#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#define LENGTH_OF_BOB 4
typedef struct bobTheBuilder
{
    char* name;
    int fixed;
    int maxFix;
}bob;
//typedef struct bobTHeBuilder bob;
void deleteBob(bob currBob);
void initBob(bob *currBob);
void copyStruct(bob* dst, bob src);
int main(void)
{
    bob currBob = {0,0,0};
    bob secondBob;
    initBob(&currBob);
    copyStruct(&secondBob, currBob);
    deleteBob(currBob);
    deleteBob(secondBob);
    system("PAUSE");    
    return 0;
}
/*
*/
void initBob(bob *currBob)
{
    char* str = (char*)calloc(LENGTH_OF_BOB, sizeof(char));
    char string[] = "bob";
    if (str)
    {
        strcat(string, "");
        str = string;
        currBob->name = str;
        currBob->fixed = 0;
        currBob->maxFix = 3;
    }
}
/*
*/
void deleteBob(bob currBob)
{
    free(currBob.name);
}
void copyStruct(bob* dest, bob src)
{
    dest->fixed = src.fixed;
    dest->maxFix = src.maxFix;
    dest->name = (char*)malloc(sizeof(char) *LENGTH_OF_BOB);
    strncpy(dest->name, src.name, LENGTH_OF_BOB);
}

Answer 1

在initBob中，您有：

char* str = (char*)calloc(LENGTH_OF_BOB, sizeof(char));
char string[] = "bob";
str = string;
currBob->name = str;

它将currBob->name设置为指向局部自动变量。不到动态分配的缓冲区。当函数退出时，自动变量超出范围，因此不再有效。当然，它不能被释放，因为它不是动态分配的内存。

我真的不确定你在那里想做什么。除了错误地将str设置为指向局部变量之外，您还有一个不必要的strcat。我猜你是想NUL终止缓冲区。但这是不必要的，因为用字符串文字初始化未大小的char数组已经保证了NUL的终止。

考虑到这些问题，initBob函数应该更像：

void initBob(bob *currBob)
{
    currBob->name = calloc(LENGTH_OF_BOB, sizeof(char));
    if (currBob->name)
    {
        strcpy(currBob->name, "bob");  
        currBob->fixed = 0;
        currBob->maxFix = 3;
    }
}

Answer 2

我不知道这是否只是一个简单的例子来学习如何做到这一点，或者这是否真的是你的范围，但如果你需要这样做，请使用：strdup（）

void initBob(bob *currBob)
{ 
if (currBob->name)
{
    currBob->name=strdup("bob");  
    currBob->fixed = 0;
    currBob->maxFix = 3;
}
}

而不是在某个地方释放（），因为malloc（）字符串。。。它是ANSI标准