На калькуляторе Сitizen все посчитал. Ошибки быть не может)

Посмотрел. Ну я думаю можно FilterableDataInterface и SortableDataInterface реализовать в коллекции. Будет полезно. Ну типа добавлять и свои фильтры и сортировки, через отдельные реализации - это хорошо.


DataWriterInterface - непонятно для чего может пригодится.

CountableDataInterface - тоже можно реализовать, ну вместо Countable

PaginatorInterface - не уверен, что нужно.

DataProcessorInterface - врядли, не могу придумать когда может пригодится

Вообще мне кажется, коллекции нужны в 2 общих случаях:
1. Коллекция которая умеет все и принимает все что угодно(и отдает все что угодно) (33 варианты сортировок, 44 варианта группировок и т д) чтобы сокртить код и избежать дублирования.
2. Типизированные коллекции, когда коллекция будет использоваться на уровне бизнес логики и доменной модели. Тут важно чтобы она делала не много, но делала это хорошо и точно.

Мне кажется это должны быть 2 разные реализации.

PaginatorInterface не нужен, т.к. это он должен использовать коллекцию. Для этого коллекции нужен интерфейс DataReader (конкретно для реализаций пагинатора)

а может наоборот - если коллекция будет поддерживать пагинацию, то почему бы и нет )

DataReaderInterface можно реализовать, но тогда получается коллекция по умолчанию будет с пагинацией насколько я понимаю.

нет, почему?

можно подумать над тем, чтобы добавить метод getDataReader, а не реализовывать интерфейс в коллекции

незнаю, если бы я увидел класс который имплементит DataReaderInterface, я бы подумал именно так

Да как вариант

наверное меня смущает DEFAULT_LIMIT)

надо у @samdark спросить, зачем он там. Вроде как это не обязывает лимитировать по умолчанию )

просто значение по умолчанию, если вдруг захотелось установить лимит, но не придумал, на сколько :D

еще обратил внимание что в KeysetPaginator не типизированы свойства, хотя в остальных сделано.

до него некому было добраться и отрефакторить

там на него и тесты надо дописывать

посмотрел, прикольно.

Понравилось разделение на map и list - не думал об этом
Понравилось что можно делать Immutable коллекции, полезно должно быть

надо начать с чего то небольшого, и посмотреть пригодится или нет. Если пригодится - развивать, нет - выкинуть.

надо только определится что это небольшое должно быть.

Можно сделать коллекцию, чтобы покрывала 2 частных случая CoockieCollection в yii-web и rbac. И посмотреть что получится на выходе.

в худшем случае получится фигня для 2 частных случаев. В лучшем - удобная штука, которая будет полезна всем.