un problème avec un tableau d'objet, erreur bete ?

Version imprimable

salut à tous...

bon voila mon problème

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

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:

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

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:

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...

12/07/2006, 22h47
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 ?
13/07/2006, 08h19
Matthieu Brucher

La mettre public ? non...
Sinon, pourquoi des char* et pas des string ?
13/07/2006, 08h24
Laurent Gomila

Citation:

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).
13/07/2006, 08h31
Matthieu Brucher

Citation:

Envoyé par Laurent Gomila

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 :D
13/07/2006, 10h01
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
13/07/2006, 10h03
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:

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:

1 2 3 tab = new int[1000000]; for(i=0, i<=1000000;i++) tab[i] = i;
Lors du lancement tous devrait bien ce passé.
13/07/2006, 10h07
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 8O , je ne sais meme pas qu'elle résultat cela peut avoir
Code:

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???
13/07/2006, 11h13
arcan2z
essaye ca et ca marchera surement
Code:

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:

tabletower[0].settx(400);

n'oublie pas de le mettre resolu by
13/07/2006, 11h19
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
13/07/2006, 11h28
Laurent Gomila

Citation:

Envoyé par arcan2z

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.

Citation:

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 ;)
13/07/2006, 12h46
harsh
Citation:
Envoyé par 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 8O , je ne sais meme pas qu'elle résultat cela peut avoir

Code:

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 :aie:
13/07/2006, 13h20
rigobert

Citation:

Envoyé par Laurent Gomila

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.