PHPのforeach初期化

###前提・実現したいこと

WordpressのデータをJSONで引っ張ってきて、別のデータベースに保存をするという処理を書いています。
jsonを取得して、配列に入れるところまでは終わっていて、Wordpressのカテゴリーがarray('hoge','hoge1','hoge2')と返ってくるのを、implode関数を使って'hoge,hoge1,hoge2'と文字列の連結をして、保存をしたいと考えています。

###発生している問題・エラーメッセージ

'hoge,hoge1,hoge2'とするところまでは出来ているのですが、変数の初期化をしているのにもかかわらず、変数が上書きされてしまいます。

###該当のソースコード

hoge.comからjsonで取得して、foreachでループ

PHP
1  $feed json_decode("hoge.com",true);
2  //初期化
3  $category = "";
4  foreach ($feed as $f){
5  // カテゴリーが多次元配列で複数取得しているので、数を取得
6    $countCategory = count($f['terms']['category']);
7      for ($i=0; $i < $countCategory; $i++){
8        $slug[] = $f['terms']['category'][$i]['slug'];
9        $category = implode(',',$slug);
10      }
11    //全ての処理が終わり次第、ループ内で値をデータベースに保存する処理を実行
12  }

###補足情報(言語/FW/ツール等のバージョンなど)
PHP5.6系で開発をしています。

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

回答1件

ベストアンサー

implodeするのはforの外側だと思います。
あと、slugの初期化もなかったですね

以下の感じでどうでしょうか

php
1$feed json_decode("hoge.com",true);
2  //初期化
3  $category = ""; // これは別にいらない
4  foreach ($feed as $f){
5  // カテゴリーが多次元配列で複数取得しているので、数を取得
6    $countCategory = count($f['terms']['category']);
7    $slug = []; // ここで$slugを初期化する
8    for ($i=0; $i < $countCategory; $i++){
9      $slug[] = $f['terms']['category'][$i]['slug'];
10    }
11    $category = implode(',',$slug); // これはforループの外側で
12    //全ての処理が終わり次第、ループ内で値をデータベースに保存する処理を実行
13  }