Discussion :

[pierre_92]

Bonjoru,

j'ai un problème de segmentation fault avec le code ci-dessous :

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
 
#include <stdio.h>
 
typedef struct {
    int data;
} st_toto;
 
typedef struct
{
    int data;
    st_toto *toto_ptr;
} st_tata;
 
int main()
{
    st_tata tata;
 
    tata.data = 123;
    tata.toto_ptr->data = 456;
 
    printf("tata data = %d\n", tata.data); 
    printf("toto data = %d\n", tata.toto_ptr->data); 
 
    return 0;
}

visiblement le problème vient du pointer qui pointe vers la première structure
quelqu'un pour m'aider ?

Merci

[Matt_Houston]

Deux questions afin d'évaluer ton niveau de connaissance : à quoi t'attendais-tu ? Que penses-tu qu'il se produise à l'exécution de ce programme ?

[sambia39]

Bonjour,

Bonjoru,

j'ai un problème de segmentation fault avec le code ci-dessous :
visiblement le problème vient du pointer qui pointe vers la première structure
quelqu'un pour m'aider ?

Merci

Effectivement, l'erreur vient de votre pointeur, vous n'avez pas alloué de la mémoire pour pouvoir stocker la valeur 456. Ce que vous avez fait en réalité, c'est que vous avez d'attribuer la valeur 456 comme l'adresse de que quelque chose au pointeur et lors de l'affichage vous lui demande d'afficher ce qui se trouve à cette adresse (ici en l'occurrence 456) donc segment.....
Le mieux est de faire une allocation, mais attention après utilisation, il faut rendre la mémoire au système en appelant delete

Code c++ :

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
 
#include <cstdlib>
#include <iostream>
 
typedef struct {
    int data;
} st_toto;
 
typedef struct{
    int data;
    st_toto *toto_ptr;
} st_tata;
 
int main( void ){
 
    int r = 0;
    st_tata x;
    x.data = 100;
    x.toto_ptr = new st_toto;
    x.toto_ptr->data = 100;
 
    r = x.data + x.toto_ptr->data;
    std::cout << "st_data.data\t:" << x.data << std::endl;
    std::cout << "st_data.st_toto->data\t:" << x.toto_ptr->data << std::endl;
    std::cout << "Resultat\t:" << r << std::endl;
 
    delete x.toto_ptr;
 
    return EXIT_SUCCESS;
}

à bientôt

[Bousk]

Y'a-t-il une raison à ce que toto_ptr soit un pointeur ?
Si oui, il conviendrait de l'initialiser à nullptr. Et de l'initialiser correctement avec un objet existant avant de l'utiliser.
Si ton initialisation revient à faire un new puis un delete, il n'a pas à être un pointeur mais devrait être un simple membre. Donc l'utilisation de std::unique_ptr n'est pas non plus bienvenue ici.

Enfin ce code est du pur C. Es-tu vraiment en C++ ?