Discussion :

[loisir1976]

Bonjour a tous,

le code ci-dessous se compile sans probleme, par contre genere une errur de "segmentation fault" quand je l'excute:

Code :

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

1
2
3
4
5
6
7
8
 
int N=10000;
float* A=new[N];
for (int i=0; i<N;i++)
{A[i]=1};
finction_save_infile(A);
 
delete [] A;

Cette erreur disprait quand j'ajoute la ligne

Code :

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

 A=NULL;

juste avant delete (en ligne 7)

j'ai les questions suivantes:
- pourquoi ca genere une erreur de segmentation ?
- je ne comprends pas pourquoi ajoute cette ligne ca fait disparaitre l'erreur!
- est que je dois faire confiance au resultat obtenu par ce programme ?

Merci de votre aide

[ram-0000]

essaye

Code :

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

float* A=new float [N];

[loisir1976]

oui j'ai fait ca

Code :

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

float* A=new float [N];

mais l'erreur de segmentation persiste.

J'ai essaye de re-excute le programme, maintenent meme avec A=NULL ca genere une erreur de segmentation a l'excution ?

[gbdivers]

Je suppose que c'est une erreur de frappe :

Code :

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

1
2
{A[i]=1};
{A[i]=1;}

A part cela, le code est "correcte" (sauf que sans raison particulière, utilise plutôt un std::vector à la place d'un tableau comme tu fais). Le problème doit venir de ta fonction finction_save_infile

[ZiGoM@r]

Ou bien le code que tu nous montres n'est pas celui qui plante.

[gl]

Que fait finction_save_infile() ?

Pourrais-tu fournir un code minimal mais compilable qui reproduit le problème ?

[3DArchi]

Salut,

Code :

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

1
2
3
int N=10000;
std::vector<float> A(N,1);
finction_save_infile(A);

les pointeurs bruts (new/delete), ça montre ceux qui ne savent pas développer en C++

Ceci dit,

- pourquoi ca genere une erreur de segmentation ?

Très concrètement, parce que tu débordes probablement au delà de l'espace réservé lors du new[].

- je ne comprends pas pourquoi ajoute cette ligne ca fait disparaitre l'erreur!

Car tu ne supprimes plus l'espace mémoire alloué. Du coup, c'est juste de la chance que le problème ne se révèle pas à cet endroit. Mais cela risque de planter ailleurs de façon assez incompréhensible. En plus, tu génères une fuite mémoire.

- est que je dois faire confiance au resultat obtenu par ce programme ?

Non.

[oodini]

Cette erreur disprait quand j'ajoute la ligne

Code :

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

 A=NULL;

juste avant delete (en ligne 7)

Ben déjà, l'affectation à NULL doit avoir lieu après la libération des ressources, dont après le delete.