Discussion :

[gnto]

Bonjour,

Voila la je commence a fatiguer,

Code :

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

1
2
3
4
5
typedef struct _Ligne Ligne
struct _Ligne
{
 char s[60];
};

dans la fonction main

Code :

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

1
2
3
4
 
Ligne * buffer = NULL;
void mafonction(&buffer);
free(buffer);

dans ma fonction

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
void mafonction(Ligne ** buffer)
{
 static int tailleBuffer=0;
 
//tailleBuffer evolue dans la fct mais j'ai simplifié le code mais tailleBuffer est non nulle et supérieur à 3
 
 *buffer = realloc(*buffer,sizeof(Ligne)*tailleBuffer);
 
buffer[2]->s[0]='a';//erreur de segmentation
 
}

La je m'embrouille, j'aimerais savoir comment on peut allouer et initialiser buffer dans mafonction et libérer dans la fonction main.

[Gruik]

Je dirais

Code :

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

(*buffer)[2].s[0]='a';

[gnto]

selon ddd, l'allocation est bien effectué mais buffer[2] pointe sur n'importe quoi ! c pas un pb de type

*buffer =>4
buffer => 60

[gnto]

nan tailleBuffer n'est pas nulle j'ai pas mis le code pour affecter une valeur a tailleBuffer pour des soucis de lisibilité

[gnto]

Je dirais

Code :

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

(*buffer)[2].s[0]='a';

je vois pas la différence entre

(*buffer[2]).s[0] = 'a';
et
buffer[2]->s[0]='a';

(*variable).champ == variable->champ Enfin je crois

[crocodilex]

Le problème vient certainement de l'appel à la fonction realloc().
Tu lui passes un pointeur non initialisé.

[gnto]

*buffer == NULL donc y'a pas de problème

l'erreur vient de l'affectation

[crocodilex]

*buffer == NULL donc y'a pas de problème

l'erreur vient de l'affectation

Oui en effet, dans ce cas là c'est équivalent à un malloc. Au temps pour moi...

[Skyounet]

tailleBuffer = 0;

realloc(*buffer,sizeof(Ligne)*tailleBuffer);

sizeof(Ligne)*tailleBuffer = 0;
Donc tu n'alloue pas grand chose la.

[Emmanuel Delahaye]

Voila la je commence a fatiguer,

Moi aussi. Ne pas recopier le code, mais faire un copié/collé. C'est plein de bugs inutiles...

A part ça, si tailleBuffer vaut 0, sizeof(Ligne)*tailleBuffer vaudra 0, donc realloc() va retourner NULL (jamais testé...)

Comportement indéfini.

De plus, si tu veux écrire en buffer[2], il faut que tailleBuffer soit > 2... (jamais testé...).

De toutes façons, cette conception est atroce (Force 9).

http://emmanuel-delahaye.developpez.com/goret.htm

Pour faire un tableau souple, il faut utiliser une structure qui regroupe

• l'adresse de base du tableau
• sa taille

Ici, ce serait

Code :

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

1
2
3
4
5
6
 
struct tabdyn
{
   Ligne *p_lines;
   size_t nb_lines;
};

[gnto]

Moi aussi. Ne pas recopier le code, mais faire un copié/collé. C'est plein de bugs inutiles...

Ah oui ? j'ai recopié par souci de simplification et pour aller à l'essentielle.

[Skyounet]

Ah oui ? j'ai recopié par souci de simplification et pour aller à l'essentielle.

Oui mais la par exemple en recopiant tu as glisser des erreurs dans ton code (oubli de ;, appel de fonction avec void devant).

Donc c'est plus simple de faire un copier/coller.

[gnto]

A part ça, si tailleBuffer vaut 0, sizeof(Ligne)*tailleBuffer vaudra 0, donc realloc() va retourner NULL (jamais testé...)

Comportement indéfini.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
void mafonction(Ligne ** buffer)
{
 static int tailleBuffer=0;
 
//tailleBuffer evolue dans la fct mais j'ai simplifié le code mais tailleBuffer est non nulle et supérieur à 3
 
 *buffer = realloc(*buffer,sizeof(Ligne)*tailleBuffer);
 
buffer[2]->s[0]='a';//erreur de segmentation
 
}

Il me semble que j'avais laissé un commentaire

[gnto]

les static sont a utiliser en dernier recours car ils sont défini en mémoire pendant la durée de tout le programme, a l'inverse des variables locales

[Emmanuel Delahaye]

les static sont a utiliser en dernier recours car ils sont défini en mémoire pendant la durée de tout le programme, a l'inverse des variables locales

Il n'y a aucune justification pour utiliser des statiques. Apprend plutôt à programmer par contexte persistants.

[gnto]

comment géres-tu les flags dans une librairie si tu n'as pas de variables statiques ?