Opened 5 years ago

Closed 5 years ago

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

WW #2

Reported by: Ruslan Salkaev Owned by: Дмитрий Свиридкин
Component: WW_strings Version: 2.0
Keywords: Cc:

Description


Change History (4)

comment:1 Changed 5 years ago by Дмитрий Свиридкин

Owner: changed from Дмитрий Свиридкин to Ruslan Salkaev

Хорошо.

  1. Код надо отформатировать. Желательно с помощью clang-format. Между аргументами и бинарной операцией желателен пробел, между определениями функций тоже стоит оставлять пустую строкую, каждый вложенный блок должен иметь больший отступ, чем блок объемлющий.
  1. Однобуквенные имена переменных -- это плохо. Особенно в заголовках функций: не понятно, что есть что.
  1. Тест strcmp покрывает только случай строк, различающихся первым символом. Строки могут различаться в самых разных местах. Одна может быть началом другой.
  1. Для теста strlen стоит проверить, что работает и для пустой строки.

comment:2 Changed 5 years ago by Egor Suvorov

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

comment:3 Changed 5 years ago by Ruslan Salkaev

Owner: changed from Ruslan Salkaev to Дмитрий Свиридкин
Type: ожидаются исправления → ожидается проверка
Version: 1.0 → 2.0

Исправил замечания

comment:4 Changed 5 years ago by Дмитрий Свиридкин

Resolution: → задача сдана
Status: assigned → closed
Note: See TracTickets for help on using tickets.