Как выделить выбранные пункты меню исходя из параметров $_GET?

Question

[# a r t u r #]

Здравствуйте!

Делаю простую сортировку по Производителю.

Имеется форма со списком производителей:

<form action="" method="get">
    <input type="checkbox" name="brands[]" value="59" > Adidas<br><br>
    <input type="checkbox" name="brands[]" value="72" > Rebook<br><br>
    <input type="checkbox" name="brands[]" value="56" > Nike<br><br>
    <input type="submit" value="Показать">
</form>

отправляющая GET-запрос, из него можно выбрать сразу несколько путем установки checkbox-а. После сохранения получаю строку вида:

... ?proizvoditel=1&proizvoditel=2&proizvoditel=3 ....

Все параметры отправляются в модель и по итогу на странице видны только товары определенных производителей, но мне нужно отметить выбранные checkbox-ы, как у всех нормальных людей.

Каким образом такое реализовать?

Спасибо за отклик.

Comments

[Remmi]

> После сохранения получаю строку вида:
> ... ?proizvoditel=1&proizvoditel=2&proizvoditel=3 ....

Это неверный запрос - переменная одна и та же, а это значит, что её значение перезатирается. И в результате будет последнее присвоенное ей значение, остальное потеряется

[# a r t u r #]

Remmi: ну а как иначе, таким образом передается массив checkbox-ов, да и сама выборка прекрасно работает, так что Вы ошибаетесь

[Remmi]

Артур Селиванов: можно попробовать передавать всё одной переменной, а все значения по производителям склеивать в одну строку, разделять запятыми и передавать через эту переменную.

Например исходный запрос переделается в такой:
?proizvoditel=1,2,3 ....

Дальше делите строку на части по разделителю запятая, вытаскиваете ИД и с ними работаете.

Но это вариант в лоб, возможно, что есть более красивые решения.

[# a r t u r #]

Remmi: Я тоже думал об этом отталкиваясь от пример: www.ulmart.ru/catalog/cases?sort=1&viewType=0&rec=...

но по-моему это слишком сложно и некрасиво

[# a r t u r #]

Remmi: да и потом в любом случае по итогу это будет массив )

[Remmi]

Артур Селиванов: это в любом случаt будет массив. В моём варианте запись будет короче, чем каждый раз дублировать название массива. Хотя в варианте с массивом будет на одно действие меньше.

[# a r t u r #]

Remmi: Тогда какая разница как передавать... проблема в другом была, читайте мой ответ ниже

Answer 1

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

css-tricks.com/snippets/javascript/get-url-variables

Comments

[# a r t u r #]

он с массивами не дружит кажется, если б не массив уже давно сделал

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

Артур Селиванов: jsfiddle.net/QW01_01/8ztsxum3

[# a r t u r #]

Александр Таратин: Вы как всегда мне все разжовываете ) спасибо! Я написал ниже ответ, там на чистом PHP, по Вашему какой вариант оптимален?

Answer 2

[Евгений Самойленко]

Каким образом применяются фильтры? Просто редиректом или аякс подгрузкой и подкидыванием урла в адресную строку?
В первом варианте - да, надо js использовать
Во втором варианте - просто при сборке в очередной раз страницы с поиском во вьюхе проверяйте, выбран ли чек бокс основываясь на тех же ГЕТ Параметрах и все!

Смогу пояснить более детально ,если пойму что за cms/cmf/etc.

UPDATE
Во-первых, у вас проблема и в гет параметре в вашем примере будет только последний производитель, чтобы там были все производители используйте гет массивы так, чтобы получилась строка такая:

... ?proizvoditel[]=1&proizvoditel[]=2&proizvoditel[]=3 ....

В таком случае в том месте, где вы выводите чек-боксы производителей проверяйте так:
Если в гет массиве есть элемент с номером один - то выводите чекбокс этого производителя с аттрибутом checked, и так далее...
p.s. Как смог объяснил, не идеально конечно. Но точней я не могу сказать, пока не увижу что и как, но общая идея должна быть более чем понятна.

Comments

[# a r t u r #]

Самописный скрипт, никаких ajax-ов, чистый php. Я бы и рад js использовать, но проблема в том что здесь массив с производителями

[# a r t u r #]

UPDATE: По поводу строки, у меня так и получается:

?proizvoditel%5B%5D=59&proizvoditel%5B%5D=72&proizvoditel%5B%5D=56

щас попробую

[Евгений Самойленко]

Артур Селиванов: обновил ответ

[# a r t u r #]

Евгений Самойленко: Смотрите, делаю так:

<?php echo in_array($key, $params['proizvoditel']) ? 'checked' : ''; ?>

Все работает, но когда $params['proizvoditel'] не задана, то выдает ошибку:

in_array() expects parameter 2 to be array, string given

т.е. второго параметра нет, тяжко все )

[Remmi]

Артур Селиванов: покажите, что в $_GET хранится

[# a r t u r #]

Remmi: в GET-е много параметров, вот что храниться в $_GET['proizvoditel']:

Array ( [0] => 72 [1] => 56 )

т.е. сейчас выбраны 2 производителя

Answer 3

[# a r t u r #]

Заработало с такой проверкой у каждого Checkbox-а:

<?php echo in_array($key, $params['proizvoditel'] ? $params['proizvoditel'] : array()) ? 'checked' : ''; ?>

т.е. форма выглядит так:

<form action="" method="get">
    <?php foreach ($brands as $key => $value) : ?>
        <input type="checkbox" name="brands[]" value="<?php echo $key; ?>" <?php echo in_array($key, $brand_sort ? $brand_sort : array()) ? 'checked' : ''; ?>> <?php echo $value; ?><br>
    <?php endforeach; ?>

    <input type="submit" value="Показать">
</form>

насколько красиво вообще?

Comments

[Евгений Самойленко]

для самописа очень даже ничего. Поздравляю с решением:)

[# a r t u r #]

Евгений Самойленко: ну да, я тоже думаю, что намного красивее тут не получится... вопрос в быстродействии, все же поиск в массиве у каждого checkbox-а и аж 2 проверки на наличие переменной... :( может проще и вправду на JS?

[Алексей Яхненко]

все правильно, только лучше вот это
$brand_sort ? $brand_sort : array()
заранее где-то отдельно вычислить.

[# a r t u r #]

Алексей Яхненко: уже функцию написал