Discussion :

[poussinphp]

Bonjour à tous !

Je suis sur un gros problème et je ne trouve pas la solution. j'ai cherché partout, rien.

J'utilise une classe:

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
 
class carte
{
      public:
              carte();
              ~carte();
              void chargement(char* mapp, SDL_Surface *screen);
              int **tableau_donnee_carte;
              int **tableau_donnee_collision;
              void afficher();
              void bouge(int x, int y);
              int collision(int x, int y, int bouge_x, int bouge_y);
      private:
              int i;
              int j;
              int k;
              int deplacement_x;
              int deplacement_y;
              FILE *fp;
              int taille_carte_x;
              int taille_carte_y;
              string tileset;
              string path;
              SDL_Rect clip_map[480];
              SDL_Surface *map;
              SDL_Rect position;
              SDL_Surface *screen;
              void chargement_png();
};

Dans cette classe j'ai une fonction.

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 carte::chargement(const char* map, SDL_Surface *screen)
{
    this->screen = screen;
    fp = fopen(map,"r");
    fscanf(fp, "%d,%d,%s", &this->taille_carte_y, &this->taille_carte_x, this->tileset.c_str());
    tableau_donnee_carte = (int**)calloc(this->taille_carte_x, sizeof(unsigned int));
    tableau_donnee_collision = (int**)calloc(this->taille_carte_x, sizeof(unsigned int));
        for (i = 0; i < taille_carte_x; i++)
        {
            tableau_donnee_carte[i] = (int*)calloc(this->taille_carte_y, sizeof(unsigned int));
            tableau_donnee_collision[i] = (int*)calloc(this->taille_carte_y, sizeof(unsigned int));
            for (j = 0; j < taille_carte_y; j++)
            {
                 fscanf(fp, "%d|", &(tableau_donnee_carte[i][j]));
                 fscanf(fp, "%d,", &(tableau_donnee_collision[i][j]));    
            }       
        }
    fclose(fp);
    chargement_png();
}

J'apelle cette fonction via :

Code :

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

1
2
 
o_carte->chargement("carte.hkm", ecran);

Après plusieurs tests. Il semble que ça plante sur la ligne

Code :

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

fp = fopen(map,"r");

J'ai essayé de mettre les paramètres de la fonction en const char*. j'ai essayé de passer en string etc... rien à faire. ça compile mais ça plante (je suis sur dev-c++). Par contre, si je declare un

Code :

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

char* test = "carte.hkm"

dans ma class et que je met test au lieu de map dans la ligne du fopen. Magie, ça marche.

Malheureusement, le programme etant un moteur de jeu. je peut pas me permettre de faire comme ça.

[Médinoc]

Houlà, tu as quelques trucs bien horribles ici.

Code :

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

fscanf(fp, "%d,%d,%s", &this->taille_carte_y, &this->taille_carte_x, this->tileset.c_str());

Très mauvais.
Pour un Xscanf("%s"), il te faut utiliser un buffer de caractères : Pas moyen d'y mettre une string standard.
Et aussi, précise la taille dans le format de scanf.

Code :

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

tableau_donnee_carte = (int**)calloc(this->taille_carte_x, sizeof(unsigned int));

Ici, tu alloues un tableau de pointeurs en demandant la taille d'entiers. Sous Win64, ça planterait. Pourquoi n'utilises-tu pas l'opérateur new ?

[poussinphp]

je vais travailler sur ça Merci pour le conseil

Par contre j'ai toujours le problème du fopen.

[loufoque]

Normal, tu ne vérifies pas la valeur de retour.

Accessoirement, ce serait pas mal d'utiliser les fonctions standards C++ plutôt que les fonctions standards C.

[JolyLoic]

Code :

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

1
2
3
 
    fp = fopen(map,"r");
    fscanf(fp, "%d,%d,%s", &this->taille_carte_y, &this->taille_carte_x, this->tileset.c_str())

En utilisant les flux C++, ce genre de code se réécrirait (sans aucune validation de format, comme ta version) :

Code :

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

1
2
3
ifstream is(map);
char virgule;
is >> taille_carte_y >> virgule >> taille_carte_x >> virgule >> tileset;

[loufoque]

Et le tileset.c_str() avec *scanf, c'est interdit.
(Ça doit faire une segfault d'ailleurs)