PHP 関数の引数バリデーションチェック

Question

前提・実現したいこと

三角形の面積の引数チェックに関して質問です。
引数に任意の数字を与える際にいずれかの引数が空の時、または数字でないときはNULLを表示し、
上記ではない時は三角形の面積結果の数字を返す。この時の引数チェックで少し迷っています。

発生している問題・エラーメッセージ

$height, $widthを文字または空文字を入れる場合
NULL(画面上ではから表記)ではなく0と表記されてしまう。

該当のソースコード

ソースコード
function triangle($h,$w){
  $total = $h*$w/2;
  return $total;
}

$height = 100;
$width = 50;
  $sum = triangle($height,$width);
  echo '三角形面積:' . $sum;

試したこと

triangle関数内に下記を試してみました。
1: is_numeric、is_null関数を使って条件分岐
if(!is_numeric($total) && !is_null($total)){
return null;
}else{
return $total;
}
2: preg_match関数を使って条件分岐
if (!preg_match('/\A[0-9]+\z/', $total)) {
return null;
} else {
return $total;
}
3:is_int関数を使って整数チェック
if(is_int($total)){
return $total;
}else{
return null;
}

追記:
上記、計算の合計値をバリデーションチェックしていた為、下記に変更。
if(is_numeric($h)){
if(is_numeric($w)){
$total = $h*$w/2;
return $total;
}else{
return NULL;
}
ご教示のほど宜しくお願い致します。

Comments

[m.ts10806]

関数の引数ではなく直接的に中でやってるのはバリデーションなので、そう表現された方が伝わりやすいと思います。

[m.ts10806]

というか、なぜ渡された引数ではなく計算結果の変数をチェックしているのですか？

[m.ts10806]

何度も追記願う形になりますが、
「NULLを表示」とはどういう結果をイメージされてますか？
"NULL"と画面に表示させたいのか、NULLを返してechoするだけなのか(画面上は空表示)、ハッキリと書かれた方が良いと思います。
現在のコードからは後者を目指しているように見受けられます。

[退会済みユーザー]

ご返信ありがとうございます。
言葉足らずで申し訳ございません。引数のバリデーションチェックに修正致します。
is_numeric関数等、１つのパラメータのみバリデーションチェックが可能な為合計値でチェックが可能か試みていました。しかしご指摘の通り、結果を変数チェックしても結果のみがバリデーションチェックされる為、引数はチェックされないことに気がつきました。
NULLに関しては後者を目指しております。
それぞれの引数のバリデーションチェックを条件分岐のネスト構造にしてみようかと思っております。
if(is_numeric($h)){
if(is_numeric($w)){
$total = $h*$w/2;
return $total;
}else{
return NULL;
}

[m.ts10806]

質問は編集できますので質問本文に追記してください。
今回の条件でネストさせるとelseも増えて可読性も落ちますし、ひとつのifで複合条件で十分と思います。

あと引数の型宣言も利用するとより厳格な作りになると思います。

[退会済みユーザー]

ご返信頂きありがとうございます。
ifの複合条件そして、引数の型宣言を利用してもう少し厳格な作りにしてみたいと思います。
引数の型宣言に関してもう少し調べていきたいと思います。

[m.ts10806]

PHPマニュアルをフル活用してくださいね

[退会済みユーザー]

ありがとうございます。
PHPマニュアル読解していきたいと思います!

[m.ts10806]

読解…というより実際は辞書がわりに使うことになります。仕様書、取扱説明書みたいなものですね。

Answer 1

どこまで厳密に評価したいかによりますね

function triangle($h=null,$w=null){
  if(!is_numeric($h) or !is_numeric($w) or is_string($h) or is_string($w)) return null;
  return $h*$w/2;
}

var_dump(triangle(10,20));//int(100)
var_dump(triangle(1.5,2.3));//float(1.725)
var_dump(triangle(0,0));//int(0)
var_dump(triangle(1e2,1));//float(50)
var_dump(triangle(1)); //null
var_dump(triangle(1,"10")); //null
var_dump(triangle(1,false)); //null
var_dump(triangle(1,[1,2,3])); //null

Comments

[退会済みユーザー]

ご回答頂きありがとうございます。
数値のみ入れたいと考えております。
文字、または空欄の場合にはnull表示をしたいと考えいます。
is_String関数を使用することは考えていなかったのでis_String関数を使ってもう少し厳密チェックにしていきたいと思います。

[yambejp]

念の為、is_stringのありなしで
var_dump(triangle(1,"10"));
の結果が違ってきます

[退会済みユーザー]

ご返信ありがとうございます。
is_stringのありなしでvar_dumpの方を試してみたいと思います。

Answer 2

バリデーションチェックを合計値で考えていた為、各引数にバリデーションチェックを行う。
バリデーションに関して以下を試みる。
Is_numericの複合条件分岐を行い、引数の型チェックを行ってみる。
IS_numeric、IS_String関数でバリデーションチェックを行ってみる。
関数を２つ用意してバリデーションチェックを試みる。

Answer 3

おす！

function is_num($value) : bool
{
    return (!isset($value) || !is_numeric($value) ? false : true;
}

function triangle($h, $w)
{
    if(!is_num($h)) { return null; }    
    if(!is_num($w)) { return null; }

    return ($h * $w) /2;
}

$result = triangle($h, $w);

echo (is_null($result))
        ? "NULL"
        : $result
        ;

こんな感じでいいか？

Comments

[退会済みユーザー]

ご回答頂きありがとうございます。
関数を２つ用意してバリデーションチェックする方法も試みていきたいと思います。