Как зарефакторить много isset вызовов в методе?

Question

[rumasterov]

Есть клиент для отправки запросов в api одной из социальных сетей, есть следующий код:

<?php

class Client
{
    public function getFriends($params)
    {
        $requestParams = [];

        if (isset($params['user_id'])) {
            $requestParams['user_id'] = $params['user_id'];
        }

        // и тут дальше множество таких же проверок для других полей
    }
}

Суть в следующем, надо сформировать запрос с набором полей, пользователь класса передает массив с параметрами, чтобы убедиться что установлены требуемые параметры вызывается isset() и если оно установлено - записывается в запрос. Таким образом какие-то неправильные параметры не попадут в запрос. Но меня смущает это множество if (isset()) или это ок?

Answer 1

[Евгений Одинец]

foreach ($params as $key => $param) {
    $requestParams[$key] = $param;
}

Comments

[rumasterov]

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

[Евгений Одинец]

rumasterov: ну можно добавить проверку на нужные поля:

if (in_array($key, ['key1', 'key2', ...., 'key'])) {
    //do something funny
}

[entermix]

rumasterov: а Вы игнорируйте все то, что не ожидаете, примерно так:

class Client
{
    public function getFriends($params)
    {
        $allowedParams = array('user_id', 'first_name', );
        
        $requestParams = array();
        
        foreach ($params as $key => $param) {
            if (in_array($key, $allowedParams)){
                $requestParams[$key] = $param;
            }
            
        }
    }
}

[YURGEN GOA]

как-то так

$params = array_intersect_key($params, array_flip(['key1', 'key2', 'key3']));

p.s. не тестил, но идея эта

Answer 2

[Денис Дерепко]

Что-то типа того:

$requiredParams = ['1', '2'];
$requestParams = [];

foreach ($requiredParams as $param) {
    if (!array_key_exists($param, $params) {
        //Кидаем исключение
    }

    $requestParams = $params[$param];
}

Comments

[DeadMemories]

$requestParams[$param] = $params[$param] :P

[Денис Дерепко]

DeadMemories: Да, проверяем наличие нужного параметра в $params и если находим, то записываем его значение в массив $requestParams (массив, который указан в условии)