質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.35%
PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

関数

関数(ファンクション・メソッド・サブルーチンとも呼ばれる)は、はプログラムのコードの一部であり、ある特定のタスクを処理するように設計されたものです。

コードレビュー

コードレビューは、ソフトウェア開発の一工程で、 ソースコードの検査を行い、開発工程で見過ごされた誤りを検出する事で、 ソフトウェア品質を高めるためのものです。

Q&A

3回答

1199閲覧

checkdate関数を自作して日付を検証したが正しくでません

patorioddo623

総合スコア12

PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

関数

関数(ファンクション・メソッド・サブルーチンとも呼ばれる)は、はプログラムのコードの一部であり、ある特定のタスクを処理するように設計されたものです。

コードレビュー

コードレビューは、ソフトウェア開発の一工程で、 ソースコードの検査を行い、開発工程で見過ごされた誤りを検出する事で、 ソフトウェア品質を高めるためのものです。

0グッド

0クリップ

投稿2021/07/28 23:59

編集2021/07/29 01:43

とある課題でcheckdate関数と同じ機能ですが、以下のような999999年まで判定可能な関数を自作して数個の日付の妥当性を検証しました。
しかし、999999年12月31日だけ存在するはずなのに正しくないとの表示になりました。
下記のコードでここを直せば正しく表示されるという箇所があればご指摘ください。

php

1<?php 2 function isLeapYear($year){ 3 return ((($year % 4) == 0) && ((($year % 100) != 0) || (($year % 400) == 0))); 4 } 5 function mycheckdate($month,$day,$year) { 6 if($month < 1 || $month > 12) { 7 return false; 8 } 9 if($day < 1 || $day > 31) { 10 return false; 11 } 12 if($year < 1 || $year > 99999) { 13 return false; 14 } 15 return true; 16 } 17 18 function debug_mcd($month, $day, $year){ 19 print($year."年".$month."月".$day."日は正しい日付"); 20 if(mycheckdate($month,$day,$year)){ 21 print("です."); 22 } else{ 23 print("ではありません."); 24 } 25 print("<br>"); 26 } 27 28 debug_mcd(7, 2, 2020); // 正しい 29 debug_mcd(7, 32, 2020); // 正しくない 30 debug_mcd(2, 29, 2400); // 正しい 31 debug_mcd(2, 29, 2401); // 正しくない 32 debug_mcd(12, 31, 99999); // 正しい 33 debug_mcd(12, 32, 99999); // 正しくない 34 debug_mcd(2, 29, 99999); // 正しくない 35 debug_mcd(2, 29, 99996); // 正しい 36 37?>

表示結果

2020年7月2日は正しい日付です.
2020年7月32日は正しい日付ではありません.
2400年2月29日は正しい日付です.
2401年2月29日は正しい日付ではありません.
99999年12月31日は正しい日付ではありません.
99999年12月32日は正しい日付ではありません.
99999年2月29日は正しい日付ではありません.
99996年2月29日は正しい日付です.

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

int32_t

2021/07/29 00:10

「HTML」タグと「関数型プログラミング」タグは質問内容にまったく関係ないので、外してください。
javahack

2021/07/29 02:00 編集

既に指摘されていますが、うるう年かどうかって正しい日付の判定に関係ありますか? 対象年の2月29日が正しい日付かの判定に必要でしたね。 失礼しました。
guest

回答3

0

if(isLeapYear($year) < 1 || isLeapYear($year) > 99999) {

が意味不明な上に、大の月小の月の判断もないし、2月の判断も無い。

このプログラムで判断できると思ったのであれば、暦に関する知識がほとんど無いということですね。
まずは、「大の月と小の月」というキーワードで調べてみましょう。

おそらく、isLeapYearの中身はコピペでしょうね。isLeapYearはその年が閏年かどうかをチェックして真か偽で返す関数です。
isLeapYear($year)が真なら2/29は正しい日付で、偽なら2/29は正しい日付で無い」
という風に使います。

投稿2021/07/29 01:42

otn

総合スコア85901

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

isLeapYearがブーリアンを返す関数なのに

if(isLeapYear($year) < 1 || isLeapYear($year) > 99999)

の処理をするのはおかしいですね

投稿2021/07/29 00:35

yambejp

総合スコア116724

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

function isLeapYear($year){

この関数は、なにを返す関数なのか考えてみよう

投稿2021/07/29 00:07

y_waiwai

総合スコア88042

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

patorioddo623

2021/07/29 00:15

指定した西暦年がうるう年かどうかを調べ,結果をコマンドの実行結果として返す関数と思います。
y_waiwai

2021/07/29 01:08 編集

で、その結果を数値と比較してますが、それはどういう意図があるんでしょうか
patorioddo623

2021/07/29 01:35

いまその部分は大きく間違っていることにきづき直しました、しかし999999年という閏年ではないが、月・日があっている場合どのように書けばいいかがいまいち理解できません。。
y_waiwai

2021/07/29 01:38

まずは間違ってる部分を正しく修正しましょう。 はなしはそれからです
patorioddo623

2021/07/29 02:36

そこの部分は修正しました
y_waiwai

2021/07/29 04:30

で、あなたの提示したコードでは、あなたの出した結果は出るはずがないです。 本当に、そのコードで出した結果なんでしょうか。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.35%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問