Discussion :

[Heero_2040]

salut à tous...


bon voila mon problème

j'essaie de créer un tableau d'objet... donc tout d'abord voici mon objet...

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
class tower{
 
private :
 
char*name;
float freq;
int tx;
int ty;
int longueur;
 
public :
tower();
tower(char*,float, int, int , int );
void settx(int);
void setty(int);
void setname(char*);
void setfreq(float);
void setlong(int);
int gettx();
int getty();
float getfreq();
int getlongueur();
 
};

ensuite voici les constructeurs

Code :

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

1
2
3
4
5
6
7
8
9
tower ::tower(){
 
 
freq=0;
tx=0;
ty=0;
longueur=0;
 
}

Code :

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

1
2
3
4
5
6
7
8
9
tower :: tower(char*nom, float fr, int x, int y, int longu){
 
 
name=nom;
freq=fr;
tx=x;
ty=y;
longueur=longu;
}

et voici mon main :

Code :

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

1
2
3
4
5
6
7
8
tower tabtower[50];
 
 
tabtower[0].settx(400);
tabtower[0].setty(400);
tabtower[0].setname("istres");
tabtower[0].setfreq(150.00);
tabtower[0].setlong(1500)

donc en fait je veux créer un tableau de 50 objets "towers" et ensuite initialiser le premier avec les fonctions set..


malheureusement le compilateur (dev c++) me dit

48 expected constructor, destructor, or type conversion before '.' token
48 `,' or `;' before '.' token


et meme chose pour les lignes suivantes...

le problème est que j'arrive pas à voir mon erreur, car j'ai bien un constructeur par défaut... mais je sais pas pourquoi je sens que j'ai oublier un détail très bete... mais bon il est tard... donc soyez indulgents svp :p


voila merci beaucoup...

[countag]

bon je ne suis pas un pro, mais j'ai était confronté à ton problème et moi j'ai choisi l'allocation dynamique et ca marche nikel :

tabtower = new tower[50];


Par contre pourquoi tu définis un Setfx et Getfx ...etc

Ne serait il pas mieu de mettre la variable public, ca reviendrait au même non et ca serait plus facile non ?

[Matthieu Brucher]

La mettre public ? non...
Sinon, pourquoi des char* et pas des string ?

[Laurent Gomila]

La mettre public ? non...

C'est discutable. Si la structure n'est là que pour aggréger plusieurs membres sans fournir de traitement particulier, alors passer par des accesseurs c'est un peu se faire ch*** pour rien.

A part ça, le code est correct et ne devrait pas générer ce genre d'erreur. A moins que je n'ai loupé quelque chose moi aussi.

Tu devrais également utiliser std::string pour tes chaînes, vu comment tu les gères (pas).

[Matthieu Brucher]

C'est discutable. Si la structure n'est là que pour aggréger plusieurs membres sans fournir de traitement particulier, alors passer par des accesseurs c'est un peu se faire ch*** pour rien.

C'est vrai

[Heero_2040]

ok donc pour résumer :

_ alllocation dynamique....
_ est mise en publique ?




cependant j'aimerais comprendre mon erreur...


en tout cas merci beaucoup pour vos précieuses réponses

[countag]

Je disais de les mettre public car dans son cas, il n'y a aucun intêret de les mettre private, car tu peux y accéder et modifier les valeurs en dehors des fonctions membre, ou est l'intêret du private ?

Par contre il faut faire gaffe au tableau static, c'est pourquoi je voulais qu'il test en allocation dynamique, exemple concret :

Code :

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

1
2
3
int tab[1000000];
 
for(i=0, i<=1000000;i++) tab[i] = i;

Normalement ce code lors du lancement, devrais faire une belle erreur

par contre :

Code :

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

1
2
3
tab = new int[1000000];
 
for(i=0, i<=1000000;i++) tab[i] = i;

Lors du lancement tous devrait bien ce passé.

[harsh]

... Dit nous pour quelle ligne l'erreur est elle donnée (et de quel fichier: .h ou .cpp)

Par contre, ta gestion des char* est peu orthodoxe... tu as un pointeur sur un char et tu comptes simplement lui assigné l'adresse d'une chaine de caractere passée en parametre , je ne sais meme pas qu'elle résultat cela peut avoir

Code :

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

1
2
 
tabtower[0].setname( "istres" );  //je suppose que tu fais juste un = comme pour ton constructeur...

Comment est géré "istres" au niveau memoire dans l'histoire???

[arcan2z]

essaye ca et ca marchera surement

Code :

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

1
2
3
tower *tabletower[50];
//tableau de pointeur c la forme la plus utilisé pr les tableux d'objets
tabletower[i]=new tower;

puis tu peux faire tous ce que tu veux

Code :

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

tabletower[0].settx(400);

n'oublie pas de le mettre resolu by

[arcan2z]

a propo t'as une autre erreur
lors de l'utilisation de char *
n'oubli pas de faire l'allocation
name =new char;
dans le constructeur sinon ca va pas marcher
et tu n'aura pas d'espace pour stcker le mot

[Laurent Gomila]

essaye ca et ca marchera surement
tower *tabletower[50];
//tableau de pointeur c la forme la plus utilisé pr les tableux d'objets
tabletower[i]=new tower;
puis tu peux faire tous ce que tu veux

tabletower[0].settx(400);
n'oublie pas de le mettre resolu by

Aucun rapport. S'il veut faire un tableau d'objets c'est très bien, pas la peine d'introduire des erreurs / fuites potentielles. Et s'il a un tableau de pointeurs il faudra utiliser tabletower[0]->xxx.

a propo t'as une autre erreur
lors de l'utilisation de char *
n'oubli pas de faire l'allocation
name =new char;
dans le constructeur sinon ca va pas marcher
et tu n'aura pas d'espace pour stcker le mot

C'est pour ça qu'on s'evertue à parler de std::string

[harsh]

Par contre, ta gestion des char* est peu orthodoxe... tu as un pointeur sur un char et tu comptes simplement lui assigné l'adresse d'une chaine de caractere passée en parametre , je ne sais meme pas qu'elle résultat cela peut avoir

Code :

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

1
2
 
tabtower[0].setname( "istres" );  //je suppose que tu fais juste un = comme pour ton constructeur...

Comment est géré "istres" au niveau memoire dans l'histoire???

Bah oui, j'insiste, je me demande ou est placé en memoire la chaine "istres" et qu'elle est sa portée... bref, quelle grosse gourde il y a a faire avec

[rigobert]

S'il veut faire un tableau d'objets c'est très bien, pas la peine d'introduire des erreurs / fuites potentielles.

Entièrement d'accord.
Et je dirais même plus, d'une façon plus générale, que l'allocation automatique d'objets est le choix qui s'impose (vs l'allocation dynamique) à chaque fois que c'est possible.