Allocation d'une structure avec tableau

Bonsoir a vous tous,
voila j'aimerais savoir si ce code est bon...il est bien passé au compilateur...
Mais voila je voulais savoir si mes allocations etaient necessaires...

Est-il necessaire de déclarer les tableaux compris dans une structure?

Merci d'avance

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 25 26 27 28

#include <stdio.h> #include <stdlib.h>   typedef struct { char *mot; int nombre; }Mot;   void Init (Mot T[]) { char *tmp=(char*)malloc(sizeof(char)); if (!tmp) { fprintf(stderr,"Erreur alloc memoire\n"); exit (EXIT_FAILURE); } tmp="\0"; T = (Mot *) malloc (sizeof(Mot)); if (!T) { fprintf(stderr,"Erreur alloc memoire\n"); exit (EXIT_FAILURE); } T->nombre=0; T->mot=tmp; }

---

Code:

---

void Init (Mot T[])

---

Surement pas ! Le mieux est de faire retourner un tableau de structure (si c'est bien ce que tu veux créer) par la fonction donc un pointeur:

Code:

---

Mot * Init (size_t size)

---

Là c'est déjà plus cohérent !

Je ne vois pas l'intérêt d'allouer mot à un seul caractère, le mieux c'est de le mettre directement à la bonne taille ou bien d'attendre de posséder la taille qui conviens puis d'allouer l'espace en conséquence !

Ceci est n'est pas tout à fait correct:

Code:

---

tmp="\0";

---

Etant donné que tu as alloué 1 char à temp donc si tu veux mettre un caractère null fait plutôt comme ca:

Code:

---

tmp[0] = '\0';

---

Mais là encore je ne vois pas l'intérêt d'une telle chaîne/allocation, autant la faire au moment où tu copies vraiment une chaîne dans cet espace car là ca te fait une allocation pour rien et plus on peut économiser des allocations mieux c'est et donc en fait pour si peu c'est du temps de gâché ;)

Au passage, les casts sur les malloc ne sont pas une bonne idée, cela peut cacher d'éventuels problèmes donc:

Code:

---

T = malloc (sizeof(Mot));

---

Sauf si tu veux que le code soit également compilable pas un compilateur C++ auquel cas il sera obligatoire !

Une version modifiée de ta fonction serait:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22

Mot * Init (size_t size, const char * mot) { Mot * T = NULL;   if (size > 0 && mot != NULL) { T = malloc (size * sizeof (* T));   if (T != NULL) { T->nombre = strlen (mot); T->mot = malloc (T->nombre + 1);   if (T->mot != NULL) { strcpy (T->mot, mot); } } }   return T; }

---

Enfin si j'ai bien compris ce que tu cherches à faire mais on peut l'améliorer un peu !

Citation:

---

Envoyé par line86

Bonsoir a vous tous,
voila j'aimerais savoir si ce code est bon...il est bien passé au compilateur...
Mais voila je voulais savoir si mes allocations etaient necessaires...
<...>

---

Horrible !

Tu fais

Code:

---

1 2 3 4

char *tmp = malloc (sizeof (char)); <...> tmp = "\0";

---

Ca signifie que tu remplaces l'adresse du bloc alloué par l'adresse d'une chaine... Tu as donc perdu l'adresse du bloc. De plus, la chaine n'est pas modifiable... Tu veux dans doute

Code:

---

strcpy (tmp, "\0");

---

ou

Code:

---

strcpy (tmp, "");

---

ou encore

Code:

---

*tmp = 0;

---

Ensuite, ceci n'a aucun sens :

Code:

---

1 2 3 4 5

void Init (Mot T[]) {   T = malloc (sizeof(Mot));

---

Tu modifies la valeur d'un paramètre, ça ne va pas modifier l'original...

http://emmanuel-delahaye.developpez....difie_variable

Bonsoir a vous,
merci pour vos réponses ! :D
Mais juste pour précisions

1- Le cast c'est par exemple

Code:

---

Liste p = (cellule *) malloc (sizeof (cellule));

---

c'est qu'on a toujours appris a faire ainsi :oops: lol... Je viens effectivement de regarder dans les tuto de c et il n'y a jamais de cast... Je vous remercie de m'avoir éclairer sur ce point !!!!

2- Le prototype de la fonction était ainsi déclaré dans mon partiel... lol c'est pour ca que je me suis trompé dans ma fonction :P je ne voyais pas comment faire :aie:

Je vous remercie je vais essayer de reprendre tout ca :D

Bonjour,

Citation:

---

Envoyé par line86

Code:

---

Liste p = (cellule *) malloc (sizeof (cellule));

---

---

Juste une question : quel est le compilateur utilisé ?
Il me semble que cette syntaxe n'est pas autorisée en C "classique" (en C++ oui).

Citation:

---

Envoyé par xavi

Juste une question : quel est le compilateur utilisé ?
Il me semble que cette syntaxe n'est pas autorisée en C "classique" (en C++ oui).

---

Non. Elle est autorisée mais déconseillée en C et elle est obligatoire en C++ (mais il y a new/delete, alors pourquoi faire...)