Проверять ли массив для foreach?

Question

[Михаил]

Чтобы избежать исключений и ошибок со времен php 5 всегда делаю проверку вида:

$data = $this->orderRepository->getList();

if ($data) {
    foreach ($data as $datum) {
    }
}

Актуально ли это в php 7.(0,1,2)?

upd: изменил код чтобы было больше понятно.
upd 2: пришел к выводу что метод getList должен возвращать только массив (или коллекцию).
Симфони как пример:

class EntityRepository implements ObjectRepository, Selectable
{
    /**
     * Finds entities by a set of criteria.
     * ...
     * @return array The objects.
     */
    public function findBy(array $criteria, array $orderBy = null, $limit = null, $offset = null)
    {
    }
}

Comments

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

Со времен 5 версии интерпретатор потерялся? :)

[Михаил]

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

[Сергей Герасимов]

Михаил, в php7 можно вот так сделать, но насколько это плохо...

foreach ($data ?? [] as $datum) {
   ... 
}

Но проверка if ($data) сама по себе не очень хороша, лучше заменить на

if (!empty($data)) {
   ... 
}

[ThunderCat]

Сергей Герасимов, а в идеале is_array(), ибо итерировать true не есть правильно

[Сергей Герасимов]

ThunderCat, тогда уж:
if (!empty($data) && is_array($data)) {
...
}

Потому что is_array() на isset() не проверит и выдаст Notice

[Александр Таратин]

ThunderCat, Итерировать не только массивы можно.
getList как бы подсказывает, что там должен быть список, а не массив.

[ThunderCat]

Александр Таратин, is_iterable() в таком случае, я обычно массивы использую, и getList может возвращать что угодно, в зависимости от реализации метода, не вижу причин почему не вернуть массив.

Answer 1

[Александр Таратин]

$this->orderRepository->getList();

Если getList вернул null вместо пустого итерируемого объекта, то это имхо неправильный дизайн самого getList

Comments

[ThunderCat]

а вот это весьма верное замечание

Answer 2

[Семён Белоглазов]

Нет. Foreach выполняется только тогда, когда в массиве есть данные.

Comments

[Михаил]

Да, про это в курсе. Хочу узнать именно как делать лучше.

Answer 3

[Михаил]

Тут был ответ Camaro67. Я приведу его потому что уже начал отвечать

Извините, а зачем вы до этого так делали? И чем не нравятся исключения, они для того и созданы? Если с массивом что-то не то, то ловите исключение и обрабатываете его.

Comments

[Михаил]

Конечно же страх исключений. Сейчас уже не на столько :) Да и не помню исключений для foreach в последних версиях php.

Например, есть метод который печатает pdf документы. Их может быть много, а может не быть вообще. И это не исключительная ситуация. Даже не вижу где тут лучше обрабатывать исключение.

[Camaro67]

Михаил, Я ответ и удалил потому-что понял, что не верно вас понял :) Сначала подумал вы так входные данные от пользователя проверяете.

Answer 4

[KTF]

По идее getList() должен возвращать коллекцию, в которой должны быть объекты ожидаемого класса (если в цикле будут вызываться какие-то методы объектов, это убережёт от NPE), тогда можно написать так:

$data = $this->orderRepository->getList();

if ($data instanceof OrderCollection) {
    foreach ($data as $datum) {
    }
}