Opened 4 years ago

Closed 4 years ago

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

WW #18

Reported by: abramov.nikita 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

Корректность 5.5/7:

  • Неверная оценка на количество цифр в числе. digits — это не всегда в десятичной системе. Посмотрите документацию.
  • Неаккуратная оценка на размер вектора: только по первому элементу.

Стиль 2/3:

  • format_size
    • Константа, должна быть большими буквами.
    • Это не размер format(), это примерная оценка по умолчанию для одного конкретного случая. Имя должно быть другое.
    • Это точно статическая приватная константа для конкретной специализации.
  • 4u/5u — не требуется, и так явно указан возвращаемый тип.
  • str.data()/str.data() + str.size() --> str.begin()/str.end().
  • FormatHelper для .format():
    • Лучше инвертировать if в estimate_size() и сделать early return для простого случая. Или сделать честно две равноценные ветки. Делать early return для сложного случая не принято.
    • append_to: лучше не сложный тернарник, а один иф. Заодно исчезнет += "".
  • Можно не исправлять: вместо проверки метода estimate_size() можно завести какую-нибудь статическую константу IS_VALID и проверять её.
Note: See TracTickets for help on using tickets.