Opened 5 years ago
Closed 5 years ago
#126 closed ожидается проверка (задача сдана)
WW_strings
| Reported by: | Roman Venediktov | Owned by: | Sokolov Viacheslav |
|---|---|---|---|
| Component: | WW_strings | Version: | 2.0 |
| Keywords: | Cc: |
Description
Change History (3)
comment:1 Changed 5 years ago by
| Type: | ожидается проверка → ожидаются исправления |
|---|
comment:2 Changed 5 years ago by
| Type: | ожидаются исправления → ожидается проверка |
|---|---|
| Version: | 1.0 → 2.0 |
comment:3 Changed 5 years ago by
| Resolution: | → задача сдана |
|---|---|
| Status: | assigned → closed |
my_strcmp так и не соответствует стандарту: строчки не сравниваются лексикографически.
my * system > 0 - плохой способ проверить совпадение знаков: здесь может быть переполнение.
Код тестов можно было бы лучше декомпозировать.
Нет проверки соблюдения контрактов: функции требуют ненулевые указатели, но не проверяют это.
Note: See
TracTickets for help on using
tickets.
Makefile: некоторые зависимости некорректно выставлены как order-only, хотя таковыми не являются. Стоит добавить для .o целей зависимости от (необходимых) заголовочных файлов, чтобы пересборка проекта происходила корректно при изменениях, затрагивающих заголовочные файлы. Лучше, чтобы флаги компиляции совпадали при компиляции разных файлов. Для некоторых флагов это необходимо для корректности работы компилятора.
Текущие реализации (кроме my_strlen) не будут корректно работать на очень длинных строках.
Нет проверки предусловий, желательно добавить assert-ы на соблюдение контрактов функций.
Поведение my_strcmp совсем не соответствует Стандарту.
Рекомендую запустить программу с fsanitize=address.