Discussion :

[rXpCH]

Hello,

Je suis nouveau dans le monde du c++, je connais surtout le java personnellement et j'ai encore un peu de mal à me faire au c++.
J'ai acheté le livre Beginning game programming third edition et donc j'apprend à utiliser la librairie directx, mais c'est une façon un peu différente de programmer et je fais encore des erreurs bêtes.

Je crée une classe qui doit m'aider à gérer les input avec directx (clavier souris, manettes) mais quand j'utilise la méthode qui m'aide a récupérer la position de la souris ou celle qui me donne les touches qui ont été appuyées je reçois cette erreur :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
First-chance exception at 0x002f13ec in xinputd3d.exe: 0xC0000005: Access violation reading location 0xfeeeff0a.
Unhandled exception at 0x002f13ec in xinputd3d.exe: 0xC0000005: Access violation reading location 0xfeeeff0a.

Mon code est toujours un peu bordélique mais je suis toujours au stade de test de la librairie :

winmain.cpp :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
/*
    Beginning Game Programming, Third Edition
    Chapter 4
    Create_Surface program
*/
 
#include <windows.h>
#include <sstream>
#include <d3d9.h>
#include <d3dx9.h>
#include <dinput.h>
#include <Xinput.h>
#include "InputHelper.h"
#include <time.h>
using namespace std;
 
#pragma comment(lib,"d3d9.lib")
#pragma comment(lib,"d3dx9.lib")
#pragma comment(lib,"dinput8.lib")
#pragma comment(lib,"dxguid.lib")
 
//application title
const string APPTITLE = "Test input program";
 
//screen resolution
#define SCREENW 1024
#define SCREENH 768
 
//Direct3D objects
LPDIRECT3D9 d3d = NULL; 
LPDIRECT3DDEVICE9 d3ddev = NULL; 
LPDIRECT3DSURFACE9 backbuffer = NULL;
LPDIRECT3DSURFACE9 surface = NULL;
InputHelper input = NULL;
 
bool gameover = false;
 
/**
 ** Game initialization function
 **/
bool Game_Init(HWND hwnd)
{
    //initialize Direct3D
    d3d = Direct3DCreate9(D3D_SDK_VERSION);
    if (d3d == NULL)
    {
        MessageBox(hwnd, "Error initializing Direct3D", "Error", MB_OK);
        return false;
    }
 
    //set Direct3D presentation parameters
    D3DPRESENT_PARAMETERS d3dpp; 
    ZeroMemory(&d3dpp, sizeof(d3dpp));
    d3dpp.Windowed = TRUE;
    d3dpp.SwapEffect = D3DSWAPEFFECT_DISCARD;
    d3dpp.BackBufferFormat = D3DFMT_X8R8G8B8;
    d3dpp.BackBufferCount = 1;
    d3dpp.BackBufferWidth = SCREENW;
    d3dpp.BackBufferHeight = SCREENH;
    d3dpp.hDeviceWindow = hwnd;
 
    //create Direct3D device
    d3d->CreateDevice( D3DADAPTER_DEFAULT, D3DDEVTYPE_HAL, hwnd,
        D3DCREATE_SOFTWARE_VERTEXPROCESSING, &d3dpp, &d3ddev);
 
    if (!d3ddev)
    {
        MessageBox(hwnd, "Error creating Direct3D device", "Error", MB_OK);
        return false;
    }
 
    //set random number seed
    srand(time(NULL));
 
    //clear the backbuffer to black
    d3ddev->Clear(0, NULL, D3DCLEAR_TARGET, D3DCOLOR_XRGB(0,0,0), 1.0f, 0);
 
    //create pointer to the back buffer
    d3ddev->GetBackBuffer(0, 0, D3DBACKBUFFER_TYPE_MONO, &backbuffer);
 
    //create surface
    HRESULT result = d3ddev->CreateOffscreenPlainSurface(
        800,                //width of the surface
        600,                //height of the surface
        D3DFMT_X8R8G8B8,    //surface format
        D3DPOOL_DEFAULT,    //memory pool to use
        &surface,           //pointer to the surface
        NULL);              //reserved (always NULL)
 
    if (!SUCCEEDED(result)) return false;
 
	input = InputHelper(hwnd);
 
    return true;
}
 
/**
 ** Game update function
 **/
void Game_Run(HWND hwnd)
{
    //make sure the Direct3D device is valid
    if (!d3ddev) return;
 
	//bb
	d3ddev->GetBackBuffer(0,0,D3DBACKBUFFER_TYPE_MONO,&backbuffer);
    //start rendering
    if (d3ddev->BeginScene())
    {    
        //stop rendering
        d3ddev->EndScene();
 
        //display the back buffer on the screen
        d3ddev->Present(NULL, NULL, NULL, NULL);
    }
 
	//acquire and get the keys
	//vector<char> keysPressed = input.getKeysPressed(hwnd);
	/*for each(char c in keysPressed)
	{
		if(c == DIK_ESCAPE)
		{
			PostMessage(hwnd, WM_DESTROY,NULL,NULL);
		}
	}*/
	//long *d = input.getMouseLocation(2);
	long d[2];
	input.getMouseLocation(d); // <-------------------------------------------------------- the error is here
	stringstream s;
	s << d[0];
	MessageBox(hwnd, s.str().c_str(),"test2",MB_OK);
}
 
/**
 ** Game shutdown function
 **/
void Game_End(HWND hwnd)
{
	input.~InputHelper();
    if (surface) surface->Release();
    if (d3ddev) d3ddev->Release();
    if (d3d) d3d->Release();
}
 
 
/**
 ** Windows event callback function
 **/
LRESULT WINAPI WinProc( HWND hWnd, UINT msg, WPARAM wParam, LPARAM lParam )
{
    switch( msg )
    {
        case WM_DESTROY:
            gameover = true;
            PostQuitMessage(0);
            return 0;
    }
    return DefWindowProc( hWnd, msg, wParam, lParam );
}
 
 
/**
 ** Windows entry point function
 **/
int WINAPI WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance, LPSTR lpCmdLine, int nCmdShow)
{
    //create the window class structure
    WNDCLASSEX wc;
    wc.cbSize = sizeof(WNDCLASSEX); 
    wc.style         = CS_HREDRAW | CS_VREDRAW;
    wc.lpfnWndProc   = (WNDPROC)WinProc;
    wc.cbClsExtra     = 0;
    wc.cbWndExtra     = 0;
    wc.hInstance     = hInstance;
    wc.hIcon         = NULL;
    wc.hCursor       = LoadCursor(NULL, IDC_ARROW);
    wc.hbrBackground = (HBRUSH)GetStockObject(WHITE_BRUSH);
    wc.lpszMenuName  = NULL;
    wc.lpszClassName = APPTITLE.c_str();
    wc.hIconSm       = NULL;
    RegisterClassEx(&wc);
 
    //create a new window
    HWND window = CreateWindow(APPTITLE.c_str(), APPTITLE.c_str(), 
       WS_OVERLAPPEDWINDOW, CW_USEDEFAULT, CW_USEDEFAULT,
       SCREENW, SCREENH, NULL, NULL, hInstance, NULL);
 
    //was there an error creating the window?
    if (window == 0) return 0;
 
    //display the window
    ShowWindow(window, nCmdShow);
    UpdateWindow(window);
 
    //initialize the game
    if (!Game_Init(window)) return 0;
 
 
    // main message loop
    MSG message;
    while (!gameover)
    {
        if (PeekMessage(&message, NULL, 0, 0, PM_REMOVE)) 
        {
            TranslateMessage(&message);
            DispatchMessage(&message);
        }
 
        Game_Run(window);
    }
 
    return message.wParam;
}

InputHelper.h

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
#include <string>
#include <XInput.h>
#include <vector>
#include <windows.h>
#include <d3d9.h>
#include <d3dx9.h>
#include <dinput.h>
#include <time.h>
#include <iostream>
using namespace std;
 
#pragma comment(lib,"d3d9.lib")
#pragma comment(lib,"d3dx9.lib")
#pragma comment(lib,"dinput8.lib")
#pragma comment(lib,"dxguid.lib")
 
class InputHelper
{
public:
 
	InputHelper(HWND hwnd);
 
	vector<char> getKeysPressed(HWND hwnd);
	long *getMouseLocation(long* l);
 
	~InputHelper(void);
 
private :
	LPDIRECTINPUT8 m_dinput;
	LPDIRECTINPUTDEVICE8 m_diKeyboard;
	LPDIRECTINPUTDEVICE8 m_diMouse;
	char m_keys[256];
	DIMOUSESTATE m_mouseState;
};

InputHelper.cpp :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
#include <windows.h>
#include <string>
#include <XInput.h>
#include <vector>
#include <d3d9.h>
#include <d3dx9.h>
#include <dinput.h>
#include <time.h>
#include <iostream>
#include "InputHelper.h"
using namespace std;
 
#pragma comment(lib,"d3d9.lib")
#pragma comment(lib,"d3dx9.lib")
#pragma comment(lib,"dinput8.lib")
#pragma comment(lib,"dxguid.lib")
#pragma comment(lib,"xinput.lib")
 
InputHelper::InputHelper(HWND hwnd)
{
	//init object
	HRESULT result = DirectInput8Create(GetModuleHandle(NULL),DIRECTINPUT_VERSION,IID_IDirectInput8,(void**)&m_dinput,NULL);
 
	if (SUCCEEDED(result)){
 
		//get device keybaord
		result = m_dinput->CreateDevice(GUID_SysKeyboard,&m_diKeyboard,NULL);
 
		//get device mouse
		HRESULT result2 = m_dinput->CreateDevice(GUID_SysMouse,&m_diMouse,NULL);
 
		if (SUCCEEDED(result) && SUCCEEDED(result2)){
 
			//set format
			result = m_diKeyboard->SetDataFormat(&c_dfDIKeyboard);
			result2 = m_diMouse->SetDataFormat(&c_dfDIMouse);
 
			if (SUCCEEDED(result) && SUCCEEDED(result2)){
 
				//set cooperative lvl (priority)
				result = m_diKeyboard->SetCooperativeLevel(hwnd, DISCL_NONEXCLUSIVE | DISCL_FOREGROUND);
				result2 = m_diMouse->SetCooperativeLevel(hwnd, DISCL_NONEXCLUSIVE | DISCL_FOREGROUND);
 
				if (SUCCEEDED(result) && SUCCEEDED(result2)){
 
				}
			}
		}
	}
}
 
vector<char> InputHelper::getKeysPressed(HWND hwnd)
{
	vector<char> keysPressed(1);
	//get keys
	if(SUCCEEDED(m_diKeyboard->Acquire()) && SUCCEEDED(m_diKeyboard->GetDeviceState(sizeof(m_keys),(LPVOID)&m_keys)))
	{
		int size = sizeof(m_keys);
		for(DWORD i = 0;i<256;++i)
		{
			if(m_keys[i] & 0x80)
			{
				MessageBox(hwnd, "PASS","TEST",MB_OK);
				keysPressed.push_back(m_keys[i]);
			}
		}
	}
 
	return keysPressed;
}
 
long *InputHelper::getMouseLocation(long* l)
{
	long *location = l;
	if(SUCCEEDED(m_diMouse->Acquire()) && SUCCEEDED(m_diMouse->GetDeviceState(sizeof(m_mouseState),(LPVOID)&m_mouseState)))
	{
		location[0] = m_mouseState.lX;
		location[1] = m_mouseState.lY;
		return location;
	}
	else
	{
		return location;
	}
}
 
InputHelper::~InputHelper()
{
	m_diKeyboard->Unacquire();
	if(m_diKeyboard)m_diKeyboard->Release();
	m_diMouse->Unacquire();
	if(m_diMouse)m_diMouse->Release();
	if(m_dinput)m_dinput->Release();
}

J'espère que vous allez pouvoir m'aider. (J'ai noté la ligne où il y a l'erreur avec un <----------------------)

Cordialement,

rXp>!<

[fregolo52]

Ce genre d'erreur avec "0xfeeeff0a" ca sent la variable non initialisée avec un programme compilé en Release.

je fais encore des erreurs bêtes.

Ce n'est pas une erreur bête, là tu t'attaques aux pointeurs.

Essaie un pointeur de pointeur, au lieu du pointeur :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

input.getMouseLocation(&d);

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

long *getMouseLocation(long** l);

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
long *InputHelper::getMouseLocation(long** l)
{
	long *location = *l;
...

PS : c'est pas beau, mais c'est un début.

[rXpCH]

Ce genre d'erreur avec "0xfeeeff0a" ca sent la variable non initialisée avec un programme compilé en Release.

Ce n'est pas une erreur bête, là tu t'attaques aux pointeurs.

Essaie un pointeur de pointeur, au lieu du pointeur :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

input.getMouseLocation(&d);

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

long *getMouseLocation(long** l);

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
long *InputHelper::getMouseLocation(long** l)
{
	long *location = *l;
...

PS : c'est pas beau, mais c'est un début.

J'ai trouvé plus d'info sur l'erreur, ce qu'aide encore moins ^^.
Alors l'envoie du tableau marche, j'ai fait un test simple et ça marche mais dès que je fait cette commande :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

m_diMouse->Acquire();

L'erreur apparaît.
Alors ce qui me pose le plus problème là dedans est que si je fais cette commande après avoir initialiser l'objet (donc dans le constructeur de la classe) il n'y a pas de problème mais si j'appelle cette méthode dans une méthode de ma classe, l'erreur apparaît.
Ai-je fait une bêtise avec la variable m_diMouse ou m_diKeyboard ?

[kessoufi]

Salut

j'ai 3 remarques :

1- dans :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

InputHelper input = NULL;

ici tu appelle le constructeur de InputHelper avec HWND = 0

2- dans le code suivant tu crée un InputHelper intermédiaire, qui passera son contenu par copy-constructor à input, en quittant Game_Init le destructeur de cet objet intermédiaire est appelé

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
 
bool Game_Init(HWND hwnd)
{
	...
	input = InputHelper(hwnd);
    return true;
}

3- là c'est peut être l'origine de ton crash : le destructeur appelé en sortie de Game_Init a libéré les ressources directX :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
InputHelper::~InputHelper()
{
	m_diKeyboard->Unacquire();
	if(m_diKeyboard)m_diKeyboard->Release();
	m_diMouse->Unacquire();
	if(m_diMouse)m_diMouse->Release();
	if(m_dinput)m_dinput->Release();
}

à mon avis input devrait être un pointeur sur InputHelper , et le constructeur de ce dernier devrait être explicit

Cdt

[rXpCH]

Salut

j'ai 3 remarques :

1- dans :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

InputHelper input = NULL;

ici tu appelle le constructeur de InputHelper avec HWND = 0

2- dans le code suivant tu crée un InputHelper intermédiaire, qui passera son contenu par copy-constructor à input, en quittant Game_Init le destructeur de cet objet intermédiaire est appelé

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
 
bool Game_Init(HWND hwnd)
{
	...
	input = InputHelper(hwnd);
    return true;
}

3- là c'est peut être l'origine de ton crash : le destructeur appelé en sortie de Game_Init a libéré les ressources directX :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
InputHelper::~InputHelper()
{
	m_diKeyboard->Unacquire();
	if(m_diKeyboard)m_diKeyboard->Release();
	m_diMouse->Unacquire();
	if(m_diMouse)m_diMouse->Release();
	if(m_dinput)m_dinput->Release();
}

à mon avis input devrait être un pointeur sur InputHelper , et le constructeur de ce dernier devrait être explicit

Cdt

D'accord c'est ce que je craignait, j’espérais en mettant = NULL qu'il n'allait pas lancer le constructeur. Donc je dois créer un pointeur et dans game_init je crée le inputhelper et point dessus ?
Comment faire ?

[kessoufi]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
InputHelper *input = NULL;
bool Game_Init(HWND hwnd)
{
	...
	input = new InputHelper(hwnd);
    return true;
}

et n'oublie pas à la fin de ton prog :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

	delete input;

Cdt

[rXpCH]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
InputHelper *input = NULL;
bool Game_Init(HWND hwnd)
{
	...
	input = new InputHelper(hwnd);
    return true;
}

et n'oublie pas à la fin de ton prog :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

	delete input;

Cdt

Cool c'est bien cela et ça m'aide a bien voire l'utilité des pointeurs ici.
Hop on continue
Merci encore !

rXp>!<