10秒立つと描画が止まってしまう

Question

描画が10秒立つと止まってしまいます。

[status.cppのworldClock()]関数で得た時間を描画していたのですが、なぜか10秒立つと止まってしまいます。(0,1,2,3,4,5,6,7,8,9という順番で描画され、最終的に9で止まってしまいます。)

---

[worldClock()]関数内で10秒経過したら初期化する処理を入れたのですが、その処理とは関係がありませんでした。5秒経過したら初期化に処理を書き換えても、0,1,2,3,4,0,1,2,3,4という順番で描画され最終的に4で止まってしまいました。

---

openglのdisplayFunc()関数
font.cppのfont_Draw()関数
status.cppのworldClock()関数
status.cppのdraw_status()関数

で処理が止まっているのかと思い、
printf関数で経過時間を描画したら、どの関数でも10秒経過以降も正常に処理されていました。

---

十の位を描画できないのかと思い
font_Draw()関数で

static int i=0;
i++;

で描画したら、5000ちょいまで描画できましたが、10秒立った瞬間にそれ以降は描画結果が変わりませんでした。（static int i の値はふえつづけているのに...）

---

動作はしているのに10秒立つととまってしまう。描画する値はふえつづけているのに...
font_Draw関数内の処理は正常なのに(10秒立っても処理は止まっていないが、10秒立つと描画がされない)、
なので、font_Draw関数になにかしらの原因があるのかと思い、色々試してみたのですががお手上げでした。

申し訳ありませんが、どこがいけないかご指摘いただければ幸いです。<(_ _)>

[status.h]

#pragma once

#define BUF_MAX 64

class status {
public:
	int level = 0;
	int nextLevel = 0;
	int hp = 0;
	int mp = 0;
	int power = 0;
	int defense = 0;
	int speed = 0;
	int jobCount = 0;
	int gender = 0;
	int intelligence = 0;

	char Name[BUF_MAX] = { 0 };
};

class Human :public status {
private:
	int hour[2] = { 0 };
	int minute[2] = { 0 };
	int second[2] = { 0 };
public:
	Human();
	void get_Name();
	void get_Job();

	void worldClock();
	void draw_status();
};

[status.cpp]

#include <stdio.h>

#include "status.h"
#include "font.h"

Human::Human() {
	for (int i = 0; i < 2; i++) {
		this->second[i] = 0;
		this->minute[i] = 0;
		this->hour[i] = 0;
	}

	this->nextLevel = 60;
	this->level = 1;
	this->hp = 10;
	this->mp = 10;
	this->power = 3;
	this->defense = 3;
	this->speed = 2;
	this->jobCount = 5;
	this->intelligence = 25;
}

void Human::get_Name() {

}

void Human::worldClock() {
	static int time = 0;
	if (time == 60) {
		this->second[1]++;
		time = 0;
	}
	if (this->second[1] == 10)
		this->second[1] = 0;

	time++;
}
int i = 0;
void Human::draw_status() {
	font_Begin();

	font_SetPosition(100, 122);
	font_SetSize(35);
	font_SetWidth(25);
	font_GetColor(0x00, 0x00, 0x00);
	font_Draw(L"%d%d:%d%d:%d%d", 
		this->hour[0], this->hour[1],
		this->minute[0], this->minute[1],
		i, this->second[1]);
	i++;
	font_SetPosition(432, 20);
	font_SetSize(35);
	font_SetWidth(25);
	font_GetColor(0x00, 0x00, 0x00);
	font_Draw(L"%d\n", this->level);

	font_SetPosition(432, 55);
	font_SetSize(15);
	font_SetWidth(20);
	font_GetColor(0x00, 0x00, 0x00);
	font_Draw(L"%d\n", this->nextLevel);

	font_SetPosition(100, 320);
	font_SetSize(15);
	font_SetWidth(20);
	font_GetColor(0x00, 0x00, 0x00);
	font_Draw(L"%d\n", this->power);

	font_SetPosition(100, 355);
	font_SetSize(15);
	font_SetWidth(20);
	font_GetColor(0x00, 0x00, 0x00);
	font_Draw(L"%d\n", this->speed);

	font_SetPosition(300, 320);
	font_SetSize(15);
	font_SetWidth(20);
	font_GetColor(0x00, 0x00, 0x00);
	font_Draw(L"%d\n", this->defense);

	font_SetPosition(300, 355);
	font_SetSize(15);
	font_SetWidth(20);
	font_GetColor(0x00, 0x00, 0x00);
	font_Draw(L"%d\n", this->intelligence);

	font_SetPosition(460, 320);
	font_SetSize(15);
	font_SetWidth(20);
	font_GetColor(0x00, 0x00, 0x00);
	font_Draw(L"%d\n", this->mp);

	font_SetPosition(460, 355);
	font_SetSize(15);
	font_SetWidth(20);
	font_GetColor(0x00, 0x00, 0x00);
	font_Draw(L"%d\n", this->hp);

	font_End();
}

[font.h]

#define FONT_DEFAULT_SIZE 20

void font_Begin();
void font_End();

void font_SetPosition(int _x, int _y);
void font_SetWidth(int _width);
void font_SetSize(int _size);

int font_GetSize();
void font_GetColor(float red, float green,float blue);

void font_Draw(const wchar_t* format, ...);

[font.cpp]

#include <Windows.h>

#include "glm/glm.hpp"
#include "glut.h"

#include "font.h"

int x, y = 0;
int width = 0;
int size = FONT_DEFAULT_SIZE;
float color[3] = { 0xff,0xff,0xff };

void font_Begin() {
	glPushMatrix();
	glPushAttrib(GL_ALL_ATTRIB_BITS);

	glMatrixMode(GL_PROJECTION);
	glLoadIdentity();

	GLint viewport[4];
	glGetIntegerv(GL_VIEWPORT, viewport);
	gluOrtho2D(0,viewport[2], viewport[3],0);

	glMatrixMode(GL_MODELVIEW);
	glLoadIdentity();

	glDisable(GL_DEPTH_TEST);
	glDisable(GL_LIGHTING);
	glDisable(GL_TEXTURE_2D);
}

void font_End() {
	glPopMatrix();
	glPopAttrib();
}

void font_SetPosition(int _x, int _y) {
	x = _x + 5;
	y = _y + 20;
}

void font_SetSize(int _size) {
	size = _size;
}

void font_SetWidth(int _width) {
	width = _width;
}

int font_GetSize() {
	return size;
}

void font_GetColor(float _red, float _green, float _blue) {
	color[0] = _red;
	color[1] = _green;
	color[2] = _blue;
}

void font_Draw(const wchar_t* format, ...) {
	va_list argList;

	if (format == nullptr)return;

	wchar_t buf[256];
	va_start(argList, format);
	vswprintf_s(buf, format, argList);
	va_end(argList);

	int length = wcslen(buf);
	int list = glGenLists(length);

	wchar_t fontType[] = { L"ＭＳ明朝" };
	HFONT Ofont = CreateFontW(
		size,				
		width,
		0,					
		0,						
		FW_REGULAR,				
		FALSE,					
		FALSE,					
		FALSE,					
		SHIFTJIS_CHARSET,		
		OUT_DEFAULT_PRECIS,		
		CLIP_DEFAULT_PRECIS,	
		DEFAULT_QUALITY,		
		FIXED_PITCH | FF_MODERN,
		fontType);				

	HDC Odc = wglGetCurrentDC();
	SelectObject(Odc, Ofont);

	wchar_t* font = buf;
	int i = 0;
	for (; (*font != '\0') && (*font != '\n'); font++) {
		wglUseFontBitmapsW(
			Odc,		
			*font,		
			1,			
			list + i	
		);
		i++;
	}
	glColor3f(color[0], color[1], color[2]);
	glPushMatrix();
	{
		glRasterPos2i(x, y);
		for (int i = 0; i < length; i++) {
			glCallList(list + i);
		}
	}
	glPopMatrix();

	if (*font == '\n' || '\0') {
		glDeleteLists(list, length);

		font_SetPosition(0, y + font_GetSize()-15);
		font_Draw(++font);
	}
}

[main.cpp]

#include <stdio.h>
#include <stdlib.h>

#include "glm/glm.hpp"
#include "glut.h"

#include "rect.h"
#include "font.h"
#include "bmp.h"
#include "status.h"

using namespace glm;
vec2 windowSize = { 600,400 };
Rect rect1 = Rect(0, 0, 600, 400);
Human human1;

void display(void) {
	glClear(GL_COLOR_BUFFER_BIT|GL_DEPTH_BUFFER_BIT);

	glMatrixMode(GL_PROJECTION);
	glLoadIdentity();
	gluOrtho2D(0, windowSize.x, windowSize.y, 0);

	glMatrixMode(GL_MODELVIEW);
	glLoadIdentity();

	human1.worldClock();
	rect1.rect_Draw();
	human1.draw_status();

	glutSwapBuffers();
}

void idle(void) {
	glutPostRedisplay();
}

void reshape(int width, int height) {
	printf("reshape: width:%d height:%d\n", width, height);
	glViewport(0,0,width,height);
	windowSize = vec2(width, height);
}

void motion(int x, int y) {
	printf("x:%d y:%d\n", x, y);
}

int main(int argc, char* argv[]) {
	glutInit(
		&argc,
		argv);
	glutInitDisplayMode(GL_DOUBLE);
	glutInitWindowPosition(100,100);
	glutInitWindowSize(windowSize.x,windowSize.y);
	glutCreateWindow("Window");

	bmp_Load("ステータスウィンドウ.bmp");

	glutDisplayFunc(display);
	glutIdleFunc(idle);
	glutReshapeFunc(reshape);
	glutMotionFunc(motion);

	glutMainLoop();

	return 0;
}

Answer 1

ぱっと見、気になった点を2つ。

vsprintf_s関数の引数が誤っている。
CreateFont関数で作成したフォントオブジェクトをDeleteObjectで解放していない。

Comments

[SHOMI]

>vsprintf_s関数の引数が誤っている。
以下があるのであっていませんか？
https://docs.microsoft.com/en-us/cpp/c-runtime-library/reference/vsprintf-s-vsprintf-s-l-vswprintf-s-vswprintf-s-l
template <size_t size>
int vswprintf_s(
wchar_t (&buffer)[size],
const wchar_t *format,
va_list argptr
);

[hikarhikar]

majiponiさんご返答ありがとうございます。
majiponiさんがおっしゃった通りDeleteObjectで開放したら正常に動作できました。
本当にありがとうございました<(_ _)>

Answer 2

動作確認はしていませんが…
コードを見た感じfont_Draw()内で毎回CreateFont()を呼びOdcにセットしており、Odcのフォントを元に戻さずDeleteObject()もしていないので、ある程度の回数呼び出された後にそれ以上フォントを作れなくなっているのでは？

今回直接関係ありませんが、書式化結果の2行目以降に書式化文字が含まれていると、font_Draw()再帰呼び出しで誤動作するので"%s"を指定しておいたほうがよいと思います。

-    if (*font == '\n' || '\0') {
+    if (*font == '\n' && *(font+1) != '\0') {
        glDeleteLists(list, length);

        font_SetPosition(0, y + font_GetSize()-15);
-        font_Draw(++font);
+        font_Draw(L"%s", ++font);
    }

Comments

[hikarhikar]

SHOMIさんご返答ありがとうございます。
SHOMIさんがおっしゃった通りに直してみたら、正常に動作しました。
何から何まで本当にありがとうございました。<(_ _)>