Discussion :

[blueLight]

Bonjour !
Je travaille sur un petit jeu et mon problème est que lorsque un niveau est fini par le joueur, j'utilise un "destroySigleton()" puis juste après pour recharger un niveau j'appelle un "GetSingleton()".
Le programme plante sur le new du "GetSingleton()" quand la variable "instance" est égale à nullptr.

Que dois je faire ? Est ce que je fais les choses d'une mauvaise façon ?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
D3D11Rendering::destroySingleton();
 
        D3D11Rendering* rendering = D3D11Rendering::getInstance();//plante ici dans le getInstance()
 
        debugFile << "before delete rendering" << endl;
 
        if (rendering)
        {
            delete rendering;
            debugFile << "after delete rendering" << endl;
        }

[Bktero]

Bonjour,

A part arrêter de faire des singletons et utiliser des pointeurs intelligents plutôt que de faire des new/delete manuellement, il faudrait nous montrer plus de code pour comprendre le problème

[dalfab]

Bonjour,

Si un new ne fonctionne pas c'est le plus souvent parce que on a cassé le mécanisme d'allocation. Pour le casser ça peut être dû a des actions telles que :
- écrire dans une zone allouée en dehors de la zone d'allocation;
- faire un delete de n'importe quoi;
- ou d'autres "undefined behaviors".
Il faut donc voir plus de code pour en dire plus. Mais déjà détruire un singleton pour le recréer n'est pas très habituel comme démarche. Est-ce que la valeur retournée par D3D11Rendering::getInstance() doit/peut être détruite, j'ai des doutes?

[blueLight]

J'ai enlevé les singletons pour la classe de rendu (j'utilise un unique_ptr à la place), je les ai laissés pour les autres classes qui en utilise et il semblerait que mon problème ait évolué.
Mon jeu plante au bout d'une douzaine de niveaux. Est que ça pourrait être dû à une fragmentation de la mémoire et si oui, que dois je faire pour résoudre cela ?
Je ne sais pas trop quelle partie du code montrer, mais voilà la partie qui gère le passage au niveau suivant:

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
 
        Sounds::getInstance()->playSound(*Sounds::getInstance()->levelWon);
 
 
        Gameplay::afterGameOver = false;
 
        //WaitForSingleObject(Sounds::handlePlaySoundThread, INFINITE);
 
 
        Gameplay::indexLevel++ ;
 
        if (Gameplay::indexLevel > 39 )
            Gameplay::indexLevel = 0;
 
 
 
        D3D11Rendering::rendering.reset();
        D3D11Rendering::rendering = make_unique< D3D11Rendering>();

[Bousk]

Les chances qu'on trouve le crash avec le code que tu fournis (pas) sont proches de 0.
Cela dit, voir que le renderer est détruit et recréé pour un changement de niveau, ou le nombre de globales que je vois dans ce très court code, ça laisse songeur sur la qualité générale du code et le nombre de bugs qui peuvent se cacher derrière ce crash.

[blueLight]

Dans ce cas comment je devrais faire pour charger le niveau suivant ?