檢查工作時間的時間 - 改善代碼/邏輯

Question

我有一個系統，檢查當前時間是否在香港工作時間。 我確定工作時間是從上午9點到下午6點。 我的代碼正在工作，但我覺得有一個更好/更漂亮的方式來編寫IF條件。

噹噹前時間不在HK工作時間時，該函數將返回True。 它也會在一天結束之前的15分鐘（6點）返回TRUE（香港下班時間），並且在開放時間前30分鐘返回FALSE（作爲香港工作時間）。

function hkOffHours($endHour = 18, $startHour = 9) { 
    $offHour = FALSE; 

    // create the DateTimeZone object for later 
    $dtzone = new DateTimeZone('Asia/Hong_Kong'); 

    // first convert the timestamp into a string representing the local time 
    $time = date('r', time()); 

    // now create the DateTime object for this time 
    $dtime = new DateTime($time); 

    // convert this to the user's timezone using the DateTimeZone object 
    $dtime->setTimeZone($dtzone); 

    // print the time using your preferred format 
    $time = $dtime->format('g:i A m/d/y'); 

    $hour = $dtime->format('G'); 
    $min = $dtime->format('i'); 
    $day = $dtime->format('D'); 

    // if weekend 
    if(($day == 'Sun' || $day == 'Sat')) 
    { 
     $offHour = TRUE; 
    } 

    // if out of office hours 
    if (
     ($admin_group == '1') && 
     (
      ($hour == ($endHour-1) && $min>=45) || 
      ($hour >= $endHour) || 
      ($hour == ($startHour-1) && $min <= 30) || 
      ($hour <= ($startHour -2)) 
     ) 
    ) 
    { 
     $offHour = TRUE; 
    } 

    return $offHour;   
} 

我會感激你的想法。

Answer 1

那麼，我的第一個想法是，如果你還沒有這樣做，你可以通過將它分解成幾個方法和一個類來重構它。

例如：

$offHour = FALSE; 

    // create the DateTimeZone object for later 
    $dtzone = new DateTimeZone('Asia/Hong_Kong'); 

    // first convert the timestamp into a string representing the local time 
    $time = date('r', time()); 

    // now create the DateTime object for this time 
    $dtime = new DateTime($time); 

    // convert this to the user's timezone using the DateTimeZone object 
    $dtime->setTimeZone($dtzone); 

    // print the time using your preferred format 
    $time = $dtime->format('g:i A m/d/y'); 

    $hour = $dtime->format('G'); 
    $min = $dtime->format('i'); 
    $day = $dtime->format('D'); 

所有上述領域可能是類變量。

if(($day == 'Sun' || $day == 'Sat')) 

這可能被移動到不同的功能，稱爲isWeekend()

if (
    ($admin_group == '1') && 
    (
     ($hour == ($endHour-1) && $min>=45) || 
     ($hour >= $endHour) || 
     ($hour == ($startHour-1) && $min <= 30) || 
     ($hour <= ($startHour -2)) 
    ) 
) 
{ 
    $offHour = TRUE; 
} 

這可移至isOutOfOfficeHours()。

全班模板：

class workingHours{ 

    private $dtzone; 
    private ... 
    (...) 

    public function __construct($dtzone....){ 
    //setup values of class members 
    } 

    private function isWeekend(){} //see above 

    private function isOutOfOfficeHours(){} //also see above 

    public function hkOffHours(){ 
    return $this->isWeekend() || $this->isOutOfOfficeHours(); 
    } 
} 

當然通知書，我寫了這快，我在工作。因此，我的代碼邏輯可能不是100％兼容你的。無論如何，我希望你能明白這個主意。

還有一個最後的想法 - 有許多關於重構的書籍。嘗試一個！他們會讓你成爲更好的編碼器。