Opened 5 years ago
Closed 5 years ago
#173 closed ожидается проверка (задача сдана)
WW #2
Reported by: | vorobeva.aleksandra | Owned by: | Артур Гулецкий (huletski) |
---|---|---|---|
Component: | WW_strings | Version: | 3.0 |
Keywords: | Cc: |
Description
Change History (5)
comment:1 Changed 5 years ago by
Type: | ожидается проверка → ожидаются исправления |
---|
comment:2 Changed 5 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|---|
Version: | 1.0 → 2.0 |
Пока сделаны только 2 функции my_strcmp и my_strlen
comment:3 Changed 5 years ago by
Type: | ожидается проверка → ожидаются исправления |
---|
my_strlen
работает -> +2;
my_strcmp
- нет. Например, падает assert(0 < my_strcmp("", "a");
.
--
Итог: +2, доделывайте, если хотите больше.
comment:4 Changed 5 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|---|
Version: | 2.0 → 3.0 |
comment:5 Changed 5 years ago by
Resolution: | → задача сдана |
---|---|
Status: | assigned → closed |
Работает, +10.
Замечания:
src/test_str.c:11-14, 22-26
: дублирование кода, проверяющего равны ли две строки; лучше завести функциюare_equal
(еслиstrcmp(..., ...) == 0
не устраивает) и вызывать ее;- отступы: где-то используете два пробела (
test_str.c
), где-то один (str.c:37
), где-то их нет внутри блоков кода (str.c:32
); лучше использовать отступы одного вида; обычно используют два/четыре/восемь пробелов/символ табуляции. Сейчас можете не присылать исправления, в след. раз за неконсистентные отступы будут сниматься баллы; - часть
my_strcat
сильно напоминаетmy_strcpy
, что должно было вас (как минимум) натолкнуть на идею вызывать последнюю, а не дублировать код.
Note: See
TracTickets for help on using
tickets.
Падает на ваших же тестах: