PHPで、平日のみをカウントするプログラムで困っています

Question

PHP初心者です。

PHPで平日だけをカウントアップする関数を作ったのですが、スタートが土日の時の計算が1日ずれてしまいます。

スタートした日の翌日から平日だけカウントするプログラムで、現在スタートが月曜から金曜なら狙ったカウントをしてくれてます。
月曜にスタートで5営業日後なら翌週の月曜日が返り値といった計算です（本当は曜日ではなくて日付です）。
スタート当日はカウントに含まず翌営業からのカウントな点がプログラムを複雑化してまして…

今のプログラムだと、スタートが土日祝日だと月曜日から1日目のカウントをしてしまって正解より1日少なくカウントされてしまいます。
土日にスタートする場合には、月曜スタート、祝日がある場合には祝日明けがスタートにしたいのですが苦戦しています。

以下書いているコードになります。
アドバイス頂けると嬉しいです、よろしくお願いします。

質問が不明瞭だったようで修正いたしました。

この既存コードに、スタートが土日祝日の場合に休日明けがスタートになるようなコードを追加したいのですがアイデアある方御教授頂けるとありがたいです。

例
スタートの日付
本日10月17日土曜にアクセス
10月19日月曜と処理したい

今のプログラムだと
3営業後は
21日水曜日になってしまいますが、狙っている結果は22日木曜日です。

function getNextBusinessDay($date_start, $count)
{
//祝日を手作業で配列にしています
    $holiday = array( "2015-09-21" => "敬老の日" ,
                      "2015-09-22" => "国民の休日",
                      "2015-09-23" => "秋分の日",
                      "2015-10-12" => "体育の日",
                      "2015-11-3" => "文化の日",
                      "2015-11-23" => "勤労感謝の日",
                      "2015-12-23" => "天皇誕生日",
                      );//祝日など納期除外の日付を連想配列で入力
    
    $date = clone $date_start;

    $oneday = new DateInterval("P1D");
   
    while ($count > 0) {
    $date->add($oneday);
    $str = $date->format("Y-m-d");
    $w = $date->format("w");
    if ($w < 1 #日曜
    || $w > 5 #土曜
    || isset($holiday[$str])) { #祝日
    continue;
    }    
    $count--;
    }
    return $date;
}

$order_time = new DateTime();
$order_time = $order_time->format('Hi');
 $date = new DateTime();

コード

Comments

[退会済みユーザー]

何がしたいのかイマイチ読み取れませんので、確認です。
たとえば、本日（2015年10月16日　金曜日）を基準にすると、次の営業日は2015年10月19日（月曜日）ですね。この場合に、どんな結果を返す関数を作りたいのですか？

[yoyoyoyogi]

わかりにくくて申し訳ありません。
次の営業日はおっしゃる通り、19日月曜です。
関数の引数$countに渡す値が「何営業日後」と言う値になるのですが、６を渡したら、19日から数えて６営業日後の27日が返ってくるようにしたいです。

[退会済みユーザー]

なるほど。翌営業日だけではなく、○営業日後が何日なのかを返す関数を作りたいということですね。では、このように関数を作りたいのだということを明確に、質問文を編集してください。補足部分は見ない人も多いので。（見逃しやすい）

[退会済みユーザー]

6営業日後だと26日だはありませんか？

Answer 1

シンプルに考えてみてはどうでしょうか
土日祝だったら1日ほしい仕様より足りない
ならば
土日祝だったら最初のカウントに+1してしまえばいい

その際に平日かどうか的な関数を作ってしまえば見た目的にもスマートです。
コードはPHPっぽい擬似コードです

if(!is_平日かどうか){
  //平日じゃなかったら
  $count++;
}

while($count > 0){

  if(is_平日かどうか){
    // 平日なら
    $count--;
  }else{
    //土日祝なら
    continue;
  }

}

fuction is_平日かどうか($date) {
  return (boolean) 平日かどうか;
}

ただ、それ以外に一点この仕様だと気になるところがあって、
祝日の日付を配列に定義しておく
だと、ハッピーマンデーの対応はどうするのかな
と思う次第です。

Answer 2

こんな感じでいかがでしょうか？

ポイントは

• 起算日を"次の平日"にしてから日数の加算を実行する。（3行目）
• 1日加算するたびに、平日でなければ"次の平日"まで移動させる。（8行目）

です。

ただし、日数の減算（$count < 0の場合）は考慮していません。

また、祝日リスト ($public_holiday) をメンテナンスし続けるのは大変なので、
Kosuke_Shibuya様の回答のように、ご使用の環境が許すなら、ライブラリの導入を検討されてはいかがでしょうか？

function getNextBusinessDay($date_start, $count)
{
    $date = toNextWeekday($date_start);

    for ($i = 0; $i < $count; $i++)
    {
        $date = $date->add(new DateInterval('P1D'));
        $date = toNextWeekday($date);
    }

    return $date;
}

/**
 * 引数で指定した日付が平日なら同じ日付を、平日でなければ"次の平日"を返却する。
 */
function toNextWeekday($date)
{
//祝日を手作業で配列にしています
    $public_holiday = array( "2015-09-21", // 敬老の日
                      "2015-09-22", // 国民の休日
                      "2015-09-23", // 秋分の日
                      "2015-10-12", // 体育の日
                      "2015-11-03", // 文化の日
                      "2015-11-23", // 勤労感謝の日
                      "2015-12-23", // 天皇誕生日
                      );//祝日など納期除外の日付を連想配列で入力

    $holiday = array(0, 6); // 0:日曜, 6:土曜

    $date = clone $date;

    while (in_array($date->format('Y-m-d'), $public_holiday)
            || in_array($date->format('w'), $holiday))
    {
        $date = $date->add(new DateInterval('P1D'));
    }

    return $date;
}

$today = new DateTime('2015-10-17');
print getNextBusinessDay($today, 0)->format('Y-m-d') . "\n";
print getNextBusinessDay($today, 1)->format('Y-m-d') . "\n";
print getNextBusinessDay($today, 2)->format('Y-m-d') . "\n";
print getNextBusinessDay($today, 10)->format('Y-m-d') . "\n";
print getNextBusinessDay(new DateTime('2015-12-22'), 1)->format('Y-m-d') . "\n";

実行結果

2015-10-19
2015-10-20
2015-10-21
2015-11-02
2015-12-24

Comments

[退会済みユーザー]

元のコードを生かしつつ、可読性も良くていいコードですね。

[KiyoshiMotoki]

Kosuke_Shibuya様、ありがとうございます。

Answer 3

PHPで平日だけをカウントアップする
スタートした日の翌日から平日だけカウントする
土日にスタートする場合には、月曜スタート、祝日がある場合には祝日明けがスタート

3行目だけが何故そのようにしたいのか不明です。
営業日という表現だと変ですし、24時間まるまる作業できる日数というわけでもないんですね。
仕様をもう一度見直したほうが良いとは思いますけど…

しかし、何はともあれそのようなルールで動作する関数にしたいならば
ループ開始の前にスタート日をチェックして、土日祝日ならcountに1を加算すれば解決すると思います。

Answer 4

祝日・振替休日を取得するには、メンテナンスも楽なので、 PEAR::Date_Holidays_Japan にお任せする。

インストールは以下のコマンドで。

# pear install Date_Holidays_Japan-alpha

Calendar クラス（Calendar.class.php）の実装は以下のように。

<?php

/**
 * カレンダークラス
 * Calendar.class.php
 */
final class Calendar
{

	/**
	 * 日曜日
	 */
	const T_SUNDAY = 0;

	/**
	 * 月曜日
	 */
	const T_MONDAY = 1;

	/**
	 * 火曜日
	 */
	const T_TUESDAY = 2;

	/**
	 * 水曜日
	 */
	const T_WEDNESDAY = 3;

	/**
	 * 木曜日
	 */
	const T_THURSDAY = 4;

	/**
	 * 金曜日
	 */
	const T_FRIDAY = 5;

	/**
	 * 土曜日
	 */
	const T_SATURDAY = 6;

	/**
	 * キャッシュに使用する祝日配列
	 * @var array(YYYY => array('YYYY-mm-dd' => '祝日名'))
	 */
	static private $_arrHolidays = array();

	/**
	 * 祝日を取得する
	 * @param integer $year
	 * @return array
	 */
	private static function getHolidays($year)
	{
		// 取得済みならキャッシュを使う
		if (isset(static::$_arrHolidays[$year])) {
			return static::$_arrHolidays[$year];
		}

		// 未取得のとき
		$errorLevel = error_reporting(); // エラー出力レベルを取得
		error_reporting(E_NOTICE); // PHP Deprecated: Non-static method... を抑止

		require_once 'Date/Holidays.php';
		$dh = Date_Holidays::factory('Japan', $year, 'ja_JP');
		$holidays = array();
		foreach ($dh->getHolidays() as $h) {
			$holidays[$h->getDate()->format('%Y-%m-%d')] = $h->getTitle();
		}
		error_reporting($errorLevel); // エラー出力レベルを戻す

		ksort($holidays);

		static::$_arrHolidays[$year] = $holidays;
		return static::$_arrHolidays[$year];
	}

	/**
	 * 祝日かどうかを判定する
	 * @param \DateTime $objDate 判定する日付
	 * @return bool 祝日ならtrue
	 */
	public static function isHoliday(\DateTime $objDate)
	{
		$year = $objDate->format('Y');
		$holidays = self::getHolidays($year);
		return array_key_exists($objDate->format('Y-m-d'), $holidays);
	}

	/**
	 * 定休日かどうかを判定する
	 * @param \DateTime $objDate 判定する日付
	 * @return bool 定休日ならtrue
	 */
	public static function isRegularHoliday(\DateTime $objDate)
	{
		//定休日リスト
		$arrRegularHoliday = array(
			self::T_SUNDAY
			, self::T_SATURDAY
		);
		return in_array($objDate->format('w'), $arrRegularHoliday);
	}

	/**
	 * ○日後の営業日を取得する
	 * @param integer $dateAfter 指定しない時は翌営業日を返す
	 * @param \Datetime $objDate 基準日
	 * @return \Datetime
	 */
	public static function getWeekdayAfter($dateAfter = 1, \DateTime $objDate = null)
	{
		$date = (is_null($objDate)) ? new \DateTime() : $objDate;
		$interval = new \DateInterval("P1D");
		for ($i = 0; $i < $dateAfter; $i++) {
			$date->add($interval);

			//	定休日・祝日のときはカウントしないので、$dateAfterをインクリメント
			if (self::isRegularHoliday($date) || self::isHoliday($date)) {
				$dateAfter++;
			}
		}
		return $date;
	}

}

利用する時はこんな感じ。

<?php

require 'Calendar.class.php';

// usage
// 2015-10-10(Friday) -> 2015-10-12(Monday) 体育の日
$res = Calendar::getWeekdayAfter(1, new \DateTime('2015-10-10'));
var_dump($res); // 2015-10-13

// 2015-05-01(金)
// 2015-05-02(土)
// 2015-05-03(日・憲法記念日)
// 2015-05-04(みどりの日)
// 2015-05-05(こどもの日)
// 2015-05-06(振替休日)
$res = Calendar::getWeekdayAfter(1, new \DateTime('2015-05-01'));
var_dump($res); // 2015-05-07

---

例
スタートの日付
本日10月17日土曜にアクセス
10月19日月曜と処理したい

今のプログラムだと
3営業後は
21日水曜日になってしまいますが、狙っている結果は22日木曜日です。

この条件おかしいのでは？？？

10/17（土）基準日
10/18（日）
10/19（月）1営業日後　- 引数には1を渡す（または未指定のときは暗黙的に翌営業日として扱う）
10/20（火）2営業日後　- 引数には2を渡す
10/21（水）3営業日後　- 引数には3を渡す
10/22（木）4営業日後　- 引数には4を渡す

このように考えたほうが、利用場面で自然だと思いますが…

Answer 5

具体的なコードがあるので、ここをこう直せば良いと書いてあげれば喜ぶとは思うのですが
初心者と書かれていたので後々のためのことを考えてアドバイスに留めておきます。
以下のサイトで同様のことを時系列で、解説しています。
少し長いですが、全てを読んだ頃には理解出来ているのではないかと思います。
http://giraffe.topaz.ne.jp/wiki/doku.php/py:lesson12