Discussion :

[sub-0]

Bonjour,
Voila mon problème: je viens de créer une structure S dans une classe C, voila la forme de ma classe:

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
class C
{
	struct S{
		Type var;
                 Type *p;
	};
private:
	S*_elmt;
	int _nbrElmt;
	int _nbrArret;
public:
	C(int,int);
	~C();
	C* inserElmt(C*,Type*);
	C* creerInstance(int,int);
	void afficher() const;
};

Quand je fait appel a la fonction creerInstance, ça me renvoit un message d'erreur au niveau de la ligne d'allocation memoire du tableau dynamique:

Code :

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

1
2
C* i=NULL;
i->_elmt= new S[n];

Merci d'avance

[Goten]

C::S pour accéder à ta struct.

[sub-0]

C::S pour accéder à ta struct.

ça fonctionne pas, voila la fonction:

Code :

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

1
2
3
4
5
6
7
8
C* C::creerInstance(int n,int m){
	C *i=NULL;
	i->_elmt= new C::S[n];
	i->_nbrElmt=n;
	i->_nbrPos=m;
 
return i;
	}

J'obtient ces erreurs:

Code :

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

1
2
3
_elmt	CXX0017: Erreur*: symbole "" introuvable	
_nbrElmt	CXX0030: Erreur*: impossible d'évaluer l'expression	
_nbrPos	CXX0030: Erreur*: impossible d'évaluer l'expression

[SofEvans]

Euh, c'est moi ou i n'est pas instancié ????

Code :

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

1
2
3
4
5
 
	C *i=NULL;
	i->_elmt= new C::S[n];
	i->_nbrElmt=n;
	i->_nbrPos=m;

Comment veut tu acceder a un champ si i pointe vers NULL ?

Code :

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

1
2
3
4
5
6
 
	C *i=NULL;
        (*i) = new C();
	i->_elmt= new C::S[n];
	i->_nbrElmt=n;
	i->_nbrPos=m;

Cela devrait etre plus dans la logique mais je ne sais pas si c'est la bonne syntaxte. De plus, en C++, les pointeurs sont de plus en plus malvenue. Voir passage par reference.

[sub-0]

Merci pour le coup de pousse, j'ai opter pour les references, ça compile mais ça m'envoie ce message d'erreur sachant que j'utilise visual studio 2005:

Code :

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

Violation d'accès lors de la lecture de l'emplacement 0x00359df0

[koala01]

Salut,

Comment veut tu acceder a un champ si i pointe vers NULL ?

Code :

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

1
2
3
4
5
6
 
	C *i=NULL;
        (*i) = new C();
	i->_elmt= new C::S[n];
	i->_nbrElmt=n;
	i->_nbrPos=m;

Cela devrait etre plus dans la logique mais je ne sais pas si c'est la bonne syntaxte. De plus, en C++, les pointeurs sont de plus en plus malvenue. Voir passage par reference.

heuu... Tant qu'à corriger le code, autant le faire correctement

Ce serait plutot

Code :

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

1
2
C *i=new C;
/* la suite */

voir

Code :

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

1
2
3
4
5
6
/* il ne sert pas à grand chose de l'initialiser à NULL ici, mais on 
 *peut le faire
 */
C *i;
i=new C;
/* la suite */

Au passage, étant donné que l'on est dans une fonction membre de C, la portée par défaut est... celle de C.

Il n'est donc pas nécessaire de rajouter C:: lorsque l'on veut utiliser le type S, à moins qu'il n'y ait conflit entre la structure imbriquée et une autre structure de même nom mais se trouvant dans l'espace de noms courent (ou dans un des espaces de noms "parent", s'il échoit)

Ceci dit, je rajouterais volontiers un constructeur pour C qui prendrait les arguments qui vont bien, à savoir: le nombre d'élément que le tableau doit contenir et la position (qui peut valoir 0 par défaut).

Ta classe ressemblerait donc à:

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
class C
{
    struct S
    {
        /*...*/
    };
    public: 
        C(unsigned int nombre, unsigned int pos = 0):
          _elem(new S[nombre]), 
          _nbrElmt(nombreà),nbrArret(pos)
        {
        }
        ~C(){delete[] _elem;}
        C* instance(unsigned int nombre, unsigned int pos)
        {
            C* ret=new C(nombre, pos);
            return ret;
        }
       /*...*/
    pritave:
	S*_elmt;
	unsigned int _nbrElmt;
	unsigned int _nbrArret;
 
};

(j'ai utilisé des entiers non signés parce qu'il n'y a aucun sens à ce qu'ils puissent être négatif)

Et mieux encore, plutôt que de me casser la tête avec l'allocation dynamique qui n'a aucun intérêt et qui n'a même que des inconvénients dans ce contexte, j'utiliserais la classe vector, qui est justement "taillée" pour éviter les problèmes dus à l'allocation dynamique de la mémoire lorsqu'il s'agit de gérer des tableaux d'éléments.

Au final, le code pourrait prendre une forme proche de

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
class C
{
    struct S
    {
        /*...*/
    };
    public:
        C(unsigned int nombre, unsigned int pos=0):
          _elem(nombre),_nbrArret(pos)
        {
        }
    private:
        std::vector<S> _elem;
        unsigned int _nbrArret;
};

Mais, ceci dit, je me demande à quoi pourra bien servir la fonction instance