ウィンドウをESCキーで閉じたい

前提・実現したいこと

ウィンドウを表示し、ESCキーでウィンドウを閉じる処理を書きましたが、
ESCキーを押してもウィンドウが閉じません。
ESCキーを押すとウィンドウが固まります。

c++
1#include <windows.h>
2
3//*****************************************************************************
4// マクロ定義
5//*****************************************************************************
6#define WND_CLASS_NAME TEXT("My_Window")
7
8#define SCREEN_WIDTH	(1280)				// ウインドウの幅
9#define SCREEN_HEIGHT	(720)				// ウインドウの高さ
10#define SCREEN_CENTER_X	(SCREEN_WIDTH/2)	// ウインドウの中心Ｘ座標
11#define SCREEN_CENTER_Y	(SCREEN_HEIGHT/2)	// ウインドウの中心Ｙ座標
12
13//*****************************************************************************
14// プロトタイプ宣言
15//*****************************************************************************
16LRESULT CALLBACK WindowProc(HWND hWnd, UINT Msg, WPARAM wParam, LPARAM lParam);
17
18// https://www.keicode.com/winprimer/wp11.php
19int WINAPI WinMain(
20    HINSTANCE hInstance,
21    HINSTANCE hPrevInstance,
22    LPSTR lpCmdLine,
23    int nCmdShow) {
24
25    HWND hWnd;
26    WNDCLASSEX wcl;
27
28    wcl.cbSize = sizeof(WNDCLASSEX);
29    wcl.hInstance = hInstance;
30    wcl.lpszClassName = WND_CLASS_NAME;
31    wcl.lpfnWndProc = WindowProc;
32    wcl.style = NULL;
33    wcl.hIcon = NULL;
34    wcl.hIconSm = NULL;
35    wcl.hCursor = LoadCursor(NULL, IDC_ARROW);
36    wcl.lpszMenuName = NULL;
37    wcl.cbClsExtra = 0;
38    wcl.cbWndExtra = 0;
39    wcl.hbrBackground = (HBRUSH)GetStockObject(WHITE_BRUSH);
40
41    if (!RegisterClassEx(&wcl)) {
42        return FALSE;
43    }
44
45    // ウィンドウの作成
46    hWnd = CreateWindowEx(
47        NULL,
48        WND_CLASS_NAME,
49        WND_CLASS_NAME,
50        WS_OVERLAPPEDWINDOW,
51        CW_USEDEFAULT,
52        CW_USEDEFAULT,
53        CW_USEDEFAULT,
54        CW_USEDEFAULT,
55        NULL,
56        NULL,
57        hInstance,
58        NULL);
59
60    if (!hWnd) {
61        return FALSE;
62    }
63
64    ShowWindow(hWnd, nCmdShow);
65
66    MSG msg;
67    BOOL bRet;
68
69    while ((bRet = GetMessage(&msg, hWnd, 0, 0)) != 0) {
70
71        if (bRet == -1) {
72            break;
73        }
74        else {
75            DispatchMessage(&msg);
76        }
77    }
78
79    return msg.wParam;
80}
81
82LRESULT CALLBACK WindowProc(
83    HWND hWnd,
84    UINT uMsg,
85    WPARAM wParam,
86    LPARAM lParam) {
87
88    switch (uMsg) {
89    case WM_DESTROY:
90        ::PostQuitMessage(0);
91        break;
92    // ここがうまくいかない 
93    case WM_KEYDOWN:				//----- キーボードが押された
94        switch (wParam) {
95        case VK_ESCAPE:					// [ESC]キーが押された
96            PostMessage(hWnd, WM_CLOSE, 0, 0);	// [x]が押されたように振舞う
97            return 0;
98        }
99        break;
100    case WM_MENUCHAR:
101        return MNC_CLOSE << 16;			// [Alt]+[Enter]時のBEEPを抑止
102    default:
103        break;
104    }
105
106    return DefWindowProc(hWnd, uMsg, wParam, lParam);
107}

補足情報（FW/ツールのバージョンなど）

Windows10
Visual Studio 2019

行動規範の内容に同意します

回答5件

おそらくこちらの事象ではないかと思います。
自分の Windows 10 環境で事象再現し IME を古いバージョンにもどしたところ ESC キーが効くことを確認しました。

■ Windows10 2004 以降のMicrosoft IMEで発生するフリーズについて
https://social.msdn.microsoft.com/Forums/sqlserver/ja-JP/6b26361e-8757-4776-b97d-c307765c07f5/windows10-2004-201973847712398microsoft?forum=vcgeneralja

■ Windows10 アップデート後に入力操作で不具合が発生した時の対処法
https://www.pasoble.jp/windows/10/ime-nyuuryoku-fuguai.html

投稿2022/02/09 13:30

cx20

総合スコア4633

提示コードをコピペして試した感じ，

GetMessage(&msg, hWnd, 0, 0)

の第2引数を NULL にすれば良さそうに見えます．
（もとの状態でデバッグ実行してるとウィンドウが突然固まる模様？）

投稿2022/02/09 02:48

fana

総合スコア11658

fana

2022/02/09 03:09

元のコードだと，どうにもキーを押すと固まってしまうようだ．また，仮にウィンドウを閉じるところまでこぎつけられたとしても，最終的に　PostQuitMessage　で生成された WM_QUIT を処理できなくて困るんじゃないかな…？

Bull

2022/02/09 07:59

> GetMessage(&msg, hWnd, 0, 0) の hWnd は NULL にするのが正しいと思います。ウィンドウが閉じられると hWnd が無効になるので GetMessage() が失敗して、-1 が返されて、結果的にプログラムが終了するような気がします。当方の環境では、元のコードでも ESC キーで終了できているようです。

fana

2022/02/09 09:01 編集

私はVS2017で試してましたがデバッグ実行だろうがなかろうが，キーを押した瞬間に（ESCキーでもaキーでも同様に）ハングな感じですね．ウィンドウプロシージャのWM_KEYDOWNイベントの処理まで行けてないっぽいです． DispatchMessage()呼び出しの直前に if( msg.message == WM_KEYDOWN ){ ... } とか入れてみてもここに来る前にハングしている．（GetMessage()が処理を返さない？）何なのだろう？（質問者はウィンドウが固まるなんていう症状が出るとは言ってないし）

futon03

2022/02/09 09:16

すみません、ESCキーを押すとウィンドウが固まります。書き忘れてました。 WndProc関数に問題があると思っていましたが、別のサイトから落としたプロジェクトを上記のWndProc関数に置き換えたところ、ESCキーで閉じることができました。どこが原因なのかわからないですが、自己解決の欄に詳しく記載しようと思います。また分かったことがあれば再度この記事を更新します。

行動規範の内容に同意します

自己解決

WndProc関数に問題があると思っていましたが、別のサイトから落としたプロジェクトを上記のWndProc関数に置き換えたところ、ESCキーで閉じることができました。
原因はまだわかっていないですが、目的を達成したので一度自己解決とさせていただきます。
また分かったことがあったらこの記事を更新します。

解答してくださった方々、ありがとうございました。

下のURLが使ったサンプルです。
https://gameworkslab.jp/2020/06/13/%e3%80%90directx11%e3%80%91%e3%83%86%e3%82%af%e3%82%b9%e3%83%81%e3%83%a3%e3%83%bc%e3%82%92%e4%bd%bf%e3%81%a3%e3%81%9f%e6%8f%8f%e7%94%bb/4/

投稿2022/02/09 09:24

futon03

総合スコア33

cx20

2022/02/11 06:13

おそらくこちらが原因ではないかと思います。 https://teratail.com/questions/blfuktkqnf3fdb#reply-guy8ui93k07ib3 Windows 10 で「新しい IME」を用いた場合、行儀の悪い(!?)メッセージループと競合しフリーズを招くようです。

futon03

2022/02/11 14:24

ご回答ありがとうございます。エラーというよりはバグの類なんですね。すっきりしました。

cx20

2022/02/12 09:51

> エラーというよりはバグの類なんですね。とはいえ１年以上経っても治らない為、バグではなく仕様、ということなのでしょう。。残念ながら、アプリケーション側での対処が必要なのではと思います。マイクロソフトのサイトに、GetMessage() の基本的なサンプルが載っていますので、こちらを参考にすると良いかと思います。質問文のコードとの主な違いは、・GetMessage() の第２引数に NULL を渡している・TranslateMessage() を処理しているの２点になります。 ■ メッセージとメッセージキューの使用 https://docs.microsoft.com/ja-jp/windows/win32/winmsg/using-messages-and-message-queues ■ GetMessage() の例 ----------------------------------------------------- while( (bRet = GetMessage( &msg, NULL, 0, 0 )) != 0) { 　　if (bRet == -1) 　　{ 　　　　// handle the error and possibly exit 　　} 　　else 　　{ 　　　　TranslateMessage(&msg); 　　　　DispatchMessage(&msg); 　　} } ----------------------------------------------------- ※ 見やすさの為、インデントに全角スペースを使用しています。

行動規範の内容に同意します

SendMessage( hWnd, WM_SYSCOMMAND, SC_CLOSE, 0 );
これが ×ボタンで実行されるメッセージのはず。

投稿2022/02/09 01:29

KOZ6.0

総合スコア2626

fana

2022/02/09 01:51 編集

Visual Studio が吐いた Win32デスクトップアプリのひな型コードのウィンドウプロシージャに case WM_KEYDOWN: if( wParam == VK_ESCAPE ){ PostMessage( hWnd, WM_CLOSE, 0,0 ); } else { return DefWindowProc(hWnd, message, wParam, lParam); } break; というcaseを追加するだけで閉じることができるようです．（つまり，PostMessage( hWnd, WM_CLOSE, 0,0 );　で良いのではないかと）

fana

2022/02/09 01:49

（そもそもメッセージを投げるんじゃなくて直接 DestroyWindow 呼ぶのではダメなのだろうか？）

KOZ6.0

2022/02/09 01:53

WM_CLOSE を送るなら、DestroyWindow が必要かと。 https://docs.microsoft.com/ja-jp/windows/win32/winmsg/wm-close

fana

2022/02/09 02:37

そこの Remarks にて > By default, the DefWindowProc function calls the DestroyWindow function to destroy the window. とされているので，WM_CLOSE が来た場合には単に DefWindowProc に流しておけば DestroyWindow されるでしょう．

KOZ6.0

2022/02/09 03:22

ふむ。とすると閉じないのは謎ですね。何らかの理由で ESC キーが検出できてないのかも。

行動規範の内容に同意します

WM_DESTROYメッセージのbreakを、return 0にしてもダメですか?

投稿2022/02/08 08:48

majiponi

総合スコア1720

futon03

2022/02/08 12:10

breakをreturn 0に変えてもウィンドウは閉じれませんでした。

majiponi

2022/02/08 22:56

ふむ…WM_KEYDOWNメッセージ自体は、正常に届いていますか? (ブレークポイントを設置する、メッセージを表示させるなどして確かめてください。)

futon03

2022/02/09 09:11

WndProc関数に問題があると思っていましたが、別のサイトから落としたプロジェクトを上記のWndProc関数に置き換えたところ、ESCキーで閉じることができました。どこが原因なのかわからないですが、自己解決の欄に詳しく記載しようと思います。また分かったことがあれば再度この記事を更新します。ありがとうございました。下のURLが使ったサンプルです。 https://gameworkslab.jp/2020/06/13/%e3%80%90directx11%e3%80%91%e3%83%86%e3%82%af%e3%82%b9%e3%83%81%e3%83%a3%e3%83%bc%e3%82%92%e4%bd%bf%e3%81%a3%e3%81%9f%e6%8f%8f%e7%94%bb/4/

行動規範の内容に同意します

あなたの回答