カスタムフィールドの値は「カンマ区切り」か「シリアライズされた配列」か

Question

カスタムフィールドの保存方法について教えてください。

保存する値として、適切なのはどちらですか？（どんなケースでどちらが適切ですか？）

【A】カンマ区切り
apple,orange

【B】シリアライズされた配列
a:2:{i:0;s:5:"apple";i:1;s:6:"orange";}

管理画面から見やすいのは【A】かと思うのですが、しかしチェックボックスの値は自動的にBの形式で保存されるので、統一感を考えれば【B】がいいのか？と悩んでいます。

また、もし【B】が良いとしたら、値がapple１つだけ（複数入ることはない）であってもそのようにしますか？

みなさんのお考えや、一般的な使い方など、メリットデメリットがあれば教えてください。

Answer 1

シリアライズする方で
シリアライズ処理はいちいち書く必要なし

WPはメタデータの関連の共通処理に
シリアライズの処理が備わっているので
配列を直接入出力する処理を書いても動くようにできてます

wp-includes/meta.php

単なるコンマ区切りは、値の一つにコンマが入ったらどうするの？
みたいな問題があるので結構注意が必要だったり面倒です

同じフィールドのデータが文字列だったり配列だったりすると
毎度型チェックしなくちゃいけなかったり面倒なので
配列と決めたフィールドは値が一つでも配列にします

Comments

[waru]

なるほど、ありがとうございます。少なくとも、投稿者のIPアドレスが１件入るフィールドが配列というのはおかしいですよね？複数のユーザーIDが入るフィールドではいかがでしょうか。「1,13,27」のような感じです。こういうケースは配列でしょうか？

[KazuhiroHatano]

配列で

WPが面倒見てくれるので、いちいちimplodeとかせずに
update_post_metaとかには配列のままブッこみましょう

[waru]

わかりました！とにかく配列でやってみることにします。ありがとうございます。

[waru]

すみません、１点だけお願い致します。

<?php var_export(get_user_meta( $current_uid, 'comment_id' ) ); ?>

としたときに下のように配列が空なのに、

array ( 0 => '', )

でもなぜか

<?php echo count( get_user_meta( $current_uid, 'comment_id' ) ); ?>

とすると

1

がechoされてしまうのですけど、

お手数ですが、これについてご意見頂けませんでしょうか。

[KazuhiroHatano]

array ( 0 => '', )は空じゃないです
それを空と判定したいならarray_filterとかして
空の文字列の要素を削除します

[waru]

array ( 0 => '', )
だった場合カウントすると１件になりますし、

array ( 0 => '123', )
の場合でも１件になりますし、困りますね。

そこでarray_fiterを使うというと、
具体的には
<?php echo count( get_user_meta( $current_uid, 'comment_id' ) ); ?>
をどう書き直せばいいのでしょうか？？

ていうか、普通はどうやってるんですか？
「それを空と判定したいなら」という言い回しからは、何かもっと普通のやり方があるような意図が感じられますが。

[KazuhiroHatano]

空ならちゃんとfalsyな値が返るようにしておきたいです
空の文字列も入ってない本当に空の配列を入れておくのでもいいですが
WPの場合は値が空の場合はdelete_user_metaとかして値を削除しておきたいです
この場合はfalseが返ることになるのでそれに対応しなくちゃいけませんが

[waru]

ありがとうございます。つまりこういうことでしょうか。
↓
空なら「delete_user_meta」されるように設定すること。
そうすれば「array ( 0 => '', )」さえ削除されるため、「count」しても１が返ることはない。

最後に、falseが返る場合への対応とは、どのようなものですか？

[KazuhiroHatano]

count(false)とかしたら「配列じゃないぞコレ」ってPHPに怒られるので
count(get_◯◯_meta(...))とか横着したらダメ
まあ止まりはしないですが怒られない方がいいので
面倒だけど丁寧に

if($hoge=get_◯◯_meta(...)){
echo count($hoge).'個あるよ';
}
else{
echo 'ひとつもないよ';
}

みたいな感じに
まあifの中で代入処理するのは
リーダブルコード的には良くないんですが
それぐらいの横着は許して

[KazuhiroHatano]

どのみち更新時に値が空の場合に
delete_user_metaをするしないに関わらず
値が未設定だとget_◯◯_metaはfalseを返すので
値がfalseだった場合への対応は必要です