【问题标题】:When do I need to deallocate memory?我什么时候需要释放内存?
【发布时间】:2010-05-25 20:58:46
【问题描述】:

我在一个类中使用这个代码来让一个浏览器控件访问一个网站:

void myClass::visitWeb(const char *url)
{
    WCHAR buffer[MAX_LEN];
    ZeroMemory(buffer, sizeof(buffer));
    MultiByteToWideChar(CP_ACP, MB_ERR_INVALID_CHARS, url, strlen(url), buffer, sizeof(buffer)-1);

    VARIANT vURL;
    vURL.vt = VT_BSTR;
    vURL.bstrVal = SysAllocString(buffer);

    // webbrowser navigate code...

    VariantClear(&vURL);
}

我从另一个 void 函数调用 visitWeb,该函数在应用程序的 handlemessage() 上调用。 我需要在这里做一些内存释放吗?,我看到 VariantClear 正在释放 vURL,但我应该为缓冲区释放内存吗? 有人告诉我,在同一个应用程序中的另一个 bool 中,我不应该释放任何东西,因为当 bool 返回 true/false 时,一切都会清除,但是在这个 void 上会发生什么?

【问题讨论】:

  • 当你的意思是“另一个函数返回 void”时,请不要说“另一个 void”。当您了解您正在处理许多函数并且唯一的区别是返回类型时,您会看到相似之处。
  • 似乎“另一个布尔值”是指“另一个返回布尔值的函数”。这反过来表明他不了解变量如何超出范围。返回类型根本不影响这一点,事实上变量甚至在函数返回之前就可以超出内部范围。
  • 顺便说一句,你有一个缓冲区溢出。 MultiByteToWideChar 的最后一个参数应该是MAX_LEN(字符数)。您当前传递了MAX_LEN * sizeof(WCHAR)(字节数)。而且你不需要通过 strlen(url);而是通过 -1。

标签: c++ visual-c++ memory-management


【解决方案1】:

我认为您对内存管理的理解存在一些基本问题。在这种情况下,不,您不需要显式释放任何内存。你从来没有打电话给new,所以你不需要打电话给deletebuffer 只存在于栈中,当该方法返回时会消失。

【讨论】:

    【解决方案2】:

    如果可以的话,我建议这样做有点不同——我会从创建一个小类开始:

    class bstr { 
        VARIANT content;
    public:
        bstr(char const *url) { 
            WCHAR buffer[MAX_LEN] = {0};
            MultiByteToWideChar(CP_ACP, 
                                MB_ERR_INVALID_CHARS, 
                                url, 
                                strlen(url), 
                                buffer, 
                                sizeof(buffer)/sizeof(buffer[0])-1);
            content.V_T = VT_BSTR;
            content.bstrVal = SysAllocString(buffer);        
        }
    
        operator VARIANT const &() { return content; }
    
        ~bstr() { VariantClear(&content); }
    };
    

    然后您的代码将更改为:

    void myClass::visitWeb(const char *url) {
        your_control.Navigate(bstr(url));
    }
    

    所有的分配和释放都会从那里自动处理。

    即使您不使用这样的类,请注意对MultiByteToWideChar 的调用的更改。最后一个参数应该是缓冲区中WCHAR 元素的数量,而不是chars 的数量。实际上,您已经设置了缓冲区溢出...

    【讨论】:

    • 我没有使用std,我刚开始学习cpp,我想在学习如何使用std命名空间之前完成这个项目。还是谢谢。
    • @extintor:在这种情况下,删除它没问题(已经完成)。
    【解决方案3】:

    我没有看到任何news,所以我不希望看到任何deletes。

    我想我会查看SysAllocString() 的描述,看看它是否分配了您需要摆脱自己的任何内存。

    【讨论】:

    • 谢谢,我读到 VariantClear() 负责 SysAllocString() 分配的内容。
    • 是的,那么您可能只需要担心这些。
    • 正确; VariantClear 知道它必须调用SysFreeString,因为vURL.vt == VT_BSTR
    猜你喜欢
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 2014-08-05
    相关资源
    最近更新 更多