Discussion :

[lucas_sophia]

Bonjour a tous,
voila j'essaye d'écrire un petit programme qui lit les tag ID3 d'un fichier mp3,
j'ai bien regardé la spec du format, et j'ai la structure suivante pour récupere le header d'un tag :

Code :

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

1
2
3
4
5
6
7
 
typedef struct {
  unsigned char		id[3];
  unsigned char		revision[2];
  unsigned char		flags;
  unsigned int		size;
} ID3V2_header_t;

Le header fait donc exactement 10 bytes, seulement voila, alors que ma structure devait en faire 10 aussi (3 + 2 + 1 + 4), lorque que je fait
sizeof(ID3V2_header_t) j'obtient 12 ...

J'ai regardé sur la FAQ C ou il est dit que cela peut venir d'un probleme d'alignement des bits ? mais comment y remédier ? Est qu'il faut que je passe par l'operateur ":" pour spécifier explicitement la taille de chacun de mes champs en bits ? cad :

Code :

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

1
2
3
4
5
6
7
 
typedef struct {
  unsigned char		id[3] : 24
  unsigned char		revision[2] : 16
  unsigned char		flags : 8
  unsigned int		size : 32
} ID3V2_header_t;

?

Voila, tout aide sera la bienvennue
(sachant que j'ai deja un autre pb en rab, mais je le garde pour le prochain post )

PS : si vous voulez plus d'informations, voila un lien vers la spec des tags :
http://www.id3.org/id3v2.3.0.html

[Emmanuel Delahaye]

voila j'essaye d'écrire un petit programme qui lit les tag ID3 d'un fichier mp3,
j'ai bien regardé la spec du format, et j'ai la structure suivante pour récupere le header d'un tag :

Code :

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

1
2
3
4
5
6
7
 
typedef struct {
  unsigned char		id[3];
  unsigned char		revision[2];
  unsigned char		flags;
  unsigned int		size;
} ID3V2_header_t;

Erreur de conception fréquente. En C, il n'est pas possible de mapper une structure sur un flux de bytes de façon portable, et ce pour de nombreuses raisons :

• La taille des types n'est pas portable
• La représentation des valeurs numériques n'est pas portable
• L'alignement des données n'est pas portable

Le header fait donc exactement 10 bytes, seulement voila, alors que ma structure devait en faire 10 aussi (3 + 2 + 1 + 4), lorque que je fait
sizeof(ID3V2_header_t) j'obtient 12 ...

Démonstration faite.

Pour être portable, extraire les données à la main (tableau de unsigned char) dans une structure interne.

Idem dans l'autre sens. Les structures ne peuvent pas servir à implémenter des interfaces externes de façon portable. Jamais.

[lucas_sophia]

Merci Emmanuel,
en fait l'idée de mapper la structure sur le header était principalement de pourvoir ensuite utiliser le code suivant :

Code :

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

1
2
3
4
5
6
7
 
FILE* fIn;
ID3V2_header_t header;
if((fIn= fopen(mp3filename, "r")) == NULL) {
	return EXIT_FAILURE;
}
fread(&header, sizeof(ID3V2_header_t), 1, fIn);

pour remplir directement toute la structure.
Si j'utilise ce code, c'est en fait parce que j'avais lu un tutorial sur le chargement de bitmap, et la personne qui l'avait rédigé utilisait cette technique pour lire le header du fichier :

Code :

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

1
2
 
fread(&bmpheader, sizeof(BITMAPINFOHEADER), 1, fIn);

Bon ben je vais remplir les structures au fur et fur, j'avais testé, mais je trouvais ca un peu long ...

J'en profite pour poser ma deuxieme question :
voila une fois que j'ai reussi a recuperer le header de mon tag, je dois récuperer les frames du tag, pour etre plus clair, voila comment se présente un tag :

-------------------------------
TAG HEADER
-------------------------------
EXTENDED HEADER (falcultatif)
-------------------------------
FRAMES
....
....
--------------------------------

Le principe et de recuperer le header de la frame, qui est constitué d'un id (4 bytes), puis sa taille (4 bytes), si la frame nous interresse (par exemple la frame qui contient le nom de l'artiste, le nom de l'album etc...), on la lit, sinon on regarde la taille, et on passe a la frame suivante.
Mon probleme, c'est que j'arrive a lire l'id de la premiere frame, "TABL" qui correspond aux informations sur la chanson , mais quand je recupere la taille, j'obient un chiffre completement exhorbitant ...

Code :

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

1
2
3
4
5
6
7
8
9
10
 
// frame header
char fid[4];
char fsize[4];
 
printf("Reading frame header\n");
fread(&fid, sizeof(fid), 1, fIn);
printf("fid read : %c%c%c%c\n", fid[0], fid[1], fid[2], fid[3]);
fread(&fsize, sizeof(fsize), 1, fIn);
printf("fsize read : %d\n", fsize);

ce qui me donne comme trace :

Reading frame header
fid read : TALB
fsize read :1310576

le probleme c'est que mon .mp3 fait seulement 258ko
PS : sachant qu'il est dit dans la spec :

The bitorder in ID3v2 is most significant bit first (MSB). The byteorder in multibyte numbers is most significant byte first (e.g. $12345678 would be encoded $12 34 56 78).

Merci, J'espere que j'ai été clair, et pas trop long

[Emmanuel Delahaye]

Merci Emmanuel,
en fait l'idée de mapper la structure sur le header était principalement de pourvoir ensuite utiliser le code suivant :

Code :

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

1
2
3
4
5
6
7
 
FILE* fIn;
ID3V2_header_t header;
if((fIn= fopen(mp3filename, "r")) == NULL) {
	return EXIT_FAILURE;
}
fread(&header, sizeof(ID3V2_header_t), 1, fIn);

pour remplir directement toute la structure.

Oui, j'avais compris ! Mais cette technique, bien que séduisante, n'est pas portable. Tu en as toi même apporté la preuve.

Si j'utilise ce code, c'est en fait parce que j'avais lu un tutorial sur le chargement de bitmap, et la personne qui l'avait rédigé utilisait cette technique pour lire le header du fichier :

D'où ma remarque 'erreur de conception courante'...

Bon ben je vais remplir les structures au fur et fur, j'avais testé, mais je trouvais ca un peu long ...

Oui, la portabilité a un coût.

[lucas_sophia]

Haleluyah !
ca y est, le probleme venait bien de "l'endianess", aprés avoir bien regardé comment ca marchait, il s'avere que mon int qui contenait la size de ma frame était lu a l'envers (Big endian), donc il faut que je passe par une convevrsion :

Code :

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

1
2
 
nsize = ntohl(fsize);

et voila, tout de suite les tailles sont plus cohérentes, 29, 25 bytes.

[Edouard Kaiser]

Si tu utilises gcc tu peux utiliser l'attribut : __attribute__ ((packed)), la taille des structures sera exactement comme tu l'attends, mais la en effet on perd la portabilité.

[Emmanuel Delahaye]

ca y est, le probleme venait bien de "l'endianess",

Ben oui. Déjà signalé :

La représentation des valeurs numériques n'est pas portable