PHPで日付のバリデーションを行いたいです

タイトルの通りです。
行いたいバリデーションの内容は以下の３つです。

ユーザーが入力した日付が
・5,10,15,20,25日である
・月末である
・過去30日以内である（１ヶ月に変更になる可能性有）

自分なりに書いてみたコードはこんな感じです。
※$check['Date']には YYYY/mm/dd の形式でDBから取ってきたデータが入っています。

//30日以内
function DateChk30( $check ) {
    $today = date("Y/m/d");
    $checkunix = strtotime($check['Date']);
    $todatunix = strtotime($today);
    $diff = ($todatunix - $checkunix ) / ( 60 * 60 * 24);

    if ($diff < 30 ) 
    { 
        return true;
    } else {
        return false;
    }
}

//5,10,15,20,25,月末
function DateChk( $check ) {
    $wk = date('Y/m/t');               //月末
    $tmp = substr($check['Date'], -2); //後ろ２文字
    $tmp = (int)$tmp;

    if ( ($tmp % 5) == 0 || $check['Date'] == $wk) 
    {
        return true;
    } else {
        return false;
    }
}

これだと31日が存在する月の30日でも通してしまうので困っています。
switchを使って1日ずつ設定するしかないでしょうか。。
何か良い方法がありましたらご教授ください。

よろしくお願いします。

行動規範の内容に同意します

回答2件

ベストアンサー

「月末である」の判断はdate関数のtを利用されているのでそれで良いと思います。
「指定月の日数」ですが、それはイコール「月の最終日」とも判断できますしね。
私もカレンダー作成の際によく使います。

PHPで月末日を取得する

原因は「5,10,15,20,25日である」チェックと「月末」であるのチェックを同時にしていることですね。
||で確認しているのでどちらかがtrueであれば通ってしまいます。
この2つは要件も違いますし、DateChk30()を別にされているので、DateMonthEndChk()のような月末チェック専用のメソッドを用意されてはどうでしょうか。

もう1つ、今回のチェックより前に入れたいのが「日付の型チェック」ですね。
$check['Date']が正しい前提で今回のチェックを入れられているようですが、
ユーザーからの入力は基本信頼しないというセキュリティ上の考え方もあります。
（※既に導入済みでしたらスルーしてください）

投稿2017/08/10 05:56

編集2017/08/10 05:57