Encore des pointeurs ! [Résolu]

[edgarjacobs]

Bonjour à tous,

J'ai besoin de vos lumières, car je cale!

Voici la déclaration d'une structure:

Code :

1
2
3
4
5
6
typedef struct {
	int nentry;
	char **fullname;
	char **shortname;
	int *pos;
} PLAYLIST;

En fonction du nombre d'entrées à mémoriser, j'alloue la structure de la manière suivante (les tests de malloc==NULL ont été supprimés):

Code :

1
2
3
4
5
6
7
8
9
10
11
12
13
PLAYLIST *plAlloc(int nentry)
{
	PLAYLIST *pl;
 
	pl=malloc(sizeof(PLAYLIST));
	if(pl!=NULL) {
		pl->fullname=malloc(nentry*sizeof(char *));
		pl->shortname=malloc(nentry*sizeof(char *));
		pl->pos=malloc(nentry*sizeof(int));
		pl->nentry=nentry;
		}
	return(pl);
}

Dans mon esprit:
pl->fullname et pl->shortname sont des 'tableaux' de nentry pointeurs vers des chaines de caractères,
pl->pos est un 'tableau' de nentry int

Enfin, lorsque je remplis ma structure, je le fais (séquentiellement, offset allant de 0 à nentry-1) de la manière suivante:

Code :

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
int plFillList(PLAYLIST *pl,int offset,char *name)
{
	char *sptr;
 
	*(pl->fullname+offset)=malloc(strlen(name)+1);
	if(*(pl->fullname+offset)==NULL) {
		fprintf(stderr,"plFillList: malloc error (%d)\n",offset);
		exit(0);	// n'arrive jamais !
		}
	strcpy(*(pl->fullname+offset),name);
	sptr=strrchr(*(pl->fullname+offset),'\\');
	if(sptr==NULL) sptr=*(pl->fullname+offset);
	*(pl->shortname+offset)=sptr;
	*(pl->pos+offset)=offset+1;
	return(0);		// provisoire
}

Le compilateur ne met aucune warning, mais lors de l'exécution mon programme se plante.

Quelqu'un at-t-il une idée? Mauvaise utilisation de ma part des pointeurs?

Merci d'avance de votre aide.

[Trademark]

Salut,

Pour moi, ta gestion des pointeurs est bonne. Je ne vois pas vraiment ton erreur à part cette ligne qui semble suspecte :

Code :

*(pl->pos+offset)=offset+1;

Pourquoi le +1 ?

Après je te conseille d'utiliser Valgrind qui, au contraire de gdb, ne demande pas de prise en main, tu fais simplement :

valgrind ./tonpgm

Et il te diras où tu as des fuites de mémoire ou lecture invalide, ce qui résoudra probablement ton pb. N'oublie pas de compiler avec l'option -g pour le débogage aussi

[edgarjacobs]

@TrademarkMerci de ta réponse.

- Le +1: juste une valeur que j'incrémente en vue d'un futur affichage pour ne pas commencer par zéro

- Valgrind: hélas, je suis sous windows....

- Je compile avec Borland bcc32

Merci à toi.

[diogene]

Je ne vois pas d'erreur dans ton code. L'erreur se situe sans doute dans le reste du programme.

Note : ce genre de choses est correct *(pl->fullname+offset)=... mais l'écriture pl->fullname[offset]=... est plus simple et plus facile à lire.

[edgarjacobs]

Je vous remerie à tous d'avoir pris le temps de me répondre.

Je vais donc revoir le reste du programme, puisque d'après vous le code montré ne présente pas d'erreur au niveau de la gestion des pointeurs. Cela doit donc certainement venir des appels ou de l'utilisation que j'en fais ensuite (zur alors

@Diogène: serais-je un peu masochiste ? Mais tu as parfaitement raison, tu l'as si souvent écrit sur le forum: tab[1]=*(tab+1)....

Encore merci,

Edgar.

[edgarjacobs]

La gestion des pointeurs était bonne. Seul hic: dans une autre fonction, je commençais par allouer à partir de 1, pas de zéro (un compteur mal placé, une ligne trop haut....). Vous pouvez imaginer le reste dès l'utilisation....

for(i=0;i<100;i++) printf("Merci\n");

[Sve@r]

Envoyé par edgarjacobs

@Diogène: serais-je un peu masochiste ? Mais tu as parfaitement raison, tu l'as si souvent écrit sur le forum: tab[1]=*(tab+1)....

Tant qu'à être masochiste, autant l'être à fond: tab[1]=*(tab+1)=*(1+tab)=1[tab]....