Discussion :

[vince3320]

Bonjour a tout le monde!!

J'ai un petit probleme....
Voila un bout de code:

mon_code.h

Code :

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

1
2
3
4
 
...
typedef struct toto *toto;
...

mon_code.c

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
 
struct toto
{
 int mon_champ;
};
...
void ma_fonction(void)
{
int i_alloc;
toto = malloc(255 * sizeof (toto));
for (i_alloc = 0; i_alloc < 255; i_alloc++)
   toto[i_alloc] = malloc(sizeof (toto));
....

Et quand je veux acceder a un des champs de ma structure, je fais par exemple:

Code :

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

1
2
 
 fscanf(mon_fichier,"%d\n", &toto[i]->mon_champ);

A la compilation, il me dit " incompatible types in assignment" a la ligne a l'interieur du for...

Merci de votre aide...

[MarcG]

je supose que ton .h est inclu avant la definition de ta structure, comment typer une variable avec un type non encore definie .
ensuite tu veut allouer "mon_champ" alors qu'il est deja alloué (int mon_champ).
si c'est un tableau de 255 entiers que tu a besoin un int mon_champ[255] suffit .....

[Emmanuel Delahaye]

mon_code.h

Code :

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

1
2
3
4
 
...
typedef struct toto *toto;
...

Si tu peux éviter les 'toto', ça a le don de m'énerver...
D'autre part, c'est pas une bonne idée de mettre une '*' dans un typedef. On ne connait plus la nature du type des objets de ce type. Il n'y a aucune honte à manipuler des pointeurs. Il ne faut pas les cacher. Je recommande que les types structures soient suffixés par 's', et que les objets pointeurs soient préfixés par 'p_'.

mon_code.c

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
 
struct toto
{
 int mon_champ;
};
...
void ma_fonction(void)
{
int i_alloc;
toto = malloc(255 * sizeof (toto));
for (i_alloc = 0; i_alloc < 255; i_alloc++)
   toto[i_alloc] = malloc(sizeof (toto));
....

'toto' n'est pas un objet, c'est un type.

Et quand je veux acceder a un des champs de ma structure, je fais par exemple:

Code :

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

1
2
 
 fscanf(mon_fichier,"%d\n", &toto[i]->mon_champ);

A la compilation, il me dit " incompatible types in assignment" a la ligne a l'interieur du for...

Normal, vu de l'extérieur, la structure est opaque. On ne peut pas accéder à ses champs. Il faut une fonction 'accesseur'.

[Gandalf]

mon_code.h

Code :

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

1
2
3
4
 
...
typedef struct toto *toto;
...

Si tu peux éviter les 'toto', ça a le don de m'énerver...
D'autre part, c'est pas une bonne idée de mettre une '*' dans un typedef. On ne connait plus la nature du type des objets de ce type. Il n'y a aucune honte à manipuler des pointeurs. Il ne faut pas les cacher. Je recommande que les types structures soient suffixés par 's', et que les objets pointeurs soient préfixés par 'p_'.

plus que le nom toto, ou le fait qu'il y ait une étoile dans un typedef, cette ligne presente une ambiguité terrible :
toto designe un pointeur sur struct toto

du coup la ligne

Code :

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

1
2
 
toto = malloc(255 * sizeof (toto));

une erreur

Code :

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

1
2
 
toto* titi = malloc(255 * sizeof (toto)

puis ensuite

Code :

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

1
2
 
 toto[i_alloc] = malloc(sizeof (toto));

deviens

Code :

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

1
2
 
titi[i_alloc] = malloc(sizeof (struct toto));

en enfin

Code :

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

1
2
 
&toto[i]->mon_champ

Code :

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

1
2
 
&(titi[i]->mon_champ)

mais evidement le mieux serait de donner des noms differents à la structure et au pointeur sur la structure...

[vince3320]

merci
effectivement j'avais ecrit

Code :

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

1
2
 
toto titi

au lieu de

Code :

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

1
2
 
toto *titi

Les autres erreur etaient des oublies pour cause de simplification
Merci pour votre aide
PS j'arrete les toto sur le forum!

[Gandalf]

....
PS j'arrete les toto sur le forum!

arrete plutot les :

Code :

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

1
2
 
typedef struct toto * toto;

Personnelement, je ne serais pas contre le retablissement du bucher pour les heretiques osant ecrire ce genre de choses

[vince3320]

ben en fait c'est ce que l'on nous apprends a l'ecole!!
lol
cela permet de masquer l'implementation et de separer l'implementation de l'interface...
Avec mes maigres connaissances, je trouve ca pas trop mal...

Pourquoi tu trouve cela horrible?

[Gandalf]

ben en fait c'est ce que l'on nous apprends a l'ecole!!
lol
cela permet de masquer l'implementation et de separer l'implementation de l'interface...
Avec mes maigres connaissances, je trouve ca pas trop mal...

Pourquoi tu trouve cela horrible?

suit la convention proposée plus haut eventuellement :

Code :

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

1
2
 
typedef struct toto *p_toto;

ou

Code :

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

1
2
 
typedef struct s_toto *toto;

ou encore

Code :

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

1
2
 
typedef struct s_toto *p_toto;

mais que l'identifiant "toto" (ou un autre) ne designe pas deux choses differentes !

[Emmanuel Delahaye]

ben en fait c'est ce que l'on nous apprends a l'ecole!!
lol
cela permet de masquer l'implementation et de separer l'implementation de l'interface...
Avec mes maigres connaissances, je trouve ca pas trop mal...

Pourquoi tu trouve cela horrible?

Je suis d'accord pour l'opacité (séparation interface/implémentation), mais pas pour cacher l'*'. Ca provoque des erreurs de codage terribles.

[vince3320]

daccord, tu as raison
c'est vrai que c'est maladroit et desagreable pour celui qui lit le code!!
Je le ferai a l'avenir
Merci pour vos conseils a tous!!

[Gandalf]

Je suis d'accord pour l'opacité (séparation interface/implémentation), mais pas pour cacher l'*'. Ca provoque des erreurs de codage terribles.

l'opacité viens du fait que tu ne publies pas la definition de la structure, mais juste son nom dans le header. Elle ne viens pas du typedef.

L'opacité est à la mode, car c'est l'encapsulation que permet de C, et cela permet d'obtenir en C une programmation objet. C'est une bonne chose.

Le typedef en est une autre. Et personnellement, qu'il y ait une étoile dans le type def ou pas cela ne me gene pas plus que cela, en particulier pour un type opaque, savoir si on a un pointeur, ou un indice dans un tabeau ou quoi que ce soit d'autre ne me fait ni chaud ni froid.

Ce qui par contre me choque c'est la double utilisation du nom de la structure. Bien sur s'il sagissait de :

Code :

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

1
2
 
typedef struct toto toto;

je ne dirait rien.

De plus, je pense que cela empeche ton code de compiler en C++, ce qui toujours dommage pour un header...