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 ?

[koala01]

Salut,

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

La question est : pourquoi veux tu détruire ton instance de D3D11Rendering

Si j'en crois le nom que tu as donné à cette classe, elle ne devrait s'occuper que d'une seule chose à savoir : le rendu 3D sous DirextX 11. Et les comportements qui s'occupent de ce rendu, les informations manipulées pour y arriver n'ont aucune raison de changer en fonction du niveau de ton jeu: quand tu veux effectuer le rendu, tu devrais fournir au comportement de rendu "un ensemble de données" qui seront utilisées pour effectuer ce rendu et "point barre".

Quant aux informations qui permettent à ton "système de rendu" d'effectuer ce rendu, elles devraient venir "d'ailleurs" (sans doute du niveau, voir, d'un "système médian" permettant de convertir les données de ton niveau en données de rendu).

Et, du coup, le passage d'un niveau à l'autre consisterait à se dire que:

1. je liquide les informations "métier" concernant le niveau terminé
2. (éventuellement) je liquide les informations de rendu (matériaux, textures, vertex et autres) du niveau terminé
3. je prend note du fait que je passe au niveau suivant (avec vérifications si besoin) (level++;
4. je charge les informations "métier" concernant le niveau suivant
5. (éventuellement) je charge les informations de rendu concernant le niveau suivant
6. j'utilise tout cela pour la suite du jeu

[blueLight]

Ok merci pour ta réponse.