Discussion :

[iPote]

Bonjour, je cherche à allouer dynamiquement un tableau de pointeurs vers une structure. J'ai essayé ceci :

Code :

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

1
2
3
4
5
6
7
8
typedef struct test{
 int x;
 int y;
} test;
…
test *tab[taille];
tab = malloc(taille * sizeof(*test));
…

mais à la compilation, j'obtiens les erreurs :

Code :

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

1
2
parse error before 'test'
incompatible types in assignment

Merci de votre aide.

[souviron34]

normal

quand tu fais un "typedef struct" le premier nom à apparaître après est le pointeur sur la structure, celui après est le nom de la structure...

Donc là tu as le même nom pour désigner le pointeur ET la structure.

En général, tu peux faire :

Code :

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

1
2
3
4
5
6
 
typedef struct Ptrtest{
 int x;
 int y;
} test;
…

Ensuite moi ce que je ferais est :

Code :

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

1
2
3
4
 
test *tab=NULL;
 
tab = malloc(taille * sizeof(test));

Si c'est bien ça que tu veux faire (un tableau de structures)

Si c'est un tableau de pointeurs vers des structures, alors c'est différent...

[Franck.H]

Salut et bienvenue sur les forums de developpez.com


Ce que tu as fait n'as aucun sens:

Code :

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

1
2
test *tab[taille];
tab = malloc(taille * sizeof(*test));

car

Code :

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

test *tab[taille];

est déjà un tableau de pointeur !

Si tu veux juste faire ceci en allocation dynamique:

Code :

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

1
2
3
4
 
test ** tab = NULL;
...
tab = malloc (taille * sizeof (* tab));

Mais il faut choisir, c'est soit l'allocation dynamique ou un tableau statique mais en aucun cas les deux !

[souviron34]

Lorsque tu définis ta structure, tu écris test avant OU après selon le cas, mais pas les deux en même temps

NON tu peux...

voir au dessus ;-)

[Ceylo]

NON tu peux...

voir au dessus ;-)

oui j'ai vu après coup, et puis 3 fois des réponses à peu près équivalentes ne servent pas à grand chose, d'où la supression de mon post.

[iPote]

Merci beaucoup pour vos réponses. Vous êtes sympa. Tout marche.

[gl]

quand tu fais un "typedef struct" le premier nom à apparaître après est le pointeur sur la structure

Ce n'est pas un pointeur au sens C du terme.

Code :

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

1
2
3
4
5
6
 
typedef struct Ptrtest{
 int x;
 int y;
} test;
…

Tout a fait, meme si dans ce cas:

Code :

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

1
2
3
4
5
 
typedef struct{
 int x;
 int y;
} test;

suffit.

Ensuite moi ce que je ferais est :

Code :

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

1
2
3
4
 
test *tab=NULL;
 
tab = malloc(taille * sizeof(test));

Ca fonctionne parfaitment comme ca, mais personnellement j'ai plutot tendance a ecrire :

Code :

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

1
2
3
4
 
test *tab=NULL;
 
tab = malloc(taille * sizeof *tab);

Ainsi, si un jour le type de tab change, il n'y a pas de modification a faire dans le malloc, d'ou une maintenance du code facilite.

Dans ce cas de figure ou l'allocation suit immediatement la definition, j'aurais meme plutot tendance a ecrire :

Code :

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

1
2
 
test *tab = malloc(taille * sizeof(test));

[crocodilex]

normal

quand tu fais un "typedef struct" le premier nom à apparaître après est le pointeur sur la structure, celui après est le nom de la structure...

L'erreur de compilation ne vient pas de là. La définition de sa structure est correct.
Le problème vient de la syntaxe :

Code :

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

tab = malloc(taille * sizeof(*test));

qui n'a aucun sens.

[souviron34]

Ce n'est pas un pointeur au sens C du terme.

si si c'est un pointeur au sens C du terme...

Exemple :

Code :

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

1
2
3
4
5
6
7
8
9
 
typedef struct  Ptr_MaStruct {
 
    int toto ;
 
    Ptr_MaStruct previous ;
    Ptr_MaStruct next ;
 
} MaStruct ;

crocodilex t'étais pas réveillé c'est bien ce qu'il avait marqué

[Médinoc]

Si si, gl a raison, et ce que tu viens de poster ne compilera pas.

[Médinoc]

Code C :

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

1
2
3
4
5
6
7
8
9
typedef struct st_maStruct MASTRUCT, *PMASTRUCT;
typedef const struct st_maStruct *PCMASTRUCT;
 
struct st_maStruct
{
	int toto;
	PMASTRUCT pPrev;
	PMASTRUCT pNext;
};

Ici, PMASTRUCT est un type pointeur (et non un pointeur lui-même). Par contre, pPrev et pNext sont des pointeurs.

[crocodilex]

crocodilex t'étais pas réveillé c'est bien ce qu'il avait marqué

C'est qui "il" ?
Peut etre que tu n'as pas bien compris ou bien lu mon post !!!

[souviron34]

Crocodilex a écrit :

C'est qui "il" ?

iPote puisque c'était bien à son code que tu faisais réference quand tu as écrit :

La définition de sa structure

non ?

Médinoc a écrit :

Si si, gl a raison, et ce que tu viens de poster ne compilera pas.

bien sûr que si.. Je l'utilise partout dans mes codes..

voir

http://vergil.chemistry.gatech.edu/r...l/structs.html

[Thierry Chappuis]

Code :

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

1
2
3
4
5
6
7
8
9
 
typedef struct  Ptr_MaStruct {
 
    int toto ;
 
    Ptr_MaStruct previous ;
    Ptr_MaStruct next ;
 
} MaStruct ;

Ici, le type Ptr_MaStruct n'est pas définit. En tout cas, mon compilateur me raconte pleins de choses.

Code :

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

1
2
3
4
5
6
7
8
typedef struct  Ptr_MaStruct {
 
    int toto ;
 
    struct Ptr_MaStruct previous ;
    struct Ptr_MaStruct next ;
 
} MaStruct ;

Thierry

[souviron34]

eh bien écoute je ne sais que te dire...

Moi j'utilises ça depuis plus de 11 ans sur HPUX, SGI Unix, Linux (redhat), et ça marche parfaitement....

Exemple pour de la triangulation :

typedef struct _Triangle {

int S1 ;
int S2 ;
int S3 ;

struct _Triangle *next ;
struct _Triangle *previous ;

} Triangle ;

[Thierry Chappuis]

eh bien écoute je ne sais que te dire...

Moi j'utilises ça depuis plus de 11 ans sur HPUX, SGI Unix, Linux (redhat), et ça marche parfaitement....

Exemple pour de la triangulation :

Là, c'est OK, mais ce n'est pas ce que tu as écris plus haut...

Thierry

[souviron34]

mujigka a écrit :

Là, c'est OK, mais ce n'est pas ce que tu as écris plus haut...

oooopppppssss t'as raison... je devais m'endormir... milles excuses...

Ce que je voulais dire c'est que le nom avant désigne le pointeur sur la structure, et que donc on peut s'en servir directement.. Si il n'y a pas de nom après, ça désigne la structure. Si il y a un nom avant ça désigne donc le pointeur sur la structure, et (contrairement à beaucoup de cours que j'ai vu sur le net) que du coup il vaut mieux lui donner un nom différent... (Ptr_Struct ou _Struct)..

Désolé encore...

[crocodilex]

[...]Si il y a un nom avant ça désigne donc le pointeur sur la structure,[...]

T'as vu ça où ? J'espère que ce n'est pas dans un bouquin qui traite du langage C.....

[souviron34]

reprenons calmement l'exemple :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
typedef struct _Triangle {
 
int S1 ;
int S2 ;
int S3 ;
 
struct _Triangle *next ;
struct _Triangle *previous ;
 
} Triangle

Si maintenant je veux créer un élément, je vais pas faire :

Code :

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

1
2
3
4
struct _Triangle *MonTriangle=NULL ;

MonTriangle = calloc ( 1, sizeof(_Triangle) ;

Mais par contre je vais faire :

Code :

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

1
2
3
4
 
struct Triangle *MonTriangle=NULL ;
 
MonTriangle = calloc ( 1, sizeof(Triangle) ;

[crocodilex]

Je trouve cette écriture assez douteuse :

Mais par contre je vais faire :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
typedef struct _Triangle { 
   int S1 ;
   int S2 ;
   int S3 ; 
   struct _Triangle *next ;
   struct _Triangle *previous ; 
} Triangle; 
 
struct Triangle *MonTriangle=NULL ;
 
MonTriangle = calloc ( 1, sizeof(Triangle) ;

Bizarre ton truc.

Moi je ferais comme ça :

Code :

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

1
2
3
 
Triangle *MonTriangle=NULL ;
MonTriangle = calloc ( 1, sizeof *MonTriangle ) ;

ou bien :

Code :

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

1
2
3
 
struct _Triangle *MonTriangle=NULL ;
MonTriangle = calloc ( 1, sizeof *MonTriangle);