Code problématique ??? Y a-t-il un problème avec mon destructeur?

Question

Mon code compile et s'exécute, mais on m'a dit que c'est assez problématique. Je ne comprends pas où je vais mal.

Aussi, n'est-ce pas supposé avoir tort de déclarer "char _arrName [2];" et pour faire l'assignation "_arrName [2] = '\ 0';" ? N'est-ce pas un bug "hors limites"?

#include <iostream> 
using namespace std; 
class Base { 
protected: 
char* _name; 
public: 
virtual ~Base() { cout << "Base dtor of " << _name << endl; }; 
}; 
class D1: public Base { 
char _arrName[2]; 
public: 
D1() { 
_name= _arrName; 
_arrName[0]= 'D'; 
_arrName[1]= '1'; 
_arrName[2]= '\0'; 
} 
virtual ~D1() { cout << "D1 dtor" << endl; } 
}; 
int main() { 
Base* arr[2]; 
arr[0]= new D1(); 
delete arr[0]; 
} 

Answer 1

Il n'y a rien de mal avec votre code. C'est peut-être parce que vous n'avez pas indenté votre code, ce qui le rend 100 fois plus difficile à lire. Cependant, quelques suggestions je suggérerais que vous utilisiez std :: string au lieu de chaînes de style C simples. Cela rend votre travail beaucoup plus facile.

Answer 2

Oui, il y a une erreur définie sur cette ligne.

_arrName[2]= '\0'; 

_arrName est un tableau de deux char de sorte que vous ne pouvez utiliser les deux valeurs _arrName[0] et _arrName[1]. _arrName[2] est hors limites.

Il existe également un problème avec cette ligne.

virtual ~Base() { cout << "Base dtor of " << _name << endl; }; 

Parce que la classe dérivée a souligné _name pour pointer sur un membre du groupe de la classe dérivée, par le temps ~Base() est appelé ce tableau aura été détruit et _name ne sera plus en montrant un tableau valide.

Il existe un potentiel d'erreur dans la classe de base. _name n'est jamais initialisé donc il dépend des classes dérivées l'initialisant. Ceci est moins que la conception idéale, bien que dans la pratique il peut ou ne peut pas causer un vrai problème.