interruption programme dans un "calloc"

Version imprimable

Bonjourr,

Mon code s'arrête en plein milieu d'un calloc (ligne 212).
Le dernier output est en effet généré à la ligne 211 (un printf). Un printf est également placé en ligne 213, mais est sans effet, ce qui me fait dire que l'arrêt du programme est causé par la ligne 212 (un calloc). J'utilise GCC.

L'output:

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
Début programme
Apres initCaracteres
TRENTEETUNMAUCHEMINDEFERDETTEHCRUOFLALENOCRAGXUEIVNREHCIUQSEEANTCOMBINAISONEULFEVURXTNCAIEBISEAUTEEOUEOLGSIAECAMPSLRUEUQRAMRIQTJADTNNENEBUSELNBNIFSAITSERMIDNLGRVELUEFEABDLSRAANNABROOIDTRIQEOTBUNQOUTNRANMEBTLUOVEAERELTISCRRIEALOIUAOPUUPTTCREILOCEEANTLEBSSPLTOOTUEUHOBSILTNNGRCRNEEIALIARRBCNRENURLTCAAEBBNCTHOERETAEURQSIRUIELPDRAAOASSURERSSMCUCSOCNUECEUTURDFCUVERSAEAHETUIOMOLCETGBOLROLEIENNEDAOGAPNBBODRIENAAERFRTTUSSVRCAFATEUNENINMPMZESTRONCAELEREHRVACRLEBESIGUEUAUEOMRESPDBEMNALUETIMRAMSRCREENSMSOMREOMERUTSURCOUPERNLUOAAAOTDNILBREVOUDLERHCUEIRCIGHEENITNOTELGNISREVERTAUQRNCNAINJAUNEIREHCIRTIMPASSEE
Dans intiGrilleMotsMeles
 hauteur 24 largeur 25
|51||52||53||54||55||56||57|0 
Dans initElementSousGrille 1: Lettre 57 Direction = 0
apres malloc: sizeof(object): 12

Dans initElementSousGrille 4, k= 3 &p= 60360C80
Dans initElementSousGrille 6a b c 1 
Dans initElementSousGrille 1: Lettre 57 Direction = 1
apres malloc: sizeof(object): 12

Dans initElementSousGrille 4, k= 18 &p= 60360C90
Dans initElementSousGrille 6a b c 2 
Dans initElementSousGrille 1: Lettre 57 Direction = 2
apres malloc: sizeof(object): 12

Dans initElementSousGrille 4, k= 22 &p= 60360CB0
Dans initElementSousGrille 6a b c 3 
Dans initElementSousGrille 1: Lettre 57 Direction = 3
apres malloc: sizeof(object): 12

Dans initElementSousGrille 4, k= 8 &p= 60360CD0
Dans initElementSousGrille 6a b c 4 
Dans initElementSousGrille 1: Lettre 57 Direction = 4
apres malloc: sizeof(object): 12

Dans initElementSousGrille 4, k= 3 &p= 60360CE0
Dans initElementSousGrille 6a b c 5 
Dans initElementSousGrille 1: Lettre 57 Direction = 5
apres malloc: sizeof(object): 12

Dans initElementSousGrille 4, k= 18 &p= 60360CF0
Dans initElementSousGrille 6a b c 6 
Dans initElementSousGrille 1: Lettre 57 Direction = 6
apres malloc: sizeof(object): 12

Dans initElementSousGrille 4, k= 8 &p= 60360D10
Dans initElementSousGrille 6a b c 7 
Dans initElementSousGrille 1: Lettre 57 Direction = 7
apres malloc: sizeof(object): 12

Dans initElementSousGrille 4, k= 3

Le code:

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
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
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
#include <stdlib.h>
#include <stdio.h>
#include <string.h>
#include <ctype.h>

#define MAX_T 100
#define DIM_MAX 60
#define EOF_LIST -1
#define MAXGRILLE 1500

/*TYPES*/
const short incr1[8]= {-1,0,1, 0,-1,1, 1,-1};
const short incr2[8]= { 0,1,0,-1, 1,1,-1,-1};
enum DIRECTION {NORD=0,EST,SUD,OUEST,NORDEST,SUDEST,SUDOUEST,NORDOUEST};


typedef int indice_t;

typedef struct
{
  char *listeCars;
  enum DIRECTION sens;
  indice_t indice;

} SousGrille_t;

typedef struct
{
  SousGrille_t *y;
} SousGrilleParCar_t;

typedef struct
{
  char *grille;
  SousGrilleParCar_t *x;
  int largeur;
  int hauteur;
} GrilleMotsMeles_t;


/*PROTOTYPES*/
GrilleMotsMeles_t *initGrilleMotsMeles(FILE *);
char *initCaracteres(FILE *,int *,int *);
SousGrilleParCar_t *initTabSousGrilleParCar(char *,int,int);
SousGrille_t *initTabSousGrille(char *,int,int,int);
SousGrille_t *initElementSousGrille(char *,int,int,int,int);
void assure(void *, const char *);



int main()
{
GrilleMotsMeles_t *grillemelimelo;
FILE* fGrille;
/*DEBUG
int i,j;
indice_t *pind;
FIN DEBUG*/
  freopen("DEBUGT.TXT","w",stdout);
  printf("Début programme\n");
  /*DEBUG
  printf("COUCOU1\n");
  fflush(stdout);
  FIN DEBUG*/
  fGrille=fopen("tableau.txt","r");
  grillemelimelo=initGrilleMotsMeles(fGrille);
  /*DEBUG
  printf("\nGRILLE:\n%s\n",grillemelimelo->grille);
  printf("\n LARGEUR: %d\n",grillemelimelo->largeur);
  printf("\n HAUTEUR: %d\n",grillemelimelo->hauteur);
  fflush(stdout);
  for (i=0;i<(grillemelimelo->largeur)*(grillemelimelo->largeur);i++)
  {
    for(j=0;j<8;j++)
    {
      printf("\n**************\n");
      printf("DIRECTION : %d\n", j);
      printf("Sousgrille : %s ; ",(((((grillemelimelo->x)+i)->y)+j)->listeCars));
      pind=(((((grillemelimelo->x)+i)->y)+j)->listeIndicesCars);
      while(*pind!=EOF_LIST)
	printf("%d ",*pind++);
      printf("\n");
    };
    fflush(stdout);
  }
  FIN DEBUG*/
  free(grillemelimelo);
  return 1;
}

GrilleMotsMeles_t *initGrilleMotsMeles(FILE *fichierGrille)
{
GrilleMotsMeles_t *p;
SousGrille_t *tmp;
int i,j;

  p=(GrilleMotsMeles_t *)malloc(sizeof(GrilleMotsMeles_t));
  p->grille=initCaracteres(fichierGrille,&(p->hauteur),(&p->largeur));
/*DEBUG*/
printf("Apres initCaracteres\n%s\n",p->grille);
fflush(stdout);
/*FIN DEBUG*/
  p->x=calloc(p->hauteur*p->largeur,sizeof(SousGrilleParCar_t *));
/*DEBUG*/
printf("Dans intiGrilleMotsMeles\n");
printf(" hauteur %d largeur %d\n",p->hauteur,p->largeur);
fflush(stdout);
/*FIN DEBUG*/
  for(i=0;i<p->hauteur*p->largeur;i++)
  {
    if(i>50){printf("|%d|",i);fflush(stdout);}
    p->x[i].y=calloc(8,sizeof(SousGrille_t *));
    for(j=0;j<8;j++)
    {
      if (i>56) {printf("%d ",j);fflush(stdout);}
      tmp=initElementSousGrille(p->grille,i,p->hauteur,p->largeur,j);
      if (i>56) {printf("a ");fflush(stdout);}
      memcpy((p->x[i].y+j),tmp,sizeof(SousGrille_t *));
      if (i>56) {printf("b ");fflush(stdout);}
      free(tmp);
      if (i>56) {printf("c ");fflush(stdout);}
    };
    if(i>56){printf("x\n");fflush(stdout);}
  };
  /*p->x=initTabSousGrilleParCar(p->grille,p->hauteur,p->largeur);*/
  return p;
}

char *initCaracteres(FILE *f,int *hauteur,int *largeur)
{

int j,k;
char *p,*tmp;
char c;
long taille;
  tmp=calloc(MAXGRILLE,sizeof(char));
  j=0;
  taille=0;
  *largeur=0;
  for (k=0;(c=getc(f))!=EOF; k++)
    if (c!='\n')
    {
      j++;
      taille++;
      *(tmp+k)=toupper(c);
    }
    else
    {
      *largeur=j;
      j=0;
      k--; /*on ne met pas '\n' dans le tableau*/
    };
  *(tmp+taille)='\0';
  *hauteur=taille/(*largeur);

  p=calloc(taille+1,sizeof(char));
  assure((void *)p,"initCaracteresGrille:erreur calloc ");
  strcpy(p,tmp);
  free(tmp);
  return p;
}

SousGrille_t *initElementSousGrille(char *grille,int i, int h, int l, int numDir)
{
char *tmpCar;
int k,increment;
SousGrille_t *p;
/*DEBUG*/ int i_=i;/*FIN DEBUG*/
/*DEBUG*/
if(i>56)
{
printf("\nDans initElementSousGrille 1: Lettre %d Direction = %d\n",i,(int)numDir);
fflush(stdout);
};
/*FIN DEBUG*/
    /*DEBUGprintf("avant malloc\n");fflush(stdout);*/
    p=malloc(sizeof(SousGrille_t));
    /*DEBUG*/if (i_>56) {printf("apres malloc: sizeof(object): %d\n",sizeof(SousGrille_t));fflush(stdout);}
    assure(p,"initElementSousGrille: erreur malloc");
/*DEBUG
if(i>56)
{
printf("\nDans initElementSousGrille 2");
fflush(stdout);
};
FIN DEBUG*/
    p->indice=i;
    p->sens=numDir;
    increment=incr1[numDir]*l+incr2[numDir];
/*DEBUG
if(i>56)
{
printf("\nDans initElementSousGrille 3");
fflush(stdout);
}
FIN DEBUG*/
    tmpCar=calloc(DIM_MAX,sizeof(char));
    k=0;
    do
    {
      tmpCar[k++]=*(grille+i);
      i+=increment;
      if(incr2[numDir]>0 && i%l==0)
	break;
      else if(incr2[numDir]<0 && i%l==l-1)
	break;

    } while (i>=0 && i<l*h);
    tmpCar[k]='\0';
/*DEBUG*/if(i_>56){printf("\nDans initElementSousGrille 4, k= %d",k);fflush(stdout);}
    p->listeCars=calloc(k,sizeof(char));
/*DEBUG*/if(i_>56){printf(" &p= %p",p->listeCars);fflush(stdout);}
    assure(p->listeCars,"initSousGrille:erreur calloc");
    strcpy(p->listeCars,tmpCar);
	free(tmpCar);
/*DEBUG*/if(i_>56){printf("\nDans initElementSousGrille 6");fflush(stdout);}

    /*printf("i= %d : %s\n",increment,p->listeCars);fflush(stdout);*/
    return p;
}





void assure(void *ptr, const char *msg)
{
/*DEBUG
printf("Dans assure\n");
fflush(stdout);
FIN DEBUG*/

  if (ptr==NULL)
  {
    fprintf(stdout,"\n");
    fprintf(stdout,msg);
    fprintf(stdout,"\n");
    fflush(stdout);
    exit(EXIT_FAILURE);
  }
}

Merci d'avance,

Olivier

09/11/2014, 12h00
Sve@r

Bonjour

Ton code sent le code d'un habitué C donc je présume que t'as déjà éliminé les hypothèses bateau comme par exemple k nul ou autre. Toutefois t'as pas une indentation super agréable à relire !!!
Or, un code déjà proprement écrit c'est un code à 50% débuggué !!!

Perso je fais l'hypothèse d'un effet de bord dû à un comportement indéterminé en amont. Par exemple ta boucle dans laquelle tu remplis tmpCar[k++] lui-même alloué à 60 caractères. N'est-il pas possible qu'il en reçoive plus de 60 ?

Sinon est-ce que t'as essayé de remplacer ton calloc par malloc voir si ça fait pareil ? Et déplacer ton calloc avant cette boucle ?

Au fait, tu peux remplacer tmpCar[k++]=*(grille+i) par tmpCar[k++]=grille[i] ça a aussi vite et c'est bien plus lisible. C'est vrai que passer par un pointeur va plus vite qu'un indice de tableau, mais c'est à condition que le pointeur pointe déjà sur l'adresse voulue. Si t'avais écrit tmpCar[k++]=*pt avec pt qui s'incrémente je veux bien mais dans ton cas, tu as un pointeur invariant et tu lui ajoutes un offset, ben tu fais exactement ce que fait grille[i]...

Et sinon je ne comprends pas pourquoi, dans ta fonction initCaracteres, tu alloues et remplis tmp, puis tu alloues p, puis tu recopies tmp dans p pour libérer tmp et renvoyer p. :?
09/11/2014, 12h32
Médinoc
Et surtout, que veux-tu dire par "s'arrête"?
- Le programme freeze?
- Le programme retourne à l'invite de commandes sans le moindre message d'erreur?
- Le programme crashe avec un message d'erreur que tu aurais connement oublié de nous communiquer?

Citation:

Envoyé par Sve@r

Bonjour
Ton code sent le code d'un habitué C donc je présume que t'as déjà éliminé les hypothèses bateau comme par exemple k nul ou autre.

Habitué C... habitué aux mauvaises pratiques paraît-il. Mais pour preuve notre échange, je m'améliore de jour en jour...

Citation:

Toutefois t'as pas une indentation super agréable à relire !!!
Or, un code déjà proprement écrit c'est un code à 50% débuggué !!!

Tu as raison, une des mauvaises pratiques. J'ai fait un effort pour nettoyer et espacer mon code (cfr ci-dessous).

Citation:

Perso je fais l'hypothèse d'un effet de bord dû à un comportement indéterminé en amont. Par exemple ta boucle dans laquelle tu remplis tmpCar[k++] lui-même alloué à 60 caractères. N'est-il pas possible qu'il en reçoive plus de 60 ?

Absolument pas. Le 60 est la dimension maximale de la grille de mots mêlés. L'input "tableau.txt" contient une grille 25*24. La plus grande grille commerciale est 50*25. Dans le cas du bug, k=3. Je demande au code d'afficher k juste avant le calloc, et il imprime k=3 (cfr listing de l'output).
J'ai mis un printf dans la boucle pour le cas k>30 et rien ne sort.

Citation:

Sinon est-ce que t'as essayé de remplacer ton calloc par malloc voir si ça fait pareil ? Et déplacer ton calloc avant cette boucle ?

Oui, c'est malheureusement pareil avec malloc.
Ouf! Un indice: si je fais p->listeCars=calloc(60,sizeof(char)) AVANT la boucle, le programme s'exécute correctement = sans arrêt inopiné.
Mais la boucle permet de déterminer k bien entendu, alors il faut la mettre après.

Citation:

Au fait, tu peux remplacer tmpCar[k++]=*(grille+i) par tmpCar[k++]=grille[i] ça a aussi vite et c'est bien plus lisible. C'est vrai que passer par un pointeur va plus vite qu'un indice de tableau, mais c'est à condition que le pointeur pointe déjà sur l'adresse voulue. Si t'avais écrit tmpCar[k++]=*pt avec pt qui s'incrémente je veux bien mais dans ton cas, tu as un pointeur invariant et tu lui ajoutes un offset, ben tu fais exactement ce que fait grille[i]...

Bien vu. Je n'ai que les livres pour m'aider et dès que je trouve une formulation, je ne me risque pas à en faire une autre. Le code a été adapté.

Citation:

Et sinon je ne comprends pas pourquoi, dans ta fonction initCaracteres, tu alloues et remplis tmp, puis tu alloues p, puis tu recopies tmp dans p pour libérer tmp et renvoyer p. :?

[/QUOTE]

Merci pour le conseil, cela n'avait en effet que peu de sens. J'ai déclaré char tmp[MAXGRILLE]. Il y avait le même style dans initSousGrille(). J'ai corrigé les deux, c'est plus élégant.

Voilà le code réindenté et allégé des fonctions debug mises en commentaires:
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 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 #include <stdlib.h> #include <stdio.h> #include <string.h> #include <ctype.h> #define MAX_T 100 #define DIM_MAX 60 #define EOF_LIST -1 #define MAXGRILLE 1500 /*TYPES*/ const short incr1[8]= {-1,0,1, 0,-1,1, 1,-1}; const short incr2[8]= { 0,1,0,-1, 1,1,-1,-1}; enum DIRECTION {NORD=0,EST,SUD,OUEST,NORDEST,SUDEST,SUDOUEST,NORDOUEST}; typedef int indice_t; typedef struct { char *listeCars; enum DIRECTION sens; indice_t indice; } SousGrille_t; typedef struct { SousGrille_t *y; } SousGrilleParCar_t; typedef struct { char *grille; SousGrilleParCar_t *x; int largeur; int hauteur; } GrilleMotsMeles_t; /*PROTOTYPES*/ GrilleMotsMeles_t *initGrilleMotsMeles(FILE *); char *initCaracteres(FILE *,int *,int *); SousGrilleParCar_t *initTabSousGrilleParCar(char *,int,int); SousGrille_t *initTabSousGrille(char *,int,int,int); SousGrille_t *initElementSousGrille(char *,int,int,int,int); void assure(void *, const char *); int main() { GrilleMotsMeles_t *grillemelimelo; FILE* fGrille; freopen("DEBUGT.TXT","w",stdout); printf("Début programme\n"); fGrille=fopen("tableau.txt","r"); grillemelimelo=initGrilleMotsMeles(fGrille); free(grillemelimelo); return 1; } GrilleMotsMeles_t *initGrilleMotsMeles(FILE *fichierGrille) { GrilleMotsMeles_t *p; SousGrille_t *tmp; int i,j; p=(GrilleMotsMeles_t *)malloc(sizeof(GrilleMotsMeles_t)); p->grille=initCaracteres(fichierGrille,&(p->hauteur),(&p->largeur)); /*DEBUG*/printf("Apres initCaracteres\n%s\n",p->grille);fflush(stdout);/*FIN DEBUG*/ p->x=calloc(p->hauteur*p->largeur,sizeof(SousGrilleParCar_t *)); /*DEBUG*/printf("Dans intiGrilleMotsMeles\n");printf(" hauteur %d largeur %d\n",p->hauteur,p->largeur);fflush(stdout);/*FIN DEBUG*/ for(i=0;i<p->hauteur*p->largeur;i++) { if(i>50){printf("|%d|",i);fflush(stdout);} p->x[i].y=calloc(8,sizeof(SousGrille_t *)); for(j=0;j<8;j++) { /*DEBUG*/if (i>56) {printf("%d ",j);fflush(stdout);} tmp=initElementSousGrille(p->grille,i,p->hauteur,p->largeur,j); /*DEBUG*/if (i>56) {printf("a ");fflush(stdout);} memcpy((p->x[i].y+j),tmp,sizeof(SousGrille_t *)); /*DEBUG*/if (i>56) {printf("b ");fflush(stdout);} free(tmp); /*DEBUG*/if (i>56) {printf("c ");fflush(stdout);} }; if(i>56){printf("x\n");fflush(stdout);} }; return p; } char *initCaracteres(FILE *f,int *hauteur,int *largeur) { int j,k; char *p,tmp[MAXGRILLE]; char c; long taille; j=0; taille=0; *largeur=0; for (k=0;(c=getc(f))!=EOF; k++) if (c!='\n') { j++; taille++; tmp[k]=toupper(c); } else { *largeur=j; j=0; k--; /*on ne met pas '\n' dans le tableau*/ }; *(tmp+taille)='\0'; *hauteur=taille/(*largeur); p=calloc(taille+1,sizeof(char)); assure((void *)p,"initCaracteresGrille:erreur calloc "); strcpy(p,tmp); return p; } SousGrille_t *initElementSousGrille(char *grille,int i, int h, int l, int numDir) { char tmpCar[DIM_MAX]; int k,increment; SousGrille_t *p; /*DEBUG*/ int i_=i;/*FIN DEBUG*/ /*DEBUG*/if(i>56){printf("\nDans initElementSousGrille 1: Lettre %d Direction = %d\n",i,(int)numDir);fflush(stdout);};/*FIN DEBUG*/ p=malloc(sizeof(SousGrille_t)); assure(p,"initElementSousGrille: erreur malloc"); p->indice=i; p->sens=numDir; increment=incr1[numDir]*l+incr2[numDir]; k=0; do { tmpCar[k++]=grille[i]; i+=increment; if(incr2[numDir]>0 && i%l==0) break; else if(incr2[numDir]<0 && i%l==l-1) break; } while (i>=0 && i<l*h); tmpCar[k]='\0'; /*DEBUG*/if(i_>56){printf("\nDans initElementSousGrille 4, k= %d",k);fflush(stdout);} p->listeCars=calloc(k,sizeof(char)); /*DEBUG*/if(i_>56){printf(" &p= %p",p->listeCars);fflush(stdout);} assure(p->listeCars,"initSousGrille:erreur calloc"); strcpy(p->listeCars,tmpCar); return p; } void assure(void *ptr, const char *msg) { if (ptr==NULL) { fprintf(stdout,"\n"); fprintf(stdout,msg); fprintf(stdout,"\n"); fflush(stdout); exit(EXIT_FAILURE); } }
Voici l'output on voit bien que k=3 avant le calloc... :
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 Début programme Apres initCaracteres TRENTEETUNMAUCHEMINDEFERDETTEHCRUOFLALENOCRAGXUEIVNREHCIUQSEEANTCOMBINAISONEULFEVURXTNCAIEBISEAUTEEOUEOLGSIAECAMPSLRUEUQRAMRIQTJADTNNENEBUSELNBNIFSAITSERMIDNLGRVELUEFEABDLSRAANNABROOIDTRIQEOTBUNQOUTNRANMEBTLUOVEAERELTISCRRIEALOIUAOPUUPTTCREILOCEEANTLEBSSPLTOOTUEUHOBSILTNNGRCRNEEIALIARRBCNRENURLTCAAEBBNCTHOERETAEURQSIRUIELPDRAAOASSURERSSMCUCSOCNUECEUTURDFCUVERSAEAHETUIOMOLCETGBOLROLEIENNEDAOGAPNBBODRIENAAERFRTTUSSVRCAFATEUNENINMPMZESTRONCAELEREHRVACRLEBESIGUEUAUEOMRESPDBEMNALUETIMRAMSRCREENSMSOMREOMERUTSURCOUPERNLUOAAAOTDNILBREVOUDLERHCUEIRCIGHEENITNOTELGNISREVERTAUQRNCNAINJAUNEIREHCIRTIMPASSEE Dans intiGrilleMotsMeles hauteur 24 largeur 25 |51||52||53||54||55||56||57|0 Dans initElementSousGrille 1: Lettre 57 Direction = 0 Dans initElementSousGrille 4, k= 3 &p= 60360C80a b c 1 Dans initElementSousGrille 1: Lettre 57 Direction = 1 Dans initElementSousGrille 4, k= 18 &p= 60360C90a b c 2 Dans initElementSousGrille 1: Lettre 57 Direction = 2 Dans initElementSousGrille 4, k= 22 &p= 60360CB0a b c 3 Dans initElementSousGrille 1: Lettre 57 Direction = 3 Dans initElementSousGrille 4, k= 8 &p= 60360CD0a b c 4 Dans initElementSousGrille 1: Lettre 57 Direction = 4 Dans initElementSousGrille 4, k= 3 &p= 60360CE0a b c 5 Dans initElementSousGrille 1: Lettre 57 Direction = 5 Dans initElementSousGrille 4, k= 18 &p= 60360CF0a b c 6 Dans initElementSousGrille 1: Lettre 57 Direction = 6 Dans initElementSousGrille 4, k= 8 &p= 60360D10a b c 7 Dans initElementSousGrille 1: Lettre 57 Direction = 7 Dans initElementSousGrille 4, k= 3
Citation:

Envoyé par Médinoc

Et surtout, que veux-tu dire par "s'arrête"?

Le programme freeze?
Le programme retourne à l'invite de commandes sans le moindre message d'erreur?
Le programme crashe avec un message d'erreur que tu aurais connement oublié de nous communiquer?

1) Je lance l'exe via l'explorateur windows. Une fenêtre d'erreur : "a.exe has encountered a problem and needs to close. We are sorry for the inconvenience. (blabla..) Please tell Microsoft about the problem: Send error report/Don't send." Choix de Don't send. Je dois cliquer sur "X" puis forcer "End Now" pour fermer la fenêtre.

2) Je lance depuis command.com. Le programme freeze sans message d'erreur et sans retourner à l'invite de commandes.

09/11/2014, 13h58
Sve@r

Oui oui, j'avais bien vu que le k était créé par la boucle donc que le calloc ne pouvait se faire qu'après. C'était juste pour voir.
Sinon histoire qu'on puisse tester, tu ne voudrais pas nous passer aussi ton fichier "tableau.txt" ???
09/11/2014, 14h18
lautrec1

1 pièce(s) jointe(s)

Citation:

Envoyé par Sve@r

Oui oui, j'avais bien vu que le k était créé par la boucle donc que le calloc ne pouvait se faire qu'après. C'était juste pour voir.
Sinon histoire qu'on puisse tester, tu ne voudrais pas nous passer aussi ton fichier "tableau.txt" ???

S'il te plaît...

Pièce jointe 160961
09/11/2014, 15h20
diogene
3 erreurs à mon sens :

1- Dans initElementSousGrille()
En sortie de boucle tmpCar[k]='\0'; donc tmpCar[] à k+1 éléments avec le '\0' . Il va être copié par le strcpy(p->listeCars,tmpCar); et p->listeCars doit avoir k+1 éléments :

Code:

p->listeCars=calloc(k+1,sizeof(char));

2- Dans initGrilleMotsMeles()
Code:

1 2 3 p->x=calloc(p->hauteur*p->largeur,sizeof(SousGrilleParCar_t )); //et non pas p->x=calloc(p->hauteur*p->largeur,sizeof(SousGrilleParCar_t * ));
3- Dans initGrilleMotsMeles()
Code:

1 2 3 p->x[i].y=calloc(8,sizeof(SousGrille_t )); //et non pas p->x[i].y=calloc(8,sizeof(SousGrille_t *));

Waaaaaouh Diogene!

Que des milliers de mains travaillent de concert à ton bonheur pour l'éternité !!!!!!!

Un immensissime point +1 !!!

Code final:

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
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
 
#include <stdlib.h>
#include <stdio.h>
#include <string.h>
#include <ctype.h>
 
/*#define MAX_T 100*/
#define DIM_MAX 60
#define MAXGRILLE 1500
 
/*TYPES*/
const short incr1[8]= {-1,0,1, 0,-1,1, 1,-1};
const short incr2[8]= { 0,1,0,-1, 1,1,-1,-1};
enum DIRECTION {NORD=0,EST,SUD,OUEST,NORDEST,SUDEST,SUDOUEST,NORDOUEST};
 
 
typedef int indice_t;
 
typedef struct
{
  char *listeCars;
  enum DIRECTION sens;
  indice_t indice;
 
} SousGrille_t;
 
typedef struct
{
  SousGrille_t *y;
} SousGrilleParCar_t;
 
typedef struct
{
  char *grille;
  SousGrilleParCar_t *x;
  int largeur;
  int hauteur;
} GrilleMotsMeles_t;
 
 
/*PROTOTYPES*/
GrilleMotsMeles_t *initGrilleMotsMeles(FILE *);
char *initCaracteres(FILE *,int *,int *);
SousGrille_t *initElementSousGrille(char *,int,int,int,int);
void assure(void *, const char *);
 
 
 
int main()
{
GrilleMotsMeles_t *grillemelimelo;
FILE* fGrille;
 
  freopen("DEBUGT.TXT","w",stdout);
  printf("Début programme\n");
 
  fGrille=fopen("tableau.txt","r");
  grillemelimelo=initGrilleMotsMeles(fGrille);
 
  free(grillemelimelo);
  return 1;
}
 
GrilleMotsMeles_t *initGrilleMotsMeles(FILE *fichierGrille)
{
GrilleMotsMeles_t *p;
SousGrille_t *tmp;
int i,j;
 
  p=(GrilleMotsMeles_t *)malloc(sizeof(GrilleMotsMeles_t));
  p->grille=initCaracteres(fichierGrille,&(p->hauteur),(&p->largeur));
 
  p->x=calloc(p->hauteur*p->largeur,sizeof(SousGrilleParCar_t));
 
  /*DEBUG*/printf("\nCHARGEMENT DES DONNEES\n"); /*FIN DEBUG*/
  for(i=0;i<p->hauteur*p->largeur;i++)
  {
    p->x[i].y=calloc(8,sizeof(SousGrille_t));
    for(j=0;j<8;j++)
    {
      tmp=initElementSousGrille(p->grille,i,p->hauteur,p->largeur,j);	  
      memcpy((p->x[i].y)+j,tmp,sizeof(SousGrille_t));
      free(tmp);
	  /*DEBUG*/printf("SENS= %d, INDICE= %d SEQUENCE= %s\n",p->x[i].y[j].sens,p->x[i].y[j].indice,p->x[i].y[j].listeCars);fflush(stdout);
    };
  };
  /*DEBUG*/printf("\nDONNEES CHARGEES\n");fflush(stdout); /*FIN DEBUG*/
 
 
 
  /*DEBUG*/printf("\nIMPRESSION DES DONNEES\n");fflush(stdout); /*FIN DEBUG*/
  for(i=0;i<p->hauteur*p->largeur;i++)
  {
    for(j=0;j<8;j++)
    {
	  /*DEBUG*/printf("SENS= %d, INDICE= %d SEQUENCE= %s\n",p->x[i].y[j].sens,p->x[i].y[j].indice,p->x[i].y[j].listeCars);fflush(stdout);
    };
  };
  /*DEBUG*/printf("\nDONNEES IMPRIMEES\n");fflush(stdout); /*FIN DEBUG*/
 
  return p;
}
 
char *initCaracteres(FILE *f,int *hauteur,int *largeur)
{
 
int j,k;
char *p;
char c;
long taille;
  p=calloc(MAXGRILLE+1,sizeof(char));
  j=0;
  taille=0;
  *largeur=0;
  for (k=0;(c=getc(f))!=EOF; k++)
    if (c!='\n')
    {
      j++;
      taille++;
      p[k]=toupper(c);
    }
    else
    {
      *largeur=j;
      j=0;
      k--; /*on ne met pas '\n' dans le tableau*/
    };
  *hauteur=taille/(*largeur);
  return p;
}
 
SousGrille_t *initElementSousGrille(char *grille,int i, int h, int l, int numDir)
{
char tmpCar[DIM_MAX];
int k,increment;
SousGrille_t *p;
 
    p=malloc(sizeof(SousGrille_t));
 
    assure(p,"initElementSousGrille: erreur malloc");
 
	p->indice=i;
    p->sens=numDir;
    increment=incr1[numDir]*l+incr2[numDir];
 
    k=0;
    do
    {
      tmpCar[k++]=grille[i];
      i+=increment;
      if(incr2[numDir]>0 && i%l==0)
		break;
      else if(incr2[numDir]<0 && i%l==l-1)
		break;
 
    } while (i>=0 && i<l*h);
    tmpCar[k]='\0';
 
    p->listeCars=calloc(k+1,sizeof(char));
    assure(p->listeCars,"initSousGrille:erreur calloc");
 
    strcpy(p->listeCars,tmpCar);
 
	return p;
}
 
 
 
 
 
void assure(void *ptr, const char *msg)
{
  if (ptr==NULL)
  {
    fprintf(stdout,"\n");
    fprintf(stdout,msg);
    fprintf(stdout,"\n");
    fflush(stdout);
    exit(EXIT_FAILURE);
  }
}