Discussion :

[sylvanus35]

Bonsoir,

Je dois réaliser une fonction qui remplace dans une liste un caractere par une liste de caractere. Elle compile mais au moment de s'éxécuter j'ai une erreur de segmentation. J'ai cru comprendre que cela venait d'un problème d'allocation de mémoire. Est-ce que quelqu'un pourrait m'en dire plus? Merci

Voici ma fonction (si quelqu'un voit d'ou vient l'erreur n'hésitez pas ; je galère dessus depuis quelques heures )

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
liste remplace (liste l, char c,liste m)
{
    //liste n = copy(liste m);
 
    liste p= (liste)malloc(sizeof(struct cellule));
    liste q= (liste)malloc(sizeof(struct cellule));
    liste o= (liste)malloc(sizeof(struct cellule)); 
    p= l;
 
 
 
    while ((*l).valeur!=c)
    {
        (*q).suivant = l;
        l=(*l).suivant;
 
    }
 
 
    o = (*l).suivant;
 
    (*q).suivant=m;
 
 
    while ((*m).suivant!=NULL)
    {
        m=(*m).suivant;
    }
 
    m=o;
    free(l);
return p;
}

[Médinoc]

1. On ne caste pas le retour de malloc() en C.
2. Tu pourrais faire des noms de variable encore moins explicites ?
3. La syntaxe (*p).valeur est lourde. p->valeur est plus lisible.
4. Fuite de mémoire: Tu fais p=l juste après avoir alloué...

[mac&cheese]

Je pense que le problème vient du fait que tu fais p=l sans controler la taille de ce que tu affectes.
Tu écris surment dans une zone mémoire, dans laquelle tu n'es pas autorisé à accéder.

Mac & Cheese

[valefor]

1. On ne caste pas le retour de malloc() en C.

Cela m'étonne. Est-ce juste inutile ? Ou est-ce une mauvaise pratique ?

[Médinoc]

En fin de compte, c'est:

• inutile,
• une mauvaise pratique potentielle car ça peut cacher un bug dans une certaine combinaison de facteurs,
• Et de toute façon, ça fait un cast explicite de plus, et d'une manière générale, je suis partisan d'éviter les casts explicites, surtout les casts C-style (qui sont, hélas, les seuls disponibles en C).