Opened 5 years ago
Closed 5 years ago
#151 closed ожидаются исправления (задача сдана)
WW #2
Reported by: | Денис Лочмелис | Owned by: | Sokolov Viacheslav |
---|---|---|---|
Component: | WW_strings | Version: | 1.0 |
Keywords: | Cc: |
Description
Change History (2)
comment:1 Changed 5 years ago by
Type: | ожидается проверка → ожидаются исправления |
---|
comment:2 Changed 5 years ago by
Resolution: | → задача сдана |
---|---|
Status: | assigned → closed |
Note: See
TracTickets for help on using
tickets.
Сейчас повторный запуск make приводит к пересборке. Некоторые цели не помечены как .PHONY, хотя таковыми являются.
Почему
#pragma once
закомментирован?Желательно, чтобы в проекте не было закомментированного кода (зачем он нужен?)
Стоит внимательнее отнестить в include-ам. Например, в str.c и str.h имеются лишние инклуды, а вот str.h в str.c не включен.
В текущей версии функции делают не то, что от них требуется по Стандарту. Это нужно исправить и добавить тесты, которые проверяют корректность.
Кажется, s2c не нужен в тестах.
Стоит аккуратнее отнестись к именованию. s, s1, s2 - это ОК в контексте абстрактных имен для однотипных функций по работе над строками. Именование же s2c уже неговорящее.
Нет проверки предусловий, желательно добавить assert-ы на соблюдение контрактов функций (в данном случае - ненулевые указатели).
В целом хотелось бы увидеть использование linux на регулярной основе, сейчас видно, что код пишется и исполняется на Windows.