Как не нагавнокодить в этом примере?

Question

[Alex_ART]

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

<?php
 $views = mysqli_query($connection, "SELECT * FROM `goods` ORDER by `views` DESC limit 5");
                    while ($top_views = mysqli_fetch_assoc($views)) { ?>
                        <article class="article">
                            <div class="article__image" style="background-image: url(/media/images/<?php
                            if ($top_views['categorie_id'] == 1) {
                                echo "tablet/";
                            } elseif ($top_views['categorie_id'] == 2) {
                                echo "phones/";
                            } else {
                                echo "nouts/";
                            }
                            echo $top_views['img'];
                            ?>
                            "></div>

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

Answer 1

[Максим Тимофеев]

Варианты
использование ООП решит эту проблему, путем добавления метода для этого
написать функцию, которая будет возвращать нужный url
хранить полный url в базе

Но и Ваш метод не супер Г. Вот только вместо if надо хотя бы switch case использовать. Или массив типа

<?php $arr = [2=>'/tablet',18=>'phones/']; ?>
<div class="article__image" style="background-image: url(/media/images/<?= $arr[$top_views['categorie_id']]?>"></div>

Comments

[Alex_ART]

подписался на канал :)

[Максим Тимофеев]

Alex_ART: ))) Спасибо

[Максим Тимофеев]

Alex_ART: Кстати Кандрашкин Алексей очень правильно заметил, надо бы еще по датам разбить, что бы через 100000 товаров не столкнуться с папками, к в которых фалов немерено. А это тот еще набор геморов даст.

[Alex_ART]

Максим Тимофеев: это учебная страница, с 10000 файлами не столкнусь, но эту деталь буду учитывать, после того как закончу страницу, исправлю этот недочёт

[Alex_ART]

Максим Тимофеев: и с массивом классное решение, запомню, что это можно так использовать

[Максим Тимофеев]

Кандрашкин Алексей: Не нашел. На самом деле нашел кучу людей. Но проекты в которые хотел подключить человека повисли в бюрократических утверждениях. И до сих пор висят. Но спасибо.