Question

我目前正在进行一项任务，我正在努力调试我的内存泄漏。我的程序运行并通过，但是当我上传它以进行评分我的代码有内存泄漏。如果有人能指出我如何解决它的正确方向，我将不胜感激！

我已经尝试了我能想到的一切！

#ifndef A2_HPP
#define A2_HPP

#include <algorithm>

class sorted_sc_array {
public:

/*
 * return: none
 * constructor with no argument assign size_ = 0 and ptr_ to null pointer
 */
sorted_sc_array() : size_(0), ptr_(nullptr) {
}

/*
 * return: none
 * destructor delete the pointer ptr_
 */
~sorted_sc_array() {
    delete[] ptr_;
}

/*
 * return: none
 * when assign an object to new object
 */
sorted_sc_array(const sorted_sc_array& A){
    const signed char* str = A.data();
    int sz = A.size_;
    this->size_ = 0;
    for(int i = 0; i < sz; i++) this->insert(str[i]);
    delete[] str;
}

/*
 * return: sorted_sc_array
 * overloading of operator =
 */
sorted_sc_array& operator=(const sorted_sc_array& A){
    const signed char* str = A.data();
    int sz = A.size_;
    this->size_ = 0;
    for(int i = 0; i < sz; i++) this->insert(str[i]);

}

/*
 * return int
 * return the size of the ptr_
 */
int size() const {
    return size_;
}

/*
 * return char*
 * return the deta stored in the pointer ptr_
 */
const signed char* data() const {
    return ptr_;
}

/*
 * return void
 * add new char to the pointer ptr_ and sort the the new string after the addition
 */
void insert(signed char c) {
    signed char *str = (signed char*)malloc((size_ + 1)*sizeof(char));
    for(int i = 0; i < size_; i++) str[i] = ptr_[i];
    str[size_++] = c;
    ptr_ = (signed char*)malloc((size_)*sizeof(char));
    for(int i = 0; i < size_; i++) ptr_[i] = str[i];
    std::sort(ptr_, ptr_ + size_);
    delete[] str;
}


private:
int size_; // size of the array
signed char* ptr_; // pointer to the array

}; // class sorted_sc_array

#endif // A2_HPP

这是测试类：

/*
* File: a2.pp
* Description: testing class a2.hpp
*/

#include <iostream>
#include "a2.hpp"


int main(int argc, char* argv[]) {
sorted_sc_array A;

{
    sorted_sc_array T;
    for (signed char c = -128; c < 127; ++c) T.insert(c);

    T = T;

    sorted_sc_array V = T;
    A = V;
}

const auto first = A.data();
const auto last = first + A.size();

auto size = A.size();
bool res = std::is_sorted(first, last);

if (!res || (A.size() != 255)) std::cout << "fail";
else std::cout << "pass";

std::cout << std::endl;

return 0;
} // main

代码编译并执行，使用＆＃34;传递＆＃34;但是某处有内存泄漏！：（

Answer 1

显示的代码中存在多个错误。

在复制构造函数中：

delete[] str;

这会删除其他对象的缓冲区。当它被破坏时，另一个对象的析构函数将再次尝试delete[]它自己的缓冲区。这将导致内存损坏和未定义的行为。

显而易见的内存泄漏发生在insert()：

中

ptr_ = (signed char*)malloc((size_)*sizeof(char));

这里有两个同步错误。

使用malloc作为析构函数最终delete[]的缓冲区。只有new - ed对象可以是delete[] d。对delete内容使用malloc是未定义的行为。
ptr_的先前内容未明确delete d，因此泄漏内存。

insert()

delete

ptr_

C ++调试帮助内存泄漏

1 个答案: