struct & pointeur fonction

1 avril 2008 à 21:26:02

Bonjour ,
J'ai un petit problème d'enregistrement dans une structure perso à partir d'une fonction.
Le problème c'est que je n'arrive pas à enregistrer toutes les données dans le tableau de la structure !!!
Voici le code simplifié pour que vous puissiez m'aider le plus vide et bien possible !!!!

#include <iostream>
struct Place
{
std::string nom;
int X;
int Y;
};
void placePerso(Place *place)
{
int X, Y;
std::string tmp, nom;
for(int j=0;j<10;j++)
{
std::cin >> nom ;
std::cin >> X ;
std::cin >> Y ;
place[j].nom = nom;
place[j].X = X;
place[j].Y = Y;
//ici je n'arrive pas à rentrer les données dans le tableau de la structure place
// comment faire ??? sachant que c'est un tableau ???
}
}
int main()
{
Place place[10];
placePerso(&place[10]);
for(int i=0;i<10;i++)
{
std::cout << place[i].nom << "\n";
std::cout << place[i].X << "\n";
std::cout << place[i].Y << "\n";
std::cout << "\n";
}
return EXIT_SUCCESS;
}

en espérant que ce soit possible
bougaloo

Nanoc

1 avril 2008 à 21:29:09

Pourquoi ne pas utiliser un std::vector<>, c'est tellement plus simple.

Pourquoi est-ce que tu passes &places[10] à la fonction et pas "place" tout simplement ?

Ta boucle doit aller de i=0 à i<10 et pas 11.
Ensuite tu fais place[i].nom = blabla...
place[i].X = ...

Co-auteur du cours de C++. ||| Posez vos questions sur le forum ||| Me contacter.

bougaloo

1 avril 2008 à 21:41:08

je viens de modifier un peu le code et après quelques petites corrections , le programme plante pendant son exécution au bout de la troisième entrée de nom !!!
pourquoi je ne sais pas !!!
quelqu'un aurait une idée ???

Nanoc

1 avril 2008 à 21:42:39

Pour les raisons que j'ai évoqué au-dessus (sans le vector).

Co-auteur du cours de C++. ||| Posez vos questions sur le forum ||| Me contacter.

bougaloo

1 avril 2008 à 22:26:13

Et bien maintenant en suivant ton conseil de ne mettre que : placePerso(place); lors de l'appel de la fonction , l'entrée des données fonctionne bien , mais de retour dans la fonction main principale , les données ne sont pas renvoyé dans celle-ci et donc çà ne sert à rien !!!
Moi j'aimerais modifier mon tableau de structure dans une fonction extérieur au main , tout en conservant les valeurs données !!!
Une solution ???

lmghs

1 avril 2008 à 22:38:38

std::istream operator>>(std::istream &is, Place & p) {
return is >> p.X >> p.y >> p.nom;
}
void readN(Place * places, int N) {
for (int i=0 ; i<N ; ++i)
{ if (!(is>>places[i]))
return i;
}
return N;
}
...
readN(&places[0], 10);

Un vecteur reste 100 fois plus propre, sûr et robuste.

bougaloo

2 avril 2008 à 9:22:21

et bien je ne connas pas les vecteur
mais en ayant mis ton code d'aide çà me fais une erreur dans le fichier ios_base.h avec environ 20 erreur dans celui ci !!!
donc ... solution ???
merci encore malgrés que je ne suis toujours pas avancé dans mon probléme !!

lmghs

2 avril 2008 à 10:52:34

Quelles erreurs ?

MatteX

2 avril 2008 à 20:00:57

#include <iostream>
struct Place;
std::istream& operator>>(std::istream &is, Place & p) {
return is >> p.X >> p.y >> p.nom;
}
int readN(Place * places, int N) {
for (int i=0 ; i<N ; ++i)
{ if (!(std::cin >> places[i]))
return i;
}
return N;
}
int readN(Place * places, int N, std::istream & is) {
for (int i=0 ; i<N ; ++i)
{ if (!(is >> places[i]))
return i;
}
return N;
}
...
//Place places[ 10 ];
readN(places, 10);
//OU
readN(places, 10, std::cin );

readN retourne un int pas un void

operator >> ne devrait-il pas retourner un std::istream& (si mes souvenirs sont bons)

is n'est pas défini dans readN, j'ai mis std::cin mais tu peux passer une std::istream& en paramètre aussi comme ça ta fonction peux lire de n'importe quel istream et ses descendants. (ex. ifstream)

&places[ 0 ] == places

lmghs a seulement été un peu trop rapide

bougaloo

2 avril 2008 à 20:13:24

J'ai omis de dire un petit truc !!!
J'ai mis des std::cin pour simplifier la chose mais c'est à partir d'une base de donnée que je remplis le tableau de structure , donc pour faire une modification de std::istream operator>> je ne sais comment faire !!!
et il n'y a pas une solution plus simple car le problème apparaît uniquement parce que je fais un tableau !!!
Enfin ... maintenant j'ai le temps ... pour attendre vos réponses

mais j'ai essayé ce que MatteX m'a dit et çà n'a pas l'air de fonctionner !!!
Merci beaucoup

lmghs

2 avril 2008 à 21:02:45

Citation : MatteX

a- readN retourne un int pas un void

b- operator >> ne devrait-il pas retourner un std::istream& (si mes souvenirs sont bons)

c- is n'est pas défini dans readN, j'ai mis std::cin mais tu peux passer une std::istream& en paramètre aussi comme ça ta fonction peux lire de n'importe quel istream et ses descendants. (ex. ifstream)

d- &places[ 0 ] == places

e- lmghs a seulement été un peu trop rapide

a-, b- Si tout à fait

c- Bonne évolution

d- Et ... non.

places est un tableau statique ... qui est implicitement converti par le compilo en &place[0]

void match(int *p) {
std::cout << "match pointer\n";
}
template <std::size_t N> void match(int (&)[N]) {
std::cout << "match array["<<N<<"]\n";
}
void match_array() {
int t[10];
match(t);
match(&t[0]);
}

(On peut faire sauter le template, mais cela perd un peu de son intérêt)

e- Hé hé :-°

Citation : bougaloo

J'ai omis de dire un petit truc !!!
1- J'ai mis des std::cin pour simplifier la chose mais c'est à partir d'une base de donnée que je remplis le tableau de structure , donc pour faire une modification de std::istream operator>> je ne sais comment faire !!!
2- et il n'y a pas une solution plus simple car le problème apparaît uniquement parce que je fais un tableau !!!

3- mais j'ai essayé ce que MatteX m'a dit et çà n'a pas l'air de fonctionner !!!

1- Passe par un std::istringstream
2- quoi ?
3- précise (ne serait pas à cause de la déclaration anticipée de Place alors que la définition est nécessaire depuis l'opérateur d'extraction ?)