récupérer les coordonnées de mon perso

depuis un fichier

Sujet résolu

15 août 2007 à 19:25:50

Bonjour,
j'ai rencontré un problème par rapport à ma fonction pour récupérer les coordonnées de mon personnage depuis un fichier nommé posheros.lvl
composé de 2 lignes , la 1ère qui définit posheros.x et la 2ème posheros.y
voilà ma fonction:

int chargerPosHeros(SDL_Rect posheros)
{
int i,j;
fstream fichier;
fichier.open("posheros.lvl");
int tmp;
if(fichier)
{
fichier >> tmp;
posheros.x = tmp;
fichier >> tmp;
posheros.y = tmp;
}
fichier.close();
}

voilà
c'est le code que j'avais mis, sûr à 100%, que j'ai remplacé par posheros.x = 0; posheros.y = 0;

Anonyme

15 août 2007 à 19:47:05

et le problème est??

ça ne te sauve pas les nouvelle coordonnée?
revois le cours sur le C (niveau pointeur) ou C++ avec les références.

Hiura

Anonyme

15 août 2007 à 20:06:57

Salut,

i et j servent à quoi ?

Dans ta fonction, il n'y a pas de problème. Et comme l'a dit hiura, il faut que tu nous disent le problème qu'il pourrait y avoir !

Xav57

chlablak

15 août 2007 à 22:07:35

- ta fonction ne retourne rien
- remplace fstream par ifstream
- tu peux faire directement fichier >> posheros.x sans passer par tmp (meme chose pour y)
- ta structure posheros ne changeras jamais, passe par un pointeur

C++ Références | Idiomes | FAQ

gogohan

16 août 2007 à 9:31:27

ça marche!!
merci!!
En plus maintenant j'ai compris pourquoi et comment utiliser les pointeurs. merci.
Je vous passe les bouts de code concernés:

int chargerPosHeros(Sint16 *pointeur1, Sint16 *pointeur2);
// plus loin à la ligne 85
SDL_Rect posheros;
Sint16 *pointeur1 = &posheros.x;
Sint16 *pointeur2 = &posheros.y;
chargerPosHeros(pointeur1, pointeur2);
*pointeur1 >> posheros.x ;
*pointeur2 >> posheros.y ;
// la fonction charger posheros
int chargerPosHeros(Sint16 *pointeur1, Sint16 *pointeur2)
{
ifstream fichier; // mettre fstream ne change rien au résultat
fichier.open("posheros.lvl");
if(fichier)
{
fichier >> *pointeur1;
fichier >> *pointeur2;
}
fichier.close();
}

Merci beaucoup!!

Anonyme

16 août 2007 à 10:46:44

SDL_Rect posheros;
Sint16 *pointeur1 = &posheros.x;
Sint16 *pointeur2 = &posheros.y;
chargerPosHeros(pointeur1, pointeur2);

Tu sais que tu peux te passer de tes deux Sint16?

chargerPosHeros(&posheros.x, &posheros.y);

C'est plus simple, non?

(Si tu crées un double d'une variable à chaque fois que tu dois en passer à une fonction, tu va faire sauter la mémoire!)

gogohan

16 août 2007 à 10:50:01

c pas grave, j'ai quand même réussi. merci

Anonyme

16 août 2007 à 10:58:44

Citation : gogohan

c pas grave, j'ai quand même réussi. merci

(je m'en excuse).

Il faut prendre de bonne habitude dès le début; sinon comment tu ferras quand tu devra créer un très gros programme? Tu ne vas pas quand même demander à l'utilisateur d'acheter un nouveau PC 4 x plus puissant?? o_O

Enfin moi je te donne des conseils, à toi de choisir de les prendre ou pas

.

Hiura

chlablak

16 août 2007 à 12:50:01

ifstream fichier; // mettre fstream ne change rien au résultat

utilise toujours ifstream pour lire dans un fichier

C++ Références | Idiomes | FAQ