Opened 5 years ago

Closed 5 years ago

#171 closed ожидается проверка (задача сдана)

WW #2

Reported by: Артём Сон Owned by: Vasily Alferov
Component: WW_strings Version: 3.0
Keywords: Cc:

Description


Change History (5)

comment:1 Changed 5 years ago by Vasily Alferov

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

Выглядит неплохо, но пока 7/10.

Корректность

5/7.

  • strcpy, кажется, делает что-то совсем неправильное, и тест совсем не содержательный. У тебя он копирует первые strlen(s1) символов строки s2 в строку s1. На самом деле, он должен копировать первые strlen(s2) символов строки s2 в строку s1. Например, в твоём тесте он должен сделать s1 равной "lol".
  • Обычно include guards пишут перед инклюдами, иначе они, на самом деле, проблему с бесконечным взаимным подключением двух файлов не решают.

Стиль

2/3.

  • А зачем gcc -v? Мне его громадный вывод немного помешал посмотреть, нет ли у тебя случайно пропущенных ворнингов линковки, например. Тебе бы тоже мог помешать.
  • Табы в функциях со строками намешались с пробелами. Видимо, в твоём редакторе табуляция показывается как четыре пробела и это незаметно. У меня она показывается как специальный символ, и мне это очень хорошо заметно.
  • for (...) {;} — в принципе, можно не писать фигурные скобки или оставить их пустыми, точка с запятой смысла не несёт и выглядит одиноко. Исправь на for (...); или for (...) {}?

comment:2 Changed 5 years ago by Артём Сон

Version: 1.02.0

comment:3 Changed 5 years ago by Vasily Alferov

Не забывай, пожалуйста, изменять статус тикета на "ожидает проверки", я твой тикет почти потерял.
Теперь 8.5/10, можно ещё в течение недели исправлять.

  • Корректность: 6/7. strcpy должен возвращать указатель на начало s1, а не на конец.
  • Стиль: 2.5/3. В strtok всё ещё мешаюся табы с пробелами.

comment:4 Changed 5 years ago by Артём Сон

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

comment:5 Changed 5 years ago by Vasily Alferov

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

Зачтено.

Note: See TracTickets for help on using tickets.