Discussion :

[Emmanuel Delahaye]

J'ai pourtant alloué de la memoire dynamique:

Code :

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

1
2
     char *coded_msg; 
    coded_msg= (char *)malloc (n * sizeof (char)) ;

est horriblement et inutilement complexe :

Code :

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

1
2
 
     char *coded_msg = malloc (n) ;

http://emmanuel-delahaye.developpez....tes.htm#malloc

Code :

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

1
2
 
//coded_msg a  bien été construit!

Prouve le !

[psgman113]

alors là , c'est la meilleure!

J'ai juste mis le code là où j'affiche le contenu de la chaine dans ma fonction

donc après la ligne char msg_decoe[1000]="\0"; , je perds le contenu du msg_code !!
Alors que cette declaration de tableau de caracteres n'a rien a voir avec le msg_code!

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
void decodage_msg(char *msg_code)
{
 
 // declaration de variables que j'utilise mais je n'ai pas affiché ici
 char buf_un[9]="\0" ;
  char buf_deux[9]="\0" ;
int T_new[9],T_chai[9],T_chaine_tab[9],T_chaine[9];

 
 printf("a%s\n",msg_code);  // m'affiche quelque chose
char msg_decoe[1000]="\0";
 printf("a%s\n",msg_code); // m'affiche plus rien :-( 
}

[Freed0]

C'est quoi ce code

[Emmanuel Delahaye]

alors là , c'est la meilleure!

J'ai juste mis le code là où j'affiche le contenu de la chaine dans ma fonction
<...>

Ceci fonctionne :

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
#include <stdio.h>
#include <stdlib.h>
 
void decodage_msg (char *msg_code)
{
   printf ("a%s\n", msg_code);  // m'affiche quelque chose
   char msg_decoe[1000] = "\0";
   printf ("a%s\n", msg_code);  // m'affiche plus rien :-(
}
 
int main (void)
{
   char coded_msg[]="hello world";
 
   printf (" %s\n", coded_msg); // m'affiche une chaine de caracteres
   decodage_msg (coded_msg);
 
   return 0;
}

Code :

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

1
2
3
4
5
6
 
 hello world
ahello world
ahello world
 
Press ENTER to continue.

Si tu postais ton code véritable, ça irait beaucoup plus vite que de jouer aux devinettes...

[psgman113]

Mon code est assez enorme .. Ca ferait mal aux yeux
En fait , j'ai remarqué que le contenu de ma chaine de caractères est légèrmeent modifié entre le moment où je l'affiche avant l'appel d'une fonction et quand je l'affiche dans la fonction (pointeur passé en parametre), il y a 3 caractères qui manque dans ma chaine.

[Emmanuel Delahaye]

Mon code est assez enorme .. Ca ferait mal aux yeux<...>

Alors réduit ton code à ce qui ne va pas et poste le. Ca doit être compilable.

On est pas devins...

[psgman113]

Voici mon code qui est long,
Le truc que je comprends pas c'est quand j'affiche le contenu de ma chaine de caractère avant et après l'appel de la fonction decodage_msg , ele differe e 3 caractères . J'ai écrit Avant après , vous pouvez le voir lors de l'exécution

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
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
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
424
425
426
427
428
429
430
431
432
433
434
435
436
437
438
439
440
441
442
443
444
445
446
447
448
449
450
451
452
453
454
455
456
457
458
459
460
461
462
463
464
465
466
467
468
469
470
471
472
473
474
475
476
477
478
479
480
481
482
483
484
485
486
487
488
489
490
491
492
493
494
495
496
497
498
499
500
501
502
503
504
505
506
507
508
509
510
511
512
513
514
515
516
517
518
519
520
521
522
523
524
525
526
527
528
529
530
531
532
533
534
535
536
537
538
539
540
541
542
543
544
545
546
547
548
549
550
551
552
553
554
555
556
557
558
559
560
561
562
563
564
565
566
567
568
569
570
571
572
573
574
575
576
577
578
579
580
 
 
 
#include <stdlib.h>
#include <stdio.h>
#define P 256
 
void decodage_msg ( char * msg_deco);
void conversion_wep_iv_un (char *cle_wep);
char  *codage_msg(char *message);
char conversion_deco (char *chaine_decod, size_t size);
int conversion_letre_decimal ( int decimal,char carac);
char *saisie_wep ();
char * conversion ( int decimal);
int size;
char * saisie_msg();
int  T_deux[20],T_ini[20], T[20];
 
int size_wep;
int n;
 
 main(void) 
{
 
char *coded_msg = malloc (n) ;
char *wep= malloc (n) ;
char *msg= malloc (n) ;
 
   wep= saisie_wep();
if (wep!= NULL)
{
 
 conversion_wep_iv_un(wep);
   msg = saisie_msg();
   if (msg != NULL)
   {
 
     coded_msg = codage_msg(msg);
 
      if (coded_msg != NULL)
      {
 
                    printf("avnt %s\n",coded_msg);
         decodage_msg(coded_msg);
         free(coded_msg);
      }
      else { printf("message non code"); }
      free(msg);
   }
 
}  
   else { printf("wep  incorrect"); }
     system("pause");
 
}
 
 
void decodage_msg(char *msg_code)
{
 
int i,z,j,n,k,y,t,err;   
 
 char buf_un[9];
  char buf_deux[9];
int T_new[9],T_chai[9],T_chaine_tab[9],T_chaine[9];
 char letter;
 printf("apre %s\n",msg_code); // m'affiche plus rien :-( 
 
 char msg_decoe[1000];
 
 
i=1;
t=0;
for(z=0;z<size;z++)   // 6 c pr lexemple
{   
j = 0;
 
j= (T_deux[i]+j)%15;
 
k= T_deux[i];
T_deux[i]=T_deux[j];
T_deux[j]=k;
n = (T_deux[i] + T_deux[j]) % 15 ;
 
 
 char *chai= conversion (T_deux[n]);
 
if (chai==NULL)
 
            printf("erreur de retour");
 
                   for (y=0;y<8;y++)
     {
 
                    char s3[2] = {msg_code[t]};   // convertir un caractère en chaine puis en nombre 
                             T_chaine_tab[y]=atoi(s3);  
                            char s4[2] = {chai[y]};                                      
                             T_chaine[y]=atoi(s4);
                              T_new[y]=T_chaine_tab[y]^T_chaine[y];
 
                              sprintf(buf_un, "%d", T_new[y]); // on a un caractere 
 
 
                              strcat(buf_deux,buf_un);// concatener
                              printf("%s\n",buf_deux);
 
                                  t++;                                            
 
                                 }
                                letter= conversion_deco(buf_deux,sizeof buf_deux); // on a la lettre
 
                                 buf_deux[0]='\0';
                               printf("la lettre decodee est %c\n",letter);
 
                                char s6[2] = {letter}; 
 
 
 
                               strcat(msg_decoe,s6);// concatener
 
 
                                printf("le msg decode est %s\n",msg_decoe);
                                 printf("\n");
 
                                 i++;
                                  }     
}
 
 
char * codage_msg(char *msg)
{
     int i,j,k,y,o,n,decimal,retour;
               //  char chaine[9],chaine_tab[9];
 
                 int   T_chaine[9],T_chaine_tab[9],T_new[9];
                 char code_un[9]="\0" ;
                 char code[1000]="\0"; 
 
char  *msg_code= malloc(n);	
char  *chaine= malloc(n);	
 
if (msg_code == NULL | chaine==NULL )
{        /*   l'allocation a echoué:    */
/*    message d'erreur et arrêt du programme    */
    printf ("l'allocation a echoue\n") ;
    exit (1) ;
}
 
 
j = 0;               
i=1;
for (y=0;y<size;y++) {
 
printf("lettre du message: %c\n",msg[y]); 
 
decimal= conversion_letre_decimal(decimal, msg[y]);   
chaine= conversion ( decimal); 
 
 
if (chaine==NULL)
printf("erreur de retour"); //ici on a la  chaine binaire en var globale corespondant à un caractère du msg 
 
j= (T[i]+j)%15;
k= T[i];
T[i]=T[j];
T[j]=k;
n = (T[i] + T[j]) % 15 ;
 
 
char *chaine_tab = conversion ( T[n]);
if (chaine_tab!=NULL)
{
printf("la chaine binaire du tableau est: %s\n",chaine_tab); // chaine binaire  de l'entier du tablo
printf("la chaine binaire du caractere est: %s\n",chaine);                   // dans chaine , on a la chaine binaire du caractère du texte
}
else printf("erreur de retour");
 
     for (o=0;o<8;o++)
     {
 
   char s1[2] = {chaine_tab[o]};   // convertir un caractère en chaine puis en nombre 
   T_chaine_tab[o]=atoi(s1);
    char s2[2] = {chaine[o]}; 
   T_chaine[o]=atoi(s2);
 
  // printf("ici %d\n",T_chaine_tab[o]);
 
     T_new[o]=T_chaine_tab[o]^T_chaine[o];
    // printf("ici %d\n",T_new[o]);
  sprintf(code_un, "%d", T_new[o]); // convertit entier en caractère
 
 
    strcat(code,code_un);
       // printf("ici %s\n",code);
     }
 
 
 
  printf("construction du message code: \n");
  printf("%s\n", code);
printf("\n");
 
 
i++;
}
 
printf("\n");
 msg_code=code;
 printf("voici le msg code enn binaire %s\n",msg_code); // msg codé 
 
 printf("decodons le maintenant\n ");
printf("\n");
 
 
return msg_code;
}    
 
/// saisie de la clé wep
 
char * saisie_wep ()
{
 char *cle_wep;
     cle_wep= (char *)malloc (n * sizeof (char)) ;
 
if (cle_wep== NULL  )
{        /*   l'allocation a echoué:    */
/*    message d'erreur et arrêt du programme    */
    printf ("l'allocation a echoue\n") ;
    exit (1) ;
}
 
cle_wep="12CA234567ABC";
size_wep=strlen(cle_wep);
return cle_wep;
}
 
///////////////////////////////////////
 
// saisie du msg au clavier
char * saisie_msg()
{
 
    char * message = malloc (n) ;
 
if (message== NULL  )
{        /*   l'allocation a echoué:    */
/*    message d'erreur et arrêt du programme    */
    printf ("l'allocation a echoue\n") ;
    exit (1) ;
}
 
 message="crist";
size=0;
size=strlen (message);
 
return message;
}
 
///////////////////////////////
/*fonction de conversion de la clé wep ( char -> entier décimal ds un tablo)
fonction de génération de la clé iv
fonction de sauvegarde du tableau (clé + clé iv)
clé wep en variable globale
*/
void conversion_wep_iv_un (char *cle_wep)
{
int i,n;
char *elem, *end;
 
for (i=0;i<size_wep;i++)
{    
 if (cle_wep[i] == 'A')
elem="10";
else if (cle_wep[i] == 'B')
elem="11";
else if (cle_wep[i]== 'C')
elem="12";
else if (cle_wep[i] == 'D')
elem="13";
else if (cle_wep[i]== 'E')
elem="14";
else 
{
 char s[2] = {cle_wep[i]}; // on crée une chaine de caractères avec un seul caractère
elem=s; 
}
 
n=strtol(elem, &end, 10); // convertir la chaine elem en un entier décimal
 
if(*end == '\0')
{
  T[i]=n;  /* La chaine a enteirement pu etre convertie */ 
 }
else{
    printf("chaine de caractere incorrect\n");
   /* La chaine contenait au moins un caractere ne pouvant pas etre converti en nombre */
}
}  
 
 
   for (i = size_wep; i<size_wep+6; i++)	// on génére un tableau de nombre aléatoires (entre 0 et N ) de 26 octets
   {      
 
   T[i] =  rand() % P; 
 
}
 
 for (i = 0; i<size_wep+6; i++)	// on génére un tableau de nombre aléatoires (entre 0 et N ) de 26 octets
   {
 
T_ini[i]=T[i];
T_deux[i]=T[i];
 
}
 
}
 
////////////////////////////////////////////////
 
//fonction de conversion lettre en binaire
 
 
int conversion_letre_decimal ( int decimal,char carac)
{
 
 
if (carac == 'a')
decimal=0;
 
if (carac == 'b')
decimal=1;
 
 
if (carac == 'c')
decimal=2;
 
 
if (carac == 'd')
decimal=3;
 
 
if (carac == 'e')
decimal=4;
 
if (carac == 'f')
decimal=5;
 
if (carac == 'g')
decimal=6;
 
if (carac == 'h')
decimal=7;
 
 
if (carac == 'i')
decimal=8;
 
 
if (carac == 'j')
decimal=9;
 
 
if (carac == 'k')
decimal=10;
 
 
if (carac =='l')
decimal=11;
 
if (carac == 'm')
decimal=12;
 
if (carac == 'n')
decimal=13;
 
 
if (carac == 'o')
decimal=14;
 
 
if (carac == 'p')
decimal=15;
 
 
if (carac == 'q')
decimal=16;
 
 
if (carac == 'r')
decimal=17;
 
if (carac == 's')
decimal=18;
 
if (carac == 't')
decimal=19;
 
 
 
if (carac == 'u')
decimal=20;
 
 
if (carac == 'v')
decimal=21;
 
 
if (carac == 'w')
decimal=22;
 
if (carac == 'x')
decimal=23;
 
if (carac == 'y')
decimal=24;
 
 
if (carac == 'z')
decimal=25;
 
 
 
return decimal;
}
 
 
 
char * conversion (  int decimal)
{
char *octet;
   int i,j,l;
  int reste;
char *tempo;
tempo = (char *)malloc (n * sizeof (char)) ;
octet= (char *)malloc (n * sizeof (char)) ;
     for (i=0;i<8;i++)
          {
              octet[i]='0';
          }  
 
	octet[8]='\0';
 
 
    l=128;// l définit le premier seuil 128
 
 
 
 
if (decimal != 0)
{
 
 
   reste=decimal%2;
 
  if (reste ==1)
{
 
      octet[7]='1';
 
  decimal=decimal-1;
}
 
   else octet[0]='0';   
 
    i=0;
 
 
while ( decimal!=0)
{
      if (decimal<l)
      {
          octet[i]='0';
          l=l/2;
          i++;
      }
 
        if (decimal>l)
      {
          octet[i]='1';
	   decimal=decimal -l;
          l=l/2;
          i++;
      }
 
             if (decimal==l)
      {
          octet[i]='1';
          i++;
          for (j=i;j<7;j++)
          {
              octet[j]='0';
          }    
		octet[8]='\0';
 
                   decimal=0;
      }
}         
 
 
 
}
else {
       for (j=0;j<8;j++)
          {
              octet[j]='0';
          }    
 
     }
 
return octet;
}
//////////////////////////////////////////////////////
 
char conversion_deco (char *chaine_decod, size_t size) // transforme une chaine en un caractere
{
     char lettre;
if (!strcmp(chaine_decod,"00000000"))
lettre='a';
 
else if (!strcmp(chaine_decod,"00000001"))
lettre='b';
 
 
else if (!strcmp(chaine_decod,"00000010"))
{ 
  lettre='c';
 
}
 
else if (!strcmp(chaine_decod,"00000011"))
lettre='d';
else if (!strcmp(chaine_decod,"00000100"))
lettre='e';
else if (!strcmp(chaine_decod,"00000101"))
lettre='f';
else if (!strcmp(chaine_decod,"00000110"))
lettre='g';
else if (!strcmp(chaine_decod,"00000111"))
lettre='h';
else if (!strcmp(chaine_decod,"00001000"))
lettre='i';
else if (!strcmp(chaine_decod,"00001001"))
lettre='j';
else if (!strcmp(chaine_decod,"00001010"))
lettre='k';
else if (!strcmp(chaine_decod,"00001011"))
lettre='l';
else if (!strcmp(chaine_decod,"00001100"))
lettre='m';
else if (!strcmp(chaine_decod,"00001101"))
lettre='n';
else if (!strcmp(chaine_decod,"00001110"))
lettre='o';
else if (!strcmp(chaine_decod,"00001111"))
lettre='p';
else if (!strcmp(chaine_decod,"00010000"))
lettre='q';
else if (!strcmp(chaine_decod,"00010001"))
lettre='r';
else if (!strcmp(chaine_decod,"00010010"))
lettre='s';
else if (!strcmp(chaine_decod,"00010011"))
lettre='t';
else if (!strcmp(chaine_decod,"00010100"))
lettre='u';
else if (!strcmp(chaine_decod,"00010101"))
lettre='v';
else if (!strcmp(chaine_decod,"00010110"))
lettre='w';
else if (!strcmp(chaine_decod,"00010111"))
lettre='x';
else if (!strcmp(chaine_decod,"00011000"))
lettre='y';
else if (!strcmp(chaine_decod,"00011001"))
lettre='z';
 
else lettre='<';
 
return lettre;
}

[Emmanuel Delahaye]

Voici mon code qui est long,

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
 
Project   : Forums
Compiler  : GNU GCC Compiler (called directly)
Directory : C:\dev\forums2\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c:10: warning: function declaration isn't a prototype
main.c:13: warning: function declaration isn't a prototype
main.c:20: warning: return type defaults to `int'
main.c: In function `decodage_msg':
main.c:101: warning: implicit declaration of function `strcat'
main.c:58: warning: unused variable `err'
main.c:62: warning: unused variable `T_chai'
main.c: In function `codage_msg':
main.c:139: warning: suggest parentheses around comparison in operand of |
main.c:129: warning: unused variable `retour'
main.c: At top level:
main.c:218: warning: function declaration isn't a prototype
main.c: In function `saisie_wep':
main.c:229: warning: assignment discards qualifiers from pointer target type
main.c:230: warning: implicit declaration of function `strlen'
main.c: At top level:
main.c:238: warning: function declaration isn't a prototype
main.c: In function `saisie_msg':
main.c:249: warning: assignment discards qualifiers from pointer target type
main.c: In function `conversion_wep_iv_un':
main.c:270: warning: assignment discards qualifiers from pointer target type
main.c:272: warning: assignment discards qualifiers from pointer target type
main.c:274: warning: assignment discards qualifiers from pointer target type
main.c:276: warning: assignment discards qualifiers from pointer target type
main.c:278: warning: assignment discards qualifiers from pointer target type
main.c: In function `conversion_deco':
main.c:514: warning: implicit declaration of function `strcmp'
main.c: At top level:
main.c:511: warning: unused parameter 'size'
main.c: In function `codage_msg':
main.c:129: warning: 'n' might be used uninitialized in this function
main.c:129: warning: 'decimal' might be used uninitialized in this function
main.c: In function `main':
main.c:52: warning: control reaches end of non-void function
Linking console executable: console.exe
Process terminated with status 0 (0 minutes, 5 seconds)
0 errors, 23 warnings

Ca fout la trouille...

Si ton compilateur ne dit rien, c'est peut être qu'il est mal configuré :

http://emmanuel-delahaye.developpez....tm#cfg_compilo

Corrige tout ça, déjà, on y verra plus clair...

[thewho]

Bonjour,

Aïe !!

des globales en veux-tu, en voilà

code mal indenté

mauvaise forme pour main

des allocations par malloc faites avec une variable non initialisée, PUIS écrasées par des retours de fonction qui ont fait leur propre allocation

...

j'en passe

pas envie d'aller plus loin

[psgman113]

Je programme sous dev c++ sous windows ,
j'ai pas eu ces warnings,
mes variables globales? y en a pas tant que ca ,
je déclare juste pas mal de fonctions

[Emmanuel Delahaye]

Je programme sous dev c++ sous windows ,
j'ai pas eu ces warnings,

Alors ça y'est, tu as modifié ta config, tu as recompilé, corrigé ?

Je ne suis pas bien sûr que tu maitrises suffisamment le C pour mener à bien ton projet ... Tu ne veux pas commencer par quelque chose de plus simple ?

Au fait tu as fait les études préalables ? spécification, conception ?

http://emmanuel-delahaye.developpez.com/dev_proj.htm

[psgman113]

c'est une application que je désire realiser . EN fait, je l'ai fait dans une premier temps en utilisant des variables globales et en mettant tout le code dans le main et ca marchait!

Je veux maintenant le structurer

J'ai compilé sous fedora core et j'ai obtenu ceci en warnings

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
 
 
essai.c: In function ‘decodage_msg’:
essai.c:119: warning: incompatible implicit declaration of built-in function
 ‘st  rcat’
essai.c:134: warning: incompatible implicit declaration of built-in function ‘st  rcat’
 
essai.c: In function ‘codage_msg’:
essai.c:228: warning: incompatible implicit declaration of built-in function ‘st  rcat’
 
essai.c: In function ‘saisie_wep’:
essai.c:268: warning: incompatible implicit declaration of built-in function ‘st  rlen’
 
essai.c: In function ‘saisie_msg’:
essai.c:291: warning: incompatible implicit declaration of built-in function ‘st  rlen’

Ne vous fiez pas aux lignes car c'est une autre version du programme.


J'en ais déduis qu'il manquait un include quelque chose pour utiliser strlen et strcat .

Sinon , quand je fais ceci , il me dit que l'allocation a échoué.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
 
 
char  *msg_code= malloc(n);	
char  *chaine= malloc(n);	
 
if (msg_code == NULL | chaine==NULL )
{        /*   l'allocation a echoué:    */
/*    message d'erreur et arrêt du programme    */
    printf ("l'allocation a echoue\n") ;
    exit (1) ;
}

[Médinoc]

c'est une application que je désire realiser . EN fait, je l'ai fait dans une premier temps en utilisant des variables globales et en mettant tout le code dans le main et ca marchait!

Tu sais, c'est encore plus inutile d'utiliser des variables globales si tout le code est dans la même fonction...

[psgman113]

j'utilisais quand meme une ou deux fonctions

Sinon dans ma fonction codage_msg , je veux créer une hcaine de caractères et la retourner (je ne connais pas sa longueur avant) donc je fais ceci :
mais il me dit bien que l'allocation a échouée
pourquoi?

Code :

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

1
2
3
4
5
6
7
8
9
 
char  *msg_code= malloc(n);	
 
if (msg_code == NULL )
{        /*   l'allocation a echoué:    */
/*    message d'erreur et arrêt du programme    */
    printf ("l'allocation a echoue\n") ;
    exit (1) ;
}

[psgman113]

J'ai simplifié mon probleme,

Je construis une chaine de caractères dans une fonction , je retourne cette chaine à l'aide d'un pointeur , je veux la réafficher dans mon main et là hic:

L'allocation mémoire doit se faire dans ma fonction , dans mon main ou dans les 2 ?

Merci

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
 
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
char * decodage_msg ()
{
     int n;
 
   char *a=malloc(n);         
if (a==NULL  )
{     
/*    message d'erreur et arret du programme    */
  printf ("l'allocation a echoue\n") ;
   exit (1) ;
}
 
       char code_msg[]="helloworld\0";
 
      a=code_msg;
 
   printf ("avant %s\n", a);  // m'affiche helloworld
return a;
}
 
int main (void)
{
char * b=  decodage_msg ();
   printf ("apres %s\n", b); // m'affiche une chaine de caracteres bizarre
 
 system("pause");
   return 0;
}

[thewho]

Bonjour,

Je te l'avais déjà signalé :

allocation avec malloc, en utilisant pour préciser la taille, une variable non initialisée.

Il faut lire les réponses, sinon il est inutile de demander de l'aide

Je pense comme Emmanuel : tu as manifestement besoin de revoir les bases, en faisant l'effort de comprendre pourquoi les exemples sont écrits comme ils le sont, et pas seulement en te contentant de les copier et vérifier si ça marche.

[psgman113]

des allocations par malloc faites avec une variable non initialisée, PUIS écrasées par des retours de fonction qui ont fait leur propre allocation

Ok , j'ai initialisé n à 300 pour 300 caractères , mais ca ne change rien .

J'ai posé une question et je la répète

Je dois faire une allocation mémoire uniquement dans la fonction , uniquement dans le main ou dans les 2? Important pour éviter d'écraser l'allocation

Merci

[Médinoc]

En théorie, dans la fonction seulement.

[gl]

Code :

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

1
2
3
4
5
6
7
8
 
char *a=malloc(n);         
if (a==NULL  )
{     
/*    message d'erreur et arret du programme    */
  printf ("l'allocation a echoue\n") ;
   exit (1) ;
}

En supposant que n est correctement intialise, jusque la ca va (enfin la sortie est un peu brutale, mais bon)

Code :

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

1
2
3
 
char code_msg[]="helloworld\0";
a=code_msg;

La tu modifies la valeur du pointeur qui ne contient plus l'adresse de la zone alloue mais celle du tableau local code_msg.
Tu n'as donc plus de lien sur la zone memoire allouee ==> fuite memoire.

En passant, le \0 dans "helloworld\0" est inutile car implicite.

Code :

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

1
2
3
 
return a;
}

a ayant maintenant l'adresse de la variable locale code_msg, tu retournes l'adresse d'un tableau local qui est detruit a la sortie de la fonction. L'adresse que tu retourne est donc invalide.

Rappel: pour copier une chaine de caractere, il faut utiliser strcpy et non =.

En outre la memoire allouee n'est pas liberee. Dans cet exemple ce n'est pas tres grave puique l'on quitte quasi immediatement mais c'est une habitude a prendre, lorsque tu n'as plus besoin de la memoire allouee, il faut la liberer.

Ceci etant, une bonne partie de ces remarques on deja ete faites, il serait peut etre bon de tenir compte des remarques que font les differents contributeurs et de poser des questions si tu ne comprends pas quelque chose dans une explication.
Je t'invite egalement a relire attentivement le chapitre sur les pointeurs et allocation dynamique de ton cours ou de ton livre.

[thewho]

Bonjour,

Je dois faire une allocation mémoire uniquement dans la fonction , uniquement dans le main ou dans les 2? Important pour éviter d'écraser l'allocation

Ta question ainsi écrite contient sa propre réponse.

Pourquoi veux-tu allouer de la mémoire à 2 endroits différents pour une même variable ?

Quand tu veux un verre d'eau, le remplis-tu à la cuisine, PUIS à la salle de bain (sans l'avoir vidé au passage, bien entendu) ?

Et pour confirmer gl : il faut libérer la mémoire allouée. Le mieux, c'est de le faire dès qu'on en a plus besoin, et AU MINIMUM, à la sortie du programme.
Si tu ne prends pas cette habitude, ne viens pas plus tard te plaindre de problèmes de mémoire.