Как это чище записать?

Question

[kaxa3201]

if (array_key_exists('$gt', $filter['created_at'])) {
            $dataArray['{date_from}'] = $filter['created_at']['$gt'];
        }
        if (array_key_exists('$lt', $filter['created_at'])) {
            $dataArray['{date_to}'] = $filter['created_at']['$lt'];
        }

слишком громоздко, как написать чище?

Answer 1

[FanatPHP]

Специально для любителей заменять array_key_exists на isset

isset() не возвращает TRUE для ключей массива, указывающих на NULL, а array_key_exists() возвращает.

Линчно я бы не стал ничего менять. Код читается, выполняет свою функцию. Можно извратиться, и сделать цикл, или использовать неподходящую функцию. Вопрос - ЗАЧЕМ.

Если сильно постараться, то можно записать этот же код короче. Но короче - не значит чище. В подавляющем большинстве случаев получается ровно наоброт. Короткая, но абсолютно нечитаемая абракадабра. или код с ошибками. или все вместе. Ответы в этом топике служат прекрасной иллюстрацией.

Работает - ничего не трогай. Не на версисаж код пишешь

Comments

[Nujabes37]

Зачем array_key_exists, если есть isset ?

[Lander]

Nujabes37,

$arr = ['a' => null];

array_key_exists('a', $arr); //true
isset($arr['a']); //false

[Nujabes37]

Lander, и, я это и так знал? isset используется не только для проверки ключей в массиве) Для стандартных кейсов он вполне пригоден)

Answer 2

[Александр]

if (isset($filter['created_at']['$gt'])) {
   $dataArray['{date_from}'] = $filter['created_at']['$gt'];
}
if (isset($filter['created_at']['$lt'])) {
   $dataArray['{date_to}'] = $filter['created_at']['$lt'];
}

Answer 3

[Lander]

$_at = $filter['created_at'];
if (isset($_at['$gt'])) {
   $dataArray['{date_from}'] = $_at['$gt'];
}
if (isset($_at['$lt'])) {
   $dataArray['{date_to}'] = $_at['$lt'];
}

Comments

[Александр]

Может тогда так?
$_at = &$filter['created_at'];

[FanatPHP]

Александр, зачем?

[Lander]

Александр, Ну можно и так. :) Главное потом с $_at ничего не делать.

[Lander]

$_at = $filter['created_at'];

switch (true) {
    case isset($_at['$gt']): $dataArray['{date_from}'] = $_at['$gt']; break;
    case isset($_at['$lt']): $dataArray['{date_to}'] = $_at['$lt']; break;
}

kaxa3201 такое ещё можно

[green_goo]

Lander, а если в массиве одновременно будет $gt и $lt?

[Lander]

green_goo, И правда. Конец пятничного рабочего дня - затупил. :)

Answer 4

[Nujabes37]

!isset($filter['created_at']['$gt']) ?:
    $dataArray['{date_from}'] = $filter['created_at']['$gt'];
!isset($filter['created_at']['$lt']) ?:
    $dataArray['{date_to}'] = $filter['created_at']['$lt'];

Comments

[Lander]

Держи хиппстера! :)

[Nujabes37]

Lander, )))

Answer 5

[green_goo]

В порядке бреда:

foreach (['$gt' => '{date_from}', '$lt' => '{date_to}'] as $k => $v) {
    if (array_key_exists($k, $filter['created_at'])) {
        $dataArray[$v] = $filter['created_at'][$k];
    }
}