当我尝试清理 STL 向量时,会发生未经处理的异常

Unhandled exception occurs when I try to do cleanup for STL vectors

本文关键字:异常 处理 向量 STL      更新时间:2023-10-16

EDIT#1:

将 wchar_t 指针的向量替换为 wstring 的向量。

我仍然得到缓冲区溢出,但现在程序中崩溃的地方数量大大减少。

编辑#1.1:

    /****************************************************/
    /*                                                  */
    /*  It was the string parser that caused buffer     */
    /*  overflow !                                      */
    /*                                                  */
    /****************************************************/

这一定是对向量的不当使用,但这将留给另一个问题。

对于所有试图提供帮助的人,您从我这里得到+1。

我试过你的答案,它们都奏效了。

由于我是初学者,我害怕使用wstring,但是由于社区的帮助,我认为我已经设法学到了一些新东西。

再次感谢大家。

再次感谢大家。

问候。

必要信息:

注意:

这是一个简化的描述,以便使这个问题尽可能简短。

欲了解更多信息,请提出其他问题,我将提供更多信息。

我有一个对话框,其中包含 2 个编辑控件、2 个按钮和组合框。

在创建对话框时,WM_INIDIALOG,会与数据库建立连接,该数据库保存有关员工年薪的数据。

然后将员工的主键加载到组合框中。

表格如下所示:

Table_Employee <#primary_key, ...>

Table_Salary <#pk_salary, $primary_key, 一月, 二月, ..., 年>

关系是一对多的,因为一个员工有关于每年月薪的数据,如下所示:

| 一月 |... |年份 |#pk_salary|$primary_密钥|| 1000.51 |... |2012年 |100025558 |48089989891 || 2000年51月 |... |2013年 |552025558 |48089989891 |...

用户从组合框中选择主键后,他可以通过在第一个编辑控件中键入有关月薪的数据来更改有关月薪的数据,并且他必须在第二个编辑控件中键入年份。

输入的数据保存在声明如下的向量中:

   INT_PTR CALLBACK dlgProcedure(HWND hwnd, UINT Message, 
           WPARAM wParam, LPARAM lParam )
   {
   static vector<wchar_t*> ee; // vector for monthly salaries and a year
   // this vector holds data for all thes
      tatic vector< vector<wchar_t*> >  Pee;  years
  ...
      case WM_INITDIALOG:
    {
                    // 12 months + 1 year = vector of 13 to store data 
                    ee.assign( 13, LoadedValue );                                   
                    ...

用户按下第一个按钮后,月份的数据保存在上面的向量中,如下所示:

      case IDC_BUTTON_MONTH:
            {
                // needed, since we have vector of wchar_t*
                wchar_t *temp = new wchar_t[50]; 
                GetDlgItemInt( ... , temp, ... );
            UINT i = // ordinal of the month taken from checkbox
            ee[ i ] = temp;

然后用户必须输入年份,按下第二个按钮后,它存储如下:

      case IDC_BUTTON_YEAR:
      {
         wchar_t *temp = new wchar_t[50]; // needed, since we have vector of wchar_t*
         GetDlgItemInt( ... , temp, ... );
         ee[12] = temp;
         // This means that all the data is collected
         // so we can store this year’s data in the vector for years
         Pee.push_back(ee);

这样,向量 Pee 保存所有年份的数据(2012 年、2013 年,...),而向量 ee 保存细节(某年的月薪)。

问题:

组合框中的选择更改后,我必须清除所有向量,以便存储新数据。

当我这样做时,我得到错误,我的程序折断。当我尝试关闭窗口时也会发生崩溃。

如果我注释掉清除向量的代码

部分,我的程序可以工作,但我不能使用它来存储新数据,因为向量没有正确清除。

重要信息:

一旦我启动程序并在组合框中更改选择,就会弹出一个对话框,其中提供了 2 个调试器,并显示以下消息:

SomeProgramName.exe[3300] 中发生了未处理的异常。

调试中,在MS Visual Studio 2008中,我单击了异常,并检查了万事。在调试模式下启动程序后,我收到带有以下消息的对话框:

这可能是由于堆损坏,这表示 MyProgramName 或它已加载的任何 DLL 中存在错误.exe该堆。

这也可能是由于用户按 F12 而 MyProgramName.exe 具有焦点。

输出窗口可能包含更多诊断信息。

正如我上面所说,在我注释掉清理代码后,错误不再发生。

这就是为什么我非常确定存在我的问题。

有问题的代码片段:

WM_CLOSE处理程序:

   case WM_CLOSE:
        {
            // cleanup 
            for( vector<wchar_t*>::size_type i = 0; i < ee.size(); i++)
                delete[] ee[i];
            ee.clear();
            for( vector< vector<wchar_t*> >::size_type i = 0; i < pee.size(); i++)
               for( vector<wchar_t*>::size_type j = 0; j < pee[i].size(); j++)
                  delete[] pee[i][j];
            pee.clear();
            DestroyWindow( hDlg );
        }
        return TRUE;

组合框的处理程序:

    case IDC_COMBO12:
        {
            if(HIWORD(wParam) == CBN_SELCHANGE )
            {
                // cleanup 
                for( vector<wchar_t*>::size_type i = 0; i < ee.size(); i++)
                   delete[] ee[i];
                ee.clear();
                for( vector< vector<wchar_t*> >::size_type i = 0; i < pee.size(); i++)
                    for( vector<wchar_t*>::size_type j = 0; j < pee[i].size(); j++)
                       delete[] pee[i][j];
                pee.clear();
                // assign neutral values to vectors
                ee.assign( 13, L”-1” );
                for( int i = 2012; i < currentYear; i++ )
                   Pee.push_back(ee);
                // other commands, like loading data from database and so on...

问题:

由于我有一个指针向量(vector < wchar_t* >),我相信我不能只使用clear()方法来清空向量,因为它会导致内存泄漏。

这就是为什么在我看来,我必须先删除指针,然后再使用clear()方法。这是我第一次使用wchar_t*向量,所以我问社区我在这里做错了什么?

我应该如何在处理程序中正确重置这些向量?

要求更多信息,我很乐意提供。

因为我有一个指针向量(向量 <wchar_t*> )

那里

有你的问题。不要使用指针向量。使用std::wstring向量。那么你就不必担心这些了。

编辑:要将std::wstringGetDlgItemText一起使用,您可以执行以下操作:

std::vector<std::wstring> ee;
wchar_t wchScratch[1024]; // should be big enough for any string you've added
GetDlgItemText(hWnd, iID, wchScratch, sizeof(wchScratch) / sizeof(wchScratch[0]));
ee.push_back(wchScratch);

或者,如果你想要更复杂的东西,你可以做:

int iLength = GetWindowTextLength(GetDlgItem(hWnd, iID));
std::wstring strTemp;
strTemp.resize(iLength + 1);
strTemp.resize(GetDlgItemText(hWnd, iID, &strTemp[0], strTemp.size()));
ee.push_back(strTemp);

基本上,从控件获取字符串的方式没有任何变化,只是更改了存储字符串的方式。

崩溃

的原因:你定义vector<wchar_t*>,用ee.assign( 13, L”-1” );wchar_t *temp = new wchar_t[50];填充你的向量,用delete[]删除。都不匹配。

delete[]运算符释放堆上分配的内存 ee.assign( 13, L”-1” );传递编译器分配的对象。这是错误的。这些不在堆上,您不应该释放该内存,但是当您调用delete[]时会这样做。

您可以将向量定义为 vector<std::wstring> 然后将其用作

std::vector<std::wstring> v;
v.assign(10,L"xyz");
std::wstring s = L"abc";
v.push_back(s);
... etc.
v.clear(); // call destructor for every std::wstring in the vector

在按钮处理程序中:

wchar_t temp[50];
GetDlgItemTxt( ... , temp, ... );
v[i] = temp; // creates a wstring object and copies temp in there

您可以像以前一样定义向量并使用new和delete,但这需要一个很好的理由。您负责为矢量中的对象分配的内存。您不能混合在堆上、堆栈上分配或静态分配的指针(例如字符串文字 L"-1")。

vector<wchar_t*> v;
v.assign(10, 0); // fill in with null pointers
wchar_t *ptr = new wchar_t[10];
... fill in ptr memory with data
v.push_back(ptr);
... etc.
// Now you need to free memory you allocated manually - You do not do that
// in case of std::vector<std::wstring>
for (vector<wchar_t*>::iterator it = v.begin(); it != v.end(); ++it)
    delete[] *it;
v.clear(); // clear function calls destructor for every object in the vector.
// There is no destructor for a pointer, hence the difference between
// deallocating vector<wstring> and vector<wchar_t*>