回答編集履歴

2

補足

2016/09/27 05:41

投稿

popobot
popobot

スコア6586

test CHANGED
@@ -1,6 +1,10 @@
1
- implodeするのはforの外側だと思います。以下の感じでどうでしょうか
1
+ implodeするのはforの外側だと思います。
2
2
 
3
3
  あと、slugの初期化もなかったですね
4
+
5
+
6
+
7
+ 以下の感じでどうでしょうか
4
8
 
5
9
  ```php
6
10
 
@@ -8,7 +12,7 @@
8
12
 
9
13
  //初期化
10
14
 
11
- $category = "";
15
+ $category = ""; // これは別にいらない
12
16
 
13
17
  foreach ($feed as $f){
14
18
 
@@ -16,7 +20,7 @@
16
20
 
17
21
  $countCategory = count($f['terms']['category']);
18
22
 
19
- $slug = [];
23
+ $slug = []; // ここで$slugを初期化する
20
24
 
21
25
  for ($i=0; $i < $countCategory; $i++){
22
26
 
@@ -24,10 +28,12 @@
24
28
 
25
29
  }
26
30
 
27
- $category = implode(',',$slug);
31
+ $category = implode(',',$slug); // これはforループの外側で
28
32
 
29
33
  //全ての処理が終わり次第、ループ内で値をデータベースに保存する処理を実行
30
34
 
31
35
  }
32
36
 
33
37
  ```
38
+
39
+ ※変更箇所にコメント書いてあります。

1

追記

2016/09/27 05:41

投稿

popobot
popobot

スコア6586

test CHANGED
@@ -1,4 +1,6 @@
1
1
  implodeするのはforの外側だと思います。以下の感じでどうでしょうか
2
+
3
+ あと、slugの初期化もなかったですね
2
4
 
3
5
  ```php
4
6
 
@@ -14,6 +16,8 @@
14
16
 
15
17
  $countCategory = count($f['terms']['category']);
16
18
 
19
+ $slug = [];
20
+
17
21
  for ($i=0; $i < $countCategory; $i++){
18
22
 
19
23
  $slug[] = $f['terms']['category'][$i]['slug'];