回答編集履歴

4

更に追記

2017/07/07 06:30

投稿

m.ts10806
m.ts10806

スコア80852

test CHANGED
@@ -56,20 +56,22 @@
56
56
 
57
57
  ```javascript
58
58
 
59
- cnt=0;
59
+ cnt=0;
60
60
 
61
- for(var i=0; i<weakArray.length; i++) {
61
+ for(var i=0; i<weakArray.length; i++) {
62
62
 
63
- if(weakArray[i]==moji[t]) {
63
+ if(weakArray[i]==moji[t]) {
64
64
 
65
- cnt++;
65
+ cnt++;
66
66
 
67
- }
67
+ }
68
68
 
69
- }
69
+ }
70
70
 
71
- console.log(moji[t]+"を"+cnt+"回間違えました。");
71
+ console.log(moji[t]+"を"+cnt+"回間違えました。");
72
-
73
-
74
72
 
75
73
  ```
74
+
75
+ ※ただし、**weakArrayが大きくなればなるほど無駄にループしていくこととなる**ので、
76
+
77
+ kei344さんのように必要なものを必要なところだけ取り出すような記述もご検討ください。

3

追記

2017/07/07 06:30

投稿

m.ts10806
m.ts10806

スコア80852

test CHANGED
@@ -41,3 +41,35 @@
41
41
  ```
42
42
 
43
43
  *jQueryの書き方をしていたので書き換えました。
44
+
45
+
46
+
47
+ # 質問追記をうけて
48
+
49
+
50
+
51
+ 0を入れるなどして初期化されてないweakCntを使っているからですね。
52
+
53
+ cntだけで十分です。
54
+
55
+
56
+
57
+ ```javascript
58
+
59
+ cnt=0;
60
+
61
+ for(var i=0; i<weakArray.length; i++) {
62
+
63
+ if(weakArray[i]==moji[t]) {
64
+
65
+ cnt++;
66
+
67
+ }
68
+
69
+ }
70
+
71
+ console.log(moji[t]+"を"+cnt+"回間違えました。");
72
+
73
+
74
+
75
+ ```

2

修正

2017/07/07 06:27

投稿

m.ts10806
m.ts10806

スコア80852

test CHANGED
@@ -26,11 +26,11 @@
26
26
 
27
27
  var array=[];
28
28
 
29
- for(ii=0;ii<moji.length;ii++){
29
+ for(i=0;i<moji.length;i++){
30
30
 
31
31
  row=[];
32
32
 
33
- row[moji[ii]] = randRange(0,30);
33
+ row[moji[i]] = randRange(0,30);
34
34
 
35
35
  array.push(row);
36
36
 

1

javascriptver

2017/07/07 05:56

投稿

m.ts10806
m.ts10806

スコア80852

test CHANGED
@@ -26,20 +26,18 @@
26
26
 
27
27
  var array=[];
28
28
 
29
- $.each(moji,function(index,val){
29
+ for(ii=0;ii<moji.length;ii++){
30
30
 
31
31
  row=[];
32
32
 
33
- row[val] = randRange(0,30);
33
+ row[moji[ii]] = randRange(0,30);
34
34
 
35
35
  array.push(row);
36
36
 
37
- });
37
+ }
38
38
 
39
39
  console.log(array);
40
40
 
41
+ ```
41
42
 
42
-
43
-
44
-
45
- ```
43
+ *jQueryの書き方をしていたので書き換えました。