Discussion :

[yo_haha]

Bonjour,
J'ai un fichier *.wkt contenant 1000 lignes sous cette forme : POINT(654875.345 68912.296)
J'ai essayé d'écrire une fonction qui permet de lire chaque ligne, d'extraire les deux nombres (qui représentent des coordonnées) et de les mettre dans des objets Point que j'ai mis dans un vector.
Dans la classe Point j'ai mis deux attributs de type double X et Y, en plus des getters et des setters.
Le problème est que je ne peux pas lire les deux doubles. Lorsque j'affiche ce que le programme a lu, je trouve des valeurs aléatoires.
Est-ce que quelqu'un a une idée de ce qui se passe ?

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
30
31
32
33
34
bool charger(char* nom, vector<Point> &liste, int taille){ //taille = 1000 : nombre de lignes du fichier
        ifstream fin;
        fin.open(nom);
 
        //Vérifier que le fichier est ouvert sans problèmes
        if(!fin){
                cout<<"Probleme a l'ouverture du fichier"<<endl;
                return false;
 
        }// Fin if
 
        // Saisie des coords dans les objets Point
        double x, y;
        char c;
        for(int j=0; j<taille; j++){
            // On saute la chaîne 'POINT('
            fin.seekg(6,ios_base::cur);
 
            // On prend le X, le Y et le caractère ')' à la fin de chaque ligne j
            fin>>x>>y>>c;
            cout<<x<<endl<<y<<endl<<c<<endl; //les valeurs de x et y ne sont pas les bonnes. Par contre dans c il y a ')' comme prévu
            // On met le X et le Y dans le point j de la liste
            liste.assign(1,Point(x,y));
            cout<<liste[j].getX()<<endl<<liste[j].getY()<<endl;//ça me donne des valeurs encore différentes
 
        }// Fin for
 
        // Fermeture du stream
        fin.close();
 
        // Valeur retournée par la fonction
        return true;
 
}// Fin charger(...)

[LittleWhite]

Bonjour,

Il se passe que le cin n'arrive pas à lire ce que vous demander.
Je vous conseille d'utiliser des floats juste pour essai ... de plus je conseille aussi de faire une surcharge de l'operateur << dans la classe Point afin de rendre le code plus simple (et plus logique C++)

[Arzar]

Bonjour :

http://www.cplusplus.com/reference/stl/vector/assign/

vector::assign
Assigns new content to the vector object, dropping all the elements contained in the vector before the call and replacing them by those specified by the parameters:

Donc la taille de ton vector vaut perpétuellement 1, et l'accès au vector "liste[j]" va taper dans une zone de mémoire non initialisée. (d'ou les valeurs aléatoires)

Il me semble que tu veux utiliser vector::push_back() ici, pas vector::assign().

Edit : Aussi, je ne pense pas que ce soit correct le coup du "fin.seekg (6,ios_base::cur);". Ca marche probablement pour la première ligne, mais ensuite il faut tenir compte du fait que chaque ligne se termine par un caractère eol ("\n")

[yo_haha]

Je vous remercie de vos réponses.

Même en déclarant x et y comme des float je n'arrive pas à lire correctement les nombres. Cette fois, il me donne des entiers en arrondissant les valeurs du fichier pour la première ligne et des valeurs erronées pour les autres.

Et j'ai ajouté

Code :

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

fin.seekg(1,ios_base::cur);

(après avoir changé assign() par push_back()) à la fin de la boucle for pour lire le caractère eol mais ça n'a rien changé

[Emmanuel Deloget]

Ca ne serait pas plus simple à parser si tu commeçais par extraire les lignes de texte (getline(fin, mystring), ou mystring est du type std::string). A partir de la, tu peux séparer le texte comme tu le souhaites, et utiliser std::stringstream pour faire la conversion des différents tokens dans leurs types respectifs).

AU moins, ça te permettrais de debugger plus aisément

[bertry]

Salut,

yo_haha : Cette fois, il me donne des entiers en arrondissant les valeurs du fichier pour la première ligne et des valeurs erronées pour les autres.

Attention! D'après microsoft :

Point, structure : Représente une paire ordonnée de coordonnées x et y entières qui définit un point dans un plan à deux dimensions.

Si tu veux utiliser des valeurs de type float ou double, il ne faut pas utiliser la structure 'System:: Drawing:: Point'
Tu peut créer tes propres points, par exemple :

Code :

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

struct MesPoints { double x, y; }

Bye

[bertry]

Re Salut,

J'ai remplacé

Code :

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

fin.seekg(6,ios_base::cur);

par

Code :

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

fin >> c >> c >> c >> c >> c >> c;

c'est pas élégant du tout mais ca permet à la fonction de fonctionner parfaitement sur mon PC!

J'ai aussi rajouté la structure 'MonPoint' pour pouvoir créer des points à coordonnées 'double'.

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
30
31
32
33
34
35
36
37
38
39
40
41
42
struct MonPoint
{
	double x, y;
};
 
bool charger(char* nom, vector<MonPoint> &liste, int taille)	 //taille = 1000 : nombre de lignes du fichier
{
    ifstream fin;
    fin.open(nom);
 
    //Vérifier que le fichier est ouvert sans problèmes
    if(!fin){
            cout<<"Probleme a l'ouverture du fichier"<<endl;
            return false;
 
    }// Fin if
 
    // Saisie des coords dans les objets Point
    double x, y;
    char c;
    for(int j=0; j<taille; j++){
		MonPoint pt;
		fin >> c >> c >> c >> c >> c >> c;	// C'est pas élégant mais du coup : bool charger( ... ) fonctionne!!
        // On prend le X, le Y et le caractère ')' à la fin de chaque ligne j
        fin >> x >> y >> c;
        cout<<x<<endl<<y<<endl<<c<<endl; //les valeurs de x et y ne sont pas les bonnes. Par contre dans c il y a ')' comme prévu
        // On met le X et le Y dans le point j de la liste
 
		pt.x = x;
		pt.y = y;
 
        liste.push_back(pt);
		cout<<liste[j].x<<endl<<liste[j].y<<endl;//ça me donne des valeurs encore différentes
 
    }// Fin for
 
    // Fermeture du stream
    fin.close();
 
    // Valeur retournée par la fonction
    return true;
}// Fin charger(...)*

Pourquoi ça ne marchait pas avant : je ne sais pas!
Pourquoi ça marche maintenant : je ne sais pas non plus!

Mais ca marche : manifestement le fautif est

Code :

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

fin.seekg ( ... );

Bon code.

[yo_haha]

Merci pour le conseil Emmanuel.

A bertry, je te remercie. Effectivement le seekg() posait des problèmes. Je l'ai retiré et les choses marchent beaucoup mieux. Par contre, le problème des valeurs qui sont prises du fichier en arrondi persiste !
Sinon, pour la structure Point, je n'utilise pas le System:: Drawing:: Point. J'ai créé une classe Point avec des attributs et méthodes que j'ai déclaré moi-même (j'ai déclaré X et Y comme des doubles).

[bertry]

Oui yo_haha,
Pour l'arrondi, Les valeurs qui s'affichent à l'écran sont arrondies, mais les valeurs qui sont dans le 'vector' sont celles qui sont dans le fichier ( le system les fait varier de + ou - 1 LSB, je ne sais pas pourquoi ).
Si toutes tes valeurs n'ont que trois décimales après la virgule, comme dans l'exemple, trouve une fonction qui les arrondissent à 10-4 ou 10-5 et ce sera bon...