ワイド文字が描画されません

Question

RPGゲームを作ろうと思い、現在ステータスウィンドウを作っているのですが
scanf_s()で取得した文字列が描画できません。

描画処理は[bool]で二つに分けています。
・名前入力画面(bool falseの場合)
・ステータス画面(bool trueの場合)

---

scanf_s()でchar型の文字列を取得
↓
ワイド文字変換(ワイド配列buffer)
↓
buffer渡す //(wchar_t buffer[NAME_MAX];)
↓
set_Name(wchar_t* _format)
this->Name=_format; //(status::Name)
↓
draw_status()
・
・
this->Name描画(描画されません)
・
・

---

main関数のscanf_sで得た文字列をワイド文字に変換して、
set_Name(wchar_t*_format)に渡す。
この[set_Name()]関数で描画した場合は正常に描画されました。

void Human::set_Name(wchar_t* _format) {
	this->Name = _format;
	font_SetSize(20);
	font_SetWidth(20);
	font_GetColor(0x00, 0x00, 0x00);
	int i = 0;
	for (; (*this->Name != '\0') && (*this->Name != '\n'); this->Name++) {
		font_SetPosition(90 + i * font_GetSize(), 40);
		font_Draw(L"%c", *this->Name);
		i++;
	}
}

---

上の描画処理を[draw_status()]関数で行うと(描画する場所が少しちらつき)描画がされません。
[draw_status()]関数では他に数字など色々描画してるのですが、set_Name()で取得した文字列だけ描画できなくなってしまいます。(描画場所に少しちらつきはあるのですが...)

---

こちらのサイトで前回質問した時には[font]関数で間違えているところを指摘され、
指摘された箇所を全て治し正常に描画できたので、[font]関数が原因ではないと思うのですが...
色々試してみたのですががお手上げでした。

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

[main.cpp]

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

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

#include "rect.h"
#include "font.h"
#include "cube.h"
#include "BONE.h"
#include "obj.h"
#include "status.h"
#include "line.h"

using namespace glm;
vec2 windowSize = { 600,400 };
Rect rect0 = Rect(0, 0, 600, 400);
Rect rect1 = Rect(0, 0, 600, 400);
Rect rect2 = Rect(322, 112, 255, 200);
Human human1;

bool login = false;
bool name = false;
wchar_t buf[NAME_MAX];
void display(void) {
	glClear(GL_COLOR_BUFFER_BIT|GL_DEPTH_BUFFER_BIT);

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

	glMatrixMode(GL_MODELVIEW);
	glLoadIdentity();

	if (login != true) {
		rect0.rect_Draw(0xff,0xff,0xff);
		
		font_Begin();
		font_SetPosition(20, 0);
		font_SetSize(20);
		font_SetWidth(0);
		font_GetColor(0x00, 0x00, 0x00);
		font_Draw(L"名前を入力してください\n");
		font_End();
	
		line_SetPosition(20,60,230);
		line_Draw(1.0);
		if (name != false) {
			size_t wLen = 0;
			errno_t err = 0;
			char numbur[NAME_MAX] = { 0 };
			wchar_t buffer[NAME_MAX];
			scanf_s("%s", numbur, NAME_MAX);

			//文字列型変換
			setlocale(LC_ALL, "japanese");
			err = mbstowcs_s(&wLen, buffer, NAME_MAX, numbur, _TRUNCATE);

			font_Begin();
			font_SetPosition(20, 30);
			font_SetSize(20);
			font_SetWidth(0);
			font_GetColor(0x00, 0x00, 0x00);
			font_Draw(L"%s\n", buffer);
			font_End();

			human1.set_Name(buffer);
			
			login = true;
			name = false;
		}
		glutSwapBuffers();
		return;
	}

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

	rect2.rect_Draw(TEXTURE_1);

	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 mouse(int button, int state, int x, int y) {
	if ((button == GLUT_LEFT_BUTTON) && (x > 0) && (x < 245)
		&& (y < 60) && (y > 30)) {
		name = true;
	}
	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", TEXTURE_0);
	//bmp_Load("キャラクター0.bmp", TEXTURE_1);

	glutDisplayFunc(display);
	glutIdleFunc(idle);
	glutReshapeFunc(reshape);
	glutMouseFunc(mouse);

	glutMainLoop();

	return 0;
}

[status.h]

#define BUF_MAX 64
#define NAME_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;

	wchar_t* Name;
};

class Human :public status {
private:
	int hour[2] = { 0 };
	int minute[2] = { 0 };
	int second[2] = { 0 };
public:
	Human();
	void set_Name(wchar_t* format);
	void set_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::set_Name(wchar_t* _format) {
	this->Name = _format;
	/*font_SetSize(20);
	font_SetWidth(FONT_DEFAULT_HEIGHT);
	font_GetColor(0x00, 0x00, 0x00);
	int i = 0;
	for (; (*this->Name != '\0') && (*this->Name != '\n'); this->Name++) {
		font_SetPosition(90 + i * font_GetSize()*1.5, 40);
		font_Draw(L"%c", *this->Name);
		i++;
	}*/
}

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

	time++;
}

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],
		this->second[0], this->second[1]);

	font_SetSize(20);
	font_SetWidth(20);
	font_GetColor(0x00, 0x00, 0x00);
	int i = 0;
	for (; (*this->Name != '\0') && (*this->Name != '\n'); this->Name++) {
		font_SetPosition(90 + i * font_GetSize(), 40);
		font_Draw(L"%c", *this->Name);
		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.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_HEIGHT;
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() {
	glPopAttrib();
	glPopMatrix();
}

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();

	SelectObject(Odc, GetStockObject(SYSTEM_FONT));
	DeleteObject(Ofont);
	if (*font == '\n' && *(font + 1) != '\0') {
		glDeleteLists(list, length);

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

Answer 1

set_Nameでbufferのポインタを記憶させていますが、bufferはローカル宣言した変数のため、変数の有効なスコープを抜けた時点で無効になり、無効になった領域のポインタを保持することになります。
文字列は std::wstring 等で保持した方がよいでしょう。

Comments

[episteme]

あ、ホントだ... ココがいちばん怪しいスね。

[hikarhikar]

radianさんご返答ありがとうございます。<(_ _)>
なるほどです！
display()→sit_Name()→抜ける→draw_status()ということですね！
radianさんがおっしゃったことを参考に処理を直していきたいとおもいます。

Answer 2

font_Drawの中からfont_Draw呼んでますけど、これで正しいんですよね?

Comments

[hikarhikar]

epistemeさんご返答ありがとうございます。<(_ _)>

[font.cpp]のfont_Drawからfont_Drawをよんでるということでしょうか？
文字列に改行コードが入っていた場合にfont_Drawを再帰的に呼んでいるので大丈夫だと思います。
前質問した際に下のように、直すようご指摘していただきました。
if (*font == '\n' && *(font + 1) != '\0') {
glDeleteLists(list, length);

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

[episteme]

ホント? CreateFontやらなんやらを何度も何度も呼ぶことになりますよ? それがチラつきの原因だったりしませんかね?

[hikarhikar]

上の処理(先に返答した)をglDeleteLists(list,length)だけに変えてみたのですが、結果は変わりませんでした。(◞‸◟)
epistemeさんがおっしゃった「createFontを何度も何度も呼ぶことになる」は、DeleteObject()関数で開放しているのですが、他に何かした方がいいのでしょうか？

[episteme]

いや、一文字ごとにCreateFont/DeleteObjectせにゃいかんのですか?

[hikarhikar]

int length = wcslen(buf);
int list = glGenLists(length);
文字の長さを取得し、文字の長さだけディスプレイを作成。
CreateFont()でフォント作成
文字列を描画(文字の長さぶん)
フォントを元に戻す。
deleteObject()でフォントを解放
glDeleteLists()で文字の長さぶん、ディスプレイを解放
という感じなのですが...

新たにfont_Load()関数みたいのを作成し、その中でCreateFontWでフォントを作成、保持し、
フォントを使用する時になったら、保持しているフォントを使用し、フォントをデフォルトに戻す。
実質CreateFontWは一度main関数で呼ばれるだけ。
みたいな感じでしょうか？

[episteme]

みたいな感じです。ひとつあれば十分なものをいくつも用意する必要ないんだから。

[hikarhikar]

epistemeさん何から何までありがとうございました。<(_ _)>
epistemeさんがおっしゃっとことを参考にコードを直していきたいと思います。
本当にありがとうございました