Opened 4 years ago

Closed 4 years ago

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

#WW_15

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

Description


Change History (1)

comment:1 Changed 4 years ago by Egor Suvorov

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

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

Стиль 1.25/3:

  • Автоформаттер нужен (это прям минус много, потому что сильно усложняет чтение):
    • НАМЕШАНЫ ТАБЫ И ПРОБЕЛЫ
    • Нужен пробел между while и (
  • Не хватает слов noexcept, rvalue-ref-qualifier у методов.
  • У enumerator стоит запретить копирование и перемещение — это точно ошибки. Перемещение можно запрещать, начиная с C++17 из-за copy elision.
  • В C++17 параметры шаблона класса могут автоматически выводиться. В частности, из параметров конструктора. Вывод: можно не сохранять лямбду в auto predicate, чтобы потом брать decltype.
  • В select_enumerator лучше сделать отдельный приватный метод, который будет обновлять elem под ифом. И вызвать его в конструкторе и в operator++. Будет симметричнее.
  • is_not_end — какое-то сложное состояние. Потому что надо ещё и parent_ проверять. А ещё лучше не использовать в названиях переменных отрицание: element_available.
Note: See TracTickets for help on using tickets.