在阅读"可能丢失"与Valgrind阻止消息似乎他们很糟糕。
我收到静态指针类成员的错误。 我想验证我们的代码没有任何问题。
我从Valgrind那里得到这个:
==27986== 76 bytes in 1 blocks are possibly lost in loss record 370 of 1,143
==27986== at 0x4C247F0: operator new(unsigned long) (vg_replace_malloc.c:319)
==27986== by 0x107CFEE8: std::string::_Rep::_S_create(unsigned long, unsigned long, std::allocator<char> const&) (new_allocator.h:94)
==27986== by 0xDDCE21F: char* std::string::_S_construct<char const*>(char const*, char const*, std::allocator<char> const&, std::forward_iterator_tag) (basic_string.tcc:140)
==27986== by 0x107D19B2: std::basic_string<char, std::char_traits<char>, std::allocator<char> >::basic_string(char const*, std::allocator<char> const&) (basic_string.h:1722)
==27986== by 0xD381F19: MyNamespace::MyClass::MyMethod() (MyClass.cpp:189)
==27986== by 0xD1A6E17: __static_initialization_and_destruction_0(int, int) (IProperty.cpp:520)
==27986== by 0xD1B2B97: _GLOBAL__sub_I_IProperty.cpp (IProperty.cpp:551)
==27986== by 0x400D4F2: call_init (in /lib64/ld-2.5.so)
==27986== by 0x400D5B4: _dl_init (in /lib64/ld-2.5.so)
==27986== by 0x4000AA9: ??? (in /lib64/ld-2.5.so)
此错误所指的我的类(简体): 我已将代码简化为在此处发布,但如果需要,我可以添加更多详细信息。
MyClass.h
class MyClass
{
private:
double _p1, _p2, _p3, _p4;
std::string _p5, _p6, _p7;
public:
MyClass(double p1, double p2, double p3, double p4, std::string p5, std::string p6, std::string p7)
{
_p1 = p1;
_p2 = p2;
_p3 = p3;
_p4 = p4;
this->_p5 = p5;
this->_p6 = p6;
this->_p7 = p7;
}
static MyClass& MyMethod();
}
MyClass.cpp
static MyClass* _myPtr = NULL;
MyClass& MyClass::MyMethod()
{
if (!_myPtr )
{
_myPtr = new MyClass(1, 2.1, 3, 4, "xxxx", "yyyyy", "zzzzz");
}
return *_myPtr ;
}
我认为我们正在使用静态指针。但是,文档说这通常是内存泄漏,除非你用你的指针做一些有趣的事情。我认为我们没有做任何有趣的事情。
这个错误实际上是指构造函数作为参数接收的字符串类的内部指针吗?
我们是否应该担心此可能会丢失阻止错误?
答案 0 :(得分:3)
这是一个内存泄漏,并不重要,因为这个对象应该和程序本身一样长,但MyClass的析构函数永远不会被调用并且指针被释放。如果MyClass使用一些可能存在问题的程序外部资源。
试
MyClass& MyClass::MyMethod()
{
static MyClass instance(1, 2.1, 3, 4, "xxxx", "yyyyy", "zzzzz");
return instance;
}
将调用析构函数(当所有静态对象都被破坏时 - 退出main之后)。
在C ++ 11中,它甚至被授予线程安全性。
答案 1 :(得分:0)
根据定义,您的程序存在内存泄漏,但这是一个良性的,因为未释放的堆内存量是有限的。除此之外,确保在程序退出时调用所有析构函数是一种好习惯。例如。考虑到,在某个时候,您希望通过记录到文件流来扩展MyClass
。如果不销毁流对象,则文件句柄仍将在每个主要操作系统上关闭,没有资源泄漏,但是将丢弃在进程内缓冲的任何数据。此外,很容易避免这种泄漏。即使您想懒惰地创建对象,也可以使用std::unique_ptr
或boost::optional
来确保在退出时正确销毁它。
也就是说,如果您确实希望保持代码不变,则可以创建suppression for Valgrind,并且在使用相应抑制文件的后续运行中将不再列出该块。