Quel serait le problème avec cette surcharge d'opérateur d'affectation

Question

Mon programme se bloque quand il essaie d'affecter un objet à l'autre, est-ce que vous voyez quelque chose de mal à cela?

Les variables sont les suivantes:

Field *fields[50]; 
int numOfFields; 
int currentField; 

La fonction est:

Screen& operator=(Screen &scr) { 
    if (this != &scr){ 
    for (int i = 0; i < 50; i++) 
     fields[i] = NULL; 

    for (int i = 0; i < scr.numOfFields; i++) 
     fields[i] = scr.fields[i]; 

    numOfFields = scr.numOfFields; 
    currentField = scr.currentField; 
    } 

    return *this; 
} 

Answer 1

Vous devez utiliser un std::vector au lieu de votre tableau dynamique et je soupçonne que votre problème disparaîtra très rapidement.

Comme suggéré par MadCap, vous devez également utiliser des pointeurs partagés car il est recommandé d'utiliser des pointeurs avec un conteneur. Cela vous donnera également la sémantique de copie que vous attendez.

Cela annulerait la nécessité de ceci:

for (int i = 0; i < 50; i++) 
     fields[i] = NULL; 

et vous pouvez le remplacer:

for (int i = 0; i < scr.numOfFields; i++) 
     fields[i] = scr.fields[i]; 

avec un std :: copie, quelque chose comme:

fields.clear() 
std::copy(scr.fields.begin(), scr.fields.end(), fields.begin()); 

Par Ce faisant, vous supprimerez la possibilité que l'incident se produise en raison d'un accès au pointeur ou d'une erreur d'initialisation.

Un conseil simple est; arrêtez d'utiliser des tableaux dynamiques et des pointeurs bruts et commencez à utiliser std::vector et boost::shared_ptr au lieu de cela, cela vous évitera de vous heurter à ce genre de problèmes.

Answer 2

Un problème peut être que scr.numOfFields dépasse le nombre de champs dans votre objet de destination.

Un autre problème est que, ou du moins il semble, vous attribuez des pointeurs à votre nouvel objet. Cela signifie que vous aurez une référence au même emplacement deux fois dans le programme. Que se passe-t-il s'il est supprimé à un endroit et que l'autre ne le sait pas? Lorsque vous essayez d'accéder à la mémoire, vous obtenez une erreur de segmentation.

Si vous avez Boost vous pouvez utiliser leurs pointeurs partagés pour éviter ceci: http://www.boost.org/doc/libs/1_43_0/libs/smart_ptr/smart_ptr.htm