Discussion :

[Vinkkk]

Bonjour à tous,

Je viens de compiler mon programme c++ avec SDL2 et SDL2_gfx et une fois que je lance le .exe Windows me dit "le programme a cessé de fonctionner". Donc je lance gdb (je suis un gros noob avec gdb) et
là je peux observer le message suivant : "Program received signal SIGSEGV, Segmentation fault". ok.
Tout d'abord j'avoue ne pas comprendre le signal SIGSEGV.
De plus j'aimerais savoir comment vous vous réglez ce genre de problème : quel est votre "procédure" pour traiter cette erreur. Je ne vous met pas de morceau de mon code ou de screen tout simplement parce que
le but n'est pas que vous trouviez la solution à mon problème à ma place mais plutôt que je comprenne la méthode pour régler ce genre de problème.

En vous remerciant.

[Bousk]

segmentation fault = tu as probablement un pointeur pourri quelquepart
Si tu lances avec gdb, il devrait s'arrêter sur la ligne en question non ?

[Vinkkk]

En effet gdb s'arrête à une ligne. Si j'ai bien compris il s'arrête à la ligne 2614 de src/video/SDL_video.c mais en même temps il me dit "No such file or directory", je ne comprend pas trop ce qu'il essaie de me dire là...

[Vinkkk]

Ok, je crois avoir compris mon erreur !

En gros j'ai un bloc SDL_Init(INIT_EVERYTHING) et si il est égal à 0 il initialise les pointer window et renderer, et une fois fais il passe le booléen run en true ce qui lui permet de faire tourner la boucle principal. Si run != true alors il clean tout. Dans mon cas SDL ne s'initialise pas et passe direct au clean() ce qui engendre un problème de pointer (SIGSEGV) au moment de du SDL_DestroyWindow(window).

Si quelqu'un à une idée de pourquoi SDL n'a pas réussi à s'initialiser je suis preneur !

[Kannagi]

Un code source serait assez utile si tu veux qu'on t'aide !

[Vinkkk]

Voilà tout d'abord le main:

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
#include "Jeu.h"
#include <iostream>
 
using namespace std;
 
//#define FPS 60
 
Jeu* jeu;
int main(int argc, char **argv){
    jeu = new Jeu();
    jeu->init("test",
               SDL_WINDOWPOS_UNDEFINED,
               SDL_WINDOWPOS_UNDEFINED,
               1280,
               720,
               false);
 
    while(jeu->run()){
        jeu->events();
        jeu->update();
        jeu->render();
    }
 
 
    jeu->clean();
    return 0;
}

et ensuite la fonction Init() de Jeu:

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
void Jeu::init(const char* title, int xpos, int ypos, int width, int heigth, bool fullscreen){
    int fl = fullscreen == true ? SDL_WINDOW_FULLSCREEN : SDL_WINDOW_SHOWN;
 
    if(SDL_Init(SDL_INIT_EVERYTHING)==0){
        window = SDL_CreateWindow(title, xpos, ypos, width, heigth, fl);
        if(window){
        }
 
        renderer = SDL_CreateRenderer(window, -1, SDL_RENDERER_ACCELERATED);
        if(renderer){
            SDL_SetRenderDrawColor(renderer, 255, 255, 255, 255);
        }
 
        isRun = true;
    }
    else{
        isRun = false;
    }
 
    joueur = new Joueur(width/2, heigth/2, renderer);
}

[Bousk]

- jeu n'a aucune raison d'être créé dans la heap
- ça aurait évité une fuite mémoire
- init qui ne retourne pas son succès ou non
- peu de vérification de succès : si la window ne se crée pas ? si le renderer ne se crée pas ?
- aucun affichage des erreurs éventuelles

Ensuite ou verra le pourquoi du problème qui t'amène, qui peut n'être qu'un problème de configuration ou que sais-je.

[Vinkkk]

Ok j'ai changé tout ça et maintenant ça marche, voilà le nouveau code d'initialisation :

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
void Jeu::init(const char* title, int xpos, int ypos, int width, int heigth, bool fullscreen){
    int fl = fullscreen == true ? SDL_WINDOW_FULLSCREEN : SDL_WINDOW_SHOWN;
 
    SDL_Init(SDL_INIT_EVERYTHING);
    if(SDL_Init(SDL_INIT_VIDEO)!=0){
        cout << SDL_GetError() << endl;
    }
    if(SDL_Init(SDL_INIT_EVENTS)!=0){
        cout << SDL_GetError() << endl;
    }
 
    if(SDL_Init(SDL_INIT_VIDEO)==0){
        window = SDL_CreateWindow(title, xpos, ypos, width, heigth, fl);
        if(window){
            renderer = SDL_CreateRenderer(window, -1, SDL_RENDERER_ACCELERATED);
            if(renderer){
                SDL_SetRenderDrawColor(renderer, 0, 0, 0, 255);
            }
        }
        isRun = true;
    }
    else{
        isRun = false;
    }
 
    joueur = new Joueur(width/2, heigth/2, renderer);
}

- jeu n'a aucune raison d'être créé dans la heap

Qu'entends-tu par là ?

[Guntha]

Qu'entends-tu par là ?

Cette ligne:

Code :

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

jeu = new Jeu();

N'est pas indispensable, surtout que "Jeu* jeu;" est une déclaration globale (si cette ligne était dans le main() et que Jeu était énorme, il serait déclaré sur la stack et serait donc potentiellement trop gros. Ça m'est déjà arrivé :p ).
Bref jeu pourrait être déclaré comme un Jeu et non un Jeu*: il est initialisé une fois, et le pointeur ne change pas pendant toute la durée de l'exécution du programme, donc pourquoi l'allouer dynamiquement?

[Vinkkk]

Ok , merci Guntha, je comprend maintenant !