c++类指针删除段错误

Question

我有一个名为 object 的简单类，但我遇到了问题。如果我调用一种方法，它会导致段错误。我不明白为什么。

typedef class object
{
   private:
      short id;
      std::string name;
      SDL_Rect offset;

   public:
      object();
      object(short i, std::string n);
      ~object();
      object(const object &o);
      object& operator = (const object &o);
      std::string get_name();
      void set_name(std::string n);
} object;

object::object()
{
   id = 0;
   name = "";
   offset.x = 0;
   offset.y = 0;
   offset.w = 0;
   offset.h = 0;
}

object::object(short i, std::string n)
{
   id = i;
   name = n;
   offset.x = 0;
   offset.y = 0;
   offset.w = 0;
   offset.h = 0;
}

object::~object()
{
   delete &offset;
   delete &id;
   delete &name;
}

object& object::operator=(const object &o)
{
   if(this != &o)
   {
      delete &name;
      name.assign(o.name);
      delete &id;
      id = o.id;
      delete &offset;
      offset = o.offset;
   }
   return *this;
}

object::object(const object &o)
{
   id = o.id;
   name = o.name;
   offset = o.offset;
}

// Functions
std::string object::get_name()
{
   return name;
}

void object::set_name(std::string n)
{
   name = n;
}

而我的 main.cpp

int main( int argc, char** argv )
{
   struct object *a = new object(0, "test");
   struct object *b = new object(1, "another test");

   printf(a->get_name().c_str());
   printf("\n");
   printf(b->get_name().c_str());
   b = a;
   printf("\n");
   printf(b->get_name().c_str());
   a->set_name("Another test");
   printf("\n");
   printf(a->get_name().c_str());

   delete a;
   printf("\nDeleted a");
   delete b;
   printf("\nDeleted b");

   return 0;
}

如果我调用 a->set_name("Another test");，我会遇到段错误。如果我不打电话，就没有问题，一切正常。我可能错过了一些简单的东西，但我找不到它。它不会在分配时出现段错误，但如果该行在那里，则在删除指针时会崩溃。

Answer 1

由于您没有在构造函数中new任何内容，因此在析构函数中delete任何内容都是错误的。只需将析构函数留空即可，或者更好的是，完全摆脱它。编译器生成的析构函数完全按照您的要求执行（什么都不执行）。您也不必手动编写复制构造函数和复制赋值运算符，只需将它们扔掉即可。

也不需要动态创建对象，并且使用本机 C++ 输入/输出工具打印字符串要容易得多。并且您可以摆脱使用默认参数重载构造函数的情况。通过引用 const 来传递和返回字符串比通过值传递字符串更有效。最后但并非最不重要的一点是，让我们保持 const 正确。这是我对您的代码的清理：

#include <iostream>

class object
{
    short id;
    std::string name;
    SDL_Rect offset;

public:

    object(short i = 0, const std::string& n = "");
    const std::string& get_name() const;
    void set_name(const std::string& n);
};

object::object(short i, const std::string& n) : id(i), name(n)
{
    offset.x = 0;
    offset.y = 0;
    offset.w = 0;
    offset.h = 0;
}

const std::string& object::get_name() const
{
    return name;
}

void object::set_name(const std::string& n)
{
    name = n;
}

int main(int argc, char** argv)
{
    object a(0, "test");
    object b(1, "another test");

    std::cout << a.get_name() << "\n";
    std::cout << b.get_name() << "\n";

    b = a;
    std::cout << b.get_name() << "\n";

    a.set_name("Another test");
    std::cout << a.get_name() << "\n";
}

Answer 2

delete &name;

您只能删除通过调用new获得的指针。 name是类的成员变量；您无法删除它，也永远不需要删除它。当类被销毁时，它也会销毁所有成员变量。

仅当您的成员是指针时（例如，如果您有 std::string* name），您就必须确保正确清理它们，但即便如此，您也应该更喜欢使用 smart诸如 scoped_ptr、shared_ptr 或 unique_ptr 之类的指针（如果您的实现支持的话）。

Answer 3

name 是一个string 并管理自己的内存。您在结构中的声明是正确的，您可以在已经存在的任何值之上分配。当结构超出范围时也无需执行任何操作。

请注意，您的 get_name 方法返回一个新副本，因为它按值返回。您可以像这样返回对结构中内容的引用，前提是在所属结构超出范围后不使用返回值：

const string& get_name() { return name; }

您应该按如下方式更改 setter 声明，因为传递副本没有任何好处（过多传递按值是新 C++ 开发人员中的常见错误）。

void set_name(const string& newValue)