teratail header banner
teratail header banner
質問するログイン新規登録

回答編集履歴

2

追記

2016/08/02 17:44

投稿

Panzer_vor
Panzer_vor

スコア1636

answer CHANGED
@@ -6,6 +6,17 @@
6
6
  iとjが同一値の時の工夫を加えるか、
7
7
  ループの回し方自体を工夫しましょう。
8
8
 
9
+ ###追記
10
+ 先ほど回答した際には見逃してましたが、
11
+ 2番目のループが10以下を継続条件としてるので、
12
+ 配列への不正アクセスが発生する疑いがあります。
9
13
 
14
+ ```PHP
15
+ // i = 0、j = 10のとき
16
+ if(data[0] === data[10]) // data[10]は未初期化
17
+ ```
10
18
 
19
+ こちらも合わせて直したほうが良いと思います。
11
20
 
21
+
22
+

1

追記

2016/08/02 17:44

投稿

Panzer_vor
Panzer_vor

スコア1636

answer CHANGED
@@ -1,4 +1,11 @@
1
1
  if文は正しく動いてますよ。
2
2
 
3
- for文の二重ループのさせ方がまずいです。
3
+ for文の二重ループのさせ方がまずいです。
4
- そのループのさせ方だとiとjの値が一致した時に同じ要素を比較して重複と判定されます。
4
+ そのループのさせ方だとiとjの値が一致した時に同じ要素を比較して重複と判定されます。
5
+
6
+ iとjが同一値の時の工夫を加えるか、
7
+ ループの回し方自体を工夫しましょう。
8
+
9
+
10
+
11
+