En C++, Passant char array à super donner des erreurs?

Question

J'ai cette structure:

class Base 
{ 
public: 
    void doACopy(char* strToCopy) { 
     strcpy(str, strToCopy); 
    } 
private: 
    char str[4]; 
}; 

class Derived : public Base 
{ 
public: 
    void doSomething() { 
     char toCopy[4]; 
     toCopy[0] = 'a'; toCopy[1] = 'b'; toCopy[2] = 'c'; 
     Base::doACopy(toCopy); // is there any problem passing toCopy here? 
    } 
}; 

Je sais que toCopy est alloué sur la pile. Y at-il un problème à passer ce tableau à super, dans ce cas Derived :: doACopy?

Answer 1

La pile de mémoire en cours de destruction ne pose aucun problème. toCopy est valide jusqu'au doSomething renvoie, et à ce moment le strcpy est terminé.

Mais ils doivent être à la fois une longueur de 4, et vous devez NUL-fin toCopy:

private: 
    char str[4]; 

// ... 
char toCopy[4]; 
toCopy[0] = 'a'; toCopy[1] = 'b'; toCopy[2] = 'c'; 
toCopy[3] = '\0'; 

Comme il est, il n'y a pas terminateur NUL (ou même chambre), et cela va causer un débordement de mémoire tampon pendant strcpy .

Answer 2

char à Copie [4]; sera disponible et existera jusqu'à la fin de la méthode doSomething. Et concernant le problème que vous avez ... Comme Naveen l'a dit ... c'est parce que vous n'avez pas mis un caractère de fin "NULL" ... pour résoudre ce problème, vous pouvez réécrire la définition de toCopy comme suit: -

char toCopy[4] = {0}; 

Je vous recommande fortement d'utiliser la chaîne au lieu de simple tableau de charbon ... donc si nous réécrivons le code avec de nouveaux changements ... ce sera comme ça ...

#include<string> 
using std::string; 

class Base 
{ 
public: 
    void doACopy(string & strToCopy) { 
     str = strToCopy; 
    } 
private: 
    string str; 
}; 

class Derived : public Base 
{ 
public: 
    void doSomething() { 
     string toCopy = "abc"; 
     Base::doACopy(toCopy); // is there any problem passing toCopy here? 
    } 
}; 

is not easy !!!