Как правильно привести значения в массиве к типу int?

Question

[Aleksandr Yurchenko]

Добрый день!
В большей степени хотелось бы понять "как правильно", поэтому решил спросить совета.
Выполняю условие:

if (!empty($filter) && in_array($attributes_id, $filter)) {
    $checked = ' checked';
} else {
    $checked = null;
}

$attributes_id имеет тип (int), в массиве $filter у значений тип (string). Я взял за правило, где это возможно, выполнять проверку не только по значению, но и по типу, in_array() дает такую возможность, если указать третий параметр true.

Но если проверять in_array($attributes_id, $filter, true) проверка естественно не проходит. Скорректировал проверку до следующего вида:
in_array((string)$attributes_id, $filter, true)
Но как мне кажется, правильнее приводить id к целочисленному значению. Ввиду этого несколько вопросов:

• Насколько корректно проверка in_array((string)$attributes_id, $filter, true), имею ввиду что attributes_id приведен к строке
  
• Если приводить значения массива к целочисленным, как правильнее всего это сделать? У одно предположение, перебрать его как-то так:
  

if ($filter){
    $data = [];
    foreach ($filter as $key => $value){
        $data[$key] = (int)$value;
    }
}

if (!empty($filter) && in_array($attributes_id, $data, true)) {
    $checked = ' checked';
} else {
    $checked = null;
}

Comments

[FanatPHP]

а зачем два массива, $filter и $data?

И откуда берется filter?

[Aleksandr Yurchenko]

FanatPHP, пробовал без data[], но в таком случае не срабатывает условие.
Если вкратце, в $filter попадают отмеченные фильтры


Само условие нужно, что бы не исчезали выбранные фильтры при перезагрузке страницы. Я изначально попробовал написать так:

if ($filter) {
    foreach ($filter as $value) {
        $value = (int)$value;
    }
}

Но в таком виде, почему-то не срабатывает проверка.

[Антон Р.]

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

- смотря в какую сторону нужно округлить, можно в меньшую а можно и в бОльшую сторону.

[Aleksandr Yurchenko]

Антон Р., тут вопрос округления не актуален, так как фактически это id категорий, соответственно числа будут без чисел после запятой.

Answer 1

[DanKud]

На счет правильности или неправильности тут вопрос мне кажется ставить не совсем корректно. Как вам удобнее, так и делайте. Тут больше вопрос в том - какой вообще по сути смысл приводить значение к (string) в вашем первом примере? При нестрогом сравнение (то есть без третьего параметра true) это по сути будет сделано автоматически. Я лично в такой ситуации вообще не вижу смысла от строго сравнения типов. Просто то преобразование типов, которое вы хотите делать, будет сделано автоматически при нестрогом сравнение самим интерпретатором.

Перебор массива можно также сделать с помощью array_map:

$arr = array_map(function($n) {
    return (int)$n;
}, $arr);

Comments

[Aleksandr Yurchenko]

Про in_array((string)$attributes_id, $filter, true) соглашусь с вами, не имеет никакого смысла.

про array_map совсем забыл. Этот вариант получается более лаконичным, спасибо.

P.S. По поводу смысла, как писал в ответе FanatPHP в большей степени для перестраховки и просто стало интересно :).

Хотя теоретически, если каким-то чудесным образом перестанет работать метод получения массива с выделенными фильтрами (именно они попадают в $filter), то особо критичного ничего не произойдет.

Answer 2

[Андрей]

а почему не

$checked = null;
if (!empty($filter) {
    foreach ($filter as $value) {
        if ((int)$value === $attributes_id) {
            $checked = ' checked';
            break;
        }
    }
}

Comments

[Aleksandr Yurchenko]

Тоже отличный вариант! Спасибо
Порой спросить полезно, одна голова хорошо, а не сколько определенно лучше :))

[FanatPHP]

А почему только 10 строчек, а не, скажем, 25?

[Андрей]

FanatPHP, идите в Perl, краткий синтаксис вам понравится.

Answer 3

[FanatPHP]

В общем, если нужно отсеять нулевые значения, то
$filter = array_filter($filter, 'intval');
если оставить, то
$filter = array_map('intval', $filter);

Но я смотрю тут уже столько умных рассуждений и заумных вариантов решения. Куда уж мне-то со своими огрызками.

Comments

[Aleksandr Yurchenko]

Спасибо!

[Андрей]

Удачи перибирать миллион строк вместо одной.
А если null придёт?)

Answer 4

[muhasa]

Ваш вариант вполне неплохой, разве что теперь отпадает вообще надобность проверять типизацию в in_array, Вы же сами весь массив привели к int :)

А вообще, какой смысл здесь в проверке со строгой типизацией?.. Конкретно в данном случае, какая разница между 1 и "1"? Я предполагаю, что тут это излишне.

Comments

[Aleksandr Yurchenko]

Ваш вариант вполне неплохой, разве что теперь отпадает вообще надобность проверять типизацию в in_array, Вы же сами весь массив привели к int :)

Об этом я не подумал, вы правы))

А вообще, какой смысл здесь в проверке со строгой типизацией?.. Конкретно в данном случае, какая разница между 1 и "1"? Я предполагаю, что тут это излишне.

Просто столкнулся сегодня с такой ситуацией и решил выяснить, а как правильнее. По большей части это из разряда "Лучше перебдеть...".

[muhasa]

Aleksandr Yurchenko, ну скажем так - перебдеть оно, конечно, можно, только если разобраться в вопросе глубже, то проверка типизации чаще применяется тогда, когда есть вариант нарваться на булевый тип или null, вот так да, там нужно.

Например, функция strpos, она может вернуть начало строки (0) и совпадения не найдено (false). Новички тут знатно обламываются.

[Aleksandr Yurchenko]

muhasa, сюда null прилететь навряд ли сможет. Он по умолчанию стоит (когда ни один фильтр не выбран).
Так что через чур видимо разогнался с проверками :)