wordpressのプラグインコードの改善について

以下のプラグイン（コード）を使用しているのですが

大変便利で使いやすいのですが、これを導入すると

wordpressの新規追加、公開、更新が以上に遅くなってしまい且つ、

「接続が切断されました再接続するまで保存を中断します。念のため、お使いのブラウザー上でこの投稿をバックアップしています。」と頻繁にでるようになってしまいました

このプラグイン（コード）自体は大変便利でこのまま使用したいのですが、

このコードを改善する方法、もしくは同じ機能で違うコードの記述の仕方はありますでしょうか？

<?php /* Plugin Name:Auto Set Category Plugin URI:http://milkveil.com/ Description:本文中にカテゴリーと同じ文字列があった場合、そのカテゴリーをセットするプラグイン。 Version:1.0 Author:taku-k Author URI:http://milkveil.com/ */ function auto_set_category ( $post_id ) { global $post; $new_post = get_post( $post_id ); $content = $new_post->post_content; /* ループ開始全てのカテゴリーを１つ１つ調べる */ $cat_all = get_terms( "category", "fields=all&get=all" ); foreach($cat_all as $value): /* もし本文にカテゴリーと同じ文字列があれば（デフォのカテゴリーを外してから）カテゴリーを付ける */ if ( stripos( $content, $value->name ) !== false ) { wp_remove_object_terms( $post_id, 1, 'category' ); wp_add_object_terms( $post_id, $value->name, 'category' ); } /* もし本文にカテゴリーと同じ文字列が無い時はカテゴリーを外す */ else { wp_remove_object_terms( $post_id, $value->name, 'category' ); } endforeach; /* ループ終了全てのカテゴリーを１つ１つ調べる */ /* もしカテゴリーが１つも無かったらエラーになるからデフォのカテゴリーを付ける */ $catcheck = get_the_category($post_id); if ( is_array($catcheck) && count($catcheck) === 0 ) { wp_add_object_terms( $post_id, 1, 'category' ); } } add_action( 'save_post', 'auto_set_category' );

izkn

2016/01/19 07:36

こちらの質問が他のユーザから「質問の範囲が広すぎる」という評価を受けていますわからない点を明確にし、調査したこと・試したことと共に記入していただくと、回答が得られやすくなります。

flat

2016/01/19 08:20

行動規範の内容に同意します

回答1件

まずプロファイラでどこの部分で遅いかを調べるのが基本です。XHProfとか使ってみるといいです。

というのが一般論ですが、見た感じの予想だと、「もし本文にカテゴリーと同じ文字列が無い時はカテゴリーを外す」というコメントのところが怪しいです。

カテゴリーの種類がたくさんあるけどそのpostについているカテゴリーの数がそれほど多くないときは、もともとついてないカテゴリーを外すwp_remove_object_termsがたくさん発行されて中でDBアクセスを毎回行うのでとても無駄な気が... この場合はあらかじめwp_get_object_termsでそのpostについているカテゴリを取得して配列にでもしまっておき、そこに入ってたときだけwp_remove_object_termsを呼ぶようにするといいかと。

追記:
wp_remove_object_terms削ると効果があるようなので、本文についているカテゴリのみ削除して無駄な削除は行わなくするコードを書いてみましたがwordpressの実行環境がないので動作確認はしてません。だいたいこんな感じだと思うので参考にしてください。

PHP
1    /* 本文が持つカテゴリーを全て取得し、チェック用の配列に記録する。 */
2    $terms = wp_get_object_terms($post_id, 'category');
3    $category_hash = array();
4    foreach ($terms as $term) {
5        $category_hash[$term->name] = true;
6    }
7
8    /* ループ開始 全てのカテゴリーを１つ１つ調べる */
9    $cat_all = get_terms( "category", "fields=all&get=all" );
10    foreach($cat_all as $value):
11
12    /* もし本文にカテゴリーと同じ文字列があれば（デフォのカテゴリーを外してから）カテゴリーを付ける */
13      if ( stripos( $content, $value->name ) !== false ) {
14        wp_remove_object_terms( $post_id, 1, 'category' );
15        wp_add_object_terms( $post_id, $value->name, 'category' );
16          }
17
18    /* もし本文にカテゴリーと同じ文字列が無い時はカテゴリーを外す */
19      else {
20          /* チェック用の配列に記録されていたときのみカテゴリーを外す処理を実行する */
21          if (isset($category_hash[$value->name])) {
22              wp_remove_object_terms( $post_id, $value->name, 'category' );
23          }
24       }
25
26    endforeach;
27    /* ループ終了 全てのカテゴリーを１つ１つ調べる */
28

投稿2016/01/19 13:56

編集2016/01/20 08:54

crhg

総合スコア1175

ryuujinn

2016/01/19 15:52

crhgさん、コメントありがとうございます読んで理解は出来るのですが、具体的なコードの記述が分かりませんお手数をおかけして大変心苦しいのですが、コードを記述して頂けないでしょうか？よろしくお願いいたします

crhg

2016/01/19 22:25 編集

まず予想が合っているか簡単に調べるため、else節のwp_remove_object_termsの呼び出しを削って試してみましょう。削って速くなるようなら予想あたり、効果が無ければハズレです。 /* もし本文にカテゴリーと同じ文字列が無い時はカテゴリーを外す */ else { /* wp_remove_object_terms( $post_id, $value->name, 'category' ); */ }

ryuujinn

2016/01/20 08:27

crhgさん、コメントありがとうございます /* もし本文にカテゴリーと同じ文字列が無い時はカテゴリーを外す */ else { /* wp_remove_object_terms( $post_id, $value->name, 'category' ); */ } 上記を削除しましたら劇的に早くなりました、ありがとうございますこの機能を残す場合はどのように記述し直せばいいのでしょうか？

行動規範の内容に同意します