Structure en C

**Flanders Weasel** · 27/03/2012, 13h55

Bonjour à toutes et à tous,

Pour mon programme, j'ai besoin d'une structure contenant une autre structure et deux tableaux de structures.

J'ai plusieurs questions :

- le nombre de structures Picture dans Metadata dépend du fichier traîté, comment puis-je faire pour l'allocation de mémoire?

- on ne connais qu'à l'exécution la taille des champs 'field' et 'value', comment puis-je faire pour l'allocation de mémoire?

- comment libérer proprement la mémoire allouée?

Voici mon code :

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
25
26
27
28
29
30
31
32
33
34
 
typedef struct {
	char	*mime_type;
	char	*description;
	int	width;
	int	height;
	int	depth;
	int	colors;
	int	data_length;
	char	*data;
} Picture;
 
typedef struct {
	unsigned int	min_blocksize;
	unsigned int	max_blocksize;
	unsigned int	min_framesize;
	unsigned int	max_framesize;
	unsigned int	sample_rate;
	unsigned int	channels;
	unsigned int	bits_per_sample;
	unsigned long int	total_samples;
	char	md5sum[16];
} Info;
 
typedef struct {
	char	*field;
	char	*value;
} Comment;
 
typedef struct {
	Info	info;
	Comment	*comments;
	Picture	*pictures
} Metadata;

Merci d'avance pour votre aide.

Edit : message corrigé.

**Bktero** · 27/03/2012, 14h07

Et ?

**Neckara** · 27/03/2012, 15h12

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
typedef struct
{
    int x;
}ma_Structure;
 
typedef struct
{
    ma_Structure a;
    ma_Structure a[45];
}mon_tableau;

**Flanders Weasel** · 27/03/2012, 15h21

Pardon, le message a été posté alors que je ne l'ai pas fini.
Je corrige et je vous renvoi le tout.

**Trademark** · 27/03/2012, 20h22

Salut,

Tu utilises les fonctions malloc (pour allouer) et free pour désallouer comme ceci :

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
int field_size;
int nb_comments;
// Récupération à l'éxecution de field_size et nb_comments...
Comment* my_comments = malloc(sizeof(Comment) * nb_comments);
 
// Imaginons que pour chaque commentaire tu veux allouer la taille nécessaire à field.
int i;
for(i=0; i < nb_comments; ++i)
  my_comments[i].field = malloc(sizeof(char) * field_size);
 
// ... Utilisation
 
 
// Libération
for(i=0; i < nb_comments; ++i)
  free(my_comments[i].field);
free(my_comments);

**Flanders Weasel** · 28/03/2012, 12h41

Merci pour ton aide Trademark.
J'ai juste un problème lorsque je souhaite faire un printf avec le champs field, j'ai une erreur de segmentation.

Pourrais-tu m'aider?

Je te donne le code concerné :

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
 
                    meta.comments = (Comment*)malloc(block->data.vorbis_comment.num_comments*sizeof(Comment));
 
                    for(i = 0; i < block->data.vorbis_comment.num_comments; ++i) {
 
                        gchar **entries = g_strsplit_set((gchar*)block->data.vorbis_comment.comments[i].entry,"=", -1);
 
                        meta.comments[i].field = malloc(sizeof(char)*sizeof(entries[0]));
                        meta.comments[i].field = UTF8(entries[0]);
                        meta.comments[i].value = malloc(sizeof(char)*sizeof(entries[1]));
                        meta.comments[i].value = UTF8(entries[1]);
 
                        g_strfreev(entries);
                    }
 
                    for(i=0; i < block->data.vorbis_comment.num_comments; ++i)
                        printf("%s\n", meta.comments[i].field);
                        free(meta.comments[i].field);
                        free(meta.comments[i].value);
                    free(meta.comments);

block->data.vorbis_comment.num_comments est le nombre de structures comment.

UF8 permet de convertir la chaîne en UTF8.

J'utilise la GLib pour ce programme.

**diogene** · 28/03/2012, 13h59

Code :

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

1
2
3
4
                        meta.comments[i].field = malloc(sizeof(char)*sizeof(entries[0]));
                        meta.comments[i].field = UTF8(entries[0]);
                        meta.comments[i].value = malloc(sizeof(char)*sizeof(entries[1]));
                        meta.comments[i].value = UTF8(entries[1]);

La ligne 2 écrase allègrement l'adresse obtenue par le malloc() de la ligne 1. Ceci ne fait pas une copie du tableau renvoyé par UTF8().
Idem pour les deux lignes suivantes

**Flanders Weasel** · 28/03/2012, 18h42

J'ai changé le code et j'ai fait ça :

Code :

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

1
2
3
4
5
6
 
meta.comments[i].field = malloc(sizeof(char)*strlen(entries[0]));
strcpy(meta.comments[i].field, UTF8(entries[0]));
 
meta.comments[i].value = malloc(sizeof(char)*strlen(entries[1]));
strcpy(meta.comments[i].value, UTF8(entries[1]));

Est-ce correct?

Ca ne me donne plus d'erreur.

**Neckara** · 29/03/2012, 06h18

Après chaque malloc comme après chaque fonction retournant une valeur il faut tester les codes de retours au cas où une erreur se serait produite.
Ceci permet d'écrire des messages d'erreurs pour savoir pourquoi le logiciel a planté et ainsi pouvoir retrouvé un peu plus vite les erreurs.

**Trademark** · 29/03/2012, 10h03

Que retourne la fonction UTF8 ? Et vu que je présumé que ça retourne un pointeur comment l'alloues-tu ?

Dans ton cas je vois deux solutions (et c'est deux erreurs) :

Soit tu retournes l'adresse d'une variable local à la fonction (ce qui provoquera une segfault à un moment ou à un autre).
Soit tu retournes un pointeur que tu as alloué dans ta fonction avec malloc, ce qui n'est pas si mal mais alors c'est en conflit avec ce que tu fais (pas de libération de mémoire, copie inutile, ..).

Ce qui me semble le plus beau, c'est de passer meta.comments[i].field en argument à UTF8 (ce qui laisse le loisir à UTF8 de retourner un code d'erreur si ça c'est mal passé) comme ceci :

Code :

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

1
2
3
4
5
6
// Si "field" est alloué avant.
UTF8(entries[0], meta.comments[i].field);
// Si field est alloué dans UTF8 :
UTF8(entries[0], &meta.comments[i].field);
// Si tu ne peux pas modifié UTF8 :
meta.comments[i].field = UTF8(entries[0]); // Ne réserve pas de place pour field avant ! (UTF8 le fait).

Je te laisse méditer le pourquoi du comment et choisir la solution qui te convient

**kwariz** · 29/03/2012, 11h09

Salut,

Envoyé par Flanders Weasel

Code :

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

1
2
3
4
5
6
 
meta.comments[i].field = malloc(sizeof(char)*strlen(entries[0]));
strcpy(meta.comments[i].field, UTF8(entries[0]));
 
meta.comments[i].value = malloc(sizeof(char)*strlen(entries[1]));
strcpy(meta.comments[i].value, UTF8(entries[1]));

En plus des dernières remarques je rajouterais que ce genre de code est faux et se retrouve souvent quand on utilise malloc/strcpy. Tu dois donner à malloc la taille de la chaine incluant le 0 terminal, strlen te donne la taille de la chaine en n'incluant pas le 0 terminal.
Il faut juste penser à faire quelquechose comme :

Code :

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

1
2
3
char* copie = malloc(sizeof(*copie)*(strlen(original)+1));
strcpy(copie,original);

Deuxième remarque : tu manipules les charsets. entries est un tableau de chaine encodées avec un certain charset et tu transforme cette chaine en utf8. Tu alloues de la place "de même taille" (cf remarque prec.) que la chaine d'origine dans le charset d'origine pour y placer la version utf8 : ATTENTION la version utf8 peut avoir une longueur différente et parfois supérieure suivant le charset d'origine. Par exemple si le charset d'origine est iso8859-1, la chaine origine="salut éric, ça va ?" contient 19 caractères et est codée sur 20 octets. La même chaine en utf8 contiendra toujours 19 caractères mais sera codée sur 22 octets car les caractères é et ç sont codés sur 2 octets.
Donc avec ton exemple tu alloues 20 et tu écris 22 => problèmes ...

**Flanders Weasel** · 30/03/2012, 17h24

Bonjour à tous et merci pour votre aide.

UTF8 est en fait une macro :

Code :

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

#define UTF8(string) g_locale_to_utf8(string, -1, NULL, NULL, NULL)

J'utilise une fonction de la glib pour m'assurer que la chaîne soit encodée correctement : g_local_to_utf8

La fonction doit s'occuper d'allouer la mémoire nécessaire, je pense.

Le tableau entries provient de la fonction g_strsplit_set qui me permet de découper une chaîne en plusieurs chaînes.

Voici le code qui la contient :

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
 
 
Metadata *metadata;
 
    for(i = 0; i < nbComments; ++i) {
 
        gchar **entries = g_strsplit_set((gchar*)comments[i].entry,"=", -1);
 
        metadata->comments[i].field = malloc(sizeof(char)*strlen(UTF8(entries[0])));
        strcpy(metadata->comments[i].field, UTF8(entries[0]));
 
        metadata->comments[i].value = malloc(sizeof(char)*strlen(UTF8(entries[1])));
        strcpy(metadata->comments[i].value, UTF8(entries[1]));
 
        g_strfreev(entries);
    }

J'ai également corrigé le code pour allouer la mémoire pour la chaîne convertie en UTF8 via la macro.

Je fais les vérifications de malloc, je ne les ai pas mises dans le code pour alléger le post.

**diogene** · 30/03/2012, 17h34

La fonction doit s'occuper d'allouer la mémoire nécessaire, je pense.

Alors, à quoi servent les malloc() ?

Code :

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

metadata->comments[i].field = malloc(sizeof(char)*strlen(UTF8(entries[0])));

**kwariz** · 30/03/2012, 17h41

Envoyé par Flanders Weasel

Bonjour à tous et merci pour votre aide.

UTF8 est en fait une macro :

Code :

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

#define UTF8(string) g_locale_to_utf8(string, -1, NULL, NULL, NULL)

J'utilise une fonction de la glib pour m'assurer que la chaîne soit encodée correctement : g_local_to_utf8

La fonction doit s'occuper d'allouer la mémoire nécessaire, je pense.

Le tableau entries provient de la fonction g_strsplit_set qui me permet de découper une chaîne en plusieurs chaînes.

Voici le code qui la contient :

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
 
 
Metadata *metadata;
 
    for(i = 0; i < nbComments; ++i) {
 
        gchar **entries = g_strsplit_set((gchar*)comments[i].entry,"=", -1);
 
        metadata->comments[i].field = malloc(sizeof(char)*strlen(UTF8(entries[0])));
        strcpy(metadata->comments[i].field, UTF8(entries[0]));
 
        metadata->comments[i].value = malloc(sizeof(char)*strlen(UTF8(entries[1])));
        strcpy(metadata->comments[i].value, UTF8(entries[1]));
 
        g_strfreev(entries);
    }

J'ai également corrigé le code pour allouer la mémoire pour la chaîne convertie en UTF8 via la macro.

Je fais les vérifications de malloc, je ne les ai pas mises dans le code pour alléger le post.

Salut,

tu as oublié le +1 pour le 0 terminal dans le malloc.

**Flanders Weasel** · 05/04/2012, 14h01

Bonjour,

Les champs field et value ont une taille qui ne sera définie qu'à l'exécution, c'est pourquoi j'utilise un malloc.

J'ai réjouté 1 dans le malloc. Tout roule. Merci pour votre aide.

**diogene** · 06/04/2012, 15h07

Si UTF8() fait l'allocation mémoire, à quoi sert le malloc() ? Il fait double emploi et en plus on a une fuite mémoire. Dans ce cas, pourquoi ne pas faire

Code :

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

metadata->comments[i].field = UTF8(entries[0]);

Structure en C

C

Discussions similaires

Partager

Partager