Как правильно обработать коллекцию?

Question

[anna_makeenko]

Есть коллекция объектов, например:

class Object
{
    protected $name
    protected $status;
}

class ObjectCollection
{
    protected $items = [];
//...
}

Нужно написать чекер, который будет проводить некоторые проверки объектов и менять их статус. Как лучше реализовать?

$collection = new ObjectCollection();

$collection->add(new Object());
$collection->add(new Object());

//...

$checker = new Checker();

$collection = $checker->process($collection);

В данном случае метод возвращает измененную коллекцию. Это правильно с точки зрения проектирования? Есть варианты как можно сделать лучше? Может быть есть примеры реализации на GitHub?

Comments

[Дмитрий]

Чисто моё мнение коллекциям в php не хватает дженериков, любые действия над коллекцией должны возвращать копию, то есть должна быть имутабельность
По вашему вопросу, вариантов есть несколько, работа с массивами и с итераторами.

[Дмитрий]

Ну и пример https://github.com/Compolomus/Collection

[anna_makeenko]

Дмитрий, то есть лучше будет, если в методе process получить все объекты, изменить их и вернуть новую коллекцию?

[Дмитрий]

anna_makeenko, конечно лучше, вдруг вам понадобиться оригинальная коллекция

[Антон Шаманов]

Максим Федоров видимо хочет реализовать что-то вроде Strategy

[Максим Федоров]

Антон Шаманов, нет, я отвечал на вполне обычный вопрос:

• как обработать члены коллекции (map())
  
• и что нужно вернуть новую коллекцию
  

по итогу вылезли чанки и внешние АПИ :):):)

При чем тут стратегия?! И полиморфизм в программах?

[Максим Федоров]

Если у вас коллекцию нужно обходить чанками и это не единичный случай, то можете подсмотреть Ларавельные коллекции, там есть чанки, потом по чанкам map() пускаете

Если это единичная реализация, то юзайте конечно приведение к массиву, только тогда не понятен вопрос по поводу новой коллекции, если вы ее все равно разобьете, к чему вопрос был?

[Антон Шаманов]

Максим Федоров, скорее стратегия наоборот - передача стратегического объекта не в коллекцию, а коллекцию в стат.объект

[anna_makeenko]

Максим Федоров, зачем Вы удалили свой ответ?

В итоге правильно будет если метод process будет принимать коллекцию, обрабатывать ее порциями и складывать результаты в новый объект? Или же нужно реализовать коллекцию таким образом, чтобы любое действие над ней возвращало новый объект и тогда можно будет передать ее в обработчик и не думать о том, что нужно вернуть новый объект?

По поводу Object - это просто пример.

[anna_makeenko]

Антон Шаманов, суть в том, чтобы программисту, который хочет проверить объекты не пришлось разбираться со всеми тонкостями проверки.

1. Собираем коллекцию объектов
2. Передаем в обработчик (который содержит методы проверки)
3. Получаем результат

$collection = $checker->process($collection);

Возможно в данном случае будет првильно, если метод будет изменять объекты в коллекции?

[anna_makeenko]

Или же другой вариант - выполнять проверки прямо в методе коллекции $collection = $collection->process(new Checker());

В таком случае, насколько я понимаю, нужно возвращать только новый объект

[Максим Федоров]

anna_makeenko,

$collection = $collection->process(new Checker());

такой вариант не приемлем, коллекция может только отвечать за работу с элементами в рамках своей ответственности, процессить что-либо -- это не про нее

[Максим Федоров]

anna_makeenko,

Возможно в данном случае будет првильно, если метод будет изменять объекты в коллекции?

Только если результатом будет новая коллекция