WW #15

[Дмитрий Свиридкин]

• в where_neq, until_eq может быть UB из-за времени жизни переданного элемента.

• реализуйте их через where/until. Зачем повторять?

• to_vector можно реализовать через copy_to и не дублировать код

• take не работает

• не копируйте функторы там, где их пожно переместить

• в where предикат может вызываться несколько раз на одном и том же элементе. Сначала в ++, потом в *

---

4 + 2

[Дмитрий Свиридкин]

• typedef std::function<bool(const T &x)> L; Зачем? тип аргумента замечательно сам вывеведется. И вместо typedef лучше использовать using.

• есть виртуальные методы, но нет виртуального деструктора

• пропуск элементов в where стоит сделать отдельным приватным методом -- он повторяется в конструкторе и в ++.

---

6 + 2.5

[Дмитрий Свиридкин]

bool state = 0;

У bool есть хорошие именованные константы. И имя поля не самое удачное. Из имени state не понятно, что означает true, а что false.

---

7 + 2.8