我的C++文件中是否有可能的内存泄漏

Is there a possible memory leak in my C++ file?

本文关键字：内存 泄漏 有可能 是否 C++ 文件 我的      更新时间：2023-10-16

我正在使用另一个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......这是个问题吗？如果这是一个问题，我该如何解决这个问题？

感谢任何帮助！

附言

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

没有看到类定义就不可能说（标题）;如果size_等是这样的 boost::shared_array，或std::unique_ptr，没有泄漏。如果它们只是int*，则存在泄漏。

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

代码有泄漏。 构造函数分配内存。析构函数或其他一些函数必须在对象被破坏之前清理它

另一个问题是你的代码不遵守三法则（链接在这里和这里）

一旦您将编写如下代码：

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

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

相关文章：