1

我继承了一些用 MFC C++ 编写的部分开发了 10 年的代码。我的任务是更新和完成应用程序。

在 CDialogChild 类中，它在 .h 文件中如下所示：

private:
static const int m_iDefaultDesktopSizeX = 1024;

static const int m_iDefaultDesktopSizeY = 740;

我仍在尝试找出代码及其背后的逻辑，但我决定动态获得解决方案。我从 .h 中删除了“静态”关键字以及幻数，然后我尝试在构造函数的初始化列表中定义它们：

CDialogChild::CDialogChild(CWnd* pParent /*=NULL*/)
: CDialog(CDialogChild::IDD, pParent) , //more variables,
m_iDefaultDesktopSizeX(GetSystemMetrics(SM_CXSCREEN)), m_iDefaultDesktopSizeY(GetSystemMetrics(SM_CYSCREEN))
{
//...
}

我的分辨率是 1366x768，我在 Visual Studio 2013 调试器中得到的数字各不相同，从“无法访问内存”到数百万的数字。

然后我决定尝试在构造函数中初始化它们，而不是在初始化列表中。我从两个变量中删除了标头中的“const”关键字，然后想我会打印到调试输出它们的值。这是我的代码（我知道它不是那么优雅，但仍在弄清楚 MFC 约定）：

#ifdef UNICODE
typedef std::wstringstream tstringstream;
#else
typedef std::stringstream tstringstream;
#endif

m_iDefaultDesktopSizeX = GetSystemMetrics(SM_CXSCREEN);
m_iDefaultDesktopSizeY = GetSystemMetrics(SM_CYSCREEN);    

#ifdef OUTPUT_DEBUG_STRING
    tstringstream ss1;
    tstringstream ss2;

    ss1 << m_iDefaultDesktopSizeX;
    //stream >> sizeX;

    ss2 << m_iDefaultDesktopSizeY;

    LPTSTR dest = _T("m_iDefaultDesktopSizeX = ");
    LPTSTR src1 = _T("");
    LPTSTR src2 = _T("");

    ss1 >> src1;
    ss2 >> src2;

    // 1000 is a magic number
    StringCchCat(dest, 1000, src1);
    StringCchCat(dest, 1000, _T(" and m_iDefaultDesktopSizeY = "));
    StringCchCat(dest, 1000, src2);
    OutputDebugString(dest);
#endif

当我运行它时，在“ss2 >> src2”上抛出一个异常，说访问冲突写入 [hex address]

那么，发生了什么以及我应该提供哪些更多信息？我遇到的问题是在对另一个问题进行故障排除期间，因此对其中一个或两个问题的帮助会很棒！

编辑

感谢 Mark 的评论，我将所有 LPTSTR 变量的初始化更改为“new TCHAR()”，从而解决了该特定问题。我能够在调试输出中运行字符串，它显示了正确的分辨率（即使它在 VS 调试器本身中是错误的），非常感谢！即使我将变量放回构造函数初始化列表并再次将它们设为“const”，这仍然有效:)

但是，我现在收到一个新错误。虽然这最终不是在这个#ifdef 块中发生的问题，但我现在得到的错误可能会在我编写的其他代码中弹出。

编码：

tstringstream ss1;
tstringstream ss2;

ss1 << m_iDefaultDesktopSizeX;
//stream >> sizeX;

ss2 << m_iDefaultDesktopSizeY;

LPTSTR dest = new TCHAR();
StringCchCat(dest, 300, _T("\nm_iDefaultDesktopSizeX = "));
LPTSTR src1 = new TCHAR();
LPTSTR src2 = new TCHAR();

ss1 >> src1;
ss2 >> src2;


StringCchCat(dest, 300, src1);
StringCchCat(dest, 300, _T(" and m_iDefaultDesktopSizeY = "));
StringCchCat(dest, 300, src2);
OutputDebugString(dest);

//error happens on the line below this one!!!!
delete dest;
dest = 0;
delete  src1;
src1 = 0;
delete  src2;
src2 = 0;

当运行时到达删除命令时会发生错误，并且 .exe 文件已触发断点。

4

1 回答 1

4

该错误很可能是由代码中的缓冲区溢出引起的，例如：

LPTSTR dest = new TCHAR();
StringCchCat(dest, 300, _T("\nm_iDefaultDesktopSizeX = "));

第一行分配一个字符，然后在第二行写大约 20-30 个字符。

不要打扰原始字符串指针，只需使用 CString 或 std::wstring：

tstringstream ss1;
tstringstream ss2;

ss1 << m_iDefaultDesktopSizeX;
ss2 << m_iDefaultDesktopSizeY;

CString dest = _T("\nm_iDefaultDesktopSizeX = ");
dest += ss1.str().c_str();
dest += _T(" and m_iDefaultDesktopSizeY = ");
dest += ss2.str().c_str();

OutputDebugString(dest);
于 2015-04-28T21:04:26.243 回答