更に細分化をしたいのですが。

質問

以前、質問をさせて頂いた内容の続きになります。Helperクラスを作ってview側でinjectさせ表示をされることは出来ました。それでも、コードが長くなりますので更に細分化したいのですが、可能でしょうか？
①例えば前回質問させていただいた下記のコードのecho $sabunの$sabunを外部classで計算させてhelperに読み込ませるようなことを考えています。また②下記のようなクラスを用意してcsvファイルを読み込ませ祝日が該当すれば表示るようなことも可能でしょうか？
①一部抜粋
$ym = date("2022-04-22");
$tm = date("n",strtotime($ym));//月
$ty = date("Y",strtotime($ym));//年
$countdate=date('t',mktime(0,0,0,$tm,1,$ty))
for ($i = 0; $i < $countdate; ++$i) {
$d = mktime(0, 0, 0, $tm, 16 + $i, $ty);
//日付ごとの配列のキー
$dd = date('Y-m-d', $d);
foreach($e_all as $e_time){
if (in_array($dd, (array)$e_time->date_time)) {
echo $e_time->start_time;
echo $e_time->end_time;
$sabun = (strtotime($e_time->end_time) - strtotime($e_time->start_time)) / 60;
echo $sabun;
②祝日がcsvファイルの中に該当すれば表示する
一部抜粋

namespace App\Helpers;
use \SplFileObject;
class SyukuHelper
{
public function syukuLogic($d)
{
$file = new SplFileObject(storage_path("app/holiday.csv"));
$file->setFlags(SplFileObject::READ_CSV);
$exist_flg = false;
foreach ($file as $line) {
if (isset($line[1])) {
$date = date("Y/m/d", strtotime($line[0]));
$name = $line[1];
if ($date == date('Y/m/d', $d)) {
echo '<td class="border border-gray-400 ..." align="center" bgcolor="lightblue">'.$name.'</td>';
$exist_flg = true;
}
}
}
//祝日一致なければ空欄
if ($exist_flg == false) {
echo '<td class="border border-gray-400 ..." align="center" bgcolor="lightblue"></td>';
}

どうぞよろしくお願い致します。

回答No.1

hogehoge78 · ベストアンサー

設計思想の問題なので、いかようにも管理しやすいように切り分ければよいと思います。

例えば、以下のように切り分けるのが良いのではないでしょうか。

■App\Helpers\SyukuHelper

これは、View(Blade)テンプレートで利用するためのいわば窓口である、という考え方。

■App\Services\HolidayService

これを祝日判定用のサービスとして機能させる。

■で、実装

app/Helpers/SyukuHelper.php
<?php
namespace App\Helpers;

use App\Services\HolidayService;

class SyukuHelper
{
private $holiday;
public function __construct()
{
$this->holiday = new HolidayService();
}

public function syuku($d)
{
$currentDay = date('Y/m/d', $d);
$result = $this->holiday->get($currentDay);
if ($result !== null) {
// 祝日判定だったら、なんかHTMLとか出力する
echo '祝日のときのHTML' . $result['name'];
} else {
echo '平日のときのHTML';
}
}
}

?>

app/Services/HolidayService.php
<?php
namespace App\Services;

use SplFileObject;

class HolidayService
{
private $holidays = [];
public function __construct()
{
// どうせ総なめするし、使いやすいようにコンストラクタで計算結果の配列つくっとく
$file = new SplFileObject(storage_path('app/holiday.csv'));
$file->setFlags(SplFileObject::READ_CSV);
foreach ($file as $line) {
if (!isset($line[1])) continue;
$date = date('Y/m/d', strtotime($line[0]));
$name = $line[1];
$this->holidays[$date] = $name;
}
}

public function get($d)
{
if (array_key_exists($d, $this->holidays) {
// ファイルから作った連想配列のキーに該当の日付があった場合、祝日の名称を返す
return ['name' => $this->holidays[$d]];
} else {
return null;
}
}
}
?>

みたいな感じで切り分けられれば良いと思います。
このような感じにしておけば、View以外で、例えばコントローラなんかで祝日チェックしたい、とか言った場合にも、HolidayServiceだけ呼び出せば使えるわけですね。

お礼

いつも、ご回答ありがとうございます。
多分また、質問をさせていただく事になるかと思いますが
よろしくお願い致します。