0

请看内联评论

bool res = false;
DWORD dwNeeded = DocumentPropertiesW(NULL, m_currPrinterHandle, (LPWSTR) m_currPrinterName.c_str(), NULL, NULL, 0); 
if (m_devmode_buf)
{
    GlobalFree(m_devmode_buf);      
}
m_devmode_buf = GlobalAlloc(GPTR, dwNeeded);
GetLastError(); // = 0;
if (m_devmode_buf)
{
    LPDEVMODEW devmode_buf = (LPDEVMODEW) GlobalLock(m_devmode_buf);        
    GetLastError(); // = 0
    if (devmode_buf)
    {           
        if (devmode_buf)
        {
            lala = DocumentPropertiesW(NULL, m_currPrinterHandle, (LPWSTR) m_currPrinterName.c_str(), devmode_buf, NULL, DM_OUT_BUFFER);
            if (lala == IDOK)
            {
                res = true;
            }
            GetLastError(); // = 122. insufficient buffer here. why????
        }
        UInt32 res1 = GlobalUnlock(m_devmode_buf); // res1 is 1. should be 0
        res2 = GetLastError(); // = 0
        if (!(res1 == 0 && (res2 == ERROR_NOT_LOCKED || res2 == NO_ERROR)))
        {
            //res = false;
        }           
    }
}
4

3 回答 3

1

如果对 DocumentProperties() 的第二次调用返回 1（即 IDOK），那么它没有失败，因此 GetLastError() 的值是没有意义的。这可能是在 DocumentProperties() 内部提出和处理的预期条件。使用 GetLastError() 的惯例是你只在失败时设置它；你通常不会在成功时清除它。由每个单独函数的文档来解释如何返回错误。DocumentProperties() 的文档甚至没有提到 GetLastError()，因此检查它可能毫无意义（尽管通常可以安全地假设所有 Win32 函数都通过 GetLastError() 返回错误）。

于 2010-05-11T18:27:41.257 回答
0

由于您将 GPTR 传递给 GlobalAlloc，因此您不需要调用 GlobalLock。只需要在传递 GMEM_MOVEABLE 时调用 GlobalLock。

但是，您不应该使用 GlobalAlloc/GlobalFree，除非您传递内存的 API 文档另有说明。更喜欢 HeapAlloc/HeapFree 或只是 new/delete。GlobalAlloc 是一个较旧的 API，旨在与 16 位 Windows 兼容。

于 2010-05-10T23:04:14.063 回答
0

在某些情况下，DocumentProperties 将为某些机器的 DEVMODE 大小返回 -1 - MS 论坛上有一个关于它的完整主题（大约从 2008 年开始）但 MS 并不认为这是一个问题，尽管他们的示例代码从未检查返回代码（或 PrintDlg() 常见对话框，它愉快地尝试分配 -1 内存并失败）。

你不能仅仅依赖这个功能，因为它可以在你的机器上工作，但在客户的机器上却失败了。检查 -1，如果它返回只是组成一个大数字（2*sizeof(DEVMODE) 或其他东西）并使用它。

于 2012-05-30T10:23:11.887 回答