Discussion :

[salseropom]

Bonjour, j'ai encore un pb avec mes #define. Voici mon code :

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
 
#include<stdlib.h>
#include<stdio.h>
#include<assert.h>
 
#ifndef MSG_ERROR
#define MSG_ERROR fprintf(stderr,"Error in file %s in function %s at line %u:\n",__FILE__,__FUNCTION__,__LINE__)
#else
#error Constant MSG_ERROR is already defined. End of compilation
#endif
 
#ifndef EXIT_PRGM
#define EXIT_PRGM exit(EXIT_FAILURE)
#else
#error Constant EXIT_PRGM is already defined. End of compilation
#endif
 
#ifndef ERROR_ALLOC_MEM
#define ERROR_ALLOC_MEM MSG_ERROR;fprintf(stderr,"Error allocation memory\nExit program\n");exit(EXIT_FAILURE)
#else
#error Constant ERROR_ALLOC_MEM is already defined. End of compilation
#endif
 
 
 
int main()
{
 
  double * p=malloc(3*sizeof(*p));
  //if(p==NULL) ERROR_ALLOC_MEM;
  assert(p!=NULL);
 
  p[0]=1; p[1]=2; p[2]=3;
  printf("%f\t%f\t%f\n",p[0],p[1],p[2]);
  return(EXIT_SUCCESS);
}

ici la sortie est correcte :

1.000000 2.000000 3.000000

en revanche si mon main devient

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
int main()
 {
 
   double * p=malloc(3*sizeof(*p));
   if(p==NULL) ERROR_ALLOC_MEM;
   //assert(p!=NULL);
 
   p[0]=1; p[1]=2; p[2]=3;
   printf("%f\t%f\t%f\n",p[0],p[1],p[2]);
   return(EXIT_SUCCESS);
 }

j'obtiens la sortie :

Error in file cste.c in function main at line 30:
Error allocation memory
Exit program

savez-vous pourquoi ?
Merci

[fearyourself]

Voilà pourquoi ne pas cacher du code dans des macros...

Il faudrait mettre des {} autour de ta macro dans ton main...

Mais je te déconseille cette pratique...

Jc

[salseropom]

OK d'accord, j'ai changé mon main en

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
int main()
{
 
  double * p=malloc(3*sizeof(*p));
  if(p==NULL) {ERROR_ALLOC_MEM;}
  //assert(p!=NULL);
 
  p[0]=1; p[1]=2; p[2]=3;
  printf("%f\t%f\t%f\n",p[0],p[1],p[2]);
  return(EXIT_SUCCESS);
}

c'est vrai que c'est vraiment laid !
mais pourquoi les accolades ?
Je prends ton conseil : pas de code dans mes macros !

[crocodilex]

mais pourquoi les accolades ?

Remplaces la macro qui se trouve dans le main par son contenu et tu auras la réponse...

[gege2061]

mais pourquoi les accolades ?

Parce qu'une fois le préprocesseur passé tu obtiens (avec l'indentation correcte) :

Code :

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

1
2
3
4
5
 
  if(p==NULL)
    fprintf(stderr,"Error in file %s in function %s at line %u:\n",__FILE__,__FUNCTION__,__LINE__);
  fprintf(stderr,"Error allocation memory\nExit program\n");
  exit(EXIT_FAILURE);

Alors que tu veux :

Code :

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

1
2
3
4
5
6
7
 
  if(p==NULL)
  {
    fprintf(stderr,"Error in file %s in function %s at line %u:\n",__FILE__,__FUNCTION__,__LINE__);
    fprintf(stderr,"Error allocation memory\nExit program\n");
    exit(EXIT_FAILURE);
  }

Pour éviter les accolades dans le code, tu peux aussi définir la macro ainsi :

Code :

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

1
2
3
4
5
6
#define ERROR_ALLOC_MEM do \
{ \
    MSG_ERROR; \
    fprintf(stderr,"Error allocation memory\nExit program\n"); \
    exit(EXIT_FAILURE); \
} while (0)

[Emmanuel Delahaye]

Bonjour, j'ai encore un pb avec mes #define. Voici mon code :
savez-vous pourquoi ?

Ben oui.

• Regle #1 Pas de source qui dépasse 80 colonnes :

Code :

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

1
2
3
4
5
6
7
8
9
 
#ifndef ERROR_ALLOC_MEM
#define ERROR_ALLOC_MEM\
MSG_ERROR;\
fprintf(stderr,"Error allocation memory\nExit program\n");\
exit(EXIT_FAILURE)
#else
#error Constant ERROR_ALLOC_MEM is already defined. End of compilation
#endif

• Regle #2 Si tu mets du code dans une macro, il doit être monolithique:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
#ifndef ERROR_ALLOC_MEM
#define ERROR_ALLOC_MEM\
do
{
   MSG_ERROR;\
    fprintf(stderr,"Error allocation memory\nExit program\n");\
   exit(EXIT_FAILURE)\
}\
while (0)
#else
#error Constant ERROR_ALLOC_MEM is already defined. End of compilation
#endif

[salseropom]

ah ah très mâlin ! merci de vos réponses !