Реализация аналога strcat() на c++

Question

[nikitosssguy]

Здравствуйте. Я начинающий в c++ программировании, и в обучающей литературе встретил задачу написать свой аналог функции strcat(). Я принял вызов, и вот что получилось:

#include <iostream>
#include <cstdio>
#include "stdafx.h"
#include "conio.h"

using namespace std;

void add(char *some, char *some2);
int main(){
	setlocale(0, "");
	char some[80] = "Слово";
	char some2[80] = "Второе";
	add(some, some2);
	cout << some << "\n";
	system("pause");
	return 0;
}
void add(char *some, char *some2){
	int first_length = strlen(some);
	for (int i = 0; i < strlen(some2); i++){
		*(some + first_length + i) = *(some2 + i);
	}
}

Хотелось бы узнать:
1)Насколько нормальным является мое решение с точки зрения производительности
2)Как можно сделать его короче и быстрее
3)Да и вообще ошибки, допущенные мной.
Заранее спасибо за ответы.

Answer 1

[Rsa97]

Огромный минус - а что будет, если длина some 79 символов? Куда будет копироваться some2?
Второй огромный минус - а какая будет длина строки в приведённом примере после конкатенации?
Вы не записали хвостовой 0 в результирующую строку.
Большой минус - перевычисление указателей и длины some2 в каждой итерации цикла. Исходные значения надо рассчитывать до цикла, в цикле делать только инкремент/декремент.
Маленький минус - нет смысла считать длину some2. Достаточно просто идти до символа с кодом 0.

Comments

[nikitosssguy]

Насчет первого огромного минуса ясно, но как записать хвостовой 0? Как реализовать то, что перечислено в большом минусе? А насчет маленького минуса - точно.
Извините за такое кол-во вопросов, хотелось бы заполнить некоторые пробелы на стадии их зарождения.

[Rsa97]

char *add(char *part1, char *part2) {
  int len;
  char *p1, *p2, *res;
    for (len = 0, p1 = part1; *p1 != 0; len++, p1++);
    for (p2 = part2; *p2 != 0; len++, p2++);
    try {
        res = new char[++len];
    } catch (...) {
        return NULL;
    }
    for (p1 = res, p2 = part1; *p2 != 0; p1++, p2++)
        *p1 = *p2;
    for (p2 = part2; *p2 != 0; p1++, p2++)
        *p1 = *p2;
    *p1 = 0;
    return res;
}