プログラマ専用SNS ミクプラ

現在、別ウインドウに情報を表示するプログラムを書いていす。
ウインドウ関係の処理を行うクラスと、実際に情報を表示させる処理を行うクラスを作って、簡単に利用できるようにしようと考えたのですが、利用する際にわざわざnShowCmdを渡すのは頻雑です。
nShowCmdを取得できる関数はないのでしょうか？

► スポイラーを表示

コード:

#define _CRT_SECURE_NO_DEPRECATE

#include <windows.h>
#include "DxLib.h"
#include <string>
#include <vector>
#include <fstream>

using namespace std;

HBRUSH hBrush;

LRESULT CALLBACK WindowProc(HWND hwnd,UINT uiMsg,WPARAM wParam,LPARAM lParam){
	int childID = (int)GetWindowLongPtr((HWND)lParam, GWLP_ID);
	COLORREF wORb[2] = {RGB(255,255,255), RGB(200,255,255)};

	switch(uiMsg){

	case WM_CTLCOLORSTATIC:
		// 背景色を水色に設定
		if(childID>=100 && childID<124){
			SetBkColor( (HDC)wParam, wORb[childID%2]);
			hBrush = CreateSolidBrush(wORb[childID%2]);
			return (BOOL)hBrush;
		}

	default:

		return DefWindowProc(hwnd, uiMsg, wParam, lParam);
	}
}

class newwin{
   WNDCLASSEX wc;
	HWND mainWH;
	int childCou;
	char textCont[101];

public:

	newwin(){}
	void init(const string &title, int width, int height, int nC){
		childCou = 100;

		wc.cbClsExtra = 0;
		wc.cbSize = sizeof(WNDCLASSEX);
		wc.cbWndExtra = 0;
		wc.hbrBackground = (HBRUSH)(COLOR_APPWORKSPACE+1);
		wc.hCursor = LoadCursor(GetModuleHandle(NULL),IDC_ARROW);
		wc.hIcon = NULL;
		wc.hIconSm = NULL;
		wc.hInstance = GetModuleHandle(NULL);
		wc.lpfnWndProc = WindowProc;
		wc.lpszClassName = "myClass";
		wc.lpszMenuName = NULL;
		wc.style = CS_DBLCLKS;

		RegisterClassEx(&wc);

		mainWH = CreateWindowEx(
			WS_EX_TOOLWINDOW,
			"myClass",
			title.c_str(),
			WS_BORDER|WS_SYSMENU,
			CW_USEDEFAULT,CW_USEDEFAULT,
			width,height,
			NULL,NULL,GetModuleHandle(NULL),NULL
		);
	 
		ShowWindow(mainWH,nC);
		UpdateWindow(mainWH);
	}
	newwin(const string &title, int width, int height, int nC){ init(title, width, height, nC); }

	HWND child(const string &mode, const string &oput, int x, int y, int width, int height){
		return CreateWindow(
			mode.c_str(),
			oput.c_str(),
			WS_CHILD|WS_VISIBLE,	
			x,y,
			width,height,
			mainWH,
			(HMENU)childCou++,
			GetModuleHandle(NULL),
			NULL
		);
	}

	const char* gets(HWND hwnd){
		GetWindowText(hwnd, textCont, 100);
		return textCont;
	}

	void sets(HWND hwnd, const string &oput){
		SetWindowText(hwnd, oput.c_str() );
	}

	~newwin(){
		for(int i=100;i<(int)childCou;i++) DestroyWindow( (HWND)i);
		DestroyWindow(mainWH);
		DeleteObject( (HGDIOBJ)hBrush);
	}
};

class view{
	newwin w;
   WNDCLASSEX wc;
	HWND mainWH;
	HWND children[24];
	int next;

public:

	view(int nC){
		w.init("Information", 320, 480, nC);

		for(int i=0;i<24;i++) children[i] = w.child("STATIC", "", 0,19*i, 320,19);
		next = 0;
	}

	void clear(){
		for(int i=0;i<24;i++) w.sets(children[i], "");
		next = 0;
	}
	void add(const string &str){ w.sets(children[next++], str); }
	void set(const string &str, int num){ w.sets(children[num], str); }
};

int WINAPI WinMain(HINSTANCE hI,HINSTANCE hPrevInst,LPSTR lpCmdLine,int nC){
	ChangeWindowMode(TRUE);
	SetOutApplicationLogValidFlag(FALSE);
	SetWindowText("DxLib");
	if(DxLib_Init() == -1) return(-1);
	SetAlwaysRunFlag(TRUE);
	SetDrawScreen(DX_SCREEN_BACK);

	view v(nC);

	while(ProcessMessage() == 0 && CheckHitKey(KEY_INPUT_ESCAPE) == 0){
        ClsDrawScreen();

		v.clear();
		v.add("test");

		ScreenFlip();
	}


	DxLib_End();
	return 0;
}

また、現在のコードは上記のようになっているのですが、できれば　WindowProc()　関数を　newwin　クラスの中に入れてしまいたいです。しかしｳｲﾝﾄﾞｳﾌﾟﾛｼｰｼﾞｬ関数をクラスの中に入れてしまうと、　W N D C L A S S E X　構造体の　lpfnWndProc　メンバ変数に関数ポインタを渡すことができませんでした。

そして、　ウインドウを作成してからしばらく経つと、水色に設定したはずの背景が灰色になり、画面がリフレッシュされなくなります。whileループで　SetWindowText()　関数を呼び出しまくっているのが原因だということはわかったのですが、これは解決できるのでしょうか？

ご教授よろしくお願い致します。

MoNoQLoREATOR さんが書きました:ウインドウ関係の処理を行うクラスと、実際に情報を表示させる処理を行うクラスを作って、簡単に利用できるようにしようと考えたのですが、利用する際にわざわざnShowCmdを渡すのは頻雑です。
nShowCmdを取得できる関数はないのでしょうか？

WinMainのnShowCmd引数は、アプリケーションに指定されたウィンドウの初期状態(最大化・最小化・通常)を受け取るためのものですから、小ウィンドウにそのまま渡すとあまりよろしくないです。
メインウィンドウの状態に応じて、適切なパラメータをShowWindowに与えてください。
いちおう説明しておくとnShowCmdを使わなくてもSW_SHOWDEFAULTを指定すると同じことになるようです。

MoNoQLoREATOR さんが書きました:また、現在のコードは上記のようになっているのですが、できれば　WindowProc()　関数を　newwin　クラスの中に入れてしまいたいです。しかしｳｲﾝﾄﾞｳﾌﾟﾛｼｰｼﾞｬ関数をクラスの中に入れてしまうと、　W N D C L A S S E X　構造体の　lpfnWndProc　メンバ変数に関数ポインタを渡すことができませんでした。

staticではないメンバ関数は暗黙の引数thisがあります。
なのでstaticメンバ関数にして何らかの方法でthisポインタを渡してやる必要があります。
「ウィンドウプロシージャ　クラス」で検索すればいろんな方法が見付かると思います。

MoNoQLoREATOR さんが書きました:そして、　ウインドウを作成してからしばらく経つと、水色に設定したはずの背景が灰色になり、画面がリフレッシュされなくなります。whileループで　SetWindowText()　関数を呼び出しまくっているのが原因だということはわかったのですが、これは解決できるのでしょうか？

WM_CTLCOLORSTATICメッセージが来る度にブラシを作り、解放していないからです。

ありがとうございます。ブラシの問題以外は解決できました！

↓ソースコード

► スポイラーを表示

コード:

#define _CRT_SECURE_NO_DEPRECATE

#include <windows.h>
#include "DxLib.h"
#include <string>
#include <vector>
#include <fstream>

using namespace std;

class newwin{
   WNDCLASSEX wc;
	HWND mainWH;
	int childCou;
	char textCont[101];

	static LRESULT CALLBACK WindowProc(HWND hwnd,UINT uiMsg,WPARAM wParam,LPARAM lParam){
		int childID = (int)GetWindowLongPtr((HWND)lParam, GWLP_ID);
		COLORREF wORb[2] = {RGB(255,255,255), RGB(200,255,255)};

		switch(uiMsg){

		case WM_CTLCOLORSTATIC:

			// 背景色を水色に設定
			if(childID>=100 && childID<124){
				if(childID%2){
					SetBkColor( (HDC)wParam, RGB(255,255,255) );
					DeleteObject(GetSysColorBrush(COLOR_WINDOW) );
					HBRUSH hBrush = CreateSolidBrush(RGB(255,255,255) );
					return (BOOL)hBrush;
				}
				else{
					SetBkColor( (HDC)wParam, RGB(200,255,255) );
					DeleteObject(GetSysColorBrush(COLOR_WINDOW) );
					HBRUSH hBrush = CreateSolidBrush(RGB(200,255,255) );
					return (BOOL)hBrush;
				}
			}

		default:

			return DefWindowProc(hwnd, uiMsg, wParam, lParam);
		}
	}

public:

	newwin(){}
	void init(const string &title, int width, int height){
		childCou = 100;

		wc.cbClsExtra = 0;
		wc.cbSize = sizeof(WNDCLASSEX);
		wc.cbWndExtra = 0;
		wc.hbrBackground = (HBRUSH)(COLOR_APPWORKSPACE+1);
		wc.hCursor = LoadCursor(GetModuleHandle(NULL),IDC_ARROW);
		wc.hIcon = NULL;
		wc.hIconSm = NULL;
		wc.hInstance = GetModuleHandle(NULL);
		wc.lpfnWndProc = WindowProc;
		wc.lpszClassName = "myClass";
		wc.lpszMenuName = NULL;
		wc.style = CS_DBLCLKS;

		RegisterClassEx(&wc);

		mainWH = CreateWindowEx(
			WS_EX_TOOLWINDOW,
			"myClass",
			title.c_str(),
			WS_BORDER|WS_SYSMENU,
			CW_USEDEFAULT,CW_USEDEFAULT,
			width,height,
			NULL,NULL,GetModuleHandle(NULL),NULL
		);
	 
		ShowWindow(mainWH,SW_SHOWDEFAULT);
		UpdateWindow(mainWH);
	}
	newwin(const string &title, int width, int height){ init(title, width, height); }

	HWND child(const string &mode, const string &oput, int x, int y, int width, int height){
		return CreateWindow(
			mode.c_str(),
			oput.c_str(),
			WS_CHILD|WS_VISIBLE,	
			x,y,
			width,height,
			mainWH,
			(HMENU)childCou++,
			GetModuleHandle(NULL),
			NULL
		);
	}

	const char* gets(HWND hwnd){
		GetWindowText(hwnd, textCont, 100);
		return textCont;
	}

	void sets(HWND hwnd, const string &oput){
		SetWindowText(hwnd, oput.c_str() );
	}

	~newwin(){
		for(int i=100;i<(int)childCou;i++) DestroyWindow( (HWND)i);
		DestroyWindow(mainWH);
		DeleteObject(GetSysColorBrush(COLOR_WINDOW) );
	}
};

class view{
	newwin w;
   WNDCLASSEX wc;
	HWND mainWH;
	HWND children[24];
	int next;

public:

	view(){
		w.init("Information", 320, 480);

		for(int i=0;i<24;i++) children[i] = w.child("STATIC", "", 0,19*i, 320,19);
		next = 0;
	}

	void clear(){
		for(int i=0;i<24;i++) w.sets(children[i], "");
		next = 0;
	}
	void add(const string &str){ if(next < 24) w.sets(children[next++], str); }
	void set(const string &str, int num){ if(next < 24) w.sets(children[num], str); }
};

int WINAPI WinMain(HINSTANCE hI,HINSTANCE hPrevInst,LPSTR lpCmdLine,int nC){
	ChangeWindowMode(TRUE);
	SetOutApplicationLogValidFlag(FALSE);
	SetWindowText("DxLib");
	if(DxLib_Init() == -1) return(-1);
	SetAlwaysRunFlag(TRUE);
	SetDrawScreen(DX_SCREEN_BACK);

	view v;
	v.add("test");

	while(ProcessMessage() == 0 && CheckHitKey(KEY_INPUT_ESCAPE) == 0){
        ClsDrawScreen();

		//v.clear();
		//v.add("test");

		ScreenFlip();
	}


	DxLib_End();
	return 0;
}

static関数はこのようなケースで使用するのですね。static関数の存在は聞いたことがありましたが、絶対に使わないだろうと思ってすぐに忘れました＾＾；

それはそうと、現在使用しているブラシのハンドルを得る関数を発見したので、ブラシを削除する際に使ってみたのですが、ちゃんと全てのブラシを削除できているのでしょうか？
※追記
whileループの中身がコメントアウトされていました。やっぱり画面がリフレッシュされなくなりました。どうやらブラシを削除できていないようです。

ブラシの設定の仕組みが未だにさっぱりわからないので、よくわかりません。ブラシの設定はどの段階で行われているのですか？
25行目～39行目の　return (BOOL)hBrush;　を　return true;　や　return false;　に変えると背景色が変わらなくなったので、ｳｲﾝﾄﾞｳﾌﾟﾛｼｰｼﾞｬから返ってきた値を親ｳｲﾝﾄﾞｳが処理した段階で設定されるのでしょうか？
でも、返しているのはBOOL値だから絶対にtrueかfalseなわけですし、CreateSolidBrush()関数をいちいち呼び出さずに、保存しておいたハンドルを渡すだけでは設定されませんでしたし・・・。

CreateSolidBrushで新しくブラシが作成されて、そのブラシのハンドルを戻り値として受け取ります。
スタティックコントロールの背景とするためにそのハンドルをWM_CTLCOLORSTATICメッセージの戻り値として返します。
その時点でブラシハンドルを受け取った変数の寿命は尽き忘れ去られますが、作成されたブラシは解放されません。

ちなみにGetSysColorBrushというのはウィンドウズにプリセットされたブラシを利用するために呼び出すものです。
プリセットされたブラシは解放する必要が無く、解放しても何も起きません。

MoNoQLoREATOR さんが書きました:でも、返しているのはBOOL値だから絶対にtrueかfalseなわけですし、

ブラシハンドルはBOOL型じゃないですよ。
BOOL型にキャストしてる解説サイトもあるみたいですけどそれはたまたまうまく動いてるだけです。
ウィンドウプロシージャの戻り値の型であるLRESULTにキャストするのが正しいと思いますけど。
#MSDNのWM_CTLCOLORSTATICのページだとなぜかINT_PTRでキャストしてますけど中身は同じ。

ちなみにwindows.hで定義されているBOOLの中身はintです。
BOOL型のGetMessage関数が0、0以外と-1を返すという訳のわからない仕様になってるのは有名かと。

MoNoQLoREATOR さんが書きました:CreateSolidBrush()関数をいちいち呼び出さずに、保存しておいたハンドルを渡すだけでは設定されませんでしたし・・・。

それはおかしいですね。
こんなふうにしたら良いと思いますけど。

コード:

	static LRESULT CALLBACK WindowProc(HWND hwnd,UINT uiMsg,WPARAM wParam,LPARAM lParam) {
		COLORREF wORb[2] = {RGB(255,255,255), RGB(200,255,255)};

		static HBRUSH hCyanBrush = NULL;

		switch(uiMsg) {
		case WM_CTLCOLORSTATIC:
			int childID = (int)GetWindowLongPtr((HWND)lParam, GWLP_ID);

			// 背景色を水色に設定
			if(childID>=100 && childID<124) {
				if(childID%2) {
					SetBkColor( (HDC)wParam, RGB(255,255,255) );
					return (LRESULT)GetStockObject(WHITE_BRUSH);
				}
				else {
					SetBkColor( (HDC)wParam, RGB(200,255,255) );
					if (hCyanBrush == NULL) {
						hCyanBrush = CreateSolidBrush(RGB(200,255,255));
					}
					return (LRESULT)hCyanBrush;
				}
			}
			break;
		case WM_DESTROY:
			if (hCyanBrush != NULL) {
				DeleteObject(hCyanBrush);
				hCyanBrush = NULL;
			}
		}

		return DefWindowProc(hwnd, uiMsg, wParam, lParam);
	}

GetStockObjectもプリセットのGDIオブジェクトのハンドルを返すので解放する必要がありません。
それと、未知のウィンドウメッセージでもlParamをウィンドウハンドルとしてGetWindowLongPtrを呼ぶのは危険です。

大変勉強になりました。

BOOL型はbool型と同じだとかってに考えていました。ということは、やはりｳｲﾝﾄﾞｳﾌﾟﾛｼｰｼﾞｬの返値を元に親ウインドウが背景色を設定しているのですね。

staticメンバ関数のなかでstatic変数を宣言すると実体を書く必要がないのですね。だから、グローバルな実体をつくらなくても済む・・・と。

解決できないだろうなと思っていたことまで解決できてしまい、驚きです。
本当にありがとうございました。

プログラマ専用SNS ミクプラ

nShowCmdを取得する関数はないのでしょうか？

nShowCmdを取得する関数はないのでしょうか？

Re: nShowCmdを取得する関数はないのでしょうか？

Re: nShowCmdを取得する関数はないのでしょうか？

Re: nShowCmdを取得する関数はないのでしょうか？

Re: nShowCmdを取得する関数はないのでしょうか？