WordPressでカスタムフィールドのfunctionを簡略化したい

Question

###実現したいこと
functions.phpでカスタムフィールドを保存しているのですが、そのコードを簡略化したいです。

下記のコードで
function create_lang() {}
function create_sex() {}
function create_age() {}
という３つを
function create_all() {}
のように一括したい感じです。

###現在の状況
現在のコードはこの「言語」に対応する形で
add_meta_box( 'post_meta_lang', '言語', 'create_post_meta_lang', 'mypost', 'side' );

このfunctionがあり、フィールドが追加されるようになっています。
function create_lang() {}

しかしこのように「性別」や「年齢」などたくさんあったときに
add_meta_box( 'post_meta_sex', '性別', 'create_post_meta_sex', 'mypost', 'side' );
add_meta_box( 'post_meta_age', '年齢', 'create_post_meta_age', 'mypost', 'side' );

このようにいちいち同じようなfunctionを書きたくないと思いました。
function create_sex() {}
function create_age() {}

これらのfunction create_xxx() {}で違うのは$keyname = 'post_meta_xxx';の部分だけだからです。

そこで教えていただきたいのは、「言語」「性別」「年齢」などのすべてに対応できるfunctionです。

たとえばこのように１つにまとめるにはどうしたらよろしいでしょうか。
function create_all() {}

###現在のソースコード
現在のソースコードはこちらです。

/*
投稿編集画面でカスタムフィールドを表示する
=======================================*/
add_action( 'admin_menu', 'add_meta_field' );
function add_meta_field() {
      add_meta_box( 'post_meta_lang', '言語', 'create_post_meta_lang', 'mypost', 'side' );
      add_meta_box( 'post_meta_sex', '性別', 'create_post_meta_sex', 'mypost', 'side' );
      add_meta_box( 'post_meta_age', '年齢', 'create_post_meta_age', 'mypost', 'side' );
}

/* これらをまとめたfunctionはどのようなコードになるか？ */
function create_lang() {
      $keyname = 'post_meta_lang';
      global $post;
      $get_value = get_post_meta( $post->ID, $keyname, true ); 
      wp_nonce_field( 'action-' . $keyname, 'nonce-' . $keyname );
      echo '<input name="' . $keyname . '" value="' . $get_value . '">';	
}
function create_sex() {
      $keyname = 'post_meta_sex';
      // create_langと同じだからまとめたい
}
function create_age() {
      $keyname = 'post_meta_age';
      // create_langと同じだからまとめたい
}

/*
投稿編集画面でカスタムフィールドを保存する
=======================================*/
add_action( 'save_post', 'save_meta_field' );
function save_meta_field( $post_id ) {
      $custom_fields = ['post_meta_lang','post_meta_sex','post_meta_age'];
      foreach( $custom_fields as $d ) {
            if ( isset( $_POST['nonce-' . $d] ) && $_POST['nonce-' . $d] ) {
                    if( check_admin_referer( 'action-' . $d, 'nonce-' . $d ) ) {
                            if( isset( $_POST[$d] ) && $_POST[$d] ) {
                                    update_post_meta( $post_id, $d, $_POST[$d] );
                            } else {
                                    delete_post_meta( $post_id, $d, get_post_meta( $post_id, $d, true ) );
                            }
                    }
            }

     }
}

Answer 1

$custom_fields=['言語'=>'lang','年齢'=>'age','性別'=>'sex'];
add_action( 'admin_menu',function()use($custom_fields){
  foreach($custom_fields as $label=>$key){
    $keyname='post_meta_'.$key;
    add_meta_box( $keyname, $label, function($post)use($label,$keyname){
      $get_value = get_post_meta( $post->ID, $keyname, true ); 
      wp_nonce_field( 'action-' . $keyname, 'nonce-' . $keyname );
      echo '<input name="' . $keyname . '" value="' . $get_value . '">'; 
    }, 'mypost', 'side' );
  }
});

save_postの方も同じ要領で$custom_fieldsを使い回す

Comments

[chacky]

どうもありがとうございます。
add_action(
の中に
add_meta_box(
を書くことなんてできたんですね。
いつもこういうWPで元からある関数は分けて使っていました。

save_postの方も
add_action(
の中ということですよね？

二行目の「use($custom_fields)」とはなんでしょうか？
検索しても「名前空間」ばかり出てきて、でもそれとは別のもののようですし、今回のコードにおける役割もよくわかりませんでした。
もしよかったら教えてください。

[KazuhiroHatano]

use()は無名関数に変数を渡すためのものです

javascriptと違ってphpは関数の中と外でスコープがちゃんと別れてるので
関数の外の変数を使うには何らかの形で中に渡してあげなきゃいけません

無名関数はPHP5.3から使えるようになったものです
WordPressが無名関数を避けているのは
多分PHP5.2対応のためだと思います
流石にもういいんじゃないかと思います

[KazuhiroHatano]

マニュアルの無名関数の項に記載があります
http://php.net/manual/ja/functions.anonymous.php#example-165

[chacky]

ありがとうございます。なんとなく、わかりました。

$custom_fieldsの配列は関数の外にあるから、
function()use($custom_fields){
によって関数の中に渡さないといけないということですね。

でもそれならば、$custom_fieldsの配列を、foreachの直前（関数の中）に書いてあげれば、
function()use($custom_fields){
でなく
function(){
だけでよくなりますよね。
そのほうがシンプルで良いということはないのでしょうか？
$custom_fieldsの配列を関数の外に出しておくことにはどのようなメリットがあるのですか？

[KazuhiroHatano]

$custom_fieldsを外に出してるのはsave_postでも使いたいからです

仮に$custom_fieldsの項目が増えたとしても$custom_fieldsの要素を一つ増やすだけで
フォームの出力と保存の処理の変更が一箇所の修正だけで済みますし
どちらかだけを変更して片方を変更し忘れて動かなくなったとかいうことが起きにくくなります

[chacky]

ナルホド！「admin_menuの$custom_fields」と「save_postの$custom_fields」とで、同じ「$custom_fields」を二か所に書くことになってしまうからですか。ご高察とお心遣いに感謝申し上げます。どうもありがとうございました。

Answer 2

$callback_args が多分使えるのでは。

【add_meta_box – WordPress私的マニュアル】
https://elearn.jp/wpman/function/add_meta_box.html

【add_meta_box() | Function | WordPress Developer Resources】
https://developer.wordpress.org/reference/functions/add_meta_box/

Comments

[chacky]

ありがとうございます。

「表示用関数のパラメータを指定」とありますが使い方がわかりませんねこれ。

質問に沿った具体的なコードではどうなるかわかりますか？

「add_meta_field() {}」で「add_meta_box」がありますけど、

その「add_meta_box」に「$callback_args」を使うだけで

「function create_lang() {}」と「function create_sex() {}」と「function create_age() {}」がまとまるというわけでもありませんよね？

[kei344]

> 使い方がわかりませんねこれ。
こっちのほうがわかりやすいかな。

【関数リファレンス/add meta box - WordPress Codex 日本語版】
https://wpdocs.osdn.jp/%E9%96%A2%E6%95%B0%E3%83%AA%E3%83%95%E3%82%A1%E3%83%AC%E3%83%B3%E3%82%B9/add_meta_box#.E3.82.B3.E3.83.BC.E3.83.AB.E3.83.90.E3.83.83.E3.82.AF.E5.BC.95.E6.95.B0

> 「$callback_args」を使うだけで(中略)がまとまるわけでもありませんよね？
まとめた関数を割り当てることが出来るだけです。

[chacky]

むつかしかったです…汗
「＞まとめた関数を割り当てることが出来る」というのも理解できませんでした。
今回は質問に沿った形でご回答を頂けまして、「$callback_args」というのは見送りらせていただこうかと思います。
ありがとうございました。