Discussion :

[Premium]

Bonjour,
j'aurais besoin d'aide car je n'arrive pas à savoir ou se situe l'erreur dans mon code.
Merci d'avance

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
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
#include<stdio.h>
#include<stdlib.h>
#include<float.h>
#include<string.h>
#include<ctype.h>
 
#define MAX 100
 
 
enum Type {Operateur, Constante, Variable};
 
 
union Info  
{
  double cte;
  char *nom;
  char op;
};
 
struct noeud{
  enum Type type;
  union Info info;
  struct noeud *gauche;
  struct noeud *droit;
};
 
typedef struct noeud Noeud;
typedef struct noeud * Expression;
 
typedef struct _Symbole{
  int type;
  char *nom;
  double val;
  int nb_parametres;
  char **parametres;
  Expression e;
} Symbole;
 
typedef struct _Environnement{
  unsigned taille;
  Symbole tab[MAX];
} Environnement;
 
 
 
 
 
 
 
/* Fonction qui gere les espaces */
void espace(char **ligne)
{
  while(*ligne[0] == ' ') ++(*ligne);
}
 
 
int EstReel(char *ligne, double *val)
{
  double v;
  char *endptr;
 
 
  v=strtod(ligne,&endptr);
 
  /* la chaine commence par un chiffre */
  if(endptr!=ligne)
    {
      /* la chaine ne contient pas que des chiffres */
      if(*endptr!='\0')
	{
	  fprintf(stderr,"erreur\n");
	  return 1;
	}
 
      /* la chaine ne contient que des chiffres */
      if(v==DBL_MIN||v==DBL_MAX)
	{
	  fprintf(stderr,"debordement\n");
	  return 1;
	}
 
      /* on recupere la valeur */
      *val=v;
      return 0;
    }
  else return 1;
}
 
 
 
/* Fonction qui teste si la chaine passee en parametre est une variable */
int EstVariable(char *ligne)
{
  if(isalpha(*ligne))
    {
      ++ligne;
      while(isalnum(*ligne))
	ligne++ ;
    }
  if(*ligne=='\0') return 0 ;
  else return 1;
}
 
 
/* Analyse d'une expression:creation de l'arbre */
Expression AnalyseExpression(char **ligne)
{
  Expression e;
  char *sep = NULL;
  double val;
  char c;
 
  espace(ligne);
 
  e = (Expression)malloc(sizeof(Noeud));
  if(e == NULL) return NULL;
 
  /* c'est un chiffre */
  if(isdigit((*ligne)[0]))
    {
      /* on cherche le premier blanc pour recuperer la valeur */
      if(!(sep = strchr(*ligne,' ')))
	{
	  fprintf(stderr,"erreur\n");
	  return NULL;
	}
      *sep = '\0';
 
      /* verifie que le reel est bien ecrit */
      if(EstReel(*ligne,&val))
	{
	  fprintf(stderr,"erreur\n");
	  return NULL;
	}
 
      e->type = Constante;
      e->info.cte = val;
      e->gauche = NULL;
      e->droit = NULL;
      *ligne = (sep+1);
    }
 
  else {
    /* c'est une lettre */
    if(isalpha((*ligne)[0]))
      {
	/* on cherche le premier blanc pour recuperer le nom */
	if(!(sep = strchr(*ligne,' ')))
	  {
	    fprintf(stderr,"erreur\n");
	    return NULL;
	  }
	*sep = '\0';
 
	/* verifie que la variable est bien ecrite */
	if(EstVariable(*ligne))
	  {
	    fprintf(stderr,"erreur\n");
	    return NULL;
	  }
 
	e->type = Variable;
	e->info.nom = *ligne;
	e->gauche = NULL;
	e->droit = NULL;
	*ligne = (sep+1);
      }
    else
      {
	/* le caractere est un operateur */
	c=(*(*ligne)++);
	if(c == '+' || c == '-' || c == '*' || c == '/')
	  {
	    e->type = Operateur;
	    e->info.op = c;
	    e->gauche = AnalyseExpression(ligne);
	    e->droit = AnalyseExpression(ligne);
	  }
	else if(c == '@' || c == '~')
	  {
	    e->type = Operateur;
	    e->info.op = c;
	    e->gauche = AnalyseExpression(ligne);
	    e->droit = NULL;
	  }
	else printf("Erreur,'%c' n'est pas un operateur prevu\n",c);
      }
  }
  return e;
}
 
int main(void)
{
  char *ligne = "           + * 1 2 7";
  Expression e;
 
  if(AnalyseExpression(&ligne)==NULL);
  {
    fprintf(stderr,"erreur\n");
    return 1;
  }
  e=AnalyseExpression(&ligne);
 
 
  return 0;
}

[zekey]

Sous quel système/compilo es tu. Si c'est avec linux/gcc par exemple: commence par compiler avec -g et regarder si il produit un core dump.
Sinon lance to prog avec gdb et fait un backtrace. Il te donnera la ligne ou ca plante.

[Premium]

Sous quel système/compilo es tu. Si c'est avec linux/gcc par exemple: commence par compiler avec -g et regarder si il produit un core dump.

je suis sous linux et j'ai fait ce que tu as écrit ,pas de core dump.

Sinon lance to prog avec gdb et fait un backtrace. Il te donnera la ligne ou ca plante.

comment cette commande s'utilise t'elle?
pour compiler,j'ai tapé cette commande :
gcc -Wall -ansi fct.c -o fct

[zekey]

gcc -Wall -ansi fct.c -g -o fct
gdb
load fct
run
backtrace

Je crois je suis pas sous linux là je peux pas vérifier mais un petit man un help dans gdb devrait t'aider

[Trap D]

A la compilation j'ai ça sous Visual C :

Code :

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

e:\visual c\essai\man_utd\man_utd.c(133) : warning C4700: local variable 'e' used without having been initialized

La ligne incriminée :

Code :

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

      e->type = Constante;

Si on regarde bien

Code :

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

1
2
3
4
5
6
7
8
9
10
11
struct noeud{
  enum Type type;
  union Info info;
  struct noeud *gauche;
  struct noeud *droit;
};
 
typedef struct noeud Noeud;
typedef struct noeud * Expression;
 
Expression e;

Comme son type ne l'indique pas (ce qui est une erreur) e est un pointeur, qui n'est pas initialisé donc plante forcée. Utilise ce bon vieux malloc sinon tu écris dans la nature et le C a horreur de la nature .

[Premium]

A la compilation j'ai ça sous Visual C :

Code :

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

e:\visual c\essai\man_utd\man_utd.c(133) : warning C4700: local variable 'e' used without having been initialized

La ligne incriminée :

Code :

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

      e->type = Constante;

Si on regarde bien

Code :

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

1
2
3
4
5
6
7
8
9
10
11
struct noeud{
  enum Type type;
  union Info info;
  struct noeud *gauche;
  struct noeud *droit;
};
 
typedef struct noeud Noeud;
typedef struct noeud * Expression;
 
Expression e;

Comme son type ne l'indique pas (ce qui est une erreur) e est un pointeur, qui n'est pas initialisé donc plante forcée. Utilise ce bon vieux malloc sinon tu écris dans la nature et le C a horreur de la nature .

J'ai toujours le pb meme apres avoir rajouter un malloc

[Trap D]

Tu as un autre pointeur,

Code :

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

 char *sep = NULL;

et comme à un moment tu fais

Code :

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

 *sep = '\0';

tu vois les dégats.

[Premium]

Tu as un autre pointeur,

Code :

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

 char *sep = NULL;

et comme à un moment tu fais

Code :

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

 *sep = '\0';

tu vois les dégats.

Cette manière de faire est du à mon prof de Td car dans un exemple,il avait fait un code de ce style.
Il disait qu'il valait mieux intialisé un pointeur à null plutot que de le laisser
De plus c'est bizzar que le compilo n'accepte pas cette ecriture car meme en enlevant l'initialisation du pointeur à null,gdb m'affiche cette meme ligne d'erreur.

[Trap D]

Et il a tout à fait raison, facile de rejeter la faute sur le prof quand on n'y arrive pas (tu n'as pas de chance j'en suis un mais de maths). Mais réfléchis, peux-tu utiliser ce pointeur s'il n'est pas initialisé ??
L'initialisation à NULL est une précaution comme ça on voit tout de suite la plante, car sinon ça peut fonctionner et c'est encore pire car tu as alors un comportement indéfini qui peut faire que par exemple ton programme tournera chez toi mais pas sur l'ordi du prof où il plantera lamentablement ; alors suis les conseils de ton prof, réfléchis comprends ce qu'il dit et tout ira bien.

De plus c'est bizzar que le compilo n'accepte pas cette ecriture car meme en enlevant l'initialisation du pointeur à null,gdb m'affiche cette meme ligne d'erreur.

Cette ligne indique que tu n'as rien compris à ce qu'est un pointeur : un pointeur est une variable qui contient une adresse mémoire, pour que cette adresse soit valide, il faut par exemple qu'elle ait été allouée par un malloc.