segmentation fault pour new

Version imprimable

19/07/2012, 12h33
dev_tours

segmentation fault pour new
bonjour,

Je suis actuellement en train de coder en CPP et j'ai un problème d'allocation avec la fonction new.

En effet l'allocation sur un pointeur libéré par delete et remis à NULL me renvoie une segmentation fault. J ai bien vérifié la valeur du pointeur avant l'allocation et elle est bien à 0.
Code:

1 2 3 4 5 delete []chrom; chrom = NULL; chrom = new float[nb_Variable];
Je précise aussi que nb_Variable est instancié et initialisé correctement.
19/07/2012, 13h04
dragonjoker59

La fonction new peut effectivement échouer, mais quand elle échoue, en fonction du compilateur elle peut retourner NULL sans lancer d'exception ou lancer une exception de type std::bad_alloc. Lorqu'elle échoue, le problème vient généralement du débordement de la mémoire autorisée pour une application (2Go de ram pour une appli Win32, si je me souviens bien, par exemple).
Le fait que tu aies une SegFault sur un new laisse malgré tout penser que le problème est ailleurs.
Ta variable chrom est elle un membre de classe ? Si oui, l'instance de cette classe est-elle bien initialisée ?
A part ça, il nous faudrait de toute façon plus de code pour répondre à ta question.

bonjour et merci de votre reponse,

chrom fait en effet partie d'une classe (Chromosome) et est initialisé à NULL dans le constructeur par défaut.

Code:

1
2
3
4
5
6
 
Chromosome::Chromosome(){
    chrom = NULL;
    borne_Sup= NULL;
    borne_Inf= NULL;             
}

l'erreur se produit dans l’opérateur d'affectation :

Code:

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
34
35
36
37
38
39
 
Chromosome &Chromosome::operator=(const Chromosome &source)
{
    int i;
 
    nb_gene = source.nb_gene;
 
    delete []chrom;
    chrom = NULL;
 
    chrom = new float[nb_gene];
    i=0;
    for(i=0;i<nb_gene;i++)
    {
        chrom[i] = source.chrom[i];
    }
 
 
    delete [] borne_Sup;
    borne_Sup = NULL;
 
    borne_Sup = new float[nb_gene];
    for(i=0;i<nb_gene;i++)
    {
        borne_Sup[i] = source.borne_Sup[i];
    }
 
 
    delete [] borne_Inf;
    borne_Inf = NULL;
 
    borne_Inf = new float[nb_gene];
    for(i=0;i<nb_gene;i++)
    {
        borne_Inf[i] = source.borne_Inf[i];
    }
 
    return *this;
}

je suis bien en dessous de la limite d'allocation. Mon programme prend maximum 1MB.

19/07/2012, 13h37
Bousk

Bonjour,

quel est le type de chrom ?
Par contre inutile de lui affecter NULL pour faire une allocation derrière.
nb_gene est bien > 0 ?
Le compteur i n'a pas besoin d'être déclaré en dehors de son scope (for), puisque tu ne l'utilises pas en dehors.
19/07/2012, 13h44
Invité

salut,

dans ton constructeur de Chromosome, tu initialises pas nb_gene.
Si c'est un int, ta valeur par défaut sera pas 0 mais aléatoire.

Donc peut être bien que tu fais un new Float[44568423146].
19/07/2012, 13h45
dev_tours

quel est le type de chrom ?
chrom est float*

Par contre inutile de lui affecter NULL pour faire une allocation derrière.
Je le sais, mais comme la réallocation ne fonctionnait pas, je l ai rajouté.

nb_gene est bien > 0 ?
oui, nb_gene est égal à 5.

Le compteur i n'a pas besoin d'être déclaré en dehors de son scope (for), puisque tu ne l'utilises pas en dehors.
vieille habitude du C natif...
19/07/2012, 13h53
dev_tours

@galerien69 : bonne observation... je l'initialise à 0 maintenant.

Cependant dans mon problème, nb_gene est à 5.
19/07/2012, 14h39
gbdivers

Tu peux donner le code appelé pour créer tes objets et faisant l'affectation ?
Une raison particulière de ne pas utiliser std::vector et std::copy ?

Une raison particulière de ne pas utiliser std::vector et std::copy ?
aucune, si ce n'est que je n'ai pas l'habitude d'utiliser des conteneurs. Il semble en effet que cela pourrait me simplifier la vie.

Tu peux donner le code appelé pour créer tes objets et faisant l'affectation ?

Code:

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
34
35
36
int main(int argc, char *argv[])
{
    int i,j,idx;
    float borne_inf,borne_sup,max_fit;
    float alea,s,temp;
    int nb_variable = 5;
    int iteration,taille;
    
    iteration = 1000;
    taille = 50;
    
    srand(time(NULL));  
    
    alea =  rand()/(double)RAND_MAX;

    borne_inf = - 5.12;
    borne_sup = 5.12;
    
    Ga essai(taille,iteration);
    Chromosome c(nb_variable);
    temp = 0;
    for(i=0;i<taille;i++)
    {
        for(j= 0;j<nb_variable;j++)
        {
            temp = (rand()/(double)RAND_MAX)*(borne_sup-borne_inf)+borne_inf;
            c.set_Gene(temp,j);
            c.set_Borne(borne_inf,borne_sup,j);
        }
         essai.set_Chromosome(c,i);
    }
    
    
    system("pause");
    
}

Code:

1
2
3
4
5
6
7
 
Chromosome::Chromosome(int taille){
    nb_gene = taille;
    chrom = new float[nb_gene];
    borne_Inf = new float[nb_gene];
    borne_Sup = new float[nb_gene];
}

Code:

1
2
3
4
 
void Ga::set_Chromosome(Chromosome& c,int pos){
     pop[pos] = c;
}

voila les initialisations.

20/07/2012, 11h34
white_tentacle

Il vient d’où pop ?

À mon avis, il n’est pas initialisé correctement, d’où tes problèmes. Qu’est-ce qui te fait dire que c’est new qui fait le core-dump ?