WPループ内のフックでmb_strlenをした箇所に不自然な動作

Question

概要

WordPressで、ユーザーが投稿したpostデータをイジっています。
その中でレビュー評価をつける機能があり、特定の商品に対するレビューを集計して出力したいのですがうまくいきません。

やりたいこと

１つの商品につき
（５段階評価の合計値）/（投稿数）
の計算を行い、総合評価値を「★★★★★」で出力したい。

その出力はsingle.postページでは問題なくクリア。
archiveページ（商品一覧）でWordPressループ内に入れたアクションフックで一部のエラーがでてしまうのを治したい。

問題点

アクションフックは機能しているのだが、たまに「★★★★★」の代わりに「8」など数字が出力される。
これは$post->page_titleの文字数であることが判明。
しかしエラーは出ておらず、明確な原因がわかりません。
推測ですが mb_strlen がうまく作動していないと考え、以下に該当部分のコードを掲載します。

global $wpdb;

$numposts = $wpdb->get_var(
  "SELECT count(*)
   FROM $wpdb->posts
    INNER JOIN $wpdb->postmeta pm
    ON ($wpdb->posts.id = pm.post_id)
    WHERE $wpdb->posts.post_status = 'publish'
    AND $wpdb->posts.post_type = 'post_type'
    AND pm.meta_key = '_keyname'
    AND pm.meta_value
    LIKE \"{$posttitleを代入した変数}%\"
    ");
  $numposts = number_format($numposts);

if (0 < $numposts)
{
  global $wpdb;
  $rtn = $wpdb->get_results("
    SELECT id 
    FROM $wpdb->posts 
    INNER JOIN $wpdb->postmeta pm 
    ON ($wpdb->posts.id = pm.post_id) 
    WHERE $wpdb->posts.post_status = 'publish' 
    AND $wpdb->posts.post_type = 'post_type' 
    AND pm.meta_key = '_keyname' 
    AND pm.meta_value 
    LIKE \"{$posttitleを代入した変数}%\" 
    ");

  foreach ($rtn as $value) {
    $variable[] = $wpdb->get_row($wpdb->prepare("
      SELECT meta_value
      from $wpdb->postmeta
      where meta_key = '_keyname'
      AND post_id = %d", $value->id) );
  }
  if(count($variable) > 0){
  	foreach ($variable as $value) {
  		$variable_work = $value->meta_value;
  	}
  	$staravg = round(mb_strlen($variable_work)/$numposts);
  	//現状のエラー、たまに☆がタイトルの文字数になる
  	if ($staravg == 0) $staravg = '';
  	if ($staravg == 1) $staravg = '★';
  	if ($staravg == 2) $staravg = '★★';
  	if ($staravg == 3) $staravg = '★★★';
  	if ($staravg == 4) $staravg = '★★★★';
  	if ($staravg == 5) $staravg = '★★★★★';
  	return $staravg;
  }
}
else
{
  $numposts = 0;
  $staravg = '';
}

回答頂きたいこと

上記のコードになにか間違った記述はございませんでしょうか。
ご教示頂けますと大変助かります。

追記

• 投稿数で割る処理をしているため、評価値$staravg が5を超えることはありません。
• $numpostsの処理を追記しました。
• foreach ($variable as $value) のループで、$variable_work = $value->meta_value; が前任者作成部分でして、ワタシも完全に理解しておらず...

Comments

[退会済みユーザー]

そのコードを示されて、そうですかと第三者が簡単に試せないということを念頭に、そもそも $staravg が5を超えてしまうことがNGなのか、それとも5を超えてしまうことがOKでその場合の回避方法を知りたいのか、どちらだろうか。また、 foreach ($variable as $value) のループで、 $variable_work = $value->meta_value; をただ繰り返しているけど、最後の $value->meta_value で単に上書きするのって想定する仕様なのかどうなのか。 $numposts をどこで定義しているかも不明だし、設計が見えてこない。

[pineappleman]

コメント誠にありがとうございます！追記修正をしてみます。

Answer 1

  $staravg = round(mb_strlen($variable_work)/$numposts);

  //現状のエラー、たまに☆がタイトルの文字数になる
  if ($staravg == 0) $staravg = '';
  if ($staravg == 1) $staravg = '★';
  if ($staravg == 2) $staravg = '★★';
  if ($staravg == 3) $staravg = '★★★';
  if ($staravg == 4) $staravg = '★★★★';
  if ($staravg == 5) $staravg = '★★★★★';
  return $staravg;

0 から 5 以外の場合は、round(mb_strlen($variable_work)/$numposts); の値をそのまま返す感じですね。
0 から 5 以外の際に $staravg にエラーなどを返すようにすればどうなりますか？

---

(追記)

どのように書くとエラーが返せるのかわかりません。。

エラーをかえすのもなんですから、範囲外は、 ★ 0個( つまり '' ) とするなら、こんな感じでしょうか。

現状の 0 で判断していた部分の条件を「1 以下または5以上」に広げた形です。

      if ($staravg < 1 or $staravg > 5 ) $staravg = '';
      if ($staravg == 1) $staravg = '★';
      if ($staravg == 2) $staravg = '★★';
      if ($staravg == 3) $staravg = '★★★';
      if ($staravg == 4) $staravg = '★★★★';
      if ($staravg == 5) $staravg = '★★★★★';

Comments

[pineappleman]

すみません...
どのように書くとエラーが返せるのかわかりません。。
ご教示いただけませんでしょうか。。