Opened 4 years ago

Closed 4 years ago

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

WW #18

Reported by: samoylov.viktor Owned by: Egor Suvorov
Component: WW_format Version: 1.0
Keywords: Cc:

Description


Change History (1)

comment:1 Changed 4 years ago by Egor Suvorov

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

Корректность 6.5/7.

  • Некорректно оценивается размер пустого вектора.

Стиль 2/3:

  • Стоит ставить пробелы до и после //, иначе всё склеивается
  • Не надо использовать тернарный оператор вместо ифа. Смотрю на FormatHelper<>. out += value ? "true" : "false" я бы ещё понял, но сейчас ужас какой-то.
  • Намешаны табы и пробелы
  • Вместо .data()/data() + size() стоит использовать begin()/end()
  • MAX_LENGTH_FOR_... — неверное название, это не максимальная длина.
  • Стоит добавлять пустые строки между реализациями методов, иначе склеивается.
  • Я бы не стал создавать FormatHelper дважды, а создал бы один раз в начале и сложил бы в вектор.
Note: See TracTickets for help on using tickets.