Problème windows! (le programme a cessé de fonctionner)

Version imprimable

salut! en travaillons avec le langage c sur les pointeurs j'ai eu le problème suivant:

http://www2.0zz0.com/2013/11/29/18/147790328.jpg

c'est quoi la cause ?
voila une partie du programme

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
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
#include <stdio.h>
#include <stdlib.h>
#include <conio.h>
#include <windows.h>
 
/*procedure pour afficher un tableau d'entiers*/
 
void afficher_tab(int t[])
{
    for(int k=0;k<9;k++)
    {printf("%d",t[k]);}
}
 
 
//structure frontiere
struct noeudf {
    int grille[9];
    int niv;
    struct noeudf *suiv;
};
 
typedef struct noeudf* frontiere;
 
//creation noeud f
frontiere creation_n_f(int vval[])
{    frontiere e = NULL ;
     e = new(noeudf);
     if(e!=NULL) {
 
         for(int i=0;i<9;i++){e->grille[i] = vval[i] ;}
         e->suiv = NULL ;
 
     }
     return e ;
}
/*insert en queu */
void enfiler_f_n(frontiere* l,frontiere* el)
{   frontiere* p,q=NULL ;
 
 
    if(*l==NULL) {
        l =el;
    }
    else {
        q = *l ;
        while(q->suiv!=NULL) q = q->suiv ;
        q->suiv = *el ;        
    }
}
 
/*defiler */
frontiere defiler_f_n(frontiere* l)
{
    frontiere q = *l ;
 
    if(q!=NULL) {
        return q;
 
    }
 
}
 
 
 
main()
{
    int goal[] = {1,2,3,4,5,6,7,8,0};
    frontiere n=creation_n_f(goal);
    frontiere f;
    enfiler_f_n(&f,&n);
    frontiere pe=defiler_f_n(&f);
    printf("%d",pe->grille[0]);
    getch();
}

29/11/2013, 19h28
Neckara

Bonjour,

Windows est très enquiquinant à ce niveau là car quand un programme plante, il ne donnera jamais l'erreur...

Essaye de lancer ton programme en mode debug dans ton IDE.
Tu peux aussi utiliser le débogueur de ton IDE pour voir ce qui ne va pas.
29/11/2013, 19h53
mohsenuss91

une violation d'accès (erreur de segmentation)
d'apres le debugger l'erreur vient de la:

Code:

printf("%d",pe->grille[0]);

mais j'ai pas trouvé une solution!

Code:

1
2
3
4
5
6
7
8
9
10
frontiere defiler_f_n(frontiere* l)
{
    frontiere q = *l ;
 
    if(q!=NULL) {
        return q;
 
    }
 
}

Et si q vaut NULL, tu retournes quoi?

Citation:

Envoyé par Neckara

Code:

1
2
3
4
5
6
7
8
9
10
frontiere defiler_f_n(frontiere* l)
{
    frontiere q = *l ;
 
    if(q!=NULL) {
        return q;
 
    }
 
}

Et si q vaut NULL, tu retournes quoi?

Code:

1
2
3
4
5
6
7
8
9
frontiere defiler_f_n(frontiere* l)
{    frontiere q = *l ;
 
    if(q!=NULL) {
	return q;
 
    }else return NULL;
 
}

29/11/2013, 20h02
Neckara
Mais n'est-ce pas au final la même chose que :
Code:

1 2 3 4 frontiere defiler_f_n(frontiere* l) { return *l; }
?

D'ailleurs, tu ne teste pas si l vaut NULL ou non.
Si l vaut NULL, ton programme risque de planter.
29/11/2013, 20h11
mohsenuss91
Citation:
Envoyé par Neckara

Mais n'est-ce pas au final la même chose que :

Code:

1 2 3 4 frontiere defiler_f_n(frontiere* l) { return *l; }

?

D'ailleurs, tu ne teste pas si l vaut NULL ou non.
Si l vaut NULL, ton programme risque de planter.
J'ai testé avec:

Code:

if(q!=NULL) {

sinon comment faire dans le cas de NULL!
29/11/2013, 20h15
Neckara

q vaut *l, là n'est pas le problème.
Le problème, c'est la valeur de l, pas la valeur de *l.

Citation:

Envoyé par Neckara

q vaut *l, là n'est pas le problème.
Le problème, c'est la valeur de l, pas la valeur de *l.

j'ai essayé avec ça et normalement le problème est réglé :)
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 void enfiler_f_n(frontiere* l,frontiere el) { frontiere p,q=NULL ; if(*l==NULL) { *l =el; } else { q = *l ; while(q->suiv!=NULL) q = q->suiv ; q->suiv = el ; } }

29/11/2013, 20h49
Neckara

Quelques remarques :
- la variable p est inutilisée ;
- attention pour défiler, tu pars du principe que l ne sera jamais égal à NULL ;
- if(*l==NULL), tu peux aussi directement écrire if(! *l) (et while(q->suiv!=NULL) -> while(q->suiv) ).

Sinon, si tout est bon, n'oublie pas de marquer le sujet en résolu.
29/11/2013, 22h30
Médinoc

C'est toujours pourri! Ton code teste la valeur pointée, pas le pointeur...
30/11/2013, 02h46
edgarjacobs
Hello,

Je vais ajouter mon grain de sel, mais sans donner de solution car avec les messages de Medinoc et Neckara, ça me parait clair. Mais attention au fait que
Code:

1 2 3 4 5 typedef struct noeudf* frontiere; // et donc avec cette déclaration void enfiler_f_n(frontiere* l,frontiere el) // l est un pointeur **
Edgar.

EDIT: c'est pour éviter ce genre de confusion que je ne fait jamais de typedef de pointeurs....

Citation:

Envoyé par Neckara

Quelques remarques :
- la variable p est inutilisée ;
- attention pour défiler, tu pars du principe que l ne sera jamais égal à NULL ;
- if(*l==NULL), tu peux aussi directement écrire if(! *l) (et while(q->suiv!=NULL) -> while(q->suiv) ).

Sinon, si tout est bon, n'oublie pas de marquer le sujet en résolu.

désolé j'ai pas bien compris :( j'ai essayé avec ça :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 /*insert en queu */ void enfiler_f_n(frontiere* l,frontiere el) { frontiere q=*l ; if(q==NULL) {q =el;} else { while(q->suiv!=NULL) q = q->suiv ; q->suiv = el ; } } /*defiler */ frontiere defiler_f_n(frontiere* l) { frontiere q = *l ; if(q!=NULL) {return q;}else return NULL; }

30/11/2013, 11h53
Neckara

Si tu n'as pas compris, dit-moi ce que tu n'as pas compris...
30/11/2013, 12h49
mohsenuss91

Citation:

Envoyé par Neckara

Si tu n'as pas compris, dit-moi ce que tu n'as pas compris...

j'ai pas compris votre idée

Code:

if(*l==NULL), tu peux aussi directement écrire if(! *l) (et while(q->suiv!=NULL) -> while(q->suiv) ).
30/11/2013, 12h51
Neckara

if( *l == NULL) est équivalent à if( ! *l ).

Et if( *l != NULL) est équivalent à if( *l).