WW #2

[Артур Гулецкий (huletski)]

Не могу собрать решение:

{lab_02}[2300]$ pwd && svn up && svn status
/home/hfx/dvl/cpp19/kuzmina.kseniya/lab_02
Updating '.':
At revision 794.
{lab_02}[2301]$ ls
include  lab2  Makefile  obj  src
{lab_02}[2302]$ make
make: *** No rule to make target 'lab_02', needed by 'all'.  Stop.

Кроме того:

• в папке с решением лежат артефакты сборки (lab2, obj);
• кое-где в исходном коде отсутствуют отступы (например, src/str.c:16).

Чините.

[Артур Гулецкий (huletski)]

Не работает my_strcat, остальные функции тесты проходят -> +6.

Замечания по тестам:

• тестовое покрытие не полно: так, например, test_mystrcpy не проверяет копирование пустых строк; текущая версия вообще не проверяет, что записалось в s1 после вызова my_strcpy;
• сравнение строк через == (str/test_str.cpp:21): если бы можно было сравнивать так строки _по содержанию_, зачем тогда было вводить в стандарт strcmp? Кроме того, assert(result != expected) должно наводить на мысль, что не то что-то сравнивается, не так ли? -> -1;
• забыли include guard в одном из .h файлов -> -0.5.

Итог: 4.5, нужно доделывать. Если зашлете исправленное сравнение, напишите в комментарии, в чем была проблема.

[kuzmina.kseniya]

В my_strcat забыла одну переменную

[Артур Гулецкий (huletski)]

Работает, +10.

Замечания:

• разный стиль отступов (в str.c - пробелы, в test_str.c - табы);
• [strlen] возвращаемое значение должно быть типа size_t; использовать имя result не самая хорошая идея, так как оно общее слишком, имя len/length лучше описывает семантику хранимого значения;
• [сравнение строк] можно было написать, например, в test_mystrcat assert(strcmp(s1, "cats") == 0), а не сравнивать посимвольно строки.