Удаление динамически распределенной памяти

Question

Мне задана задача, где мне нужно создать функцию string_copy. Обратите внимание, что тело функции и прототипы были предоставлены источником и должны быть сохранены. Части, написанные мной, после комментария write your code here.

#include <iostream> 
using namespace std; 
int string_length(const char* string_c); 
char* string_copy(const char* string_c); 

int main() 
{ 
    const char* string_c = "This is a string and is a long one so that we can create memory leaks when it is copied and not deleted"; 

    // write your code here 
    int length = string_length(string_c); 
    cout << "Copied String: " << string_copy(string_c) << endl; 
    return 0; 
} 

int string_length(const char* string) { 
    int length = 0; 
    for (const char* ptr = string; *ptr != '\0'; ++ptr) { 
     ++length; 
    } 
    return length; 
} 

char* string_copy(const char* string) { 
    // we need to add 1 because of ’\0’ 
    char* result = new char[string_length(string) + 1]; 

    // write your code here (remember zero-termination !) 
    int i; 
    for (i = 0; string[i] != '\0'; ++i) 
    { 
     result[i] = string[i]; 
    } 
    result[i] = '\0'; 
    return result; 
} 

Теперь задача говорит мне

, что очень важно, чтобы любая память, выделенная с е = новым типом выпущен позже с удалением е (и = новым типом [размером] с удалением [ ] a) иначе это приведет к ошибке.

Непонятно, существует ли ошибка compile/runtime error или ошибка, как в моей задаче не соответствовала требованию ошибки.

Мой вопрос: в этом коде как удалить промежуточный динамически созданный массив result? Если я удалю result, не будет ли это результатом задачи? Тогда как я должен соблюдать приведенную выше цитату или, может быть, имитировать утечку памяти, как указано в строковой константе long?

Спасибо.

EDIT: Почему отрицательные голоса? Пожалуйста, объясните хотя бы причину! Я не прошу какого-либо решения или чего-то еще, но простое предложение, если я пропущу какой-то момент или нет!

Answer 1

Вызывающий абонент string_copy будет нести ответственность за освобождение памяти, когда это будет сделано, позвонив по телефону delete[].

Это, кстати, ужасный способ написать код на C++. Вы должны использовать std::string или std::vector<char> или что-то в этом роде.

Вот почему:

int length = string_length(string_c); 
char* copy = string_copy(string_c); 
cout << "Copied String: " << copy << endl; 
delete[] copy; 
return 0; 

Тьфу.

Answer 2

Фактически идеальным решением является использование std::string, а не char *. В вашем примере нет необходимости в using char * вместо std::string. С std::string:

• Вам не нужно ничего нового
• Вам не нужно ничего удалять
• Вы можете сделать все с станд :: строка, что вы делаете с гольца *.