Question

我正在使用另一个C ++编码器为我提供的示例代码。我是C ++语言的新生，我想知道给我这个类文件中是否存在内存泄漏/错误（PlacementHead.cpp）：

#include "PlacementHead.h"
#include <string>
#include <iostream>
#include <string.h>

PlacementHead::PlacementHead(int width, int height, int gap, char* s) {
    width_ = width;
    height_ = height;
    gap_ = gap;
    size_ = (width*height)+1;
    set_ = new char[size_ + 1];
    from_ = new int[size_ + 1];
    original_ = new char[size_ + 1];
    strcpy(set_,s);
    strcpy(original_,s);
}

PlacementHead::~PlacementHead() {

}

int PlacementHead::getSize() { return size_; }
int PlacementHead::getHeight() { return height_; }
int PlacementHead::getWidth() { return width_; }
int PlacementHead::getGap() { return gap_; }

// Palauttaa indeksissä i olevan suuttimen
char PlacementHead::getNozzle(int i) {
    return set_[i-1];
}

// Asettaa indeksissä i olevan suuttimen
void PlacementHead::setNozzle(int i, char c) {
    set_[i-1] = c;
}

// Merkitsee suuttimen poimituksi poistamalla sen listasta
void PlacementHead::markNozzle(int i, int bankPos) {
    set_[i-1] = ' ';
    from_[i-1] = bankPos;
}

// Palauttaa seuraavan poimimattoman suuttimen indeksin
int PlacementHead::getNextUnmarkedPos() {
    for (int i=0; i<size_; i++) {
        if (set_[i]!=' ') {
            return i+1;
        }
    }
    return 0;
}

// Palauttaa suuttimen alkuperäisen sijainnin pankissa
int PlacementHead::getBankPos(int i) {
    return from_[i-1];
}

// Plauttaa alkuperäisen ladontapaan suutinjärjestyksen
void PlacementHead::reset() {
    //for (int i=0; i<size_; i++) {
    //  set_[i] = original_[i];
    //}
    strcpy(set_,original_);
}

// Tulostusmetodi
void PlacementHead::print() {
    std::cout << "ladontapaa:\n";
    for (int h=height_; h>0; h--) {
        for (int w=width_; w>0; w--) {
            int i = ((h-1)*width_)+w;
            std::cout << getNozzle(i);
        }
        std::cout << "\n";
    }
}

PlacementHead.h：

#ifndef PLACEMENTHEAD_H
#define PLACEMENTHEAD_H

class PlacementHead {
    public:
        PlacementHead(int size, int rows, int gap, char* s);
        ~PlacementHead();
        int getSize();
        int getHeight();
        int getWidth();
        int getGap();
        char getNozzle(int i);
        void setNozzle(int i, char c);
        void markNozzle(int i, int bankPos);
        int getNextUnmarkedPos();
        int getBankPos(int i);
        void reset();
        void print();
    private:
        char* set_;
        int* from_;
        char* original_;
        int size_;
        int width_;
        int height_;
        int gap_;
};

#endif

我注意到内存有动态分配，但我没有看到delete任何地方......这是一个问题吗？如果这是一个问题，我怎么能解决这个问题？

Thnx任何帮助！

P.S。

我注意到此示例中没有使用关键字class？...您能定义这样的类吗？

Answer 1

如果没有看到课程定义，就不可能说出来报头）;如果size_等等 boost::shared_array或std::unique_ptr，没有泄漏。如果它们只是int*，则会发生泄漏。

当然，没有C ++程序员会编写这种代码无论如何。该类将包含std::vector<int>和 std::string。从我们在这里看到的情况来看，作者不懂C ++。

Answer 2

另一个问题是您的代码不遵守三条规则（链接here和here）

一旦你编写了如下代码：

{
PlacementHead a(0,0,0,"asdsa");
PlacementHead b(0,0,0,"asdsa");
a = b; // line 1
} // here segfault

你将得到段错误，在line 1中，指针将从b复制到a，一旦你最终有了析构函数，指针将被删除两次，这是错误的。这称为浅拷贝，您需要深拷贝，其中将分配新数组。

Answer 3

代码泄漏。构造函数分配内存.Destructor或其他一些函数必须在对象被销毁之前清除它

我的C ++文件中是否存在内存泄漏？

3 个答案: