Libération de mémoire dans le problème C++ (Visual Studio 2010)

Question

J'essaie d'apprendre C++, dans le processus j'ai essayé d'écrire une fonction qui obtient deux pointeurs de char et concaténer le second au premier (je sais qu'il y a strcat pour ce).
Mais - ce que je veux accomplir est de modifier le premier pointeur de paramètre afin qu'il pointe vers le résultat. pour cette raison j'ai utilisé une référence au pointeur dans le premier paramètre.

Avant de revenir à la fonction, je souhaite libérer la première mémoire de paramètres, mais j'obtiens une erreur.

Voici le code:

void str_cat(char*& str1, char* str2) 
{ 
if (!str1) 
{ 
    str1 = str2; 
    return; 
} 
if (!str2) 
    return; 
char * new_data = new char[strlen(str1) + strlen(str2) +1]; 
char * new_data_index = new_data; 
char * str1_index = str1; 
char * str2_index = str2; 

while(*str1_index) 
    *new_data_index++ = *str1_index++; 
while(*str2_index) 
    *new_data_index++ = *str2_index++; 
*new_data_index = NULL; 

delete str1; //ERROR HERE (I also tried delete[] str1) 

str1 = new_data; 
} 

Je ne comprends pas pourquoi.
Des suggestions?

Merci,
Itay \

EDIT Voici comment utiliser la fonction

char * str1 = NULL; 
char * str2 = NULL; 
str_cat(str1, "abc"); 
str_cat(str2, "def"); 
str_cat(str1, str2); 

Answer 1

Vous ne pouvez supprimer des choses qui ont été attribuées à nouveau - si votre code ressemblait à ceci:

str_cat("foo", "bar"); 

ce serait illégal. Fondamentalement, votre fonction telle qu'elle est est complètement dangereuse. Une meilleure conception consisterait à renvoyer la nouvelle chaîne via la valeur de retour de la fonction. Encore mieux, oubliez l'idée et utilisez std :: string.

Bien qu'apprendre à utiliser des références aux pointeurs soit une chose louable à faire, sachez qu'ils sont très rarement utilisés en programmation C++. Il vaudrait mieux que vous passiez votre temps à apprendre à utiliser les fonctionnalités de la bibliothèque standard C++.

Answer 2

Le premier appel à str_cat() résultats en str1 étant attribué l'adresse de la chaîne littérale "abc" que vous avez passé dans.
Avec le troisième appel cela devient un problème que vous essayez de deletestr1 qui, comme Neil l'a souligné, est illégal pour les littéraux de chaîne.