Opened 5 years ago
Closed 5 years ago
#162 closed ожидается проверка (задача сдана)
WW #2
Reported by: | podoprigora.ivan | Owned by: | Артур Гулецкий (huletski) |
---|---|---|---|
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 |
Ок, +2
Замечания по стилю:
- str.c:18. обычно с обеих сторон от оператора присваивания ставят пробелы:
int i=0;
->int i = 0;
; - str.c:19. размещать несколько разных концептуально инструкций (пусть и "простых") в одной строке - плохая идея, тк такой код становится труднее воспринимать. Для сравнения: написать инструкции в строках 28 и 29 в одной строке - нормально, так как инструкции "подобны": обе перемещают указатель на следующий элемент и приводят к "продвижению" по массиву;
- str.c:20. лишний символ табуляции после statement'a;
- str.c:26. используйте тернарный оператор:
if (cond) { return 1; } else { return -1; }
->return cond ? 1 : -1;
, так лаконичнее.
Note: See
TracTickets for help on using
tickets.
Тесты прошли, +10.
Замечания:
my_strlen
должна возвращать значение типаsize_t
;my_strcat
похожи наmy_strcpy
иmy_strlen
-> поведение функции можно было реализовать через вызовы "более простых", что избавило бы от дублирования кода.Итог: 8/10, исправляйте отступы, чтобы получить полный балл.