WW #2

[Дмитрий Свиридкин]

0. Контрольный набор тестов не пройден.

---

1. Безмолвные программы очень хороши, но лучше все-таки сообщить, что ничего не упало.

2. Имена аргументов стоит изменить на более информативные.

3. Надо пройтись форматированием кода.

4. Не понятно, что проверяет тест strcpy: недачные подбор входных данных и сравнение результатов.

5. Остальные тесты проверяют всего лишь один набора входных данных. Нужны и другие.

[shirokov.kirill]

Я исправил "безмолвие", по-устранял найденные крайние случаи(для корректной работы), добавил assert-ов, немного изменил наборы входных данных.

[Дмитрий Свиридкин]

Контрольные тесты пройдены.

1. тест strcmp проверяет только случай равенства строк. Строки не всегда бывают равны.

2. значение, возвраящаемое strcmp, можно сравнивать на равенство только с 0. Посмотрите внимательно что именно гарантирует стандарт.

3. У вас очень странный способ проверки strcpy. Вы уверены, что вам нужен для этого библиотечный strcpy?

4. Оцените информативность имен аргументов следующих функций по пятибальной шкале:

void memcpy(void* bts1, const void* bts2, int n)

int strspn(const char* str1, const char* str2)

После сделанной оценки, исправьте имена аргументов в своих реализациях.

[shirokov.kirill]

Я исправил имена(на стандарт), сделал доп assert-ы на strcmp, не использовал strcpy() в тесте my_strcpy()(кроме как для сравнения)

[Дмитрий Свиридкин]

Результат strcmp сравнивать на равенство можно только с 0. Внимательно посмотрите описание возвращаемого значения.

Я не знаю, где вы смотрите выдержки из стандарта, но тут ​https://en.cppreference.com/w/c/string/byte/strcpy точно аргумены не Str1, Str2.

Использование strcpy в тесте my_strcpy все равно не ясно. Для проверки нужно проверить, что исходная строка и строка-копия совпадают. Для этого не нужен библиотечный strcpy.

7.5