質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.77%

ボタンを押すともう一つウィンドウが開くプログラムを作りたい

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 601

tometome

score 26

そのプログラムを書いたのですが、なぜかボタンを押すとウィンドウが開く時と開かずに終了してしまう時があります。どこを修正すればいいでしょうか。win apiなのでコードが長くなりますが、(ファイルをアップロードする方法も分からないので)一応全部ここに載せます。あと面倒でなければこのコードの中で「こういう風には書いてしまうのは絶対避けるべき」などの点があったらそれも教えてください。


header.hpp

#include<windows.h>

//window placement
#define COORD_X 700
#define COORD_Y 0

//window size
#define W_LENTH 800
#define W_WIDTH 700

//Window Style
#define MAIN_WND_STYLE WS_OVERLAPPEDWINDOW
#define BSTYLE BS_PUSHBUTTON | WS_VISIBLE | WS_CHILD

class WINDOWOBJECT{
    HWND Button;
    HWND Wnd_s1;
public:
    void WM_CREATEfunc(HWND, LPCREATESTRUCT);
    void Button_func(HWND, LPCREATESTRUCT);
};

LRESULT CALLBACK WindowProc(HWND, UINT, WPARAM, LPARAM);
LRESULT CALLBACK WindowProc_s1(HWND, UINT, WPARAM, LPARAM);
void WM_COMMANDfunc(HWND, WPARAM, LPARAM, WINDOWOBJECT&);


WinMain.cpp

#include "header.hpp"

int WINAPI WinMain(HINSTANCE hInstance, HINSTANCE hPrevInst, LPSTR lpCmdLine, int nCmdShow){
    WNDCLASSEX wc;
    wc.cbSize = sizeof(WNDCLASSEX);
    wc.style = CS_HREDRAW | CS_VREDRAW;
    wc.lpfnWndProc = WindowProc;
    wc.cbClsExtra = 0;
    wc.cbWndExtra = 0;
    wc.hInstance = hInstance;
    wc.hIcon = LoadIcon(NULL, IDI_APPLICATION);
    wc.hCursor = LoadCursor(NULL, IDC_ARROW);
    wc.hbrBackground = (HBRUSH)GetStockObject(BLACK_BRUSH);
    wc.lpszMenuName = NULL;
    wc.lpszClassName = "MainWndClass1";
    wc.hIconSm = NULL;

    if(!RegisterClassEx(&wc))    return FALSE;

    HWND hWnd;
    hWnd = CreateWindowEx(WS_EX_CLIENTEDGE, wc.lpszClassName, "Title",
        MAIN_WND_STYLE, COORD_X, COORD_Y, W_WIDTH, W_LENTH,
        NULL, NULL, hInstance, NULL);

    if(hWnd == NULL)    return FALSE;

    ShowWindow(hWnd, nCmdShow);
    UpdateWindow(hWnd);

    MSG msg;
    BOOL bRet;

    while(bRet = GetMessage(&msg, NULL, 0, 0)){ 
        if(bRet == -1){
            MessageBox(NULL, "Message loop error", "alert", MB_OK);
            return 0;
        }

        TranslateMessage(&msg);
        DispatchMessage(&msg); 
    }

    return msg.wParam;    
}

LRESULT CALLBACK WindowProc(HWND hWnd, UINT uMsg, WPARAM wParam, LPARAM lParam){
    static WINDOWOBJECT WndOb;
    switch(uMsg){
        case WM_CREATE:    WndOb.WM_CREATEfunc(hWnd, (LPCREATESTRUCT)lParam);    break;
        case WM_COMMAND:    WM_COMMANDfunc(hWnd, wParam, lParam, WndOb);    break;
        case WM_DESTROY:    PostQuitMessage(0);    break;
        default:    return DefWindowProc(hWnd, uMsg, wParam, lParam);
    }

    return 0;
}


WM_COMMANDfunc.cpp

#include "header.hpp"

void WM_COMMANDfunc(HWND hWnd, WPARAM wParam, LPARAM lParam, WINDOWOBJECT &WndOb){
    switch(LOWORD(wParam)){
        case 10: WndOb.Button_func(hWnd, (LPCREATESTRUCT)lParam);
    }
}


WM_CREATEfunc.cpp

#include "header.hpp"

void WINDOWOBJECT::WM_CREATEfunc(HWND hWnd, LPCREATESTRUCT lpCreateStruct){
    Button = CreateWindow("button", NULL, BSTYLE, 250, 380, 60, 30, hWnd, (HMENU)10,
            lpCreateStruct->hInstance, NULL);
    UpdateWindow(Button);
}


Button_func.cpp

#include "header.hpp"

void WINDOWOBJECT::Button_func(HWND hWnd, LPCREATESTRUCT lpCreateStruct){
    WNDCLASSEX wc;
    wc.cbSize = sizeof(WNDCLASSEX);
    wc.style = CS_HREDRAW | CS_VREDRAW;
    wc.lpfnWndProc = WindowProc_s1;
    wc.cbClsExtra = 0;
    wc.cbWndExtra = 0;
    wc.hInstance = lpCreateStruct->hInstance;
    wc.hIcon = LoadIcon(NULL, IDI_APPLICATION);
    wc.hCursor = LoadCursor(NULL, IDC_ARROW);
    wc.hbrBackground = (HBRUSH)GetStockObject(WHITE_BRUSH);
    wc.lpszMenuName = NULL;
    wc.lpszClassName = "SubWndClass1";
    wc.hIconSm = NULL;

    RegisterClassEx(&wc);
    Wnd_s1 = CreateWindowEx(WS_EX_CLIENTEDGE, wc.lpszClassName, NULL,
        WS_VISIBLE | WS_CAPTION | WS_SYSMENU,
        COORD_X + 10, COORD_Y, W_WIDTH - 50, W_LENTH - 50,
        hWnd, NULL, lpCreateStruct->hInstance, NULL);

    ShowWindow(Wnd_s1, SW_SHOW);
    UpdateWindow(Wnd_s1);

    MSG msg;
    BOOL bRet;

    while(bRet = GetMessage(&msg, NULL, 0, 0)){ 
        if(bRet == -1){
            MessageBox(NULL, "Message loop error", "alert", MB_OK);
            break;
        }

        TranslateMessage(&msg);
        DispatchMessage(&msg); 
    }

    DestroyWindow(Wnd_s1);
}

LRESULT CALLBACK WindowProc_s1(HWND hWnd, UINT uMsg, WPARAM wParam, LPARAM lParam){
    switch(uMsg){
        case WM_DESTROY:    PostQuitMessage(0);    break;
        default:    return DefWindowProc(hWnd, uMsg, wParam, lParam);
    }

    return 0;
}

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

0

Windowシステムに対するいくつかの誤解があるようです。

  1. lParamは常にLPCREATESTRUCTというわけではない
  2. umsg == WM_COMMAND && LOWORD(wParam) == ボタンのID だからといって常にボタンが押下された通知とは限らない
    以上2点は、Notification Messages from Buttonsこちらを参照してください
  3. ウィンドウを開くたびにWindowClassの登録が必要ではない
    同一のウィンドウプロシージャであるならば使いまわしてもよい

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/08/13 17:49

    私はインターネットのサイトを見て、winapiの基本的な部分を勉強して、そのサイトにlParamをLPCREATESTRUCTにキャストしてhInstanceを使っていた部分があったので、何も考えずそれを続けてきて今まで何も起こらなかったので間違いに気が付かなかったです。hInstanceは元のをそのまま使うことにしました。指摘された3つのことを一気に直したので動作がおかしかった原因がどれなのかは分かりませんでしたが、修正後はウインドウが開かないということはなくなりました。回答ありがとうございます。おかげで解決しました。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.77%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る