Нужен совет. Является ли это нормальным switch case в контроллере Laravel 5.4?

Question

[bizgrof]

Хочу спросить совета по использованию switch case в laravel контроллере.

Суть такая: У меня есть таблица Posts где хранится столбец status

Для своего удобства я в статусах поставил такие возможные варианты ('published', 'approval', 'draft')

В контроллере создал функцию в которой буду проверять статус и переводить на человеческий язык:

public function status($status) {
    	switch ($status) {
		    case 'draft':
		    	$status = 'Черновик';
		    	break;
		    case 'published':
			    $status = 'Опубликовано';
			    break;
		    case 'approval':
		    	$status = 'На утверждении';
		    	break;
		    default:
		    	$status = null;
		    	break;
	    }
    	return $status;
    }

Является ли это нормальным, или есть какие-то более хорошие способы данной реализации?

Answer 1

[Максим Федоров]

лучше в модели сделайте константу-массив, в котором и переводите это дело, а также сделайте метод тут ж е в моделе для получения статуса

public const STATUSES = [
    'draft' => 'Черновик',
    'published' => 'Опубликовано',
    'approval' => 'На утверждении',
];

public function getRuStatus(string $status): string
{
    if(!isset(self::STATUSES[$status])){
        throw new InvalidArgumentException('Status not found.');
    }

    return self::STATUSES[$status];
}

далее использовать:
$post->getRuStatus($status);

Ну и конечно вам этот массив пригодится например в админке в выпадалке выводить список статусов:
Posts::STATUSES

Comments

[bizgrof]

Интересное решение, благодарю.

[bizgrof]

Спасибо еще раз за наводку на интересную мысль. Только у вас есть некоторые нюансы:

in_array проверяет значение, а не ключ. В нашем случае нужно использовать array_key_exists

и в $post->getRuStatus прописали как множественное число (то есть getRuStatuses).

Попрошу вас поправить это для других людей, чтобы не возникало у них вопросов

[Максим Федоров]

bizgrof, да, спасибо :) поправил

[Роман Сохарев]

Раз уж речь о ларе...

return array_get(self::STATUSES, $status);

[Максим Федоров]

Роман Сохарев, не работал с Ларой, я так понимаю используя эту конструкцию не нужно првоерять на наличие значения в массиве?

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

[Роман Сохарев]

Просто вернет null, если не найдет такого ключа. Третьим аргументом можно передать значение по умолчанию, если хочется не null, а какой-то "case default"
Да и выглядит лаконичнее.