Вывод нужного количества элементов на странице Битрикс?

Question

[SteepNET]

Добрый день! Есть метод который выводит заданное количество элементов.
Перед вызовом bitrix:catalog.section

<?
$pageElementCount = "10";
if (array_key_exists("showBy", $_REQUEST)) {
	if ( intVal($_REQUEST["showBy"]) && in_array(intVal($_REQUEST["showBy"]), array(18, 36, 54, 72)) ) {
		$pageElementCount = intVal($_REQUEST["showBy"]);
		$_SESSION["showBy"] = $pageElementCount;
	} elseif ($_SESSION["showBy"]) {
		$pageElementCount = intVal($_SESSION["showBy"]);
	}
}
?>

В компоненте
"PAGE_ELEMENT_COUNT" => "$pageElementCount",

В шаблоне

<div class="text-right">
                    <span class="show_title">Показать по </span>
                    <span class="number_list">
                <? for ($i = 18; $i <= 72; $i+=18) : ?>
                    <a rel="nofollow" <? if ($i == $pageElementCount): ?>class="current"<? endif; ?>
                         href="<?= $APPLICATION->GetCurPageParam('showBy=' . $i, array('showBy', 'mode')) ?>"
                    >
                        <span><?= $i ?></span>
                    </a>
                <? endfor; ?>
                </span>
                </div>

Как избавиться от цикла for() ,просто задавать фиксированные значения?
И вроде бы в сессию не загоняется значения, правильно ли делаю?

Answer 1

[PetrPo]

В сессию пишется, проверил, только не понятно зачем именно в сессию писать и логика в целом...

По умолчанию $pageElementCount = "10", далее
Если в $_REQUEST есть showBy, то 
        Если showBy соответствует значению 18, 36, 54, 72,
                pageElementCount = $_REQUEST['showBy']
                пишем в $_SESSION
        Если не соответствует
                 pageElementCount = $_SESSION['showBy']

Ну записали в $_SESSION showBy, затем переходим на другую страницу каталога или убираем из адресной строки параметр showBy, тогда $pageElementCount становится снова 10.
Вопрос. Какой тогда смысл записывать в сессию?

А в целом хоть и код не красивый и с непонятной логикой, но работает)))

Мой вариант:

$arAvailableCounts = array(18, 36, 54, 72);

$pageElementCount = isset($_COOKIE['showBy']) && intval($_COOKIE['showBy']) ? $_COOKIE['showBy'] : $arParams['PAGE_ELEMENT_COUNT'];

if(isset($_GET['showBy']) && $_GET['showBy'] && in_array($_GET['showBy'], $arAvailableCounts)) {
	$pageElementCount = $_GET['showBy'];
	setcookie('showBy', $_GET['showBy'], 0, SITE_DIR);
}

<div class="text-right">
	<span class="show_title">Показать по </span>
	<span class="number_list">
		<?foreach($arAvailableCounts as $value):?>
			<a class="<?=($value == $pageElementCount ? 'current' : '');?>" href="<?=$APPLICATION->GetCurPageParam('showBy='.$value, array('showBy'));?>" rel="nofollow"><span><?=$value;?></span></a>
		<?endforeach;?>
	</span>
</div>

Comments

[SteepNET]

Изначально статья из https://habr.com/ru/post/271307/

Кстате твой код тоже почему то не работает ,не пойму почему,товар вообще не выводится, в отладке ничего нет в переменных...
Размещаю твой первый код перед вызовом bitrix:catalog.section, второй в шаблоне соответственно, не выводит "кол-во элементов"

Компонент не комплексный, выводит на главной bitrix:catalog.section после умного фильтра...

[PetrPo]

Ясно, я думал комплексный компонент...
Не очень хорошая практика такие вещи просто на индексной странице размещать, вынеси код хотя бы в include файлик какой-нибудь

Измени строчку

$pageElementCount = isset($_COOKIE['showBy']) && intval($_COOKIE['showBy']) ? $_COOKIE['showBy'] : $arParams['PAGE_ELEMENT_COUNT'];

на вот такую

$pageElementCount = isset($_COOKIE['showBy']) && intval($_COOKIE['showBy']) ? $_COOKIE['showBy'] : 10;

потому что $arParams['PAGE_ELEMENT_COUNT'] у тебя просто нет, так как компонент простой

Во втором коде, если ты его размещаешь внутри шаблона компонента, тогда переменная $arAvailableCounts = array(18, 36, 54, 72) у тебя отсутствует, ее надо снова объявить внутри шаблона
ПРОВЕРЕНО, все работает.

На мой взгляд, не логично разделять эти два кода. Потому что, первый код без второго, ровно как и второй без первого никакого функционала из себя не представляют. Я бы поместил все во включаемый файл

[PetrPo]

По поводу статьи. Писать говнокод и говорить смотрите все работает никто не запрещает, плохо что такие люди других этому учат...
Тут уже сам смотри, что именно ты хочешь от своего кода, я тебе описал, что происходит в этом примере. Если ты делаешь это с простым компонентом, может быть тебе так и надо, чтобы юзер менял количество товара пока он пользуется этой страницей, зайдет в следующий раз, надо будет, снова тыкнет на нужное количество товара.
Но запись в сессию в этом коде просто херня какая-то, если у тебя есть showBy в $_REQUEST то так и так $pageElementsCount станет равно тому что в $_REQUEST, а если showBy нет, то код с $_SESSION просто не отработается.

[SteepNET]

PetrPo, Да, все просто превосходно! Все работает! Снова выручил!

[SteepNET]

PetrPo, Да теперь доперло в плане сессий , без комплексного компонента смысла нет в записи в сессию.., в этих статьях об этом умалчивается....

[SteepNET]

PetrPo, Кстати ,сперва не заметил , запись в сессию действительно работает, сайт закрыть или перейти в каталог, вернувшись выводит то кол-во которое установлено

[SteepNET]

PetrPo, Решил по твоему совету заинклюдить код в файл, сделал так ..
в index.php

<?$APPLICATION->IncludeFile(SITE_DIR."include/showby_include.php", Array(), Array("MODE" => "php")); ?>

В include/showby_include.php

<?
$arAvailableCounts = array(1, 2, 3, 4);
$pageElementCount = isset($_COOKIE['showBy']) && intval($_COOKIE['showBy']) ? $_COOKIE['showBy'] : 10;

if (isset($_GET['showBy']) && $_GET['showBy'] && in_array($_GET['showBy'], $arAvailableCounts)) {
    $pageElementCount = $_GET['showBy'];
    setcookie('showBy', $_GET['showBy'], 0, SITE_DIR);
}
?>

Но в catalog.section ноль товаров выводит, как только из инклюда в index код возвращаю все работает...
В чем проблема может быть?

Кстати также сделал с сортировкой, там все норм!

[PetrPo]

Ты сделал не по моему совету, а сам додумал...
Хочу тебе че-нить пафосное написать, типа... проблема в том, что ты ищешь проблему там где ее нет)) Ну да, ладно, то что делаешь, на пальцах выглядит так:
- первый вариант

function a() {
	$a = 1;
}

$a = a();
echo 'a = '.$a;

- второй вариант

function a() {
	return $a = 1;
}

$a = a();
echo 'a = '.$a;

Надеюсь, смысл понятен, если нет сюда

Мой вариант просто использовать incude

[SteepNET]

PetrPo, Сделал так

require($_SERVER['DOCUMENT_ROOT']."/include/inc_sort_include.php");

Работает! =)
Прости, но как я не пытался сделать include ,как простой так и Битриксовский так ничего и не вышло, может все таки покажешь пример рабочий? Все перерыл не нашел как что бы работал =(

[PetrPo]

Ну так все правильно ты сделал, либо require либо include, в твоем случае без разницы какую функцию использовать. В примере выше, я хотел простой пример показать, почему у тебя не работает. Область видимости всех переменных, которые ты объявляешь и используешь внутри функции, находится внутри функции (если переменные конечно не глобальные), в отличии от php include и require joxi.ru/l2ZW1d3tz0MWxA
То есть при использовании битриксового IncludeFile твоя переменная $pageElementCount находится внутри, а снаружи ее нет и в компонент catalog.section ты передаешь пустоту, поэтому и товаров ноль

[PetrPo]

Я тут подумал, в использовании битриксового $APPLICATION->IncludeFile есть свои плюсы. Конечно придется сделать return $pageElementCount, но зато ты, как путевый программист получаешь возможность гибкого переиспользования кода. Конечно при таком малом объеме кода польза от этого мало ощутима, но все равно круто..
.
Вот, что я имею ввиду:
Допустим ты решил сделать под своим catalog.section вывод новостей с той же логикой отображения количества товара, но количество должно быть не 18, 36, 54, 72, а к примеру 5,10,15. Сейчас получается, что ты будешь делать - скопируешь свой include файлик изменишь в нем $arAvailableCounts и подключишь его перед компонентом новостей.

Что можно сделать используя $APPLICATION->IncludeFile. Вторым параметром в эту функцию можно передавать массив параметров, которые будут доступны внутри файла в массиве $arParams, тогда можно сделать что-то вроде такого:

$pageElementCount = $APPLICATION->IncludeFile(SITE_DIR."/include/inc_sort_include.php", Array('COUNTS' => array(18, 36, 54, 72), 'DEFAULT_COUNT' => 10), Array("MODE" => "php", "NAME" => ""));

А внутри файла:

<?
$arAvailableCounts = $arParams['COUNTS'];

$pageElementCount = isset($_COOKIE['showBy']) && intval($_COOKIE['showBy']) ? $_COOKIE['showBy'] : $arParams['DEFAULT_COUNT'];

if(isset($_GET['showBy']) && $_GET['showBy'] && in_array($_GET['showBy'], $arAvailableCounts)) {
  $pageElementCount = $_GET['showBy'];
  setcookie('showBy', $_GET['showBy'], 0, SITE_DIR);
}
?>
<div class="text-right">
  <span class="show_title">Показать по </span>
  <span class="number_list">
    <?foreach($arAvailableCounts as $value):?>
      <a class="<?=($value == $pageElementCount ? 'current' : '');?>" href="<?=$APPLICATION->GetCurPageParam('showBy='.$value, array('showBy'));?>" rel="nofollow"><span><?=$value;?></span></a>
    <?endforeach;?>
  </span>
</div>

<?return $pageElementCount;?>

Тогда для новостей достаточно просто вызвать $APPLICATION->IncludeFile со своими параметрами

$pageElementCount = $APPLICATION->IncludeFile(SITE_DIR."/include/inc_sort_include.php", Array('COUNTS' => array(5, 10, 15), 'DEFAULT_COUNT' => 10), Array("MODE" => "php", "NAME" => ""));

[PetrPo]

И плюсани уже ответ, мы уже вышли за рамки вопроса)

[SteepNET]

PetrPo, Да, вот как раз этого как примера я нигде не мог найти, в основном просто инклюдят либо просто текст либо компонент..
аспро например все вообще делают инклюдами полностью, не нашел информацию какая практическая польза от такого метода?