Change History (3)

comment:1 Changed 5 years ago by Артур Гулецкий (huletski)

Type: ожидается проверкаожидаются исправления

Тесты прошли, +10.

Замечания:

  • отсутствуют отступы перед statement'ами в теле функций/в блоках if/while (-2 балла, отступы должны быть, с ними код воспринимать удобнее);
  • my_strlen должна возвращать значение типа size_t;
  • части my_strcat похожи на my_strcpy и my_strlen -> поведение функции можно было реализовать через вызовы "более простых", что избавило бы от дублирования кода.

Итог: 8/10, исправляйте отступы, чтобы получить полный балл.

comment:2 Changed 5 years ago by podoprigora.ivan

Type: ожидаются исправленияожидается проверка
Version: 1.02.0

Добавил отступы

comment:3 Changed 5 years ago by Артур Гулецкий (huletski)

Resolution: задача сдана
Status: assignedclosed

Ок, +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.