[英]Valgrind found 3 memory leaks but I can't figure out where they are
我嘗試自己實現基本的String類,並且運行良好,但是Valgrind說存在3個內存泄漏,我不知道在哪里以及為什么。 我真的想刪除不再使用的所有內容(我今天開始使用Valgrind)。 現在,我真的很擔心我的基本C / C ++內存管理知識。 我對代碼中Valgrind發現泄漏(// VALGRIND)的位置進行了注釋。 我還上傳了此錯誤消息的屏幕截圖, 請點擊以查看屏幕截圖 。
編輯:我已經更新了屏幕截圖,所以您可以看到完整的輸出。
StringT.h
template<typename char_type = char>
class StringT {
public:
explicit StringT(const char_type *str) {
if (str != nullptr) {
size_t len = strlen(str);
m_str = new char_type[len + 1]; //VALGRIND: 6 bytes in 1 blocks are definitely lost in loss record 1 of 3
strcpy(m_str, str);
}
}
~StringT() {
delete [] m_str;
}
StringT(const StringT & other) {
size_t len = 0;
if (other.m_str) len = strlen(other.m_str);
m_str = new char_type[len + 1]; //VALGRIND: 6 bytes in 1 blocks are definitely lost in loss record 2 of 3
strcpy(m_str, other.m_str);
}
StringT(StringT && other) noexcept {
m_str = other.m_str;
other.m_str = nullptr;
}
StringT & operator+=(const StringT &other) {
if (other.m_str == nullptr) //when other str is empty just return current Str
return *this;
const size_t mysize{m_str ? strlen(m_str) : 0}; // check if not null then call strlen
const size_t osize{other.m_str ? strlen(other.m_str) : 0};
char *newStr = new char_type[osize + mysize + 1]; //VALGRIND: 11 bytes in 1 blocks are definitely lost in loss record 3 of 3
newStr[0] = '\0'; //strcat searches for '\0', so newStr has to be a valid String
if (m_str) strcat(newStr, m_str);
if (other.m_str) strcat(newStr, other.m_str);
delete[] m_str; //delete old string
m_str = newStr; //set member to new concatenated str
return *this;
}
size_t length() const {
if (!m_str) return 0;
return strlen(m_str);
}
friend
std::ostream &operator<<(std::ostream &out, StringT<> &other) {
if (other.m_str) out << other.m_str;
return out;
}
private:
char_type *m_str{nullptr};
};
main.cpp中
int main() {
const char *cArr = "Hello";
const char *cArr2 = "World";
StringT<char> hello(cArr);
StringT<char> world(cArr2);
StringT<char> emptyStr;
std::cout << "hello: " << hello << std::endl;
std::cout << "world: " << world << std::endl;
std::cout << "emptyStr: " << emptyStr << std::endl;
StringT<char> hCopy(hello);
StringT<char> wMove(std::move(world));
std::cout << "hCopy: " << hello << std::endl;
std::cout << "hCopy: " << hCopy << std::endl;
std::cout << "world: " << world << std::endl;
std::cout<< "wMove: " << wMove << std::endl;
std::cout<< "lenMove: " << wMove.length() << std::endl;
std::cout<< "lenEmptyStr: " << emptyStr.length() << std::endl;
hello += wMove;
std::cout<< "hello += world: " << hello << std::endl;
return 0;
}
您的刪除在這里:
StringT() {
delete [] m_str;
}
但這是構造函數,而不是析構函數。 😉
正如已經回答的, delete
需要在析構函數中。 但是,解決此問題的正確方法是在這種情況下不手動進行內存管理。 您應該為您的m_str
成員使用std::unique_ptr
:
std::unique_ptr<char_type[]> m_str;
這使您不必手動new
和delete
。 這也有助於防止異常情況下的內存泄漏。 即使delete
分配的所有內容,如果new
和delete
之間發生異常,仍然可能會發生內存泄漏。 unique_ptr
有助於防止此類泄漏。
您的課程只需要進行一些小改動:
template<typename char_type = char>
class StringT {
public:
StringT()
{}
explicit StringT(const char_type *str)
{
if (str != nullptr) {
size_t len = strlen(str);
m_str = std::make_unique<char_type[]>(len + 1);
strcpy(m_str.get(), str);
}
}
StringT(const StringT & other)
{
size_t len = 0;
if (other.m_str)
len = strlen(other.m_str.get());
m_str = std::make_unique<char_type[]>(len + 1);
strcpy(m_str.get(), other.m_str.get());
}
StringT(StringT && other) noexcept
{
m_str = std::move(other.m_str);
}
StringT & operator+=(const StringT &other)
{
if (other.m_str == nullptr)
return *this;
const size_t mysize{m_str ? strlen(m_str.get()) : 0};
const size_t osize{strlen(other.m_str.get())};
auto newStr = std::make_unique<char_type[]>(osize + mysize + 1);
newStr[0] = '\0';
if (m_str)
strcat(newStr.get(), m_str.get());
strcat(newStr.get(), other.m_str.get());
m_str = std::move(newStr);
return *this;
}
size_t length() const
{
if (!m_str)
return 0;
return strlen(m_str.get());
}
friend
std::ostream &operator<<(std::ostream &out, StringT<> &other)
{
if (other.m_str)
out << other.m_str.get();
return out;
}
private:
std::unique_ptr<char_type[]> m_str;
};
您會注意到此代碼中沒有對new
或delete
調用。 如果需要, m_str
會自動自動刪除分配的內存。
聲明:本站的技術帖子網頁,遵循CC BY-SA 4.0協議,如果您需要轉載,請注明本站網址或者原文地址。任何問題請咨詢:yoyou2525@163.com.