【问题标题】:C++ default constructor, initializing pointer with new objectC++ 默认构造函数,用新对象初始化指针
【发布时间】:2023-03-14 11:34:01
【问题描述】:

我有以下问题: 在 myClass 中,我想使用新的 yourClass 地址默认初始化指向 yourClass 的指针。 不幸的是,如果我想在任何时候删除指针,我都会得到一个(核心转储)。

class myClass
{
      protected:
      yourClass * yc;

      public:
      myClass() { yc = new yourClass(); }

      myClass(yourClass * tyc ) { delete yc; yc = tyc; }

      ~myClass() { delete yc; yc = NULL; }

      void setMyClass (yourClass * tyc) { delete yc; yc = tyc; }

      void print () { yc->print(); }
};

int main()
{
  yourClass b (//parameter);
  myClass * a = new myClass();
  a->print();
  a->setMyClass(&b)
  a->print();

  delete a;
  return 0;
}

a 的 print() 应该产生两个不同的打印,取决于 //parameters。

我认为 yourClass yc;而不是 yourClass* yc,但我想知道是否可能。

编辑: 我以以下方式重新编写了代码并且它可以工作。看起来仍然很复杂,智能指针似乎很有希望,我仍然没有应用“三法则”。 这里是代码。谢谢大家。

class myClass
{
      protected:
      yourClass * yc;
      bool dynamic;

      public:
        myClass() { dynamic = true; yc = new yourClass (); }
        myClass (yourClass * tyc ) 
        { 
          // dynamic init (like default)
          if (tyc == NULL ) { dynamic = true; yc = new yourClass (); }
          // static use of yc
          else { dynamic = false; yc = tyc; } 
        }
        // because only if dynamic is true, we need to erase
        ~blu () { if (dynamic) { delete yc; dynamic = false; } } 

        void setMyClass(yourClass* tyc) 
        { 
          // leaving unchanged if new-stuff is NULL or like old-stuff
          if ( tyc == yc || tyc == NULL ) return;
          else // treating dynamic and static differently
          { 
            if (dynamic) // if flag is set, must be deleted 
            {
              delete yc; yc = tyc; dynamic = false;
            }
            else // must not be deleted, dynamic is still false
            {
              yc = tyc;
            }
          }
        }
        void print () { yc->print(); }
};

【问题讨论】:

  • 这应该是可能的 - yourClass 的析构函数包含什么?
  • 哦,我认为参数化构造函数中不应该有 delete yc。

标签: c++ class pointers dynamic default-constructor


【解决方案1】:

那是因为你试图删除太多:

  • 您正在删除第二个构造函数中未分配的对象(删除delete yc;
  • 您正试图删除堆栈分配的对象,b。 delete a; 将尝试删除指向 b 的指针,该指针是堆栈上的一个对象;会发生什么取决于您的操作系统(我预计会出现异常/核心转储/无论如何)

编辑:我发现的另一个问题.. a->setMyClass(NULL)

我建议:

  • this post 智能指针
  • this blog post 在 RAII 上
  • 任何解释堆栈与堆分配(静态与动态?)的 C/C++ 入门书

【讨论】:

  • 解决第二个问题的方法可能是为 'ownsYc' 存储一个标志,您可以在调用 setMyClass () 时将其设置为 false,如果标志为 true,则仅在析构函数中删除 yc
  • @Crog ...嗯...是的,但它仍然是一个非常非常危险的设计。在 C++ 中,我相信拥有 RIIA 总是更好;此外,还有智能指针(stroustrup.com
【解决方案2】:

你违反了三原则。

What is The Rule of Three?

这也是灾难的秘诀:

 myClass(yourClass * tyc ) { delete yc; yc = tyc; }

如果tyc==yc 会发生什么?正确的。不漂亮:)

 myClass(yourClass * tyc ) { if (yc!=tyc) { delete yc; yc = tyc; } }

【讨论】:

  • 这比这更灾难:) 在构造函数中,yc 将永远不会被初始化:如果你幸运的话,它是 delete NULL (segfault);否则它会删除一些随机的东西
  • 所以你提出的代码无论如何都会崩溃(假设编译器将 yc 初始化为 0...)
  • @dema80 我同意,我没有时间真正阅读该代码。为我感到羞耻。我只是想强调一下这个经典的陷阱,因为它稍微微妙一些(甚至 Scott Meyer 曾经在一本书中发表过一个带有这种错误的 smartptr)
猜你喜欢
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 2019-10-07
  • 2012-01-02
  • 2015-11-05
  • 2018-12-08
  • 2021-05-28
  • 2011-06-12
相关资源
最近更新 更多