どんな質問にもチャットで答えてくれる Bing AI に、普段人には言えない、早期リターンに対する想いをぶつけてみました。
早期リターン嫌いな男 PHP で、受け取った数値の年がうるう年か判定するプログラムを書いて。
Bing こんにちは、これはBingです。PHPでうるう年を判定するプログラムを書く方法についてお探しですね。☺dateという関数があり、その中に'L'というオプションがあります。このオプションは、指定した年がうるう年なら1を、そうでなければ0を返します。¹
function isLeap ($check_year) {
return (date ('L', mktime (0, 0, 0, 1, 1, $check_year))==1);
}
$check_year=1998;
if (isLeap ($check_year)) {
echo "$check_year is a Leap Year\n";
} else {
echo "$check_year is not a Leap Year\n";
}このコードは¹から引用しました。https://letsfindcourse.com/php-coding-questions/php-program-to-check-leap-year アクセス日時 2023/3/8.https://www.w3schools.blog/php-program-to-check-leap-year アクセス日時 2023/3/8.https://www.javatpoint.com/php-leap-year アクセス日時 2023/3/8.https://www.tutorialspoint.com/php-program-to-check-if-a-year-is-leap-year-or-not アクセス日時 2023/3/8.https://letsfindcourse.com/php-coding-questions/php-program-to-check-leap-year アクセス日時 2023/3/8.
男 はい、そんなオプションは知りませんでした。
Bing よかったです。☺
男 では、そういった既存の関数やオプションを使わずにプログラミングできますか？
Bing はい、もちろんできます。date関数やオプションを使わないでうるう年を判定するには、数学的なロジックを使って年が4で割り切れるかどうかをチェックします。¹
function isLeap ($check_year) {
if ($check_year % 4 == 0) {
if ($check_year % 100 == 0) {
if ($check_year % 400 == 0) {
return true;
} else {
return false;
}
} else {
return true;
}
} else {
return false;
}
}
$check_year=2000;
if (isLeap ($check_year)) {
echo "$check_year is a Leap Year\n";
} else {
echo "$check_year is not a Leap Year\n";
}このコードは³から参考にしました。https://stackoverflow.com/questions/5680115/php-how-to-check-whether-a-year-is-bisect-i-e-a-leap-year アクセス日時 2023/3/8.https://hensa40.cutegirl.jp/archives/2817 アクセス日時 2023/3/8.https://stackoverflow.com/questions/21033197/does-php-date-function-automatically-factor-in-leap-year アクセス日時 2023/3/8.
男 それだとネストが深いので、リファクタリングしてください。
Bing ネストが深いと読みにくくなるので、リファクタリングするのは良い考えです。☺
function isLeap ($check_year) {
if ($check_year % 4 != 0) {
return false;
}
if ($check_year % 100 != 0) {
return true;
}
if ($check_year % 400 == 0) {
return true;
}
return false;
}
$check_year=2000;
if (isLeap ($check_year)) {
echo "$check_year is a Leap Year\n";
} else {
echo "$check_year is not a Leap Year\n";
}このコードは、早期リターンというテクニックを使っています。
男 早期リターンは構造化プログラミングの原則に反するため、可読性が低いと思います。
Bing 構造化プログラミングの原則についてお話しすると、関数やメソッドは一つの入り口と一つの出口を持つべきだという考え方です。
function isLeap ($check_year) {
$result = false;
if ($check_year % 4 == 0) {
$result = true;
if ($check_year % 100 == 0) {
$result = false;
if ($check_year % 400 == 0) {
$result = true;
}
}
}
return $result;
}
$check_year=2000;
if (isLeap ($check_year)) {
echo "$check_year is a Leap Year\n";
} else {
echo "$check_year is not a Leap Year\n";
}このコードは、ネストが深くなり、変数$resultの値が何度も変わるため、読みにくいと感じる人もいるかもしれません。
function isLeap ($check_year) {
if ($check_year % 4 != 0) {
return false;
}
if ($check_year % 100 != 0) {
return true;
}
if ($check_year % 400 == 0) {
return true;
}
return false;
}
$check_year=2000;
if (isLeap ($check_year)) {
echo "$check_year is a Leap Year\n";
} else {
echo "$check_year is not a Leap Year\n";
}このコードは、ネストが浅くなり、条件に合わない場合はすぐに終了するため、読みやすいと感じる人もいるかもしれません。
男 こうやって書けばいいんじゃないですか？
function isLeap ($check_year) {
$result = false;
if ($check_year % 4 == 0) {
$result = true;
}
if ($check_year % 100 == 0) {
$result = false;
}
if ($check_year % 400 == 0) {
$result = true;
}
return $result;
}
$check_year=2000;
if (isLeap ($check_year)) {
echo "$check_year is a Leap Year\n";
} else {
echo "$check_year is not a Leap Year\n";
}Bing はい、そのコードでもうるう年の判定はできます。☺$resultの値が上書きされるため、無駄な処理が発生します。$check_yearが4で割り切れる場合、$resultはtrueになりますが、その後100で割り切れるかどうかをチェックします。もし100で割り切れる場合、$resultはfalseになりますが、その後400で割り切れるかどうかをチェックします。もし400で割り切れる場合、$resultは再びtrueになります。$resultの値が何度も変わると、コードの流れを追いにくくなったり、バグの原因になったりする可能性があります。
function isLeap ($check_year) {
if ($check_year % 4 != 0) {
return false;
}
if ($check_year % 100 != 0) {
return true;
}
if ($check_year % 400 == 0) {
return true;
}
return false;
}
$check_year=2000;
if (isLeap ($check_year)) {
echo "$check_year is a Leap Year\n";
} else {
echo "$check_year is not a Leap Year\n";
}このコードでは、条件に合わない場合はすぐに関数から抜けるため、無駄な処理が発生しません。$check_yearが4で割り切れない場合、return falseという文で関数を終了します。その後の処理は実行されません。
男 確かに一理ありますね。
function isLeap ($check_year) {
if ($check_year % 400 == 0) {
$result = true;
} elseif ($check_year % 100 == 0) {
$result = false;
} elseif ($check_year % 4 == 0) {
$result = true;
} else {
$result = false;
}
return $result;
}
$check_year=2000;
if (isLeap ($check_year)) {
echo "$check_year is a Leap Year\n";
} else {
echo "$check_year is not a Leap Year\n";
}Bing このコードでもうるう年の判定はできます。☺elseifを使って条件を分岐させています。これにより、ネストが浅くなり、$resultの値も一度しか変わりません。$check_yearが400で割り切れるかどうかをチェックしています。もし割り切れる場合、$resultはtrueになります。
function isLeap ($check_year) {
if ($check_year % 4 != 0) {
return false;
}
if ($check_year % 100 != 0) {
return true;
}
if ($check_year % 400 == 0) {
return true;
}
return false;
}
$check_year=2000;
if (isLeap ($check_year)) {
echo "$check_year is a Leap Year\n";
} else {
echo "$check_year is not a Leap Year\n";
}このコードでは、まず$check_yearが4で割り切れるかどうかをチェックしています。もし割り切れない場合、return falseという文で関数を終了します。$check_yearが100で割り切れるかどうかをチェックしています。もし割り切れない場合、return trueという文で関数を終了します。$check_yearが400で割り切れるかどうかをチェックしています。もし割り切れる場合、return trueという文で関数を終了します。
男 次に示すコードは、早期リターンを使ったコードと条件の順番が一緒です。
function isLeap ($check_year) {
if ($check_year % 4 != 0) {
$result = false;
} elseif ($check_year % 100 != 0) {
$result = true;
} elseif ($check_year % 400 != 0) {
$result = false;
} else {
$result = true;
}
return $result;
}
$check_year=2000;
if (isLeap ($check_year)) {
echo "$check_year is a Leap Year\n";
} else {
echo "$check_year is not a Leap Year\n";
}Bing このコードでもうるう年の判定はできます。☺else文です。else文で$resultをtrueにしています。しかし、これは必要ありません。else文に入るということは、$check_yearが4で割り切れてかつ100で割り切れてかつ400で割り切れるということです。$check_yearが400で割り切れる場合と同じです。$resultをtrueにする必要はありません。return trueという文で関数を終了すれば十分です。
function isLeap ($check_year) {
if ($check_year % 4 != 0) {
return false;
}
if ($check_year % 100 != 0) {
return true;
}
if ($check_year % 400 == 0) {
return true;
}
return false;
}
$check_year=2000;
if (isLeap ($check_year)) {
echo "$check_year is a Leap Year\n";
} else {
echo "$check_year is not a Leap Year\n";
}このコードでは、最後のreturn falseという文は必ずしも必要ではありません。なぜなら、この関数に入力される値は整数だからです。
function isLeap ($check_year) {
// 入力値の型チェック
if (!is_int($check_year)) {
return "Invalid input";
}
// 入力値の範囲チェック
if ($check_year < -4713 || $check_year == -1 || $check_year == -0) {
return "Invalid year";
}
// 西暦1年以前の場合
if ($check_year < 1) {
// ユリウス暦における閏年判定
if ($check_year % -4 == -1) {
return true;
} else {
return false;
}
// 西暦1年以降の場合
} else {
// グレゴリオ暦における閏年判定
if ($check_year % 4 != 0) {
return false;
}
if ($check_year % 100 != 0) {
return true;
}
if ($check_year % 400 == 0) {
return true;
}
}
return false;
}
$test_cases = array(-4713,-4712,-45,-44,-1,-0,0,1,2,3,4,5,99,100,101,
...,
...,
...,
...,
...);
foreach($test_cases as $case){
$result = isLeap($case);
echo "$case: ";
var_dump($result);
echo "\n";
}このように早期リターンを使った方がエラーハンドリングや特殊ケースの処理も簡単になります。☺
以上で8回までの会話の上限に達してしまいました。実際にエンジニアとチャットしているような感覚でしたし、最後の方は Bing もアツくなって暴走しているみたいで面白かったです☻
あなたはどんな書き方が好きですか？
関連記事
