回答編集履歴
2
edit
test
CHANGED
|
@@ -30,7 +30,7 @@
|
|
|
30
30
|
|
|
31
31
|
|
|
32
32
|
|
|
33
|
-
print_r(eto(1993)); // returnの
|
|
33
|
+
print_r(eto(1993)); // returnで関数が終了するのでここは通らない行。それにeto()になってる
|
|
34
34
|
|
|
35
35
|
|
|
36
36
|
|
|
@@ -75,3 +75,5 @@
|
|
|
75
75
|
|
|
76
76
|
|
|
77
77
|
つまり、バリデーションするならもっと前。
|
|
78
|
+
|
|
79
|
+
関数の引数をチェックすること。
|
1
edit
test
CHANGED
|
@@ -66,11 +66,11 @@
|
|
|
66
66
|
|
|
67
67
|
「何が無効なのか」をきちんと定義されたほうが良いと思います。
|
|
68
68
|
|
|
69
|
-
「
|
|
69
|
+
「文字列かどうか」をチェックしたいなら$zyunishiではなく$seirekiをチェックすべきです。
|
|
70
70
|
|
|
71
|
+
$seirekiが「年数としてありえない」なら、[checkdate関数](https://www.php.net/manual/ja/function.checkdate.php)する必要があるでしょうしね。
|
|
71
72
|
|
|
72
|
-
|
|
73
|
-
|
|
73
|
+
[引数の型宣言](https://www.php.net/manual/ja/functions.arguments.php)も有効かもしれません。（[TypeError](https://www.php.net/manual/ja/class.typeerror.php)）
|
|
74
74
|
|
|
75
75
|
|
|
76
76
|
|