Discussion :

[salseropom]

Bonjour, à chaque fois que l'on fait un malloc, il faut immédiatement faire un assert(pointeur!=NULL); Peut-on faire une macro du genre

Code :

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

1
2
 
#define MALLOCN(p,N) assert(N>0); type * p=malloc(N*sizeof(*p)); assert(p!=NULL)

en gros c'est une idée, mais comment définir le type ?

Merci.

[Emmanuel Delahaye]

Bonjour, à chaque fois que l'on fait un malloc, il faut immédiatement faire un assert(pointeur!=NULL); Peut-on faire une macro du genre

Code :

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

1
2
 
#define MALLOCN(p,N) assert(N>0); type * p=malloc(N*sizeof(*p)); assert(p!=NULL)

en gros c'est une idée, mais comment définir le type ?

Merci.

Ce n'est pas une bonne idée, parce que un malloc() qui échoue n'est pas forcément une erreur grave. L'idée des concepteurs du C est de laisser l'application prendre les bonnes décisions.

Un exemple d'usage massif de malloc() pour construire des objets plus ou moins complexes est donné ici:

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

En aucun cas, assert() n'est utilisé pour quitter brutalement (sauf dans les endroits où la condition ne doit pas se produire, et serait le résultat d'un mauvais codage, notamment dans les fonctions privées dont l'appelant est parfaitement connu, donc hors 'callbacks).

[fearyourself]

Je trouve que cela serait une mauvaise idée. Premièrement, faire trop de macro est une façon de masquer la programmation sous-jacente et cela aura une tendance à faire oublier ce qu'il faut faire derrière un malloc.

Deuxièmement, lorsqu'il faut lire du code, il faut pouvoir le lire comme si c'est un livre, dès l'utilisation de macro avec paramètres (si cela est mal fait), le code devient illisible...

Finalement, cela n'ajoute pas grand chose et le temps d'écrire

"type *p = malloc(N*sizeof(*p)); assert(p!=NULL);" m'a pris moins de 20 secondes...

Mais bon, en supposant que mes mises en garde (fondées ou non) seront suivies, tu peux très bien passés le type en paramètre à la macro:

Code :

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

1
2
3
 
#define MYMALLOC(type,var,size) type* var = malloc((sizeof *var)*size);\
                                assert (var!=NULL);

en l'utilsant comme ceci:

Code :

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

1
2
 
MYMALLOC(int,tableau,n);

Remarque que le dernier ";" de la macro n'est pas nécessaire mais je préfère le mettre...

Jc

[salseropom]

bon ben j'oublie mon idée alors. Merci de vos remarques.

[Thierry Chappuis]

Je rejoins les les commentaires de mes prédécesseurs. Récemment, j'ai commencé à étudier les sources d'un logiciel libre pour en étudier la structure, la conception, etc. Perso, je trouve que moins on utilise de directives pré-processeur, plus le code est lisible. Dans ton cas particulier, je ne crois pas qu'il y a avantage déterminant à écrire une macro.

Meilleures salutations

Thierry