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>

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