PHP 関数内で別の関数を呼び出したい

Question

Integer(整数)の配列を渡すと、配列の中身が３乗される関数
かつ関数の中で引数に必要だと思うvalidationも作成したい。

バリデーションチェックは
・空チェック
・配列であること
・配列の中身が数字であること

バリデーションがOKならメイン処理をして期待値を返したい。

<?php

$nums = [1, 2, 3, 4];

function returnArr($int){
    check($nums);
    return pow($int ,3);
};

$result = array_map('returnArr', $nums);
print_r($result);

function check($nums){
    if(!is_array($nums)){
        echo 'not array';
        return true;
    }
    if(empty($nums)){
        echo 'empty';
        return true;
    }
    foreach($nums as $num){
        if(!is_numeric($num)){
            echo 'not numeric';
           return true;
        }
    }

    
}

処理の状況としまして、
・３乗された値を格納した配列を返す関数
・check関数にてバリデーションは正常に動作します。

しかしreturnArr関数の中でcheck関数を呼んでバリデーションをしたいのですが
関数の呼び出し方がわからず困っております。

こちらで動作確認したところ、ブラウザ上で

not arraynot arraynot arraynot arrayArray ( [0] => 1 [1] => 8 [2] => 27 [3] => 64 )

と表示されます。

教えていただけると大変助かります。
よろしくお願い致します。

Comments

[gpsoft]

ちなみに、バリデーションチェックがNGの場合は、どうしたいですか?

また、バリデーションチェックを行うのは、returnArr関数の中じゃないとダメですか?
returnArr関数は4回呼ばれるので、バリデーションチェックも4回行うことになってしまいますね。

[yambejp]

https://teratail.com/questions/306641

と、どう違うのでしょうか？
１個でもエラーなら全体を弾くのか、エラーの箇所だけ弾くのか？
いまいち見えてこないです

[yukinari302]

バリデーションチェックが一個でもNGなら全体を弾くようにしたいです。

先程returnArr関数の外でcheck($nums);を実行すると期待値が帰ってきました。

yambejpさん
前回はバリデーションチェックと配列の中身が３乗される関数をまとめていたのですが、
今回はcheck関数をバリデーション用の関数として、
メイン処理は別にしたかったのです。

Answer 1

変数のスコープと、変数の「参照渡し」「値渡し」 というのを学ぶといいでしょう。
参照渡し、値渡しについては検索していくつか記事を読んでみるほうがいいかと思います。
「php 参照渡し 値渡し」あたりをキーワードに検索するといろいろ記事が出てきます。

同じファイル内に書いていても、

• 関数外（global）で書かれている変数 $nums
• 関数 returnArr 内で利用されている変数 $nums
• 関数 check の引数として定義されている変数 $nums

はすべて別物です。

関数内で global キーワードで $nums を宣言する、関数の返り値を利用するなど代替案はありますが、少なくとも今のソースでは、 check() の結果、エラーメッセージは表示されますがエラーかどうかで次の処理を行うかどうかとはつながっていませんので、必ず return pow($int ,3); が実行されることになり、エラーチェックの用をなしていません。

スコープ、参照渡し、値渡しを理解したうえでソースコードを読めばすぐわかると思いますので、ちょっと調べてみてください。

Comments

[yukinari302]

ご丁寧な解説ありがとうございます。
またご返信遅くなりまして申し訳ありません。

kaz.Suenagaさんの仰られる通り、
・変数のスコープ
・変数の参照渡し、値渡しについて調べてみて理解が深まりました。

ローカルスコープやグローバルスコープという事は知らずにコードを書いていたのでとても勉強になりました。

関数は自分の苦手なとこなのでコードをもっと書いて勉強していきたいと思います。

Answer 2

先程returnArr関数の外でcheck($nums);を実行すると期待値が帰ってきました。

ということなので、既に解決済みのような気もしますが、
私も書いてみました(php7.4です)。

// バリデーションチェック。
// NGならエラーメッセージ、OKなら''を返す。
function check($nums){
    if ( !is_array($nums) ) return 'not array';
    if ( empty($nums) ) return 'empty';
    if ( count(array_filter($nums, 'is_numeric'))
        != count($nums) ) return 'not numeric';
    return '';
}

// 各要素を3乗する。
// エラー時は[false, エラーメッセージ]を
// 成功時は[true, 結果]を返す。
function pow3($nums) {
    $err = check($nums);
    if ( !empty($err) ) return [false, $err];
    $res = array_map(fn($n) => pow($n, 3), $nums);
    return [true, $res];
}

// 各要素を3乗して、結果を出力する。
// エラー時はエラーメッセージを出力する。
function run($nums) {
    list($ok, $resOrError) = pow3($nums);
    if ( !$ok ) {
        echo $resOrError.PHP_EOL;
        return;
    }
    echo 'ANSWER: ';
    print_r($resOrError);
}

// 呼び出し例。
run(1);
run([]);
run(['hello']);
run([1, 2, 3, 4]);

出力は、↓こんな感じです。

not array
empty
not numeric
ANSWER: Array
(
    [0] => 1
    [1] => 8
    [2] => 27
    [3] => 64
)

Comments

[yukinari302]

ご丁寧な解説ありがとうございます。
またご返信遅くなりまして申し訳ありません。

gpsoftさんのおかげで解決することができました。
今回ベストアンサーとさせていただきました。