Discussion :

[laulaurent]

Salut,

J'ai un problème en utilisant un tableau d'objets.
Je dois lire un fichier texte contenant des données et créer un certain nombres d'objets en fonction de ce que je trouve dans ce fichier. Je ne connais donc pas la taille de mon tableau à l'avance. Je crée donc une référence vers un objet dans le main, je la transmet à ma fonction qui analyse le fichier et crée le tableau de la taille souhaitée. Cette dernière fonction alloue la place nécessaire et crée le tableau d'objets. Mais je n'arrive pas à comprendre pourquoi je n'arrive pas à accéder à mes objets après cette fonction dans le main(). J'utilise pourtant bien l'opérateur new.
Pour plus d'information, merci de jeter un oeil au sources ci-jointes (volontairement raccourcies).

Merci par avance de votre aide,
Lau

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
 
using namespace std;
 
class FrameCameraParams {
private:
        double focalLength; 
public:
        FrameCameraParams();
        void setFocalLength(double f);
        double getFocalLength();
};
 
FrameCameraParams::FrameCameraParams()
{
        focalLength=0;
}
 
void FrameCameraParams::setFocalLength(double f)
{
        focalLength=f;
}
 
double FrameCameraParams::getFocalLength()
{
        return focalLength;
}
 
 
void parseCameraFile( FrameCameraParams* framesCameraParams)
{
        framesCameraParams=new FrameCameraParams[5];
	framesCameraParams[0].setFocalLength(37.);
	cout << framesCameraParams[0].getFocalLength() << endl;
	cout << framesCameraParams[1].getFocalLength() << endl;
}
 
 
int _tmain(int argc, _TCHAR* argv[]) {
	FrameCameraParams* frames=NULL;
        parseCameraFile( frames); // ici tout fonctionne bien
        cout << (frames[0]).getFocalLength() << endl; // Cet appel plante...
        return 0;
}

[Kujara]

Code :

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

FrameCameraParams & * framesCameraParams

Tu lui passe un pointeur mais tu a oublié de le renvoyer ...

[laulaurent]

Merci de ta réponse, mais je n'ai pas tout compris. En quoi ai-je besoin de renvoyer le pointeur. Je modifie son adresse. N'est ce pas suffisant (EDIT:ah ok c'est une copie qui est transmise?). Et sinon que signifie le & dans une déclaration?
Merci par avance.

[darrylsite]

Code :

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

1
2
3
4
5
6
7
8
9
/* je ferais plutot*/
void parseCameraFile( FrameCameraParams* framesCameraParams)
{
        framesCameraParams=new FrameCameraParams[5];
	framesCameraParams[0]->setFocalLength(37.);
	cout << framesCameraParams[0]->getFocalLength() << endl;
	cout << framesCameraParams[1]->getFocalLength() << endl;
}
/* les "." deviennent "->" pour les pointeurs*/

[laulaurent]

Ok donc j'ai fais comme cela.

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
 
using namespace std;
 
class FrameCameraParams {
private:
        double focalLength; 
public:
        FrameCameraParams();
        void setFocalLength(double f);
        double getFocalLength();
};
 
FrameCameraParams::FrameCameraParams()
{
        focalLength=0;
}
 
void FrameCameraParams::setFocalLength(double f)
{
        focalLength=f;
}
 
double FrameCameraParams::getFocalLength()
{
        return focalLength;
}
 
 
FrameCameraParams* parseCameraFile( FrameCameraParams* framesCameraParams)
{
        framesCameraParams=new FrameCameraParams[5];
	framesCameraParams[0].setFocalLength(37.);
	cout << framesCameraParams[0].getFocalLength() << endl;
	cout << framesCameraParams[1].getFocalLength() << endl;
        return framesCameraParams;
}
 
 
int _tmain(int argc, _TCHAR* argv[]) {
	FrameCameraParams* frames=NULL;
        FrameCameraParams* frames2=NULL;
        frames2=parseCameraFile( frames);
        cout << (frames[0]).getFocalLength() << endl;
        return 0;
}

Mais ca ne m'arrange pas en fait car je remplis plusieurs (tableau d'objets) différents. Comment je peux contourner le pb dans le cas suivant où j'ai deux tableaux d'ojets différents (sans créer plusieurs fonctions pour lire le fichier...). Je souhaiterais que les arguments transmis contiennent vraiment les adresses au final:

EDIT: J'ai enfin compris. Il faut que je transmette le pointeur par adresse.
Ca donne donc cela:

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
 
 
parseCameraFile( FrameCameraParams* & framesCameraParams, FrameSuperCameraParams2* & frames2 )
{
  framesCameraParams=new FrameCameraParams[10];
  frames2 =new FrameSuperCameraParams2[50];
  //...
}
 
 
int _tmain(int argc, _TCHAR* argv[]) {
	FrameCameraParams1* frames1=NULL;
        FrameSuperCameraParams2* frames2=NULL;
        parseCameraFile( frames1,frames2);
// ...
        return 0;
}

Merci à tous les deux pour votre aide!!

[darrylsite]

En fait ton programme ne fait pas un peu ce que tu dis. A vrai dire je ne te comprend pas très bien. Mais commence par changer les "." en "->". pour ton programme s' execute sans erreur.
Je ne vois meme pas les fichier que tu veux traiter. Exprime toi plus clairement.

[laulaurent]

En fait ton programme ne fait pas un peu ce que tu dis. A vrai dire je ne te comprend pas très bien. Mais commence par changer les "." en "->". pour ton programme s' execute sans erreur.
Je ne vois meme pas les fichier que tu veux traiter. Exprime toi plus clairement.

Les . ne posent aucunement problème dans le code. Sous linux comme sous VC++, l'erreur ne venait pas de là. A l'avenir je les remplacerai par des ->. Sinon, je n'ai pas mis les fichiers que je veux traiter car sinon le code serait beaucoup trop gros. J'ai juste mis les informations nécessaires pour comprendre mon problème vis à vis d'un point précis sur les tableaux d'objets ainsi qu'un infime bout de code correspondant. C'est sans doute pour cela que ca rendait le problème un peu difficile à comprendre...

Merci pour ton aide en tout cas

[Biozic]

Il ne faut pas changer les . en -> dans ce code. Ces opérateurs ne sont pas interchangeables.
Ici, framesCameraParams* est un pointeur sur le début d'un tableau, mais framesCameraParams[0] n'est pas un pointeur. Ta première syntaxe était bonne.

Sinon pour les tableaux, il y a aussi std::vector, qui serait peut-être plus pratique ici puisque la longueur du tableau n'est pas connue à l'avance, et que cette classe permet justement de gérer cette situation.

Enfin, je suppose que le "delete[] frames;" a été omis dans un soucis de place ...

Et pendant que j'y suis, j'aurais mis:

Code :

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

1
2
3
FrameCameraParams::FrameCameraParams() : focalLength(0)
{
}