Laravel 5: как лучше выводить данные из бд?

Question

[Евгений Старцев]

Суть такая
В бд ест колонка status (integer)
И возможные значения в ней
1- новый заказ
2 - обработан
3 - выполнен
4 - отказ

Нужно вывести эти значения к тому же выделять цветом
Новый - зелёный
Выполнен - синий
Отказ - красный

Как лучше организовать вывод?

Answer 1

[N]

class Order extends Model
{

    // ... Тут стандартное начало модели
    
    protected $appends = [
        'status_color'
    ];

    
    public function getStatusColorAttribute() {

        switch($this->status) {
        
            case 1:
            
                return 'зелёный';
            
            break;
            
            
            case 2:
            
                return 'синий';
            
            break;
            
            case 3:
            
                return 'красный';
            
            break;
            
            default:
            
                return 'хз';
            
            break;
        
        
        }
    }

}

Comments

[Евгений Старцев]

эмм, а так чтобы и цвет и значение?

public function getStatusColorAttribute() {
switch($this->status) {
case 1: return '<span class="green">Новый</span>'; 
break;
case 3: return '<span class="blue">Выполнен</span>'; 
break;
case 4: return '<span class="red">Отказ</span>'; 
break;
....
}
}

просто мне кажется что так не совсем правильно... запихивать html в методы...

[N]

Евгений Старцев, Да никто не просит html пихать...
У тебя всегда при получении записи через модель будет два ключа:

status - который и был (числовой)
status_color - который мы с помощью мутаторов замутили...тут можешь либо сразу класс CSS прописать, либо код цвета...

[jazzus]

Title и другие данные тоже через геттеры получать?)

Answer 2

[JhaoDa]

if (status === 1) {
    echo '<зелёный>Новый</зелёный>';
} elseif (status === 2) {
    echo '<синий>Выполнен</синий>';
} elseif (status === 3) {
    echo '<красный>Отказ</красный>';
} else {
    echo 'Што?';
}

Comments

[Евгений Старцев]

Спасибо, а более элегантный выриант есть?

[JhaoDa]

Евгений Старцев, это офигенно элегантный вариант.

[N]

JhaoDa, Я бы так не сказал...

https://laravel.com/docs/8.x/eloquent-mutators

Автор, взгляни на доку...если не поймёшь, то спроси, что не понял именно...
Можно НАМНОГО элегантнее! :)

[JhaoDa]

N, ну если доку читать, то можно. А если ждать, пока за него напишут (а вопрос задан именно в духе «дайте готовый код»), то и так хорошо.

[N]

JhaoDa, Так и живём :)
Мне то тем более фиолетово)))

[Евгений Старцев]

N, JhaoDa,
да, мутаторы думаю подойдут
другой вопрос - элегантно ли будет в эти функции запихивать код html?

public function getStatusAttribute()
{
    if (status === 1) {
      echo '<зелёный>Новый</зелёный>';
    } elseif (status === 2) {
      echo '<синий>Выполнен</синий>';
    } elseif (status === 3) {
      echo '<красный>Отказ</красный>';
    } else {
      echo 'Што?';
    }
}

[Евгений Старцев]

N, про "Можно НАМНОГО элегантнее! :)" можно чуть подробнее?

[N]

Евгений Старцев, Смотри ответ...

Теперь при получении любой записи через модель в поле status_color будет твой цвет...

[N]

Евгений Старцев,

другой вопрос - элегантно ли будет в эти функции запихивать код html?

Ты туда код цвета (например #008000) записывай...без всякого html...ну или там какое-то символьное обозначение, чтобы потом при выводе понимать какой цвет показывать...

P.S.: Можно например название класса (CSS) туда пихать...а при выводе подставлять из status_color...

[Евгений Старцев]

N, и выводить 2 переменные?

<span class="{{$order->status_color}}">{{$order->status}}</span> // $order->status - это getStatusAttribute()

[N]

Евгений Старцев, ну да...тут уже твоя импровизация как там дальше печатать это на странице)

[Евгений Старцев]

N, СПАСИБО)))) будем пробовать

[N]

Евгений Старцев, Удачи)

Answer 3

[jazzus]

Не нужны здесь никакие геттеры. Автору нужно освоить связи Ларавел и записывать всю необходимую инфу в бд.

Делаем таблицу statuses с полями title, color и любые другие, предусмотренные тз
Забиваем статусы
В модели Order делаем belongsTo связь status и в запросе к заказам цепляем статусы через with

$orders = Order::with('status')
    ->get();

используем в шаблоне $order->statsus->title $order->statsus->color и тд
В таком варианте ты сможешь не только это все удобно юзать, не обращаясь тысячу раз к геттерам, но и использовать статусы для других моделей и в другой логике

Comments

[N]

Это, если ему там вздумается цвета динамически менять, название статусов и т.д.
Тоже имеет место быть.
Но у него может не такая глобальная задача и мутить связь ради трёх ключей статуса...ну хз...))

[jazzus]

N, да это бред. Нет смысла изображать бд геттерами, где можно и нужно использовать бд. То что в принятом ответе 100% костыль. Я представляю как он потом извращаться будет с фильтрами. Т.е. будет составлять вручную массив из статусов где будет прописывать ид константами, а потом геттерами получать тайтлы и цвета. Идиотизм. И еще 200 вызово геттеров с проверками в каждом на 100 заказов. Только бд и отношения. Составить отношения это 5 минут времени. Гораздо проще чем мутить геттеры с проверками на каждое поле и массивы из данных. А если отношения не используете то и в Ларавел нет смысла т.к. это его основной функционал. Автору советую такому не учиться и делать по нормальному.

[N]

jazzus, Да мы даже полностью контекст задачи не знаем.
Там может три с половиной заказа за всё время будет и никаких доработок не предвидится...

Мне тут "ответы" в статистике в письку не упёрлись!))
Я ж говорю, твой ответ норм. Но я, по задачи и по времени на эту задачу, а так же по дальнейшим доработкам выбирал бы решение.

У меня есть кнопка "Отметить решением"...я хз как оно повлияет...но нажму на твой ответ))

Расходимся :)

[jazzus]

N, какая бы задача не была. Написать отношение 3 строчки кода, а не пол страницы switch case в модели и массив из статусов с константами.

[N]

jazzus, Три строчки?
Ну раз ты такой правильный, то накидай ему ещё функции для управления этими статусами (создание/редактирование)...не ну а че?:)
Ну там чтобы цвета он менял из админки...названия...

Так-то можно тут бесконечно фантазировать...

[jazzus]

N, в документации накидано. Вот как вы программистами работаете, а элементарных вещей не знаете? Наверняка еще деньги берете.

[N]

jazzus, Остапа понесло...

в документации накидано

Ну и к чему тогда вообще твой ответ? На полфёдора? С девушками так же?

элементарных вещей не знаете?

Слова ради слова? Если бы я что-то в этом плане не знал, я бы про связи тебе не раскидывал вообще...

КОНТЕКСТ ЗАДАЧИ (ВОПРОСА) ИЗУЧИТЕ...

Прощаемся...

[jazzus]

N, а девушки тут причем?) ты сам написал про три строчки и что отношения для глобальных проектов. Хотя они созданы чтобы упросить разработку и делать их 5 минут. Тебе походу с девушкой нужно встретиться, а не на Тостере сидеть. а то скоро на капс полностью переключишься)

[N]

jazzus, Ясна. Понятна.