teratail
回答率
85.35%
teratail header banner
teratail header banner
質問するログイン新規登録
トップJavaScriptに関する質問JavaScript/jQueryで同じ処理をできる限りスマートにしたい。

編集履歴

回答編集履歴

4

edit

2021/10/07 21:57

投稿

m.ts10806
m.ts10806

スコア80875

test CHANGED
@@ -34,7 +34,7 @@
34
34
 
35
35
  $insert[i] += '<li>';
36
36
 
37
- for (this_i = 0; this_i < 6; this_i++) {
37
+ for (this_i = 0; this_i <=5; this_i++) {
38
38
 
39
39
  $insert[i] += '<div>' + this[this_i] + '</div>';
40
40
 
@@ -82,17 +82,11 @@
82
82
 
83
83
  $insert[num] += '<li>';
84
84
 
85
- $insert[num] += '<div>' + this[0] + '</div>';
85
+ for (this_i = 0; this_i <= 5; this_i++) {
86
86
 
87
- $insert[num] += '<div>' + this[1] + '</div>';
87
+ $insert[num] += '<div>' + this[this_i] + '</div>';
88
88
 
89
- $insert[num] += '<div>' + this[2] + '</div>';
89
+ }
90
-
91
- $insert[num] += '<div>' + this[3] + '</div>';
92
-
93
- $insert[num] += '<div>' + this[4] + '</div>';
94
-
95
- $insert[num] += '<div>' + this[5] + '</div>';
96
90
 
97
91
  $insert[num] += '</li>';
98
92
 

3

edit

2021/10/07 21:57

投稿

m.ts10806
m.ts10806

スコア80875

test CHANGED
@@ -55,3 +55,55 @@
55
55
  }
56
56
 
57
57
  ```
58
+
59
+
60
+
61
+ もしくは「カテゴリn」と捉えて
62
+
63
+ ```js
64
+
65
+ $insert = [];
66
+
67
+ $(csv).each(function() {
68
+
69
+ if(this.length === 0){
70
+
71
+ return true;
72
+
73
+ }
74
+
75
+ num = this[0].replace('カテゴリ','');
76
+
77
+ if(!$insert.indexOf(num)){
78
+
79
+ $insert[num] = '';
80
+
81
+ }
82
+
83
+ $insert[num] += '<li>';
84
+
85
+ $insert[num] += '<div>' + this[0] + '</div>';
86
+
87
+ $insert[num] += '<div>' + this[1] + '</div>';
88
+
89
+ $insert[num] += '<div>' + this[2] + '</div>';
90
+
91
+ $insert[num] += '<div>' + this[3] + '</div>';
92
+
93
+ $insert[num] += '<div>' + this[4] + '</div>';
94
+
95
+ $insert[num] += '<div>' + this[5] + '</div>';
96
+
97
+ $insert[num] += '</li>';
98
+
99
+ });
100
+
101
+
102
+
103
+ for(i in $insert){
104
+
105
+ $('#slick_'+i+' .slider').append($insert[i]);
106
+
107
+ }
108
+
109
+ ```

2

edit

2021/10/07 21:53

投稿

m.ts10806
m.ts10806

スコア80875

test CHANGED
@@ -1,4 +1,4 @@
1
- コピペで現状再現できないので未検証ですし、ループばかりでお世辞にも（パフォーマンス的に）スマートとは言えませんが、数字が見た目短くするのは難しくはないです。
1
+ コピペで現状再現できないので未検証ですし、ループばかりでお世辞にも（パフォーマンス的に）スマートとは言えませんが、数字が固定のようなので見た目短くするのは難しくはないです。
2
2
 
3
3
 
4
4
 

1

edit

2021/10/07 21:37

投稿

m.ts10806
m.ts10806

スコア80875

test CHANGED
@@ -1,4 +1,4 @@
1
- コピペで現状再現できないので未検証ですし、ループばかりでお世辞にも（パフォーマンス的に）スマートとは言えませんが、見た目短くするのは難しくはないです。
1
+ コピペで現状再現できないので未検証ですし、ループばかりでお世辞にも（パフォーマンス的に）スマートとは言えませんが、数字が見た目短くするのは難しくはないです。
2
2
 
3
3
 
4
4
 
@@ -14,51 +14,43 @@
14
14
 
15
15
  $insert = [];
16
16
 
17
- for(i of list){ //初期化
17
+ for (i of list) { //初期化
18
18
 
19
- $insert[i] = '';
19
+ $insert[i] = '';
20
20
 
21
21
  }
22
22
 
23
- $(csv).each(function() {
23
+ $(csv).each(function () {
24
24
 
25
- if(this.length === 0){
25
+ if (this.length === 0) {
26
26
 
27
- return true;
27
+ return true;
28
28
 
29
- }
29
+ }
30
30
 
31
- for(i of list){
31
+ for (i of list) {
32
32
 
33
- if(this[0] === 'カテゴリ'+i){
33
+ if (this[0] === 'カテゴリ' + i) {
34
34
 
35
- $insert[i] += '<li>';
35
+ $insert[i] += '<li>';
36
36
 
37
- $insert[i] += '<div>' + this[0] + '</div>';
37
+ for (this_i = 0; this_i < 6; this_i++) {
38
38
 
39
- $insert[i] += '<div>' + this[1] + '</div>';
39
+ $insert[i] += '<div>' + this[this_i] + '</div>';
40
40
 
41
- $insert[i] += '<div>' + this[2] + '</div>';
41
+ }
42
42
 
43
- $insert[i] += '<div>' + this[3] + '</div>';
43
+ $insert[i] += '</li>';
44
44
 
45
- $insert[i] += '<div>' + this[4] + '</div>';
45
+ }
46
46
 
47
- $insert[i] += '<div>' + this[5] + '</div>';
48
-
49
- $insert[i] += '</li>';
50
-
51
- }
47
+ }
52
-
53
- }
54
48
 
55
49
  });
56
50
 
51
+ for (i of list) {
57
52
 
58
-
59
- for(i of list){
60
-
61
- $('#slick_'+i+' .slider').append($insert[i]);
53
+ $('#slick_' + i + ' .slider').append($insert[i]);
62
54
 
63
55
  }
64
56