读的时候偶然间发现了书中源代码潜在的错误

[villager]

while (GetMessage(&msg, NULL, 0, 0))
{
TranslateMessage (&msg) ;
DispatchMessage (&msg) ;
}
return msg.wParam ;

上面这行代码是该书中常用的消息处理方式。当GetMessage函数调用失败的时候会返回-1。这个时候while循环的条件就成了:

while(-1) ;

编译出来的应用程序会出现致命错误！大家来想一想怎么来去掉这个潜在的错误。

[villager]

while (GetMessage(&msg, NULL, 0, 0) > 0)

怎么样？

[skywinner]

GetMessage does not remove WM_PAINT messages from the queue. The messages remain in the queue until processed.

Note that the function return value can be TRUE, FALSE, or -1. Thus, you should avoid code like this:

while (GetMessage( lpMsg, hWnd, 0, 0)) ...


The possibility of a -1 return value means that such code can lead to fatal application errors.

上面是delphi里的帮助文件中的

[villager]

VC6有问题，可能是编译器的问题。

[feiyingw]

为自作聪明干杯

[villager]

楼上那位什么意思？？？

[villager]

觉得不对就直说。

别像个太监一样阴阳快气的

[fireseed]

while(-1)

我想不出有什么问题……

while后面跟的是一个bool值，而按照C++语义，标准数据类型转换规则，从其它类型转为bool时，0是false，非0是true

[fireseed]

请villager(为什么我这么绝顶聪明 却还是菜鸟？) 自重，切莫引起争吵

谢谢合作！

[jeckyz]

// Start the message loop.

while( (bRet = GetMessage( &msg, NULL, 0, 0 )) != 0)
{
if (bRet == -1)
{
// handle the error and possibly exit
}
else
{
TranslateMessage(&msg);
DispatchMessage(&msg);
}
}

[villager]

试一试while(-1)会出现什么结果！~！！！！

不试就没有发言权。那个奶奶狗。我管你是不是版主。

[villager]

别的环境不知道，Win32下的HelloWin一定会出错。这是MSDN上就说过的。严禁直接使用GetMessage()放在while里的。

[fireseed]

to villager(为什么我这么绝顶聪明 却还是菜鸟？) 有技术问题可以讨论，但请不要骂人，说脏话。

我不知道你的编译环境是怎么设置的……

/* 以下代码在VC.net环境下编译通过，并正常运行 */
/* Creamdog保留所有权力 */

#include <iostream>
using namespace std;

void main( void )
{
while( -1 )
{
}
system( "pause" );
}

[fireseed]

to villager(为什么我这么绝顶聪明 却还是菜鸟？) 有技术问题可以讨论，但请不要骂人，说脏话。

我不知道你的编译环境是怎么设置的……

/* 以下代码在VC.net环境下编译通过，并正常运行 */
/* Creamdog保留所有权力 */

#include <iostream>
using namespace std;

void main( void )
{
while( -1 )
{
}
system( "pause" );
}

[XBug]

关注

[晨星]

不管怎么说，这确实是个问题，因为既然是错误，最好做特殊处理，以免引起更严重的问题。
我同意jeckyz(深圳)的方案。