Discussion :

[Fafa_Bouillé]

Bonjour

Je Suis nouvelle sur le forum. et j'aurais aimé solliciter votre aide car je suis au bord du précipice. Je te trouve pas de solution.
Suis une débutante du C++ et je rame un peu.

VOilà dans le cadre de mon stage et de mon programme je dois récupérer des coordonnées de vertex et de faces dans un fichier obj.
Dans mon inexpérience, le code que j'ai écris récupére les données du fichier et stocke chaque point comme un objet d'une classe POINT3D avec comme attributs : identifiant, x,y et Z;

le même principe vaut pour les faces.

après je procède à des calculs.

Le Hic c'est que jusqu'ici je testais avec un misérable et minime fichier obj et ca tournait . mais dès que je teste avec des fichiers lourds (des millions de points et des millions de faces) ca ne va plus du tout.

Mon erreur c'est d'avoir stocké tout cela dans des classes mais je ne sais pas comment faire autrement

Quelqu'un peut m'aider.?

parce que suis censée finir mon stage bientôt, trop bientôt

Mais quand je tes

[Luc Hermitte]

Ce n'est pas une erreur. Un objet de trois entiers ne va pas couter plus cher qu'une structure de trois entiers ou même trois entiers. C'est exactement le même poids.
Ce qui faut voir ce sont tes algos et les structures utilisées pour organiser tes ensembles de données.

Et là difficile de parler dans le vide. De plus, de ce que j'en sais, ce n'est pas un domaine connu pour être simple à optimiser quand le volume de données augmente. Ils n'ont pas d'expérience sur le sujet dans ta boite ?

[Fafa_Bouillé]

Leproblème c'est que dans ma boîte il n'y pas d'informaticien. Ce qui finalement est une très mauvaise chose.

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
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
char lettre;
float x, y, z;
FILE *pfichier_obj;
 
int R,V,B;
int f1= 0,f2=0,f3=0, i =0, j=0, f6= 0,f4=0,f5=0;
// int f1= 0,f3=0, i =0, j=0, f5=0;
 
pfichier_obj = fopen("fichier_mesh.obj","r");
 
if(!pfichier_obj)
{
    // MessageBox(NULL,"Erreur Fichier OBJ","Gestion Erreur",MB_OK);
    cout << "Chargement du Fichier échoué"<< endl;
    return 1;
}
 
while(!feof(pfichier_obj))
{
    // cout << "i = " i<< endl;
 
    lettre = fgetc(pfichier_obj);
 
    switch(lettre)
    {
        default:
            break;
 
        case 'v':
            lettre = fgetc(pfichier_obj);
 
            switch(lettre)
            {
                default:
                    break;
 
                case ' ':
                    fscanf(pfichier_obj," %f %f %f %i %i %i   \0",&x,&y,&z,&R,&V,&B);
                    // fscanf(pfichier_obj," %f %f %f    \0",&x,&y,&z);
 
                    cPoint3D sommet ; // création de l'objet temporaire//
                    i = i+1;
                    std::cout<<"i="<<i<<std::endl ;
                    sommet.IDs = i;
                    sommet.X = x;
                    sommet.Y = y;
                    sommet.Z = z;
                    VarGlob::POINT3D.pAJOUT_ELEMENT(sommet); 
                    break;
            }
            break;
 
        case 'f':
            fscanf(pfichier_obj," %i // %i %i // %i  %i // %i   \0",&f1,&f2,&f3,&f4,&f5,&f6);
            // fscanf(pfichier_obj," %i %i %i   \0",&f1,&f3, &f5);
            cTriangle3D triangle;
            j = j+1;
            std::cout<<"j="<<j<<std::endl ;
            triangle.IDt = j;
            triangle.S1 = f1;
            triangle.S2 = f3;
            triangle.S3 = f5;
            VarGlob::TRIANGLE3D.pAJOUT_ELEMENT(triangle);
            break;
    }
}
 
fclose(pfichier_obj);
VarGlob::POINT3D.pAFFICHE_TABLE();
VarGlob::TRIANGLE3D.pAFFICHE_TABLE();
 
return 0;

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
cTriangle3D::cTriangle3D(unsigned int IDt, unsigned int S1, unsigned int S2, unsigned int S3) :
   Taille ( 65536*sizeof(char) + 2*sizeof(unsigned int) ),
   NomObj("cTriangle3D")
{
   this->IDt = IDt;
   this->S1 = S1;
   this->S2 = S2;
   this->S3 = S3;
}
 
cTriangle3D::~cTriangle3D()
{
 
}

[Aurelien.Regat-Barrel]

Commence par lire les règles:
http://club.developpez.com/regles/
et mettre plus de soin à la rédaction de tes messages (balises codes, moins de sauts de lignes...).
Parcours la FAQ, les cours...

[Fafa_Bouillé]

Désolée pour la formulation et la forme de mes posts!

D'autre part, si j'ai fait appel à ce forum, c'est que justement j'ai passé plusieurs heures à tenter d'éplucher tutoriaux, cours, etc et que finalement, du fait de mon inexpérience ou autres (...) je n'ai pooint trouvé la solution!!
D'où l'idée de poster sur ce forum....
Merci quand même

[Laurent Gomila]

J'ai donné à ton code une mise en page plus lisible.

A l'avenir, pense :

- Que tu peux éditer tes posts si tu as un ajout ou une correction à faire
- A le faire toi-même


Ensuite pour ton problème, quelques remarques :

- Ca ne changera pas grand chose, mais puisque ta classe triangle possède un constructeur pourquoi ne pas l'utiliser ?
- Puisque tu codes en C++, utilise les flux (fstream) ainsi que la surcharge d'opérateur >>, le code n'en sera que plus clair.
- Comment sont stockés tes triangles ?
- Que signifie leur donnée membre Taille ?
- C'est quoi toutes ces variables globales ?
- Il faudrait aussi voir le format de tes fichiers (un petit exemple).

[Fafa_Bouillé]

Merci pour tes conseils, que j'ai suivis.

J'ai épluché de nouveau la doc que j'avais pu accumuler et décidé de ne pas céder à l'abattement; résultat, c'est bon je l'ai : merci mes amis les vector.

[Fafa_Bouillé]

Voilà ce que ca donne. Merci encore

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
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
 
 
class cPoint3D 
 
{
public : 
	int IDs;
	float  X;
	float  Y; 
	float  Z;
};
 
class cTriangle3D 
 
{
public : 
	int IDt;
	int  S1;
	int S2; 
	int  S3;
};
 
 
vector <cPoint3D> Lptot3D;
vector <cTriangle3D> LFace3D;
 
 
pfichier_obj = fopen("fichier_mesh.obj","r");
 
if(!pfichier_obj)
{
 
    cout << "Chargement du Fichier échoué"<< endl;
   return 1;
 
}
 
while(!feof(pfichier_obj))
{
 
     lettre = fgetc(pfichier_obj);			
    switch(lettre)
   {
       default:					
       break;
 
      case 'v':
      lettre = fgetc(pfichier_obj);
 
      switch(lettre)
     {
	default:
                break;
                case ' ':
       fscanf(pfichier_obj," %f %f %f %i %i %i   \0",&xt,&yt,&zt,&R,&V,&B);
       i2 = i2+1;
 
      sommet.IDs = i2;				
      sommet.X = x;
      sommet.Y = y;
      sommet.Z = z;
      Lptot3D.push_back(sommet);
      break;
 
     }
 
    break;
 
     case 'f':
   fscanf(pfichier_obj," %i // %i %i // %i  %i // %i    \0",&f1,&f2,&f3,&f4,&f5,&f6);
 
     j2 = j2+1;
     triangle.IDt = j2;
     triangle.S1 = f1;
     triangle.S2 = f3;
     triangle.S3 = f5;
     LFace3D.push_back(triangle);
 
     break;
 
 
     }
}
 
     fclose(pfichier_obj);

[bountykiller]

Tu es obligé de tous stocker en même temps? Parce que sinon tu peux choisir de ne traiter qu'un partie de ton fichier à la fois (quite à garder en mémoire les résultats temporaires par ex)
Sinon, comme l'a dit Luc Hermitte, c'est plus un problème de structure de données que de programmation pure.

[Luc Hermitte]

En passant aux flux C++, et en écrivant les opérateurs d'extraction depuis les flux, ton code gagnera en simplicité et facilité de maintenance. Ne jamais hésiter à faire des fonctions pour réduire une fonction à moins de 15 lignes.

C'est faux en C++ (avec le flux), et je crois bien que cela l'est également en C : il ne faut jamais faire une lecture tant que non eof. (FAQ!)