Allocation de pointeur dans le constructeur ou pas ?

Bonjour tout le monde,

Je souhaiterais savoir quelquechose qui m'intrigue assez, faut-il allouer dynamiquement un pointeur appartenant à une classe dans le constructeur ou faut-il juste l'initialiser a NULL ? Je vous donne un petit exemple :

classe.hpp

Code:

---

1 2 3 4 5 6

class A{ public: A(); private: int* p; };

---

classe.cpp

Code:

---

1 2 3

A::A(){ p=new int; OU p=NULL; ? }

---

Merci d'avance.
Beulard.

Salut,

Ça dépend de ce que tu veux en faire...

Au passage il vaut mieux utiliser la liste d'initialisation.

MAT.

Salut, et bienvenue sur le forum

Les deux sont possibles, en fonction de ce que tu veux faire précisément.

Mais ton exemple prête à confusion à cause de l'utilisation d'un int, ce qui rend encore la réponse plus compliquée :aie:

Si tu envisage d'utiliser le pointeur pour allouer dynamiquement un tableau C style, le meilleur conseil que tu recevra sera... d'éviter cette méthode au profit de la classe vector, disponible dans l'espace de noms std (comme tout ce qui est fourni par le standard) par simple inclusion du fichier d'en-tête <vector>.

Cette classe présente tous les avantages des tableaux "C style" sans en présenter les inconvénients.

Si le recours à un pointeur se justifie, il y a deux grandes possibilités:

• Soit, tu sais exactement ce que tu dois créer, et il est alors préférable de le créer directement
• Soit tu ne sais pas ce que tu dois créer, et il est alors préférable d'initialiser ton pointeur à NULL, car l'invocation de delete sur NULL est garantie sans effet.

Cependant, si ton pointeur a pour but de mettre l'instance de ta classe en relation avec une autre instance (de la même classe ou non), qui peut exister ou non, il y a fort à parier que la dite instance aura été créée par ailleurs, et qu'il s'agira donc de transmettre un pointeur vers celle-ci (avec, le cas échéant, une valeur par défaut à NULL) au constructeur, et d'initialiser directement le pointeur avec celui reçu en argument ;)

En bref: aide nous à t'aider : plus tu nous donnera d'informations, plus nous pourrons répondre précisément à ta question, qui, en l'état, est malheureusement trop vague pour que nous puissions le faire ;)

Bonjour, et merci pour vos réponses.

D'accord, prenons un exemple concret.
En copiant sur un tutoriel, donnons une classe Personnage qui aurait dans ses attributs un pointeur sur une instance de la classe Arme.

Disons que si l'on lance le "jeu", le Personnage créé dans le main n'a aucune arme de départ. Cet attribut sera bien sûr modifié au fil du temps. Dans le constructeur de Personnage, doit-on initialiser le pointeur sur Arme à NULL ou doit-on directement l'allouer dynamiquement ?

J'éspère avoir été plus clair cette fois. :ccool:

Merci d'avance,
Beulard.

Ton personnage est il sensé avoir directement un arme, fusse-t-elle de base, ou devra-t-il commencer par se battre à mains nues pour obtenir la moindre branche pouvant servir d'arme:question:

S'il est sensé voir directement une arme de base, s'agit il d'une arme pré définie, ou y a-t-il une quelconque sélection de l'arme effectuée (ex: un guerrier commence avec une épée, un archer avec un arc et un magicien avec un baton) :question:

Si tu réponds à ces deux questions, tu réponds à celle de l'initialisation de ton pointeur ;):

S'il commence tout à fait désarmé, tu initialisera le pointeur à NULL, s'il a déjà une arme de base en commençant, ou bien fournis un pointeur dessus dans le constructeur et tu initialise donc le pointeur avec celui fourni comme argument, ce qui permet de "sélectionner" l'arme de départ, ou bien tu effectue l'allocation dynamique de la mémoire directement dans le constructeur (ou plutôt dans la liste d'initialisation), si tout le monde commence avec la même ;)

Comme tu peux le voir, même en présentant une situation qui peut sembler claire, il reste un certain nombre de questions auxquelles il peut être utile d'apporter une réponse, de manière à être tout à fait certain du choix que tu fera en définitive ;)

C'est d'ailleurs le propre de la programmation de manière générale: il faut toujours être particulièrement attentif à se poser les bonnes questions, de manière à être tout à fait sur de la situation tout à fait particulière dans laquelle on se trouve ;)

Ou tu instancies une arme de type "pas d'arme". ;)

MAT.

Citation:

---

Envoyé par Mat007

Ou tu instancies une arme de type "pas d'arme". ;)

MAT.

---

LOLLL Aussi... disons que cela fait bizarre à écrire et à lire, et, bien que ce soit tout à fait cohérent en programmation, cela peut faire douter les débutants de la santé mentale de ceux qui leur répondent:D

Mais c'est, effectivement, une solution qui permettrait d'éviter d'avoir un pointeur NULL, et donc d'éviter de devoir tester le pointeur pour déterminer ce qu'utilise le personnage pour se battre ;)

MAT à raison, si tu n'as pas d'armes, la plupart du temps tu as "arme à main nue" au minimum sauf si il est manchot.

Bonjour,

Je suis d'accord avec vous sur le fait que l'on mettrait plutôt "Mains nues" si le personnage n'a pas d'arme, mais mon exemple n'etait qu'un exemple et je ne comptait pas l'utiliser dans mon programme, je l'ai ecrit de cette façon seulement pour simplifier le problème.

Sinon, merci à vous tous de m'avoir aidé, je crois avoir mieux compris tout ça, et il ne semble plus y avoir de memory leaks dans mon programme à cause d'initialisations hasardeuses. :aie:

Je vous dis salut et à bientôt ! ;)
Beulard.