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';

[Skyounet]

tailleBuffer = 0;

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

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

[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...

[gnto]

qd je créé une variable temp

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
void mafonction(Ligne ** buffer)
{
 Ligne * temp;
 static int tailleBuffer = 0;
 
 ...
 
 temp = malloc(temp,sizeof(Ligne)*tailleBuffer);
 
 temp[2]->s[0] = 'a'; // la ca fonctionne
 
}

le probleme c'est que je n'ai pas de retour

[crocodilex]

Pourquoi tu ne fais pas quelque chose de plus simple du genre :

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
Ligne *mafonction(void)
{
    static Ligne *buffer = NULL;
    static int tailleBuffer=3;
 
    if ( (buffer = realloc(buffer,sizeof(Ligne)*tailleBuffer)) != NULL)
    {
        buffer[2].s[0]='a';
    }
    return buffer;
}
 
int main(void)
{
    [...]
    Ligne * buffer = NULL;
    buffer = mafonction();
    free(buffer);
    [...]
    return 0;
}

[gnto]

j'attendais la question,

j'aimerais avoir une signature de ce type

int mafonction(Ligne ** buffer,size_t count)

[Skyounet]

Tout ca pour dire que la solution t'avais ete donné par Gruik

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
 
#include <stdio.h>
#include <stdlib.h>
 
typedef struct _Ligne Ligne;
struct _Ligne
{
    char s[60];
};
 
void mafonction(Ligne ** buffer)
{
    static int tailleBuffer=5;
 
    //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';
}
 
int main(void)
{
    Ligne * buffer = NULL;
    mafonction(&buffer);
    free(buffer);
	return 0;
}

[gnto]

peut on me donner la difference entre

(*variable).champ et variable->champ

parce que j'ai toujours cru que c'étais la même chose !!

J'allais oublier de vous remercié tous.

[Skyounet]

peut on me donner la difference entre

(*variable).champ et variable->champ

parce que j'ai toujours cru que c'étais la même chose !!

J'allais oublier de vous remercié tous.

C'est effectivement la meme chose, c'est juste une ecriture simplifié.

Mais ici avec ton probleme
(*buffer)[2]...
(*buffer[2])...
*buffer[2]...

Ce n'est pas la meme chose. Quand tu as un doute avec la priorité des opérateurs met des parenthèses.

[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.

[Emmanuel Delahaye]

Pourquoi tu ne fais pas quelque chose de plus simple du genre :

Code :

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

1
2
3
4
Ligne *mafonction(void)
{
    static Ligne *buffer = NULL;
    static int tailleBuffer=3;

Goretisation de force 9 !
http://emmanuel-delahaye.developpez.com/goret.htm

[Emmanuel Delahaye]

peut on me donner la difference entre

(*variable).champ et variable->champ

parce que j'ai toujours cru que c'étais la même chose !!

Certes, mais le problème, c'est que tu as dans ton code une atrocité appelée 'douple pointeur', qu'il est préférable de fuir pour ne pas avoir ces écritures gores...

Je recommande fortement l'usage de la structure qui nous ramènes à des choses connues et lisibles du style : p->element...