Notice: Undefined offset: 0について

Question

Notice: Undefined offset: 0 in C:\Users\xxxxxxx\Desktop\xxxxxxx\wordpress\wp-content\plugins\auto-se
t-category\auto-set-category.php on line 35

wordpress4.4.1で以下のコードのプラグインを使用したら上記のエラーがでるようになってしまいました

これはどのように改善したらいいのでしょうか？

<?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) && is_null($catcheck[0]) ) { wp_add_object_terms( $post_id, 1, 'category' ); } } add_action( 'save_post', 'auto_set_category' );

Answer 1

auto-set-category.phpの35行目で未定義のエラーがでています。
記載されているソースがauto-set-category.phpだとして35行目は大体しかわかりませんが、
$catcheck = get_the_category($post_id);
ここで値がとれない場合（たとえば戻り値が空のarray()の場合）、その下の
if ( is_array($catcheck) && is_null($catcheck[0]) ) {
では$catcheck[0]が定義されていないことになります。
もし原因がここだとしたらis_nullの代わりにempty()を使ってはどうでしょう？

Comments

[退会済みユーザー]

あんまり読んでいないのですが、

> Notice: Undefined offset: 0

ってことは、配列の0番が定義されていないエラーですから該当箇所は配列に関する

>if ( is_array($catcheck) && is_null($catcheck[0]) ) {

ここだと思います。なので。

>もし原因がここだとしたらis_nullの代わりにempty()を使ってはどうでしょう？

では、やはり未定義エラーが出るので

>if ( is_array($catcheck) && !isset($catcheck[0]) ) {

となるかと思いますが、どうでしょうか？

[退会済みユーザー]

>/* もしカテゴリーが１つも無かったらエラーになるからデフォのカテゴリーを付ける */

ということなので、

>if ( is_array($catcheck) && count($catcheck) === 0 ) {

の方がいいかな。

[test_hf]

> >もし原因がここだとしたらis_nullの代わりにempty()を使ってはどうでしょう？

> では、やはり未定義エラーが出るので
一応情報として、emptyはissetと同じで未定義エラーでませんよ。未定義の場合はtrueが返ります。

[ryuujinn]

test_hfさん、luckerさん、コメント、解決方法ありがとうございます
if ( is_array($catcheck) && count($catcheck) === 0 ) {
と記述したらエラー表示が見事に消えました
何日も困っていたので大変助かりました
ありがとうございました

[退会済みユーザー]

そしたらぜひ、雑草を刈っておいてくれたtest_hfさんのコメントに▲で+1して下さい。

[退会済みユーザー]

>一応情報として、emptyはissetと同じで未定義エラーでませんよ。未定義の場合はtrueが返ります。

失礼しました。そうなりますね。PHPのempty()は何年使っていても間違えます…

[ryuujinn]

▲で+1出来るの知りませんでした
luckerさん、情報ありがとうございます

[test_hf]

luckerさんお気遣いすみません。ありがとございます。