ユーザーの入力した文字列が日付として有効かどうかをチェックする方法

Question

PHPのプログラミングについての質問です。

日付を入力するフォームがあり、送信された文字列が日付として有効かどうかをチェックしています。
年月日を別々に入力させてcheckdate関数でチェックする方法が無難だと思いますが今回はできません。

現状はこんな感じで書いています

//送信された日付文字列
$date = $_POST('date');
//日付かどうかチェック
if(strtotime($date) === false){
    //ダメなときの処理
    $date = null;
}else{
    //下記フォーマットに統一する
    $date = date('Y-m-d',strtotime($date));
}

同じ様なチェックをDateTimeオブジェクトを使ってシンプルに書くことはできるでしょうか？

ユーザーからの入力は、YYYY-MM-DD、YYYY/MM/DD、YYYYMMDDを想定しています。

よろしくお願いします。

Comments

[yuba]

2017-02-31は拒絶しますか、受理しますか？

[msx2]

2月31日はエラーとします

Answer 1

ユーザーからの入力は、YYYY-MM-DD、YYYY/MM/DD、YYYYMMDDを想定

であればパターンを設定して正規表現でチェックしてください
そうでない場合他の想定していないパターンにもマッチします
またフォーマットがあっていればstrtotimeは無理やり日付として
解釈するので、極端な値を設定したり0月や20月とか0日とか99日などを
チェックしてもそれなりに解釈します。

$a=["2017-02-14","2017/02/15","20170216",//問題なくマッチ
    "2017/2/17","2017020180","2017/02-19","xxxx-yy-zz",//エラー
    "0000-00-00","9999-99-99" //極端な数字でもマッチ
    ];
/* $pattern="#^\d{4}[/-]{0,1}\d{2}[/-]{0,1}\d{2}$#"; */
/* おかしなパターンを拾うのでいかに修正 */
$pattern="#^\d{4}([/-]?)\d{2}\\1\d{2}$#";
foreach ($a as $val){
  print $val." is ";
  if(preg_match($pattern,$val,$match)){
    print date("Y-m-d",strtotime($val))."<br>";
  }else{
    print "ng<br>";
  }
}

おまけ

POSTで受けたデータを前提とするのであれば、filter_inputで先に
バリデートするとか、日付形式の文字かどうかをcheckdate()するとか
色々やりようはあります

<form method="post"><input type="text" name="a" value="2017-02-14"><input type="submit" value="go"></form>
<form method="post"><input type="text" name="a" value="2017/02/15"><input type="submit" value="go"></form>
<form method="post"><input type="text" name="a" value="20170216"><input type="submit" value="go"></form>
<form method="post"><input type="text" name="a" value="2017/2/17"><input type="submit" value="go"></form>
<form method="post"><input type="text" name="a" value="2017020180"><input type="submit" value="go"></form>
<form method="post"><input type="text" name="a" value="2017/02-19"><input type="submit" value="go"></form>
<form method="post"><input type="text" name="a" value="xxxx-yy-zz"><input type="submit" value="go"></form>
<form method="post"><input type="text" name="a" value="0000-00-00"><input type="submit" value="go"></form>
<form method="post"><input type="text" name="a" value="0001-01-01"><input type="submit" value="go"></form>
<form method="post"><input type="text" name="a" value="9999-12-31"><input type="submit" value="go"></form>
<form method="post"><input type="text" name="a" value="2017-00-01"><input type="submit" value="go"></form>
<form method="post"><input type="text" name="a" value="2017-01-40"><input type="submit" value="go"></form>

$pattern='#^(\d{4})([/-]?)(\d{2})\2(\d{2})$#';
$a=!isset($_POST["a"])?NULL:filter_input(INPUT_POST,'a',FILTER_VALIDATE_REGEXP,[
  'options'=>[
    'default'=> "",
    'regexp'=> $pattern
    ],
  ]);
if(!is_null($a)){
  if(preg_match($pattern,$a,$m) and checkdate($m[3],$m[4],$m[1])){
    print date("Y-m-d",strtotime($a))."<br>";
  }else{
    print "bad data!";
  }
}

Comments

[msx2]

やはり入力形式を絞ると正規表現のチェックが必要になるんですね。
正規表現が苦手で避けていましたが少し勉強してみます。。
ありがとうございました！

[KiyoshiMotoki]

横から失礼します。

　#^\d{4}[/-]{0,1}\d{2}[/-]{0,1}\d{2}$#
だと、
　2017/02-14
や
　201702/14
のような、区切り文字がチグハグな文字列にもマッチしてしまいます。

そのため、正規表現を使うのであれば、
以下のようにそれぞれの区切り文字で個別にチェックする方が良いと思います。
　#^\d{4}/\d{2}/\d{2}$#
　#^\d{4}-\d{2}-\d{2}$#
　#^\d{8}$#

[yambejp]

KiyoshiMotokiさんフォローありがとうございます。
たしかにおっしゃるパターンがありますね
ただしパターンはそんなに増やす必要はないので
元の文を修正しておきます

[KiyoshiMotoki]

b

[msx2]

まず入力の書式チェックとして正規表現で検証してから、日付としての妥当性は別途チェックするという流れですね。こうなると一連のチェック関数としてまとめた方がよさそうですね。

[msx2]

おまけの追記ありがとうございます。
filter_inputなんて初めて見ました。
日付チェックの質問からこんなテクニックが出てくるとは、、
有益なご回答を本当にありがとうございます。

[退会済みユーザー]

strtotime のマニュアル見ても良く分からなかったので、
自身のスクリプトの検証と合わせて、こちらのスクリプトにも
気になる日付を試してみました。

2017/02/8 is ng
2017/02/30 is 2017-03-02
2017028 is ng
20000000 is 1999-11-30
20170230 is 2017-03-02
20170431 is 2017-05-01
2017/04/31 is 2017-05-01

フォーマットチェックがある分、おかしな記述は受け付けませんが
月末系はちゃんと処理してやる必要がありそうです。

[msx2]

検証ありがとうございます！！
dは月関係なく31まで受け入れるんですね

[退会済みユーザー]

strtotime を利用するなら checkdate が必要そうです。

Answer 2

$d = DateTime::createFromFormat('Y-m-d', $date) or
     DateTime::createFromFormat('Y/m/d', $date) or
     DateTime::createFromFormat('Ymd', $date);

if ($d === false) {
    $date = null;
} else {
	$date = $d -> format('Y-m-d');
}

Comments

[msx2]

こんな風に書けるんですね。
書式が増えても対応できて、失敗はfalseが返ってきて、何よりもわかりやすい！
ありがとうございました！！

[KiyoshiMotoki]

横から失礼します。

$d = DateTime::createFromFormat('Y-m-d', $date) or
　　DateTime::createFromFormat('Y/m/d', $date) or
　　DateTime::createFromFormat('Ymd', $date);

について、PHP の 'or' 演算子は '=' 演算子より優先順位が低いので、最も左の
　DateTime::createFromFormat()
の結果だけが、変数 $d に代入されることになりますよ。
http://php.net/manual/ja/language.operators.precedence.php

'or' の代わりに '||' を使うと、意図通りの動作になるかと思います。

--------------------------------------
<?php
$d = false || false || true;
var_dump($d);

$d = false or false or true;
var_dump($d);
--------------------------------------

実行結果
bool(true)
bool(false)

[yuba]

あ、そしたら||にするのでなく( )で囲ってもらうのがいいです。
|| だとDateTimeオブジェクトでなくboolが出てきてしまうので。

[msx2]

KiyoshiMotoki様
度々のフォローありがとうございます。
演算子の優先順位なんてあまり考えたことがなかったのですが、なるほどこういう事なんですね。
今までわからなかったらとりあえずカッコ（）で括っていましたけど演算子の優先順位を意識することで無用な（）を減らせそうです。（）括りは読みやすさというメリットもありますけど。

[msx2]

> || だとDateTimeオブジェクトでなくboolが出てきてしまうので
そんな違いがあるとは！？
理由はわかりませんが勉強になります！奥が深い…

[KiyoshiMotoki]

再び横から失礼します。

改めて確認してみたところ、
'||' と 'or' のいずれも、() で囲ったとしても
（演算の評価順は意図通りになるものの） $d には boolean値が代入されてしまいました。
--------------------------------------------------------
<?php
$d = (false || false || new Datetime('now', new DateTimeZone('Asia/Tokyo')));
var_dump($d);

$d = (false or false or new Datetime('now', new DateTimeZone('Asia/Tokyo')));
var_dump($d);
--------------------------------------------------------

実行結果
bool(true)
bool(true)

考えてみれば '||' も 'or' も実行結果として boolean を返すため、これは当然の動作ですね(^^;
（すなわち、私の
　「'or' の代わりに '||' を使うと、意図通りの動作になるかと思います。」
　という指摘も誤りでした）
http://php.net/manual/ja/language.operators.logical.php

というわけで、以下のようにすると、今度こそ意図通りの結果が得られます。
しかし、ちょっと無理矢理感が否めませんね、、
--------------------------------------------------------
<?php
($d = false) || ($d = new Datetime('now', new DateTimeZone('Asia/Tokyo'))) || ($d = false);
var_dump($d);

($d = false) or ($d = new Datetime('now', new DateTimeZone('Asia/Tokyo'))) or ($d = false);
var_dump($d);
--------------------------------------------------------

実行結果
object(DateTime)#1 (3) {
["date"]=>
string(26) "2017-02-14 17:02:03.000000"
["timezone_type"]=>
int(3)
["timezone"]=>
string(10) "Asia/Tokyo"
}
object(DateTime)#1 (3) {
["date"]=>
string(26) "2017-02-14 17:02:03.000000"
["timezone_type"]=>
int(3)
["timezone"]=>
string(10) "Asia/Tokyo"
}

[msx2]

となると回答にあるコードの最初の部分を以下に修正すればOKですね
$d = DateTime::createFromFormat('Y-m-d', $date) or
$d = DateTime::createFromFormat('Y/m/d', $date) or
$d = DateTime::createFromFormat('Ymd', $date);

短絡評価で$dにfalse以外が入った時点で以降の式が評価されないのでどの書式がきても$dにDateTimeオブジェクトが入ることを確認しました。

[yuba]

すみません、その通りでした。

[yuba]

あ、msx2さんの最後のその書き方かっこいい。

[msx2]

この回答が私のような初級者にとってわかりやすく、他の方にも役立つ情報になると思いますので回答を修正いただけるとありがたいです。
ベストアンサーにしたいですし。

[KiyoshiMotoki]

b

Answer 3

終わった後の検証作業
dについて
01 から 31 あるいは 1 から 31 とあるので最後の日付だけ通るてしまうのではないでしょうか
print_r(date_parse_from_format ( $format ,$date ));

<?php

$dates = [ 
		'2017028',
		'2017208'
];
function chk_date($date) {
	$formats = [ 
			'Ymd' 
	];
	foreach ( $formats as $format ) {
		DateTime::createFromFormat ( $format, $date );
		echo "<pre>";
		print_r(date_parse_from_format ( $format ,$date ));
		echo "</pre>";
		$result = DateTime::getLastErrors ();
	
		if (! $result ['warning_count'] && ! $result ['error_count']) {
			return TRUE;
		}
	}
	return FALSE;
}

foreach ( $dates as $date ) {
	echo $date . ':';
	echo chk_date ( $date ) ? 'OK' : 'NG';
	echo PHP_EOL;
}

Array
(
    [year] => 2017
    [month] => 2
    [day] => 8
    [hour] => 
    [minute] => 
    [second] => 
    [fraction] => 
    [warning_count] => 0
    [warnings] => Array
        (
        )

    [error_count] => 0
    [errors] => Array
        (
        )

    [is_localtime] => 
)

OK 2017208:

Array
(
    [year] => 2017
    [month] => 20
    [day] => 8
    [hour] => 
    [minute] => 
    [second] => 
    [fraction] => 
    [warning_count] => 1
    [warnings] => Array
        (
            [7] => The parsed date was invalid
        )

    [error_count] => 0
    [errors] => Array
        (
        )

    [is_localtime] => 
)

Comments

[退会済みユーザー]

ちゃんとマニュアルにあたるべきでした＾＾；書いてありますね。
d 日。二桁の数字（先頭にゼロがつく場合も）
http://php.net/manual/ja/function.date.php

つかない場合もあるってことですね。ここの回答2回めの指摘ありがとうございます。
お恥ずかしい＾＾；

[msx2]

(^o^)b

[退会済みユーザー]

'20172008'が通るんですけど、何かわかりますか？
[month] => 20
が入っています。
===追記
失礼しました。上は誤りでした。
'20172008' は通りません。
'Ym' のフォーマットで、'201720'が '[month] => 20'で通ってしまいます。
なにかご存知であれば＾＾；

[msx2]

'20172008'は通りましたよ
[month] => 20
[day] => 8
PHPは5.6です

[退会済みユーザー]

私も 5.6 と 7.0 で試しましたが、通りませんでした。
'20172008' と 'Ymd' の組み合わせで
[warning_count] => 1
が出ています。
msx2 さんの環境では出ていませんか？

Answer 4

PHP DateTime
DateTimeクラスで、不正な日付フォーマットの場合にException投げるようですよ。

Comments

[msx2]

ありがとうございます。
とりあえず入れてみて例外が発生するかどうかでチェックできますね。
ただ今回は日付として有効でない文字列（ほどんどのケースは空ですが）も想定していて、そういった入力を異常値ではなく正常な入力の一つとして取り扱いたいと考えています。

Answer 5

せっかくなんで、備忘録兼ねて回答します。

<?php
$dates = [
'2017-02-28',
'2017/02/28',
'20170228',
'2016-02-29',
'1800-02-28',
'2017-02-31',
'17-02-29',
'hogehoge',
];

function chk_date($date)
{
    $formats = [
    'Y-m-d',
    'Y/m/d',
    'Ymd',
    ];
    foreach ($formats as $format){
        DateTime::createFromFormat($format, $date);
        $result = DateTime::getLastErrors();
        if(!$result['warning_count'] && !$result['error_count'])
        {
            return TRUE;
        }
    }
    return FALSE;
}

foreach ($dates as $date){
    echo $date . ':';
    echo chk_date($date)?'OK':'NG';
    echo PHP_EOL;
}

勉強になりました。

Comments

[msx2]

まとめていただいてありがとうございます。
PHPを始めた当初からどうするのがいいものか疑問に思っていて、今回ようやく納得のいく答えに行き着いた気がします。
ほんと何でも質問してみるものですね、とても勉強になりました。
この質問に投稿してくれた人たちのおかげです、ありがとうございました。
ベストアンサー悩みますが明日にでも付けたいと思います。

[退会済みユーザー]

'2017028' が `2017-02-08` として通る。。。何でだろ？
'2017208', はちゃんと NG だけど。。。

[退会済みユーザー]

ベストアンサーは非常に光栄なのですが、要求仕様を満たしてませんorz
どなたか、'2017028' が `2017-02-08` として通る件を解決していただけると幸いです。`d`の挙動が意図通りじゃないんですよねぇ。。。

[退会済みユーザー]

date さんの指摘通り、私が `d` の挙動を勘違いしていました。
(Blog 情報は鵜呑みにしちゃダメですね。。。)

http://php.net/manual/ja/function.date.php
d 日。二桁の数字（先頭にゼロがつく場合も）

ということで、許容しても良い気がしますが、厳密にやるなら`d` の取扱は要チェックです＾＾；

Answer 6

日付の正当性以前に $date = $_POST('date');が気になりますが、それは置いておいて、kunai さんの提案のように、DateTimeクラスで良くないですか？

exit しなければ、やりたいことは実現できる気がするのですが。

Comments

[msx2]

ですね(^^
いい方法を教えてもらえましたのでDateTimeクラスを使うことにしました。

[date]

一つ注意しないといけないのですが
$date = new DateTime('2017-02-31');
仮のこのようなデータが入ると
2017-03-03
と通ってしまうためこのような場合の対処はしないといけません

[退会済みユーザー]

マニュアルから FALSE が返ると思ってました。
フォーマットの正当性のみの確認なんですね。
ということは、DateTime を使用した回答は全滅ですね＾＾；

[退会済みユーザー]

幾つか手元で試してみました。
YYYY-MM-DD、YYYY/MM/DD、YYYYMMDD のみであれば、入力を正規表現で拾い、DateTime で同じ出力結果を出せるなら正しい。とかやってみましたが、それなら、素直に checkdate 使えよ！って感じでした。
あんまりきれいにならないですね＾＾；

指摘ありがとうございました。

[msx2]

なんてこった（汗）確かに通ってしまいますね。
実際に試してみると、new DateTime('2017-01-32')は例外になりますが、DateTime::createFromFormat('Y-m-d','2017-01-32')は通ってしまいます。
もうあり得ない日付でもいいかなと思いつつも調べてみたらDateTimeクラスのgetLastErrorsメソッドでチェックできるみたいです。

[退会済みユーザー]

'warning_count' と 'error_count' 見ることで、OKっぽいですね。
いやぁ、勉強になりました。

[退会済みユーザー]

備忘録兼ねて、別回答作成しました。