Discussion :

[Chamimi]

Bonsoir,

J'ai une structure définie ainsi :

Code :

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

1
2
3
4
typedef struct {
  int nbcoups;
  coup coups[];
}partie;

Mais quand je fais ça :

Code :

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

1
2
partie part;
part.coups=calloc(sizeof(coup), 3);

Il me dit :

Code :

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

invalid use of flexible array member.

Je fais comment pour réserver la place en mémoire qu'il faut pour mon tableau?

Merci d'avance !

[kwariz]

Salut,

tu demandes à ton ami (?) Raikyn, ou tu fais une recherche sur le forum http://www.developpez.net/forums/d12...f/#post6633092

EDIT:
Par curiosité, la structure a été définie par un prof dans un sujet, ou vous faites tous parti d'une même équipe et quelqu'un de cette équipe a choisi cette structure ?

[Chamimi]

Oh tiens, je vais penser à le contacter !

Ça a été défini par le prof.

Et merci au fait

[Chamimi]

Finalement, j'ai quand même un problème qui me rend dingue :

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
 
partie coups_possible(othellier p, joueur j) {
    partie *part;
    partie res;
    // Allocation de l'espace mémoire nécessaire :
    part = malloc(sizeof(partie)+3*sizeof(coup));
    // Exemple de modification de mon tableau de tableau dans la structure part :
    part-> coups[0][0]='a';
    part-> coups[0][1]='a';
    part-> coups[2][0]='a';
 
    res=*part;
    free(part);
    return res;
}

Lorsque j'affiche res.coups[0][0] ça m'affiche bien 'a' mais lorsque j'essaie d'afficher (le résultat de la fonction).coups[0][0], ça m'affiche des caractères bizarres.
Quelqu'un sait pourquoi?

[kwariz]

Tu libères la mémoire que tu vas utiliser ensuite.

[Chamimi]

J'ai fais exprès de copier la structure dans une autre variable (res) pour pouvoir libérer la mémoire. C'est bon normalement non?

Code :

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

1
2
3
res=*part;
free(part);
return res;

Quand j'ai mis ça à la place, ça ne fonctionne pas non plus...

Code :

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

return *part;

Finalement, j'ai fais d'autre tests et ce qu'il m'affiche avec :

Code :

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

printf("  %c  ", (*part).coups[0][0]);

n'est pas la même chose qu'avec :

Code :

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

printf("  %c  ", res.coups[0][0]);

dans la fonction.

Alors que j'ai bien fait avant :

Code :

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

res=*part;

Je me suis trompée tout à l'heure.


C'est très bizarre...

[Bousk]

J'ai fais exprès de copier la structure dans une autre variable (res) pour pouvoir libérer la mémoire. C'est bon normalement non?

Code :

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

1
2
3
res=*part;
free(part);
return res;

Quand j'ai mis ça à la place, ça ne fonctionne pas non plus...

Code :

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

return *part;

Finalement, j'ai fais d'autre tests et ce qu'il m'affiche avec :

Code :

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

printf("  %c  ", (*part).coups[0][0]);

n'est pas la même chose qu'avec :

Code :

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

printf("  %c  ", res.coups[0][0]);

dans la fonction.

Alors que j'ai bien fait avant :

Code :

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

res=*part;

Je me suis trompée tout à l'heure.


C'est très bizarre...

Je suis sceptique quant au fait que cette ligne fasse une copie de l'objet initial.

[Bktero]

Tu n'as pas attentivement lu l'utilisation des flexible array members. Tu ne dois pas créer de variable de ce type directement, uniquement des pointeurs, comme expliqué dans ce lien :

When using structures with a flexible array member you should never directly declare an instance of the structure.

[diogene]

J'ai fais exprès de copier la structure dans une autre variable (res) pour pouvoir libérer la mémoire. C'est bon normalement non?

Non ici, ça ne marche pas parce que la structure a un tableau "flexible" et donc que sa taille réelle est inconnue du compilateur.
Tu ne peux pas construire une partie correcte avec la simple déclaration partie res; . C'est bien évident puisque tu as pris le soin d'en construire une par le malloc(). Si on pouvait le faire, on ne se serait pas cassé le chou à la créer par allocation dynamique .

Le res=*part; ne fera que copier le début des structures, mais pas le contenu du tableau flexible.

On doit avoir :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
partie * coups_possible(othellier p, joueur j) {
    partie *part = malloc(sizeof(partie)+3*sizeof(coup));
    // Il faudrait tester ici la réussite de l'allocation
    part-> coups[0][0]='a';
    part-> coups[0][1]='a';
    part-> coups[2][0]='a';
    // free(part);  il faudra penser à le faire ailleurs
    return part;
}
.... 
partie *part = coups_possible(....,....);
if(part != NULL) printf("  %c  ", part->coups[0][0]);
free(part);

[Chamimi]

Merci pour vos réponses !
Il est donc bien impossible de renvoyer le structure même.

En fait, la structure et la prototype de la fonction nous ont été imposés dans ce projet, d'où mon entêtement. Bon, ça veut dire qu'ils se sont trompés (génial !).

Encore une fois merci

[kwariz]

Salut,

je pense que pour la santé mentale de tout le monde, une âme charitable pourrait poster le sujet, non ?

Car entre une mauvaise lecture du sujet, une mauvaise interprétation, des erreurs éventuelles, la fin du monde en décembre il y a beaucoup d'alternatives.

Enfin, si vous voulez une aide utile ...

EDIT: à essayer avec gcc :

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
/* test.c */
#include <stdio.h>
 
typedef struct {
  int a;
} ma_struct;
 
ma_struct f(ma_struct m)
{
  m.a=2;
  return m;
}
 
int main()
{
  ma_struct m;
  ma_struct n;
 
  m.a=1;
  n=f(m);
 
  printf("m.a=%d\n", m.a);
  printf("n.a=%d\n", n.a);
}

compilation :

Code :

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

gcc -o test test.c

À votre avis ? ça compile ? ça donne un résultat ? lequel ? Est-ce propre ?

[Chamimi]

Héhé, kwariz, pour votre santé mentale, j'ai fait l'effort de poster le sujet

http://www.pps.jussieu.fr/~rifflet/e...projet-lc4.pdf

Voilà. Mais quelqu'un a posé la question à notre chargé de TD et effectivement, il y avait une erreur dans le sujet. Ils vont (j'espère avant la date limite ) corriger le sujet et nous le faire savoir.

à essayer avec gcc :

À votre avis ? ça compile ? ça donne un résultat ? lequel ? Est-ce propre ?

C'est un avis ou un test que vous demandez?
Si c'est un avis, je pense que ça compile, que ça fonctionne mais que c'est moche.
Mais mon problème était plus une question de modifs de tableaux flexibles dans une structure que de modifs de types simples dans une structure.

Si c'est un test :
Ca compile et ça donne bien :

Code :

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

1
2
m.a=1
n.a=2

[kwariz]

Merci Chamimi
Tu as sauvé ma santé mentale, et non ... "ils" ne se sont pas trompés.
La conception "imposée" n'est certainement pas celle que j'aurais choisie, mais elle est imposée donc vous devez faire avec ...

EDIT: peut être pourriez-vous "négocier" un changement d'API ?

[Chamimi]

"ils" ne se sont pas trompés

mais elle est imposée donc vous devez faire avec ...

Dans la mesure où l'on ne peut pas faire fonctionner cette fonction (partie coups_possible(othellier p, joueur j)) avec la structure partie donnée, je pense qu'ils se sont bien plantés oui

Oui, j'ai demandé au prof si on ne pouvait pas modifier les prototypes des fonctions. J'attends sa réponse...

[kwariz]

Bien sûr on peut, mais cela va impliquer des fuites mémoires inévitables, car tu vas allouer dynamiquement une structure dont tu ne renverras pas l'adresse et dont le contenu se fera copié en perdant tout espoir de récupérer l'adresse du bloc alloué.
C'est donc faisable, mais absolument pas propre ...

Maintenant, je pense effectivement que personne ne donne ce genre de contraintes volontairement.

[Chamimi]

C'est possible? Tu pourrais me dire comment parce que j'ai tout essayé, le tableau n'est jamais copié correctement...