Как реализовать вывод статей блога по категориям?

Question

[Hey One]

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


и набросал код, но в итоге он не пашет, var_dump показывает следующее:

При клике на любую из категорий, результат вывода функции var_dump не меняется. Подскажите пожалуйста в чём проблема, где я накосячил? Заранее благодарю.
Исходный код:
- Вывод категорий в меню и осуществление GET-запроса по id категории:

<?php foreach($category as $categ){ ?>
                    <li><a href="category.php?category_id=<?php echo $categ['id']?>"><?php echo $categ['title']; ?></a><li>
                    <?php } ?>

- function.php

//Выборка из таблицы "Категории"
$category = mysqli_query($connect, "SELECT * FROM category");
$categ = mysqli_fetch_assoc($category);

//Выборка из таблицы "Посты"
$posts = mysqli_query($connect, "SELECT * FROM posts");
$post = mysqli_fetch_assoc($posts);

function get_post_by_category($category_id){
global $connect;
global $post;
$sql = "SELECT * FROM category WHERE id =".$post['category_id'];
$result = mysqli_query($connect, $sql);
$post = mysqli_fetch_assoc($result);
return $post; 
}

- category.php

<?php

require_once 'header.php';

$category_id = $_GET['category_id'];
$post = get_post_by_category($category_id);

?>



<div class="container">
	<div class="alert alert-danger">      
	</div>
</div>

<?php var_dump($post); ?>
<div class="container">
	<div class="row">
		<div class="col-md-9">
			<div class="page-header">
				<h1>Все записи:</h1>
			</div>
			<?php foreach($posts as $post){ ?>
				<div class="row">
					<div class="col-md-3">
						<div class="thumbnail">
							<img src="<?php echo $post['image']; ?>" alt="">
						</div>
					</div>
					<div class="col-md-9">
						<h4><a href="post.php?post_id=<?=$post['id']?>"><?php echo $post['title']; ?></a></h4>
						<p>
							<?php echo mb_substr($post['text'], 0, 200) . '...'; ?>
						</p>
						<p><a class="btn-info btn-sm" href="post.php?post_id=<?=$post['id']?>">Читать полностью</a></p>
						<br/>
						<ul class="list-inline">
							<li><i class="glyphicon glyphicon-list"></i> Категория:  | </li>
							<li><i class="glyphicon glyphicon-calendar"></i> <?php echo $post['datetime']; ?> | </li>
							<li><i class="glyphicon glyphicon-comment"></i> by <a href="#">3 comments</a> | </li>
						</ul>
					</div>
				</div>
				<hr>
			<?php } ?>
		</div>
		<?php require_once 'sidebar.php'; ?>
	</div>
</div>

Answer 1

[Петр Ш]

Вот от этого лучше избавиться:

//Выборка из таблицы "Категории"
$category = mysqli_query($connect, "SELECT * FROM category");
$categ = mysqli_fetch_assoc($category);

//Выборка из таблицы "Посты"
$posts = mysqli_query($connect, "SELECT * FROM posts");
$post = mysqli_fetch_assoc($posts);

выполняй те запросы, которые тебе необходимы и в тех местах, где они необходимы.

- function.php

<?php
// передавай соединение в параметре, избегай использования global-ов
function get_post_by_category(\mysqli $db, $category_id)
{
    // правильный запрос. нам надо все посты конкретной категории
    // $category_id приводим к int, чтобы обезопасить себя от sql-инъекций
    $sql = "SELECT * FROM posts WHERE category_id = " . (int)$category_id;
    $result = $db->query($connect, $sql);

    // у нас несколько результатов, читаем их в цикле и собираем в переменную $posts
    $posts = [];
    while ($row = $result->fetch_assoc()) {
        $posts[] = $row;
    }

    return $posts;
}

- category.php

<?php
require_once 'header.php';

$category_id = $_GET['category_id'];

// тут была вторая ошибка: $post вместо $posts
// не забываем передать $connect
$posts = get_post_by_category($connect, $category_id);

?>
// ну и тд., далее без изменений

Comments

[Hey One]

а что должна содержать в себе переменная $db? Если подключение к БД, то оно есть в переменной $connect.

[Hey One]

и чем плохи global-ы?

[Петр Ш]

Hey One,
- По переменной $db.
Есть переменная $connect, но она внутри функций не видна. Чтобы использовать её внутри функции, надо либо сделать глобальной (плохая идея), либо передать как параметр (хорошая идея).
Параметр можна называть любим именем: хоть $connect, хоть $db, хоть $i_am_vasya. Параметр будет локальной переменной, которая существует только внутри функции.

Я назвал $db потому, что мне так показалось удобнее и понятнее (DB - DataBase). Это мое личное предпочтение в именовании. А название $connect считаю не совсем корректным, поскольку это не ресурс (тип resourse, как было в mysql), а обьект.

[Петр Ш]

Hey One,
- Глобалы.
Они плохи не только в php, они вообще плохи. Об этом в интернете можно найти много информации.
Но чтобы не отправлять в гугляндексы, попробую свои примеры придумать.
1.

<?php
$taxPercent = 20; // налог 20%

function calculateTax($price) {
    global $tax;
    return $price * $tax / 100;
}

echo calculateTax(120); //  выведет 24

но вот если окажется, что на текоторые группы товаров действует другой налог, возникает проблема.
намного проще передать переменную $tax параметром.

2. Если изменить глобальную переменную в каком-то месте, она изменится повсюду. Такую ошибку очень сложно найти. Сложность эта растёт експоненциально с ростом проэкта.
Посмотрим в твой код:

<?php
//Выборка из таблицы "Посты"
$posts = mysqli_query($connect, "SELECT * FROM posts");
$post = mysqli_fetch_assoc($posts);
...
function get_post_by_category($category_id){
global $connect;
global $post;
$sql = "SELECT * FROM category WHERE id =".$post['category_id'];
$result = mysqli_query($connect, $sql);
$post = mysqli_fetch_assoc($result);
return $post; 
}

все другие проблемы откинем, нас сейчас интересует переменная $post.

<?php
var_dump($post); // начальное значение, данные одного поста
$postsList = get_post_by_category(1);
var_dump($post); // уже другое значение, в котором даже не пост, а категория

Почему так? Потому что мы изменили значение глобальной переменной внутри функции.

Надеюсь этих примеров достаточно. В любом случае, в интернете должны быть доводы и лучше.

[Hey One]

ошибку выдаёт:

Fatal error: Uncaught Error: Call to a member function fetch_assoc() on null in D:\OpenServer\OSPanel\domains\lesson\function.php:26 Stack trace: #0 D:\OpenServer\OSPanel\domains\lesson\category.php(8): get_post_by_category(Object(mysqli), '1') #1 {main} thrown in D:\OpenServer\OSPanel\domains\lesson\function.php on line 26

[Петр Ш]

in ...\function.php on line 26

смотри в этой строке.
вероятно, там $result->fetch_assoc()
но перед этим нет присвоения переменной $result результата запроса.

Answer 2

[Антон Р.]

Как-то сложно. Не увидел у тебя JOIN таблиц, тебе нужно объединить таблицы и потом выборка по категории, я делаю так:

if(isset($_GET['category'])){
	
	$category = $_GET['category'];

	$sql='SELECT items.id, name, description, price, date, image, address, categoryname, category_id 
	FROM items 
	
	INNER JOIN category 
	ON category_id = category.id
	
	WHERE  category.id = ' . $category . '
	
	ORDER BY id ASC';
	$result=$pdo->query($sql);
	
	while($row=$result->fetch())
	{
		$items[]=array(
		'id'=>$row['id'],
		'name'=>$row['name'],
		'description'=>$row['description'],
		'price'=>$row['price'],
		'date'=>$row['date'],
		'image'=>$row['image'],
		'address'=>$row['address'],
		'categoryname'=>$row['categoryname'],
		'category_id'=>$row['category_id']
		);
	}
}

Comments

[Петр Ш]

JOIN тут явно лишний.

[Антон Р.]

kester, а где я возьму тогда 'categoryname' ?

[Петр Ш]

Антон Р., тут да, есть categoryname, но в ТС нет :)