Как в функции чтобы возвращал foreach всю выборку?

Question

[hollanditkzn]

Я использую функцию, для того чтобы сократить код и чтобы не было повторяющего кода во вьюшке, но не получается внутри функции реализовывать foreach. Если я верну значение foreach то выборка будет только одна, а все остальное он не обрабатывает. Если echo делать вместо return то запись попадается не там где надо. То есть нужно чтобы записывалась внутри ячейке td А отображается за этим тегом

function shift($shop_id, $day){
    $monday = Shifts::find()->andWhere(['shop_id' => $shop_id])->andWhere(['>=', 'start_date', date('Y-m-d 00:00:00', strtotime($day))])->andWhere(['<=', 'start_date', date('Y-m-d 23:59:59', strtotime($day))])->all();
    foreach ($monday as $employee){
        return Yii::$app->formatter->asTime($employee->start_date, 'php:H:i').'-'.Yii::$app->formatter->asTime($employee->end_date, 'php:H:i').' '.$employee->user->nameEmployee.'<br>';
    }
}
<table class="table">
        <tr>
            <th>Магазины</th>
            <th>Пн <?= date("d.m", strtotime("last Monday")) ?></th>
            <th>Вт <?= date("d.m", strtotime("last Tuesday")); ?></th>
            <th>Ср <?= date("d.m", strtotime("last Wednesday")); ?></th>
            <th>Чт <?= date("d.m", strtotime("Thursday")); ?></th>
            <th>Пт <?= date("d.m", strtotime("Friday")); ?></th>
            <th>Сб <?= date("d.m", strtotime("Saturday")); ?></th>
            <th>Вс <?= date("d.m", strtotime("Sunday")); ?></th>
        </tr>
        <?php foreach ($shops as $shop){
            echo '<tr>
                <td>'.$shop->name.'</td>';
            echo '<td>'.shift($shop->id, 'last Monday').'</td>
                <td></td>
                <td></td>
                <td></td>
                <td></td>
                <td></td>
                <td></td>
            </tr>';
        } ?>
    </table>

Comments

[Zhainar]

что нужно?

[hollanditkzn]

Zhainar, Вот эта функцию запихнуть в таблицу, он будет постоянно повторяться у меня

function shift($shop_id, $day){
    $monday = Shifts::find()->andWhere(['shop_id' => $shop_id])->andWhere(['>=', 'start_date', date('Y-m-d 00:00:00', strtotime($day))])->andWhere(['<=', 'start_date', date('Y-m-d 23:59:59', strtotime($day))])->all();
    foreach ($monday as $employee){
        return Yii::$app->formatter->asTime($employee->start_date, 'php:H:i').'-'.Yii::$app->formatter->asTime($employee->end_date, 'php:H:i').' '.$employee->user->nameEmployee.'<br>';
    }
}

То есть возвращает так только одно значение в массиве, а если в foreach постовить echo

echo Yii::$app->formatter->asTime($employee->start_date, 'php:H:i').'-'.Yii::$app->formatter->asTime($employee->end_date, 'php:H:i').' '.$employee->user->nameEmployee.'<br>';

То будет отображаться вообще за пределами таблицы

[Zhainar]

что-то я не догоняю

[hollanditkzn]

Zhainar, Вот как сейчас строится мой код

<?php function day($shop_id, $day){
    return Shifts::find()->andWhere(['shop_id' => $shop_id, 'date' => date('Y-m-d 00:00:00', strtotime($day))])->all();
}
function cell($start, $end, $employee){
    echo Yii::$app->formatter->asTime($start, 'php:H:i').'-'.Yii::$app->formatter->asTime($end, 'php:H:i').' '.Html::a($employee->user->lastNameEmployee, ['shifts/update', 'id' => $employee->id], ['class' => 'button-editShifts', 'data-pjax' => 0]).'<br>';
}


<table class="table">
        <tr>
            <th>Магазины</th>
            <th>Пн <?= date("d.m", strtotime("last Monday")) ?></th>
            <th>Вт <?= date("d.m", strtotime("last Tuesday")); ?></th>
            <th>Ср <?= date("d.m", strtotime("last Wednesday")); ?></th>
            <th>Чт <?= date("d.m", strtotime("Thursday")); ?></th>
            <th>Пт <?= date("d.m", strtotime("Friday")); ?></th>
            <th>Сб <?= date("d.m", strtotime("Saturday")); ?></th>
            <th>Вс <?= date("d.m", strtotime("Sunday")); ?></th>
        </tr>
        <?php foreach ($shops as $shop){
            echo '<tr>
                <td>'.$shop->name.'</td>';
            echo '<td>';
            foreach (day($shop->id, 'last Monday') as $employee){
                cell($employee->start_time, $employee->end_time, $employee);
            }
            echo '</td>
                <td>';
            foreach (day($shop->id, 'last Tuesday') as $employee){
                cell($employee->start_time, $employee->end_time, $employee);
            }
            echo '</td>
                <td>';
            foreach (day($shop->id, 'last Wednesday') as $employee){
                cell($employee->start_time, $employee->end_time, $employee);
            }
            echo '</td>
                <td>';
            foreach (day($shop->id, 'Thursday') as $employee){
                cell($employee->start_time, $employee->end_time, $employee);
            }
            echo '</td>
                <td>';
            foreach (day($shop->id, 'Friday') as $employee){
                cell($employee->start_time, $employee->end_time, $employee);
            }
            echo '</td>
                <td>';
            foreach (day($shop->id, 'Saturday') as $employee){
                cell($employee->start_time, $employee->end_time, $employee);
            }
            echo '</td>
                <td>';
            foreach (day($shop->id, 'Sunday') as $employee){
                cell($employee->start_time, $employee->end_time, $employee);
            }
            echo '</td>
            </tr>';
        } ?>
    </table>

Видишь сколько foreach их много на каждый день недели, а мне бы их все сократить в одну функцию и все. То есть обычный рефакторинг сделать.
Если внутрь функции поместить foreach и там значение выводить то получится что из массива 5 ключей, выведит не 5, а 1. Вот что я полагаю

[Zhainar]

hollanditkzn, Я бы сделал такой рефакторинг

function day($shop_id, $day){ 
    return Shifts::find()->andWhere(['shop_id' => $shop_id, 'date' => date('Y-m-d 00:00:00', strtotime($day))])->all();
}

// Эту функцию можно перенести в модель как метод Shops::getShiftsByDay($day)
//{
//	return Shifts::find()->andWhere(['shop_id' => $this->id, 'date' => date('Y-m-d 00:00:00', strtotime($day))])->all();
//};

function cell($start, $end, $employee){
    echo Yii::$app->formatter->asTime($start, 'php:H:i').'-'.Yii::$app->formatter->asTime($end, 'php:H:i').' '.Html::a($employee->user->lastNameEmployee, ['shifts/update', 'id' => $employee->id], ['class' => 'button-editShifts', 'data-pjax' => 0]).'<br>';
}

// Эту функцию тоже можно перенести в модель Shifts::getFormattedTime()
//{
//	return sprintf('%s - %s %s', Yii::$app->formatter->asTime($this->start_time, 'php:H:i'), Yii::$app->formatter->asTime($this->end_time, 'php:H:i'), Html::a($this->user->lastNameEmployee, ['shifts/update', 'id' => $this->id], ['class' => 'button-editShifts', 'data-pjax' => 0]));
//}

$weekDays = [
'Пн' => "last Monday",
'Вт' => 'last Tuesday',
// и тд
];

<table class="table">
        <tr>
            <th>Магазины</th>
	    <?php foreach($weekDays as $shortname => $time): ?>
	            <th><?= $shortname ?> <?= date("d.m", strtotime($time)) ?></th>
            <?php endforeach; ?>
        </tr>
        <?php foreach ($shops as $shop): ?>
		<tr>
			<td><?= $shop->name ?></td>
		    <?php foreach($weekDays as $time): ?>
		            <td>
					<?php 
					$employeeData = array_map(function(Shifts $employee){ return $employee->getFormattedTime(); }, $shop->getShiftsByDay($time));
					echo implode('<br>', $employeeData);
					?>
			    </td>
	            <?php endforeach; ?>
		</tr>
        <?php endforeach; ?>
        } ?>
    </table>

В целом должно быть понятно.

Answer 1

[ilyale]

Сам подход, где смешиваются верстка и логика, скажем так, имеет право на существование в ограниченных специфических задачах...

По поводу решения, насколько я смог понять поток мысли автора, одного цикла по записям $shops не достаточно.
Ваш первый цикл перебирает строки в массиве $shops и формирует строку в верстке для отображения строки данных. Это в границы строки
Между ними (на месте троеточия) должен быть цикл по дням недели, где и вызывается ваша функция, которая обрабатывает данные одного дня.

Надеюсь, поможет.

P.S. ИМХО. Вообще, подход, когда вы извлекаете массив данных (у вас это массив $shops), а затем идете по нему циклом и каждая итерация цикла лезет в БД - это низкопроизводительно и избыточно грузит БД (проще говоря, это говнокод).

Comments

[hollanditkzn]

Ну да, просто когда задача стоит и опыта недостаточно чтобы такие задачи можно было решать на таком уровне чтобы был чистый код и более производительный.

[hollanditkzn]

Мое мнение лучше говнокод чем вообще ничего. Результат все равно нужен хот какой-то. А чтобы взял и научил кто-то, это предел мечтаний

[ilyale]

hollanditkzn, Скажем так, абстрактно попробую проиллюстрировать.

Ваше решение: выгребаем все нужные строки данных из базы, потом делаем цикл по ним, для каждой строки делаем запрос в базу. Итог: один большой запрос и куча мелких. каждый запрос жрет время сервера на коннект с БД, гоняет данные между РНР и сервером БД. Это неэффективно, но работает. На низкозагруженных проектах с малыми объемами данных это сгодится

Более элегантное решение будет попробовать сразу выгрести из БД все данные, использовав связи между таблицами. тогда получаем один тяжелый запрос, но всего один. И дальше по циклу идем и просто отображаем данные

Есть и другие способы оптимизации, но они уже подбираются под задачу и их не опишешь абстрактно.

Надеюсь, поможет.