Discussion :

[mohsenuss91]

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



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

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
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();
}

[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.

[mohsenuss91]

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

Code :

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

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

mais j'ai pas trouvé une solution!

[Neckara]

Code :

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

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?

[mohsenuss91]

Code :

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

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 :

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

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;
 
}

[Neckara]

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

Code :

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

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.

[mohsenuss91]

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

Code :

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

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 :

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

if(q!=NULL) {

sinon comment faire dans le cas de NULL!

[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.

[mohsenuss91]

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 :

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

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 ;
 
 
    }
}

[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.

[Médinoc]

C'est toujours pourri! Ton code teste la valeur pointée, pas le pointeur...

[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 :

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

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....

[mohsenuss91]

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 :

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
/*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;
}

[Neckara]

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

[mohsenuss91]

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

j'ai pas compris votre idée

Code :

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

if(*l==NULL), tu peux aussi directement écrire if(! *l) (et while(q->suiv!=NULL) -> while(q->suiv) ).

[Neckara]

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

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