Question

所以..我一直在努力解除分配数组我不知道为什么会有记忆泄漏，但不知何故有一个记忆泄漏除了在main函数中，我还没有在任何地方分配任何内存。

#include <iostream>
#include "Motorboat.h"
#include "Sailboat.h"

using namespace std;
void printSailBoats(Boat* arr[], int nrOfElements);
int main() {
    _CrtSetDbgFlag( _CRTDBG_ALLOC_MEM_DF | _CRTDBG_LEAK_CHECK_DF); // used to check for memoryleaks in debug mode

    Boat* test[4];
    int nrOfElements = 4;

    test[0] = new Motorboat("heeelllooo",15000,"v100");
    test[1] = new Sailboat("saaailboat",1004,43.5);
    test[2] = new Motorboat("ASDK",4932,"Blabla");
    test[3] = new Sailboat("DKEOK",4992,103.4);

    printSailBoats(test,nrOfElements);

    for(int i=0; i<4; i++) {
        delete test[i];
    }

    return 0;
}

void printSailBoats(Boat* arr[], int nrOfElements) {
        // prints all sailboats
}

编辑：添加了课程。 Boat.h：

#ifndef BOAT_H
#define BOAT_H
#include <string>
using namespace std;

class Boat {
    public:
        virtual void setModel(string newModel) = 0;
        virtual void setPrice(int newPrice) = 0;
        virtual string getModel() const = 0;
        virtual int getPrice() const = 0;
        virtual string getType() const = 0;
        virtual string toString() const = 0;
};
#endif

Sailboat.h：

#ifndef SAILBOAT_H
#define SAILBOAT_H
#include "Boat.h"

class Sailboat: public Boat {
    private:
        double sailArea;
        string model;
        int price;

    public:
        Sailboat(string model, int price, double sailArea);
        void setSailArea(double newSailArea);
        double getSailArea() const;
        string toString() const;
        void setModel(string newModel);
        void setPrice(int newPrice);
        string getModel() const;
        int getPrice() const;
        string getType() const;
};
#endif

Sailboat.cpp：

#include "Sailboat.h"

Sailboat::Sailboat(string model, int price, double sailArea) {
    this->model = model;
    this->price = price;
    this->sailArea = sailArea;
}

// Setters, getters and toString...

除了有一个字符串变量来存储引擎名称而不是sailarea之外，对于摩托艇类来说几乎是一样的。

Answer 1

漏洞在这里：

for(int i=0; i<4; i++) {
        delete test[i];
    }

它正在删除元素，就好像它们与基类的类型相同。 I.E.如果您在派生类中有任何“额外”信息，它将被泄露。

例如：

delete (Sailboat*)test[i]

与

不同

delete (Boat*)test[i]

您需要在删除之前将test [i]强制转换为适当的类型。回收你实例化的类型可能很困难，所以我建议你只使用智能指针，而不用担心删除。

编辑：此外，虚拟析构函数将解决此问题。我仍然都是聪明的指针;）

Answer 2

这可能是你的构造函数中的泄漏。我的建议是为你定义的每个类创建析构函数，以确保删除在构造函数中创建的任何对象。

Answer 3

你可以做的是添加

#define _CRTDBG_MAP_ALLOC
#include <Crtdbg.h>

在内存泄漏输出的情况下，它应该为您提供分配了剩余块的文件和行。

此外，在每个破坏者（船，帆船，摩托车）中放置printf("Destructor of xxxx\n");等。这些应该在删除时调用/打印。

但是如果基本调用的析构函数（Baot）被标记为虚拟，则只会调用它们。否则你只会被称为船舶破坏者（并且可能会丢失在帆船和摩托艇中分配的记忆）

Answer 4

看到定义后添加：

class Boat {
    public:
        Boat() 
            { }
        virtual ~Boat()  // <--- this is the real catch!
                  { } 

        ...
};

释放指针数组 - C ++

4 个答案: