Как оптимизировать данный PHP код?

Question

[mars130]

Приветствую коллеги, столкнулся с проблемой оптимизации кода, имеется вот такой код:

// Проверяем дату на выходной день
									switch ($today_day_of_week) 
									{
										case 1:
											if($monday_output == '1' ) {  // Понедельник
												echo '<a class="disabled"  aria-expanded="false">Выходной</a>';
											}
											else {
												echo '<a href="#today" onclick="today()"  data-toggle="tab" aria-expanded="false">Сегодня</a>';
											}
										break;
										case 2:
											if($tuesday_output == '1') { // Вторник
												echo '<a href="" class="disabled"   data-toggle="tab" aria-expanded="false">Выходной</a>';
											}
											else {
												echo '<a href="#today" onclick="today()"  data-toggle="tab" aria-expanded="false">Сегодня</a>';
											}
										break;
										case 3:
											if($wednesday_output == '1') {  // Среда
												echo '<a href=""  class="disabled"  data-toggle="tab" aria-expanded="false">Выходной</a>';
											}
											else {
												echo '<a href="#today" onclick="today()"  data-toggle="tab" aria-expanded="false">Сегодня</a>';
											}
										break;
										case 4:
											if($thursday_output == '1') {  // Четверг
												echo '<a class="disabled"  aria-expanded="false">Выходной</a>';
											}
											else {
												echo '<a href="?partner='.$_GET['partner'].'&date='.$today_form.'&tab=today" aria-expanded="false">Сегодня</a>';
											}
										break;
										case 5:
											if($friday_output == '1') { // Пятница
												echo '<a href="" class="disabled"   data-toggle="tab" aria-expanded="false">Выходной</a>';
											}
											else {
												echo '<a href="#today" onclick="today()"  data-toggle="tab" aria-expanded="false">Сегодня</a>';
											}
										break;
										case 6:
											if($saturday_output == '1') { // Суббота
												echo '<a class="disabled"  aria-expanded="false">Выходной</a>';
											}
											else {
												echo '<a href="#today" onclick="today()"  data-toggle="tab" aria-expanded="false">Сегодня</a>';
											}
										break;
										case 0:
											if($sunday_output == '1') { // Воскресение
												echo 
												'<a href="" class="disabled"  data-toggle="tab" aria-expanded="false">Выходной</a>';
											}
											else {
												echo '<a href="#today" onclick="today()"  data-toggle="tab" aria-expanded="false">Сегодня</a>';
											}
										break;
									}		
								?>

Который проверяет дату на выходной день и если день выходной, то выводит сообщение "выходной" таких кусков кода аж 7 штук, есть ли способы как-то оптимизировать данный код? Заранее Благодарю всех откликнувшихся.

Answer 1

[Антон Шаманов]

подставлять в шаблон переменную с названием дня или 'сегодня'. в индию надо переезжать, будешь гуру)))

Comments

[mars130]

Не совсем понял предложение, в case сейчас записаны порядковые номера дня недели, т.е. если например today_day_of_week = 1 это значит понедельник и выводим сегодня, немного выше этого скрипта стоит скрипт получающий текущую дату и определяющий порядковый номер дня в неделе. Дальше это записывается в переменную today_day_of_week и таких переменных 7 штук ) т.е. выводится ряд ссылок по типу:
Сегодня
Завтра
24.11
25.11
26.11
27.11
28.11

и вот не могу дойти до решения как можно оптимизировать эти куски кода т.к. их аж 7 штук

[Антон Шаманов]

mars130, зачем тебе 7 переменных? храни массив дней недели - номер => имя

[mars130]

Антон Шаманов, прошу прощения за тугость, а можно хотя быть кусочек реализации попросить? Не могу сообразить как это прикрутить.

Если с массивом всё понятно, то как его использовать при проверке?

$array = array(
    "1" => "monday",
    "2" => "tuesday",
    "3" => "wednesday",
    "4" => "thursday",
    "5" => "friday",
    "6" => "saturday",
    "0" => "sunday",
);

[Антон Р.]

mars130, Вынеси ПОВТОРЯЮЩИЙСЯ код куда-то в одно место и управляй вариантами этого кода через вставку данных посредством переменных:

<a href="$linkDestination" class="$linkClass">$linkText</a>

Answer 2

[globalmac]

Для работы с датой рекомендую - https://carbon.nesbot.com/docs/ и в частости - https://carbon.nesbot.com/docs/#api-week

Carbon::parse('2019-07-14')->isWeekend(); // true

// ИЛИ

$ru = CarbonImmutable::now()->locale('ru_RU');

var_dump($ru->firstWeekDay); 
var_dump($ru->lastWeekDay);
var_dump($ru->startOfWeek()->format('Y-m-d H:i'));
var_dump($en->endOfWeek()->format('Y-m-d H:i'));