Question

我是C ++编码和STL库使用的新手。我对在我开发的C ++程序中使用STL库非常感兴趣。但是最近我尝试使用Visual Leak探测器并发现了许多泄漏，特别是在哪里我使用过STL库和new运算符。

由于它是一个运行时应用程序，我还发现当我连续运行代码4-5天时，CPU内存不断增加。在一些论坛和其他注意事项上学习，我了解到我需要在调用相应的std::map之前删除/删除std::unordered_map，std:set，clear中的内容。我的问题是，如果我在析构函数调用中执行所有那些永远不会被调用的函数（因为我的程序员永远不会停止），那么每次访问它时STL库都会创建内存泄漏吗？

有替代方案吗？

示例：common.h

struct stTagElem_t {
    uint32_t  m_unOffset;
    uint32_t  m_unArraySize;
    string    m_acTagName;
    string    m_acTagValue;
    string    m_acDataType;
} ;

typedef std::unordered_map<uint8_t, stTagElem_t> tagsList_t;

struct stMappingElem_t {
    uint16_t     m_unMemSize;
    tagsList_t   m_stTagsList;
    string       m_acEventName;
    string       m_acMapTypeName;
} ;

typedef std::unordered_map<uint32_t, stMappingElem_t> eventDataMap_t;

dataMapManager.cpp //声明为全局列表，因为dataMapManager.cpp具有所有静态函数

eventDataMap_t sm_eventDataMapList;

所以当我使用Visual Leak Detector时它会说：

---------- Block 596 at 0x00DE8DB8: 120 bytes ----------


Leak Hash: 0xB26C3A34, Count: 1, Total 120 bytes
  Call Stack (TID 11580):
    c:\program files (x86)\microsoft visual studio 14.0\vc\include\xmemory0 (977): example.exe!std::_Wrap_alloc<std::allocator<std::_List_node<std::pair<unsigned __int64 const ,stMappingElem_t>,void *> > >::allocate()
    c:\program files (x86)\microsoft visual studio 14.0\vc\include\list (730): example.exe!std::_List_alloc<std::_List_base_types<std::pair<unsigned __int64 const ,stMappingElem_t>,std::allocator<std::pair<unsigned __int64 const ,stMappingElem_t> > > >::_Buynode0() + 0x11 bytes
    c:\program files (x86)\microsoft visual studio 14.0\vc\include\list (716): example.exe!std::_List_alloc<std::_List_base_types<std::pair<unsigned __int64 const ,stMappingElem_t>,std::allocator<std::pair<unsigned __int64 const ,stMappingElem_t> > > >::_Buyheadnode()
    c:\program files (x86)\microsoft visual studio 14.0\vc\include\list (649): example.exe!std::_List_alloc<std::_List_base_types<std::pair<unsigned __int64 const ,stMappingElem_t>,std::allocator<std::pair<unsigned __int64 const ,stMappingElem_t> > > >::_List_alloc<std::_List_base_types<std::pair<unsigned __int64 const ,stMappingElem_t>,std::al() + 0x8 bytes
    c:\program files (x86)\microsoft visual studio 14.0\vc\include\list (826): example.exe!std::_List_buy<std::pair<unsigned __int64 const ,stMappingElem_t>,std::allocator<std::pair<unsigned __int64 const ,stMappingElem_t> > >::_List_buy<std::pair<unsigned __int64 const ,stMappingElem_t>,std::allocator<std::pair<unsigned __int64 const ,stMappin()
    c:\program files (x86)\microsoft visual studio 14.0\vc\include\list (899): example.exe!std::list<std::pair<unsigned __int64 const ,stMappingElem_t>,std::allocator<std::pair<unsigned __int64 const ,stMappingElem_t> > >::list<std::pair<unsigned __int64 const ,stMappingElem_t>,std::allocator<std::pair<unsigned __int64 const ,stMappingElem_t> >()
    c:\program files (x86)\microsoft visual studio 14.0\vc\include\xhash (197): example.exe!std::_Hash<std::_Umap_traits<unsigned __int64,stMappingElem_t,std::_Uhash_compare<unsigned __int64,std::hash<unsigned __int64>,std::equal_to<unsigned __int64> >,std::allocator<std::pair<unsigned __int64 const ,stMappingElem_t> >,0> >::_Hash<std::_Umap_tra() + 0xF bytes
    c:\program files (x86)\microsoft visual studio 14.0\vc\include\unordered_map (119): example.exe!std::unordered_map<unsigned __int64,stMappingElem_t,std::hash<unsigned __int64>,std::equal_to<unsigned __int64>,std::allocator<std::pair<unsigned __int64 const ,stMappingElem_t> > >::unordered_map<unsigned __int64,stMappingElem_t,std::hash<unsigned __int6()
    g:\mywork\src\datamapmanager.cpp (15): example.exe!`dynamic initializer for 'sm_eventDataMapList''() + 0xD bytes
    d:\rs1\minkernel\crts\ucrt\src\appcrt\startup\initterm.cpp (22): ucrtbased.dll!_initterm()
    f:\dd\vctools\crt\vcstartup\src\startup\exe_common.inl (221): example.exe!__scrt_common_main_seh() + 0xF bytes
    f:\dd\vctools\crt\vcstartup\src\startup\exe_common.inl (296): example.exe!__scrt_common_main()
    f:\dd\vctools\crt\vcstartup\src\startup\exe_main.cpp (17): example.exe!mainCRTStartup()
    KERNEL32.DLL!BaseThreadInitThunk() + 0x24 bytes
    ntdll.dll!__RtlUserThreadStart() + 0x2F bytes
    ntdll.dll!_RtlUserThreadStart() + 0x1B bytes
  Data:
    38 09 E6 00    F0 98 E8 00    CD CD CD CD    CD CD CD CD     8....... ........
    CD CD CD CD    CD CD CD CD    CD CD CD CD    CD CD CD CD     ........ ........
    CD CD CD CD    CD CD CD CD    CD CD CD CD    CD CD CD CD     ........ ........
    CD CD CD CD    CD CD CD CD    CD CD CD CD    CD CD CD CD     ........ ........
    CD CD CD CD    CD CD CD CD    CD CD CD CD    CD CD CD CD     ........ ........
    CD CD CD CD    CD CD CD CD    CD CD CD CD    CD CD CD CD     ........ ........
    CD CD CD CD    CD CD CD CD    CD CD CD CD    CD CD CD CD     ........ ........
    CD CD CD CD    CD CD CD CD                                   ........ ........

更多地图：common.h

using namespace std;

#ifdef USE_64BIT
typedef uint64_t   ID;
#else
typedef uint32_t   ID;
#endif

typedef ID EVENT_UID;

struct stEventParams_t {
    EVENT_UID  m_unEventID;
    uint8_t    *m_ptrMemAddr;
    size_t     m_dataSize;
} ;

struct stTagElem_t {
    uint32_t  m_unOffset;
    uint32_t  m_unArraySize;
    string    m_acTagName;
    string    m_acTagValue;
    string    m_acDataType;
} ;

typedef std::unordered_map<uint8_t, stTagElem_t> tagsList_t;

struct stMappingElem_t {
    uint16_t     m_unMemSize;
    tagsList_t   m_stTagsList;
    string       m_acEventName;
    string       m_acMapTypeName;
} ;

typedef std::unordered_map<EVENT_UID, stMappingElem_t> eventDataMap_t;

struct stAppBlocInfo_t {
    string   m_acAppBlocName;
    string   m_acInstanceName;
    string   m_acDataMap_r_SPError;
    string   m_acTypeName;
} ;

typedef std::unordered_map<string, string> instanceTypeList_t;

typedef struct {
    stAppBlocInfo_t    m_acServiceProvider;
    instanceTypeList_t m_acParamsList;
} stSP_Params_t;

typedef std::unordered_map<string, stSP_Params_t> serviceProvider_t;

struct stServiceParams_t {
    bool               m_bIsItOutput;
    uint8_t            m_unIdxToEvtClient;
    EVENT_UID          m_unEventID;
    string             m_acInstanceName;
    string             m_acTypeName;
    instanceTypeList_t m_acParamsList;
} ;

typedef std::unordered_map<ID, stServiceParams_t> serviceParams_t;

struct stStatusElem_t {
    bool                m_bErrorPresent;
    std::string         m_acSPError;
    instanceTypeList_t  m_ServiceStatus;
};

typedef std::unordered_map<CSI_base *, stSP_Params_t > spInstanceList_t;
typedef CSI_base* (*CREATE_SI)(stSP_Params_t, eventDataMap_t, serviceParams_t);
typedef std::map<std::string, HINSTANCE> extnSiObjList_t;
struct AppBloc_Elements {
APPBLOCNAME      m_acAppBlocName;
CFBNwExecutor   *ptrFbnExec;
};

typedef std::map<APPLET_ID, AppBloc_Elements> AppBlocElemList;
typedef std::map<APPBLOCNAME, APPLET_ID> AppBlocsList;

我的整个应用程序包含50多个cpp文件，我在大多数用例中都使用标准STL库。

Answer 1

回答你的问题＆＃34;每次访问时，STL库都会造成内存泄漏吗？＆＃34; 否。标准库容器会根据操作需要分配内存，但是您必须非常努力地使它们实际泄漏自己的内存。您提供的VLD输出显示由unordered_map分配的内存;它没有被泄露，它被地图用来存储你的数据。如果清除地图，将释放此内存。

为什么内存使用量增加有两种可能性。首先，您可能会无限期地将越来越多的数据推送到某个容器中。例如。您可能正在创建新事件并将其放入sm_eventDataMapList而不删除旧事件。在这种情况下，无论是标准容器还是手卷结构，内存都必须上升。这很容易被VLD发现，因为随着时间的推移它会在某个特定的地方显示越来越多的分配。这不是容器中的泄漏，这是代码中的错误。

第二种可能性，也许这就是你发生的事情 - 是你泄漏了由new分配的内存。删除分配有new的所有对象是否责任是否已不再使用。容器不会自动完成。智能指针可以帮助您完成此任务，但我建议您仔细查看您使用new的所有位置，并考虑您确实需要以这种方式分配对象。通常，您可以采用不同的方式做事，并获得更简单，更安全的代码。

E.g。在这种情况下：unordered_map<CSI_base *, stSP_Params_t > - 使用指向对象的指针作为键。在您从地图中删除该元素后，您是delete或以其他方式处置该对象吗？

另一个例子：

struct stEventParams_t {
    EVENT_UID  m_unEventID;
    uint8_t    *m_ptrMemAddr;
    size_t     m_dataSize;
} ;

我将假设此结构拥有m_ptrMemAddr指向的内存，m_dataSize是该内存块的大小。在这种情况下，每次移除delete[]实例时都需要记住stEventParams_t这个内存，这很容易忘记。

更好的方法 - 让结构删除自有资源：

struct stEventParams_t {
    ~stEventParams_t() {delete[] m_ptrMemAddr;}
    EVENT_UID  m_unEventID;
    uint8_t    *m_ptrMemAddr;
    size_t     m_dataSize;
} ;

更好 - 使用智能指针：

struct stEventParams_t {
    EVENT_UID  m_unEventID;
    std::unique_ptr<uint8_t[]> m_ptrMemAddr;
    size_t     m_dataSize;
} ;

甚至更好 - 使用适当的标准容器（不能泄漏内存，不需要单独的size，因为容器知道它的长度）：

struct stEventParams_t {
    EVENT_UID  m_unEventID;
    std::vector<uint8_t> m_data;
} ;

STL替代内存泄漏

1 个答案: