strcopy
не должна ни выделять, ни очищать. В функциях очищать память передаваемых аргументов вообще нельзя. Например должен работать такой код (в начале строки хотите записать длину строки) :
const char* src = "hello"
size_t len = strlen(src)
char* dst = new char[len+9];
*reinterpret_cast<uint64_t*>(dst) = len;
strcopy(dst + 4, src);
И на delete[] dest;
в strcopy у вас всё упадет. Когда вы пишете функцию, вы НЕ знаете как очищать память преданного аргумента. Выделять память в функции в принципе можно, но по возможности стоит избегать. Ибо тому, кто будет в вашем коде разбираться придется выяснять, а как её потом очищать. Желательно выделять и очищать на одном уровне абстракции.
Другой пример. Иногда используют свой аллокатор. Чтобы реже использовать медленный вызов new, выделяют через new большой кусок памяти(арену), а потом просят с этой арены выдать свободный кусок. Тогда ваш new в strcopy будет вообще не к месту, а delete вообще уронит программу.
Если посмотреть требования к функции strcpy, в ответе товарища @eanmos, то реализация в с++ должна быть какой-то такой.
static size_t strlen(char* str)
{
size_t len = 0;
while (str[len] != 0) {
++len;
}
return len;
}
static char* strcpy(char* dest, const char* src)
{
for (size_t i = 0; ; ++i) {
dest[i] = str[i];
if (str[i] == 0) {
return dest;
}
}
}
count
?count
показывает кол-во элементов с "лишними" или без?char dest[10]; strcopy(dest,"какая-то хрень");
? что получится? :) А вообще - то, что вас расстраивает - решается одной единичкой:int count = strlen(src)+1;
...