回答編集履歴

指摘事項を追記しました。

2018/10/25 10:55

投稿

atata0319

スコア881

test CHANGED Viewed

@@ -595,3 +595,59 @@
 }
 ```
+---
+**2018/10/25 追記**
+ダイアログのコードを拝見しましたが、一見問題になりそうなところはありませんでした。
+私が以前に指摘したのは disp_star_xy 関数における以下のコードブロックの一番下の 2 行です。おそらく星の X 座標を何らかの形で計算しているものと思われますが、X 座標から**(rect.right-rect.left)/2** つまりは画面の半分の幅を減算しているため、この関数が 2 回呼び出された場合、画面の表示範囲内に収まっている X 座標はすべて範囲外になるという指摘です。それが InvalidateRect を2回と言ってたわけですが、実際のところ InvalidateRect を呼び出す以外に WM_PAINT が発行されることがあるので、InvalidateRect だけに限定されるわけではありません。
+y 座標の計算も怪しい感じがします。
+```C++
+// 星の座標データを１チャンネル分ウィンドウに表示する関数
+BOOL disp_star_xy(HDC hdc, RECT rect, HWND hDlg) //ただし、xy平面に描画する関数
+{
+   COLORREF color;
+   HPEN hPen_xy;
+   int i;
+   color = RGB(255,255, 255);/*RGB(255,241,0)*/   // チャンネル１は黄色
+   hPen_xy = CreatePen(PS_SOLID, 9, color);
+   SelectObject(hdc, hPen_xy);                   // hdcで上記のペンを使うために選ぶ
+   HBRUSH hBrush_xy, h01dBrush_xy;
+   hBrush_xy = CreateSolidBrush(color);
+   h01dBrush_xy = (HBRUSH)SelectObject(hdc, hBrush_xy);
+   int nop = rect.right - rect.left + 1; //横方向のピクセル数を求める?
+   double ppd = (rect.bottom - rect.top) / 10.0;       // 縦方向1div当たり何ピクセルか
+         for(i=0; i<=total_n2; i++){
+           star[i].x = star[i].x - (rect.right-rect.left)/2;
+           star[i].y = (rect.bottom - rect.top) / 2 - star[i].y;
+```

記載漏れを追記

2018/10/25 10:55

投稿

atata0319

スコア881

test CHANGED Viewed

@@ -24,7 +24,7 @@
 ![サンプル画像](6d56dc7557446f47f2c50d684af7faec.png)
-コード的に変えたのは月の描画部分はめんどくさいのでカットしています。星は移動しないように disp_star_xy 関数の座標更新部分を変更してあります。それ以外の部分は提示されているコードのままです。これで問題がある場合、さらに提示されていない部分を提示していただければ、回答しやすいかと思います。
+コード的に変えたのは月の描画部分はめんどくさいのでカットしています。星は移動しないように disp_star_xy 関数の座標更新部分を変更してあります。単色の星だと寂しいので、色が着くようにしてみました。それ以外の部分は提示されているコードのままです。これで問題がある場合、さらに提示されていない部分を提示していただければ、回答しやすいかと思います。
 ```C++

回答に追記しました。

2018/10/23 17:18

投稿

atata0319

スコア881

test CHANGED Viewed

@@ -13,3 +13,585 @@
 ```
 InvalidateRect を 2 回以上呼び出すと X 座標がマイナスとなって描画位置がウィンドウの範囲に入らずに消えています。該当関数を呼び出すかどうかでそのような差になっているのではないでしょうか？
+---
+**2018/10/24 追記**
+すみません。最初に伝えるのを忘れてましたが、描画処理に関しては提示されている元のコードで問題なく星は描画できています。
+![サンプル画像](6d56dc7557446f47f2c50d684af7faec.png)
+コード的に変えたのは月の描画部分はめんどくさいのでカットしています。星は移動しないように disp_star_xy 関数の座標更新部分を変更してあります。それ以外の部分は提示されているコードのままです。これで問題がある場合、さらに提示されていない部分を提示していただければ、回答しやすいかと思います。
+```C++
+#include <windows.h>
+#include <math.h>
+#define MAX_LOADSTRING 100
+#define ID_MYTIMER      33
+#define MAX_STAR       520
+typedef struct {
+	HINSTANCE hInst_bitmap;
+	HWND hWnd_bitmap;
+	BYTE *lpBmpData;
+	BITMAPINFO bmi;
+	RGBQUAD dummy[3];
+	char *fname;
+	int fsize;
+} IMG;
+struct STAR : public POINT {
+	double Mag;
+};
+// グローバル変数:
+HINSTANCE hInst;
+WCHAR szTitle[] = L"teratail153158";
+WCHAR szWindowClass[] = L"teratail153158";
+STAR star[MAX_STAR];
+// このコード モジュールに含まれる関数の宣言を転送します:
+ATOM                MyRegisterClass(HINSTANCE hInstance);
+BOOL                InitInstance(HINSTANCE, int);
+LRESULT CALLBACK    WndProc(HWND, UINT, WPARAM, LPARAM);
+BOOL disp_star_xy(HDC hdc, RECT rect);
+int APIENTRY wWinMain(_In_ HINSTANCE hInstance,
+                     _In_opt_ HINSTANCE hPrevInstance,
+                     _In_ LPWSTR    lpCmdLine,
+                     _In_ int       nCmdShow)
+{
+    UNREFERENCED_PARAMETER(hPrevInstance);
+    UNREFERENCED_PARAMETER(lpCmdLine);
+	for (int i = 0; i < MAX_STAR; i++) {
+		star[i].x = 988 * rand() / RAND_MAX;
+		star[i].y = 741 * rand() / RAND_MAX;
+		star[i].Mag = 1.0 - 4.0 * rand() / RAND_MAX;
+	}
+    // グローバル文字列を初期化する
+    MyRegisterClass(hInstance);
+    // アプリケーション初期化の実行:
+    if (!InitInstance (hInstance, nCmdShow))
+    {
+        return FALSE;
+    }
+    MSG msg;
+    // メイン メッセージ ループ:
+    while (GetMessage(&msg, nullptr, 0, 0))
+    {
+		TranslateMessage(&msg);
+		DispatchMessage(&msg);
+	}
+    return (int) msg.wParam;
+}
+//
+//  関数: MyRegisterClass()
+//
+//  目的: ウィンドウ クラスを登録します。
+//
+ATOM MyRegisterClass(HINSTANCE hInstance)
+{
+    WNDCLASSEXW wcex;
+    wcex.cbSize = sizeof(WNDCLASSEX);
+    wcex.style          = CS_HREDRAW | CS_VREDRAW;
+    wcex.lpfnWndProc    = WndProc;
+    wcex.cbClsExtra     = 0;
+    wcex.cbWndExtra     = 0;
+    wcex.hInstance      = hInstance;
+    wcex.hIcon          = NULL;
+    wcex.hCursor        = LoadCursor(nullptr, IDC_ARROW);
+    wcex.hbrBackground  = (HBRUSH)(COLOR_3DSHADOW);
+    wcex.lpszMenuName   = NULL;
+    wcex.lpszClassName  = szWindowClass;
+    wcex.hIconSm        = NULL;
+    return RegisterClassExW(&wcex);
+}
+//
+//   関数: InitInstance(HINSTANCE, int)
+//
+//   目的: インスタンス ハンドルを保存して、メイン ウィンドウを作成します
+//
+//   コメント:
+//
+//        この関数で、グローバル変数でインスタンス ハンドルを保存し、
+//        メイン プログラム ウィンドウを作成および表示します。
+//
+BOOL InitInstance(HINSTANCE hInstance, int nCmdShow)
+{
+   hInst = hInstance; // グローバル変数にインスタンス ハンドルを格納する
+   HWND hWnd = CreateWindowW(szWindowClass, szTitle, WS_OVERLAPPEDWINDOW,
+      CW_USEDEFAULT, 0, CW_USEDEFAULT, 0, nullptr, nullptr, hInstance, nullptr);
+   if (!hWnd)
+   {
+      return FALSE;
+   }
+   ShowWindow(hWnd, nCmdShow);
+   UpdateWindow(hWnd);
+   return TRUE;
+}
+//
+//  関数: WndProc(HWND, UINT, WPARAM, LPARAM)
+//
+//  目的: メイン ウィンドウのメッセージを処理します。
+//
+//  WM_COMMAND  - アプリケーション メニューの処理
+//  WM_PAINT    - メイン ウィンドウを描画する
+//  WM_DESTROY  - 中止メッセージを表示して戻る
+//
+//
+LRESULT CALLBACK WndProc(HWND hWnd_bitmap, UINT msg, WPARAM wp, LPARAM lp)
+{
+	IMG g_img;
+	g_img.lpBmpData = NULL;
+	PAINTSTRUCT ps;
+	HDC hdc, hdc_mem;
+	HBITMAP hBmp;
+	int a, b, ab;
+	switch (msg) {
+	case WM_CREATE:
+		break;
+	case WM_PAINT:
+		//月をビットマップ形式で描画していく作業
+		g_img.bmi.bmiHeader.biSize = sizeof(BITMAPINFOHEADER);
+		g_img.bmi.bmiHeader.biWidth = 988;
+		g_img.bmi.bmiHeader.biHeight = 741;
+		g_img.bmi.bmiHeader.biPlanes = 1;
+		g_img.bmi.bmiHeader.biBitCount = 32;
+		g_img.bmi.bmiHeader.biCompression = BI_RGB;
+		g_img.bmi.bmiHeader.biSizeImage = 988 * 741 * 4;
+		g_img.bmi.bmiHeader.biXPelsPerMeter = 0;
+		g_img.bmi.bmiHeader.biYPelsPerMeter = 0;
+		g_img.bmi.bmiHeader.biClrUsed = 0;
+		g_img.bmi.bmiHeader.biClrImportant = 0;
+		g_img.lpBmpData = (BYTE *)malloc(988 * 741 * 4);
+		RECT rect;
+		hdc = BeginPaint(hWnd_bitmap, &ps);
+		GetClientRect(hWnd_bitmap, &rect);
+		hdc_mem = CreateCompatibleDC(hdc);
+		for (a = 0; a < 741; a++) {
+			for (b = 0; b < 988; b++) {
+				ab = a * (988 * 4) + b * 4; //始めの項が行数、次の項が左から何ビット目かを表す。（blueの位置を決めている）
+				// 以下は月の縁（円）の外側に対する処理（宇宙空間なので黒色で塗りつぶしている）
+				int centerX = 988 / 2;
+				int centerY = 741 / 2;
+				if (sqrt(pow(b - centerX, 2) + pow(a - centerY, 2)) < 250) {
+					g_img.lpBmpData[ab] = 0xff;
+					g_img.lpBmpData[ab + 1] = 0xff;
+					g_img.lpBmpData[ab + 2] = 0xff;
+					g_img.lpBmpData[ab + 3] = 0;
+				} else {
+					g_img.lpBmpData[ab] = 0;
+					g_img.lpBmpData[ab + 1] = 0;
+					g_img.lpBmpData[ab + 2] = 0;
+					g_img.lpBmpData[ab + 3] = 0;
+				}
+			}
+		}
+		hBmp = CreateDIBitmap(hdc, &(g_img.bmi.bmiHeader), CBM_INIT, g_img.lpBmpData, &(g_img.bmi), DIB_RGB_COLORS);
+		free(g_img.lpBmpData);
+		SelectObject(hdc_mem, hBmp);
+		SetStretchBltMode(hdc, COLORONCOLOR);
+		StretchBlt(hdc, 0, 0, 988, 741, hdc_mem, 0, 0, 988, 741, SRCCOPY);
+		//メモリ軸を描画する関数を作り、ここで呼び出している（これはうまく描画できた）
+		//disp_axes2(hdc, rect);
+		//星の座標データを描画する関数を作り、ここで呼び出している（月をビットマップ形式で描画しない時は星がちゃんと描画されるが、月をビットマップ形式で描画すると消える）
+		disp_star_xy(hdc, rect);
+		DeleteDC(hdc);
+		DeleteDC(hdc_mem);
+		DeleteObject(hBmp);
+		EndPaint(hWnd_bitmap, &ps);
+		ReleaseDC(hWnd_bitmap, hdc);
+		break;
+	case WM_DESTROY:
+		PostQuitMessage(0);
+		break;
+	default:
+		return (DefWindowProc(hWnd_bitmap, msg, wp, lp));
+	}
+	return 0;
+}
+// 星の座標データを１チャンネル分ウィンドウに表示する関数
+BOOL disp_star_xy(HDC hdc, RECT rect) //ただし、xy平面に描画する関数
+{
+	const int total_n2 = 40;
+	COLORREF color;
+	int i;
+	color = RGB(255, 255, 255);/*RGB(255,241,0)*/    // チャンネル１は黄色
+	int nop = rect.right - rect.left + 1; //横方向のピクセル数を求める?
+	double ppd = (rect.bottom - rect.top) / 10.0;        // 縦方向1div当たり何ピクセルか
+	for (i = 0; i <= MAX_STAR; i++) {
+//		star[i].x = star[i].x - (rect.right - rect.left) / 2;
+//		star[i].y = (rect.bottom - rect.top) / 2 - star[i].y;
+		int m;
+		if (star[i].Mag < 0.0) {
+			for (m = 0; m <= 8; m += 1) {
+				SetPixel(hdc, star[i].x - m, star[i].y, RGB(228, 0, 127)/*RGB(255, 255, 255)*/);
+				SetPixel(hdc, star[i].x, star[i].y - m, RGB(228, 0, 127)/*RGB(255, 255, 255)*/);
+				SetPixel(hdc, star[i].x, star[i].y, /*RGB(255, 255, 255)*/RGB(228, 0, 127));
+				SetPixel(hdc, star[i].x + m, star[i].y, RGB(228, 0, 127)/* RGB(255, 255, 255)*/);
+				SetPixel(hdc, star[i].x, star[i].y + m, RGB(228, 0, 127)/*RGB(255, 255, 255)*/);
+			}
+		}
+		else if (star[i].Mag < 2.0) {
+			for (m = 0; m <= 6; m += 1) {
+				SetPixel(hdc, star[i].x - m, star[i].y, RGB(255, 255, 255));
+				SetPixel(hdc, star[i].x, star[i].y - m, RGB(255, 255, 255));
+				SetPixel(hdc, star[i].x, star[i].y, RGB(255, 255, 255)/*RGB(173, 255, 47)*/);
+				SetPixel(hdc, star[i].x + m, star[i].y, RGB(255, 255, 255));
+				SetPixel(hdc, star[i].x, star[i].y + m, RGB(255, 255, 255));
+			}
+		}
+		else if (star[i].Mag < 4.0) {
+			for (m = 0; m <= 4; m += 1) {
+				SetPixel(hdc, star[i].x - m, star[i].y, RGB(255, 255, 255));
+				SetPixel(hdc, star[i].x, star[i].y - m, RGB(255, 255, 255));
+				SetPixel(hdc, star[i].x, star[i].y, RGB(255, 255, 255)/*RGB(173, 255, 47)*/);
+				SetPixel(hdc, star[i].x + m, star[i].y, RGB(255, 255, 255));
+				SetPixel(hdc, star[i].x, star[i].y + m, RGB(255, 255, 255));
+			}
+		}
+		else {
+			SetPixel(hdc, star[i].x - 2, star[i].y, RGB(255, 255, 255));
+			SetPixel(hdc, star[i].x, star[i].y - 2, RGB(255, 255, 255));
+			SetPixel(hdc, star[i].x, star[i].y, RGB(255, 255, 255)/*RGB(173, 255, 47)*/);
+			SetPixel(hdc, star[i].x + 2, star[i].y, RGB(255, 255, 255));
+			SetPixel(hdc, star[i].x, star[i].y + 2, RGB(255, 255, 255));
+		}
+	}
+	return TRUE;    // 戻り値TRUEを返す
+}
+```