2

我在以下类型的布局中有一些代码，我相信topExample/botExample调用时没有正确设置addTopBotExample。我认为这是由于函数堆栈上的顶部 bot 变量因此在函数结束时被清除？我有一种感觉，也许我需要先malloc记住记忆，但我不确定我将如何去做这件事，即使它是正确的方法。

typedef struct Example Example;
struct Example {
   /* normal variables ...*/
   Example *topExample;
   Example *botExample;
};

....

void addTopBotExample(Example **example, int someVariable) {
    Example top = createTopExample(int someVariable); //(createTopExample returns a
                                                      //type Example based on some input)
    Example bot = createBotExample(int someVariable);
    (*example)->topExample = ⊤
    (*example)->botExample = ⊥
    return;
}
4

3 回答 3

2

如果createTopExample不分配内存，那么当它被多次调用时，这将导致问题。重写createTopExampleandcreateBotExample使用malloc并返回一个Example*. 像这样的东西：

Example* createTopExample(stuff)
{
    Example *example = malloc(sizeof(Example)); 
    // ... stuff you do
    return example;
}

然后你addTopBotExample会看起来像这样：

 void addTopBotExample(Example **example, int someVariable) {
     if ((*example)->topExample)
         free((*example)->topExample)
     if ((*example)->botExample)
         free((*example)->botExample)
     (*example)->topExample = createTopExample(int someVariable);
     (*example)->botExample = createBotExample(int someVariable);
     return;
 }

请注意，这addTopBotExample将free在再次调用之前分配内存，malloc但在程序结束之前，您需要调用free任何Example使用此addTopBotExample函数的 lingering ：

free(exampleInstanceThatWasPassedIntoAddTopBotExampleAtSomePoint.topExample);
free(exampleInstanceThatWasPassedIntoAddTopBotExampleAtSomePoint.botExample);
于 2013-02-08T23:38:07.090 回答
0

哦，这很糟糕。addTopBotExample() 函数中的表达式“Example top”在堆栈上分配了该对象。退出该功能后将被丢弃。（下一行的“Example bot”也是如此。）这样的事情会更好：

void addTopBotExample(Example **example, int someVariable) {
  Example *top = createTopExample(someVariable); // NOTE THE *
  Example *bot = createBotExample(someVariable); // NOTE THE *

  (*example)->topExample = top; // NOT &top !!
  (*example)->botExample = bot; // NOT &bot !!
  return;
}

并且您需要编写 createTopExample 和 createBotExample 以便它们返回指针：

#include <stdlib.h> // For malloc!
Example *createTopExample(stuff)  // Note *. It's returning a pointer.
{
  Example *example = malloc(sizeof(Example)); // Allocate on the HEAP. Lives after this function call.

  // Fill in the fields of example.
  example->field1 = 25; // Note the "->": you're dereferencing a pointer.
  example->title = "Example title";

  return example;
}
于 2013-02-08T23:40:38.803 回答
0

你已经拥有了一切。分配ExampleincreateTopExample或createTopExample 

Example *createTopExample(int someVariable)
{
    Example *x = malloc(sizeof(Example));
    /* initialize x */
    return x;
}

并且在addTopBotExample 

void addTopBotExample(Example *example, int someVariable) {
    Example *top = createTopExample(int someVariable); //(createTopExample returns a
                                                       //type Example based on some input)
    Example *bot = createBotExample(int someVariable);
    example->topExample = top;
    example->botExample = bot;
    return;
}
于 2013-02-08T23:37:07.887 回答