Hello world!
Самоучка, тренируюсь писать код, сейчас делаю интернет магазин
вообщем суть вопроса
имею
Планшеты - картинка в А/tablets/..
Ноутбуки - картинка в А/noutes/..
Телефоны -картинка в А/phones/..
А вот мне например нужно вывести топ 5 просмотренных товаров - а для каждого товара картинка лежит в отдельной папке . Я вот так как то накрутил


"> Всё работает, но это сильно похоже на кусочек г*** а не на код, как можно сделать лучше?
Подскажите нубу)

Предыдущий
вопрос Следующий
вопрос

Конечно, я могу помочь улучшить ваш код. Вот как можно сделать его более читаемым и эффективным:

Используйте массив категорий и путь к папкам для удобства и лучшей читаемости кода:$categories = [
1 => "tablet",
2 => "phones",
3 => "nouts"
];
while ($top_views = mysqli_fetch_assoc($views)) {
$category_id = $top_views['categorie_id'];
$img_path = $categories[$category_id] . "/" . $top_views['img'];
?>Разделите логику отображения картинки от логики получения данных в отдельные функции:function getCategoryImage($category_id, $img) {
$categories = [
1 => "tablet",
2 => "phones",
3 => "nouts"
];
return $categories[$category_id] . "/" . $img;
}
while ($top_views = mysqli_fetch_assoc($views)) {
$img_path = getCategoryImage($top_views['categorie_id'], $top_views['img']);
?>Используйте шаблонизаторы для более чистого HTML кода:<div class="article__image" style="background-image: url(/media/images/<?= $img_path; ?>"></div>

Таким образом, вы можете улучшить читаемость и эффективность вашего кода, делая его более поддерживаемым и масштабируемым. Надеюсь, это поможет вам в разработке интернет-магазина.