Ваше мнение о таком PHP коде?

Question

[king555]

В большом классе есть функция

public function findByClient($client_data, $conditions = [])
    {
        return array_map(function ($row) use ($client_data) {
            $row = $row['param'];
          //... действия с $row
            return $row;

        }, $this->getData(...));
    }

какие недостатки вы видите в таком коде?

Answer 1

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

создание объекта - замыкания при каждом вызове метода

Answer 2

[MintTea]

Я бы в сигнатуре функции добавил тайпхинт ко второму аргументу.

А так, обычный код. Возможно, там он и не к месту (непонятно, какие именно действия происходят внутри замыкания), но с помощью array_map зачастую можно довольно красиво переписать те участки, в которых раньше был обычный foreach.