Opened 3 years ago

Closed 3 years ago

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

HW #14

Reported by: Georgiy Angeni Owned by: Святослав Власов
Component: WW_linq Version: 3.0
Keywords: Cc:

Description


Change History (5)

comment:1 Changed 3 years ago by Святослав Власов

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

Упал тест на select

  1. Если вызвать select на пустом инвервале всё свалится
  2. Не работает с нечисловым типом U (например std::string)

По стилю:

  1. Ты делаешь std::move для функторов, но почему-то принимаешь их по значению (а значит происходит копирование)
  2. until_eq можно отнаследовать от until и уменьшить копипасту (аналогично для where_neq
  3. to_vector и copy_to тоже хорошо выражается одно через другое

7/10

comment:2 Changed 3 years ago by Georgiy Angeni

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

comment:3 Changed 3 years ago by Святослав Власов

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

Предикаты в until_eq и where_neq принимают аргумент по значению -- это не очень хорошо, т.к. будет приводить к избыточному копированию, а на крупных объектах это может быть неприятно.

В остальном всё клёво, 9/10

comment:4 Changed 3 years ago by Georgiy Angeni

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

comment:5 Changed 3 years ago by Святослав Власов

Resolution: → задача сдана
Status: assigned → closed

10/10

Note: See TracTickets for help on using tickets.