WW #2

[Egor Suvorov]

Корректность:

1. my_strcpy делает немного не то (идея верная).

Стиль:

1. Вместо '\0' лучше использовать просто 0.
2. Вместо while (*s != 0) лучше while (*s) (и в Си, и в C++, и во всех лабах).
3. Функция min не используется (и там скобочки не нужны).
4. В my_strcmp можно избавиться от одного return.
5. stl --> len
6. В тестах лучше не используйте my_strcmp/my_strlen/strcmp/strlen для проверки. Проверяйте прямо содержимое массивов напрямую.

[Egor Suvorov]

Корректность:

1. my_strcpy принципиально не поменялся, осталась та же самая проблема. Подсказка: в my_strcat такой проблемы нет.

Стиль:

1. test_my_strcpy проверяет только первый символ копирования. Улучшите тест, заодно повысится вероятность поймать проблемы корректности.
2. test_str.c не использует string.h
3. Ставьте отступы либо только пробелами, либо только табами, иначе смешиваются и код становится невозможно читать (см. личку). В любом нормальном редакторе можно поставить флажок вроде "автоматически заменять табы на пробелы".
4. Непонятно, зачем в my_strcmp нужен if с break внутри while, можно просто условие расширить.
5. Ставьте пробелы между while и ( консистентно: либо везде (рекомендую), либо нигде.
6. Вместо *s1 != 0 лучше писать просто *s1.

Баллы за корректность:

• Корректный makefile и структура папок +1
• Корректные заголовки +1
• Ваши тесты проходят +1
• Корректный strcat и тест +1
• Корректный strcmp и тест +1
• Корректный strlen и тест +1

Итого пока 6/7 по корректности, стиль ещё не ставлю в надежде, что вы поправите.

[Egor Suvorov]

Корректность есть: 7/7.

По стилю 2.5/3:

1. Используйте везде одинаковый отступ. Вы почти везде используете 4 пробела, а в while в my_strcmp — восемь.
2. Вместо '\0 используйте просто 0.
3. В test_my_strcat незачем заводить лишнюю переменную s2.

[Egor Suvorov]

10/10, успех!