Discussion :

[poussinphp]

Bonjour à tous !

Je suis sur le développement d'un jeu. j'utilise une class "Carte" permettant le chargement d'un fichier contenant les données d'une carte. ce fichier est un fichier au format texte et il sont nom est contenu dans un "const char* fichier) déclaré dans la class.

je fais donc :

Code :

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

1
2
3
4
5
6
7
	int largeurImage = 0;
	int hauteurImage = 0;
	char* fichierImage = NULL;
	char delimiter;
 
	std::ifstream donnees(this->fichier);
donnees >> largeurImage >> delimiter >> hauteurImage >> delimiter >> fichierImage;

pour lire le contenu du fichier. J'ai une erreur que je ne comprend pas au moment de lire les informations (invalid null pointer)

Je comprend pas trop d'ou ça peut venir. j'ai bien vérifier le chemin du fichier, ça fonctionne et la première ligne du fichier est:

Code :

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

20,15,test.png

ça devrais fonctionner... je comprend pas

[loufoque]

ça devrais fonctionner...

Qu'est-ce qui te permet de prétendre une chose pareille ?

Comment tu veux écrire les données de ton fichier en mémoire, puisque ton pointeur est nul ?

[koala01]

Salut,

Cela n'a aucune chance de fonctionner, vu que tu essaye de faire placer une valeur lue dans une pointeur initialisé à NULL...

On le dit et on le répète, ces temps, si à longueur de journée:

POUR LA GESTION DE CHAINE DE CARACTERES, IL FAUT UTILISER LA CLASSE STD::STRING QUI REND LES CHOSES BIEN PLUS FACILES QUE LES CHAINE DE CARACTERES "C STYLE"

Est-ce écrit, cette fois-ci suffisemment grand pour que cela rentre enfin dans la tete de tous

Si, vraiment, il faut que tu récupère une chaine C style, pour l'utilise comme nom de fichier pour ouvrir un flux, par exemple, tu dispose de la fonction membre c_str() qui fera cela tres bien...

[poukill]

POUR LA GESTION DE CHAINE DE CARACTERES, IL FAUT UTILISER LA CLASSE STD::STRING QUI REND LES CHOSES BIEN PLUS FACILES QUE LES CHAINE DE CARACTERES "C STYLE"

Est-ce écrit, cette fois-ci suffisemment grand pour que cela rentre enfin dans la tete de tous

Là je pense que c'est bon !

[poussinphp]

oula oula !

J'avais tendance à utliser du char* comme habitude (langage C). Je vais essayer d'utiliser les std::string mais c'est un peu embettant dans certains cas car l'api que j'utilise (SDL) est en C.

De l'autre, je pensais que ça fonctionnerais car j'initialise à NULL mais ensuite je lis le fichier : je place dans le char* le contenu...

si je fais :

Code :

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

1
2
3
 
char* toto = NULL;
toto = "coucou";

ça pose aucun problème...

Enfin bon, je vais me mettre au string

Encore merci

[Laurent Gomila]

std::string peut te sortir un const char* quand tu veux avec sa fonction membre c_str(). Bosser avec des APIs C n'est pas une excuse pour ne pas les utiliser

Ensuite quand tu fais toto = "coucou", c'est une erreur car "coucou" est une chaîne littérale certainement placée dans un espace en lecture seule. Il faudrait donc que toto soit un const char*.

Ensuite concernant la lecture, l'opérateur >> ne vas pas allouer d'espace pour la chaîne qu'il va lire, il va supposer qu'il y a suffisamment de place au bout du pointeur. Comme tu ne connais pas la taille de la chaîne à lire... autant dire que std::string est inévitable.

[koala01]

oula oula !

J'avais tendance à utliser du char* comme habitude (langage C). Je vais essayer d'utiliser les std::string mais c'est un peu embettant dans certains cas car l'api que j'utilise (SDL) est en C.

Ca, c'est typiquement un mauvais argument...

Comme, malgré tout, tu peux toujours obtenir la chaine C style à ta bonne convenance, utilise la std::string tant que tu peux, et passe en C style seulement quand tu as réellement besoin du C style (SDL, nom de fichier à fournir pour ouvrir un flux...)

La seule exception est, finalement, quand tu dois lire le contenu d'un fichier dans un buffer en vue de le traiter et que tu ne peux pas etre qu'il n'y aura pas un signe "eof" ou autre singificateur de fin (typiquement, les fichier images et son)...

A ce moment là, le fichier sera bien souvent encodé en binaire, d'ailleurs, et ce qu'il te faudra faire, c'est de déclarer un pointeur de type char, de lui allouer la taille pour contenir la totalité du fichier, et de lire le fichier "d'une seule traite"...

De l'autre, je pensais que ça fonctionnerais car j'initialise à NULL mais ensuite je lis le fichier : je place dans le char* le contenu...

si je fais :

Code :

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

1
2
3
 
char* toto = NULL;
toto = "coucou";

ça pose aucun problème...

ce que tu fais là, c'est:
déclarer un pointeur et l'initialiser à NULL
fournir à ce pointeur l'adresse à laquelle se trouve la chaine (définie dans la mémoire "read only") "coucou"...

D'ailleurs, c'est bien simple, il te sera normalement impossible d'écrire un code du genre de

Code :

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

1
2
3
4
 
char* toto = NULL;
toto = "coucou";
toto[2]='a';

Si on initialise un pointeur à NULL, c'est simplement:

• parce que NULL est la seule valeur connue et testable dans n'importe quelle application
• parce que lors de la déclaration d'un pointeur, on ne peut jamais etre sur du contenu, du fait d'une utilisation antérieure de la mémoire

Enfin bon, je vais me mettre au string

Encore merci

Ca, c'est une tres bonne initiative
[EDIT]grillé