WW #2

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

Решение падает на собственных тестах:

{lab_02}[2382]$ pwd && svn up && svn status
/home/hfx/dvl/cpp19/berbat.georgiy/lab_02
Updating '.':
At revision 828.
{lab_02}[2383]$ make
mkdir obj
gcc -c -Iinclude -Wall -Wextra -Werror src/test.c -o obj/test.o
gcc -c -Iinclude -Wall -Wextra -Werror src/str.c -o obj/str.o
gcc -c -Iinclude -Wall -Wextra -Werror src/test_str.c -o obj/test_str.o
gcc obj/test.o obj/str.o obj/test_str.o -o lab2
{lab_02}[2384]$ ./lab2 
lab2: src/test_str.c:9: test_my_strcpy: Assertion `*testing != '\0'' failed.
Aborted (core dumped)

Еще вы забыли отступы перед statement'ами внутри функций.

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

Компилируется -> +0.01

Падают собственные тесты, нужно доделывать:

{lab_02}[2090]$ pwd && svn up && svn status
/home/hfx/dvl/cpp19/berbat.georgiy/lab_02
Updating '.':
At revision 858.
{lab_02}[2091]$ make
mkdir obj
gcc -c -Iinclude -Wall -Wextra -Werror src/test.c -o obj/test.o
gcc -c -Iinclude -Wall -Wextra -Werror src/str.c -o obj/str.o
gcc -c -Iinclude -Wall -Wextra -Werror src/test_str.c -o obj/test_str.o
gcc obj/test.o obj/str.o obj/test_str.o -o lab2
{lab_02}[2092]$ ./lab2 
Segmentation fault (core dumped)

Совет: запустите valgrind ./lab2

[berbat.georgiy]

Спасибо большое, valgrind очень помог. Прошу прощения за просроченный дедлайн(((

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

Тесты прошли, +10.

Замечания:

• size_t не надо было определять самостоятельно, тип определен в stddef.h;
• не везде расставлены отступы (e.g. test.c:10);
• зачем нужна строка str.c:6? В цикле перед следующим присваиванием зануляется элемент, который перезапишется на следующей итерации все равно, это лишнее: '\0' запишется непосредственно перед выходом из цикла;
• тесты: при тестировании my_strcat проверяется записался ли один символ, тогда как логичнее было бы проверить всю строку; при тестировании my_strcpy проверяется копирование пустой строки (что хорошо), но это частный случай - странно было увидеть этот тест и не увидеть копирование одной содержательной строки в другую.