私はいくつかの助けと私のPHPコードを簡素化する方法を説明する必要があります。このPHPコードを単純化するには
これは機能していますが、繰り返しコードがほとんどないようです。
基本的には、ショップが閉鎖されているときに書式設定済みのテキストを表示する日時と、デスクトップやモバイルデバイス上の異なるテキストをチェックしたいとします。
読んでいただきありがとうございます。
<?php
include 'Mobile_Detect.php';
$detect = new Mobile_Detect();
//Get time
$hour = date("H", mktime(date("H")));
$day = date("D");
if($day == 'Mon' && ($hour >= 6 && $hour < 22)) {
if ($detect->isMobile())
{
include 'online_mobile.php';
} else
{
include 'online_desktop.php';
}
}
elseif($day == 'Tue' && ($hour >= 6 && $hour < 22)) {
if ($detect->isMobile())
{
include 'online_mobile.php';
} else
{
include 'online_desktop.php';
}
}
elseif($day == 'Wed' && ($hour >= 6 && $hour < 22)) {
if ($detect->isMobile())
{
include 'online_mobile.php';
} else
{
include 'online_desktop.php';
}
}
elseif($day == 'Thu' && ($hour >= 6 && $hour < 22)) {
if ($detect->isMobile())
{
include 'online_mobile.php';
} else
{
include 'online_desktop.php';
}
}
elseif($day == 'Fri' && ($hour >= 6 && $hour < 22)) {
if ($detect->isMobile())
{
include 'online_mobile.php';
} else
{
include 'online_desktop.php';
}
}
elseif($day == 'Sat' && ($hour >= 8 && $hour < 18)) {
if ($detect->isMobile())
{
include 'online_mobile.php';
} else
{
include 'online_desktop.php';
}
}
elseif($day == 'Sun' && ($hour >= 8 && $hour < 18)) {
if ($detect->isMobile())
{
include 'online_mobile.php';
} else
{
include 'online_desktop.php';
}
} else {
if ($detect->isMobile())
{
include 'closed_mobile.php';
} else
{
include 'closed_desktop.php';
}
}
?>
よろしくR.C.
[簡素化PHPコード](https://stackoverflow.com/questions/6779163/simplifying-php-code) – Jacob
の可能性のある重複した私はそれを言うと思いますスタックオーバーフローではなく、[CodeReview](https://codereview.stackexchange.com/)で議論する必要があります。 – agrm