function_existsを楽に使いたい

Question

WordPressでプラグイン依存の関数を使うとき、function_existsを使いますよね。

そのときに、普通にifを書くのが面倒だと感じたのですが、下記のような感じで楽に使うことはできませんでしょうか？

↓普通にifを書く

if ( function_exists( 'new_my_func1' ) ) {
    new_my_func1();
}
if ( function_exists( 'new_my_func2' ) ) {
    new_my_func2();
}

↓楽かなと思う書き方（の案）

function ex( $func ){
    if ( function_exists( $func ) ) {
        $func;
    }
}
ex( new_my_func1() );
ex( new_my_func2() );

しかしこれでは引数をどうするんだという話もあってダメなのですが、このようなイメージで巧い方法がございましたら教えてください。宜しくお願い致します。

Answer 1

ex( new_my_func1() );

引数もそうですが、こう書いた場合、new_my_func1の実行結果をexの引数として実行する
と解釈できます。そのため、お望み結果を得られません。

CHERRYさんのいう通り、第一パラメーラに関数名（文字列として）を与え、第二引数以降に可変変数を利用し
call_user_func_arrayで処理すればできなくはないと思いますが

見た目は概ね以下のようになります

ex('new_my_func1', $arg1, $arg2)

が、変数スコープとか変わると思うので結果微妙だと思います

Comments

[hana_hana]

たしかに、そうならざるをえなそうですね。ありがとうございます。

Answer 2

そもそも論になりますが…
その関数がなかったらニッチもサッチもいかないんだったら
最初にチェックしておく方がいいと思いますし、
関数名を配列にしといてループでチェックするなら
if 文はひとつで済みますし、
無ければ実行しなくていいやって話なら
フラグ立てといて判断すれば
多少短くなる…
とは思いますが、それがご希望に沿えるかどうかは、ね。

$func_list = ['new_my_func1','new_my_func2'];
foreach ($func_list as $func_name) {
  if (function_exists($func_name)) {
    //エラー処理 or フラグ立て
  }
}

//(中略)

if (フラグ) new_my_func1();

っていうか、バグ対策ですよね?
ならデバッグで対応するのがスジだと思いますが
違ったら申し訳ないです。

Comments

[KazuhiroHatano]

WPは管理画面でユーザーが簡単にプラグインの停止などができるし、何か問題が起きたらまずプラグインを停止してみるというのが一般的なトラブル対応の手順なので、もしプラグインを停止してもFatal Errorにならないようにするために、プラグイン依存の処理はFail Safeにしておくのがお作法

[hana_hana]

ありがとうございます。

[takasima20]

KazuhiroHatanoさん、フォローありがとうございます。

Answer 3

第1パラメータを関数名、第2パラメータ以降に引数として、可変変数 を利用して関数を実行すればできたりするかなと思ったけど試したことはないなぁ。

Comments

[hana_hana]

ありがとうございます。

Answer 4

『テーマのテンプレートからプラグインで定義されている関数を呼び出したいけど、プラグインが有効化されていない場合にもCall to undefined functionとならないようにしたい、でもfunction_existsを毎回書くのは面倒だ』というのであれば、functions.php で何もしない関数として同名の関数を定義しておいて、その関数をそのまま呼び出すというのは一つの手かと思います。

if ( ! function_exists( 'new_my_func1' ) ) {
    function new_my_func1(/* 引数は元の関数と合わせる */) {}
}

Comments

[mikkame]

読み込み順でプラグインより先にロードした時に意図しない動作をすると思います

[YukiYamashina]

'after_setup_theme'以降のアクション／フィルターでフックさせた関数内で定義されてるとダメですね。

[hana_hana]

現実的ですね。ありがとうございます。mikkame様もご忠告ありがとうございます。