Не могу разобраться с переключателем страниц для новостей

@apaicer · Регистрация: 16.09.2016

Author24 — интернет-сервис помощи студентам

Добрый вечер, товарищи! Проблема в следующем, есть достаточное количество новостей, чтобы не вываливать все, я сделал вывод первых 5ти новостей, но дальше нужно было сделать переключение новостей с помощью pageselector, который почему-то у меня не работает. при нажатии на 1/2/3 вылетает url/news/main/page/3 и 404.
Структура URL: url/page/module
Сначала подключил страницы

PHP

1
2
3

 else if ($Page == 'news') {
if (!$Module or $Page == 'news' and $Module == 'category' or $Page == 'news' and $Module == 'main') include('module/news/main.php');
else if ($Module == 'material') include('module/news/material.php');

в main.php у нас

PHP

<?php 
if ($Module == 'category' and $_GET['id'] != 1 and $_GET['id'] != 2 and $_GET['id'] != 3) MessageSend(1, 'Такой категории не существует.', '/news');
$_GET['page'] += 0;
Head('Новости');
?>
<body>
<div id="wrapper">
 
<?php  Menu();
MessageShow()?> 
<?php 
if ($_SESSION['USER_GROUP'] == 2) echo '<a href="/news/add"><div class="button15">Добавить новость</a></div>'
?>
    
<?php 
if (!$Module or $Module == 'main') {
$Param1 = 'SELECT `id`, `name`, `added`, `date` FROM `news` ORDER BY `id` DESC LIMIT 0, 5';
$Param2 = 'SELECT `id`, `name`, `added`, `date`, FROM `news` ORDER BY `id` DESC LIMIT START, 5';
$Param3 = 'SELECT COUNT(`id`) FROM `news`';
$Param4 = '/news/main/page/';
} else if ($Module == 'category') {
$Param1 = 'SELECT `id`, `name`, `added`, `date` FROM `news` WHERE `cat` = '.$_GET['id'].' ORDER BY `id` DESC LIMIT 0, 5';
$Param2 = 'SELECT `id`, `name`, `added`, `date`, `active` FROM `news` WHERE `cat` = '.$_GET['id'].' ORDER BY `id` DESC LIMIT START, 5';
$Param3 = 'SELECT COUNT(`id`) FROM `news` WHERE `cat` = '.$_GET['id'];
$Param4 = '/news/category/?id='.$_GET['id'].'/page/';
}
 
$Count = mysqli_fetch_row(mysqli_query($CONNECT, $Param3));
 
if (!$_GET['page']) {
$_GET['page'] = 1;
$Result = mysqli_query($CONNECT, $Param1);
} else {
$Start = ($_GET['page'] - 1) * 5;
$Result = mysqli_query($CONNECT, str_replace('START', $Start, $Param2));
}
 
PageSelector($Param4, $_GET['page'], $Count);
 
while ($Row = mysqli_fetch_assoc($Result)) echo '<a href="/news/material/?id='.$Row['id'].'"><div class="NewsBlock"><span>'.$Row['name'].'</span><br>Добавил: '.$Row['added'].' | '.$Row['date'].'</a> </div>';
?>
</div>

в material.php у нас

PHP

<?php 
$_GET['id'] += 0;
if ($_GET['id'] == 0) MessageSend(1, 'URL адрес указан неверно.', '/news');
$Row = mysqli_fetch_assoc(mysqli_query($CONNECT, 'SELECT `name`, `added`, `date`, `read`, `text` FROM `news` WHERE `id` = '.$_GET['id']));
if (!$Row['name']) MessageSend(1, 'Такой новости не существует.', '/news');
mysqli_query($CONNECT, 'UPDATE `news` SET `read` = `read` + 1 WHERE `id` = '.$_GET['id']);
Head($Row['name']);
?>
<body>
<div id="wrapper">
 
<?php  Menu();
MessageShow()?> 
 
<?php 
if ($_SESSION['USER_GROUP'] == 2) $EDIT = '| <a href="/news/edit/?id='.$_GET['id'].'">Редактировать новость</a> | <a href="/news/delete/?id='.$_GET['id'].'">Удалить новость</a>';
    
echo '<div class="gucci">'.$Row['name'].'<br><br>'.$Row['text'].'<br><br> Просмотров: '.($Row['read'] + 1).' | Добавил: '.$Row['added'].' | Дата: '.$Row['date'].' '.$EDIT.' </div>';
?>

Сама функция pageselector лежит в index.php

PHP

function PageSelector($p1, $p2, $p3, $p4 = 5) {
/*
$p1 - URL (Например: /news/main/page)
$p2 - Текущая страница (из $Param['page'])
$p3 - Кол-во новостей
$p4 - Кол-во записей на странице
*/
$Page = ceil($p3[0] / $p4); //делим кол-во новостей на кол-во записей на странице.
if ($Page > 1) { //А нужен ли переключатель?
echo '<div class="PageSelector">';
for($i = ($p2 - 3); $i < ($Page + 1); $i++) {
if ($i > 0 and $i <= ($p2 + 3)) {
if ($p2 == $i) $Swch = 'SwchItemCur';
else $Swch = 'SwchItem';
echo '<a class="'.$Swch.'" href="'.$p1.$i.'">'.$i.'</a>';
}
}
echo '</div>';
}
}

Я понимаю, что объем кода большой, могу только извиниться.

@ИмяПользователя · 14.02.2018, 22:46

Безликие имена переменных, "магческие" числа в теле функций, нет: тут извиняться мало.
Тут нужно либо оставить только проблемное место, либо переписать код, что бы его было приятно читать.
Сомневаюсь, что кто-то станет разбираться с чужими портянками столь безрадостно оформленными...

Вот это:

PHP

$p1 - URL (Например: /news/main/page)
$p2 - Текущая страница (из $Param['page'])
$p3 - Кол-во новостей
$p4 - Кол-во записей на странице

Почему изначально нельзя назвать как-то так:

PHP

$url - URL (Например: /news/main/page)
$currentPage - Текущая страница (из $Param['page'])
$countOfNews - Кол-во новостей
$countOfPost - Кол-во записей на странице

Кстати, почему в функции PageSelector в коментах $p3 описана как все остальные, а в теле функции обращение к ней идет как к массиву (элементу массива)? Что фактически передается в $_GET['page'] - массив?
Касательно всех остальных индексов - тоже не понятно: почему в одной строке "2", а в другой "5" - зачем эти цифры там?
"Вездесущая" пятёрка (я так понимаю кол-во записей по умолчанию) если вдруг это кол-во нужно будет изменить - придётся "бегать" по всем скриптам и надеяться, что нигде её не пропустили??!
Что бы не было такого "блошиного цирка" стоит предусмотреть единый файл-конфиг, в котором объяввить, например: $defaultPostCount = 5; проинклудить этот конфиг во всех скриптах, где планируется использование этого значения и в теле скриптов писать уже не "магическую пятёрку/семёрку/сотку", а имя соответствующей переменной, в данном примере: $defaultPostCount.

PS. Если я правильно понимаю происходящее, то ошибка в восьмой строке файла index.php, удачи! )))

	14.02.2018, 22:46