回答編集履歴
2
これで最後。もうこの質問では答えない
test
CHANGED
|
@@ -167,3 +167,89 @@
|
|
|
167
167
|
$('.tabs-content div[id="tabs-' + selector.slice(-1) + '"]').show();
|
|
168
168
|
|
|
169
169
|
```
|
|
170
|
+
|
|
171
|
+
|
|
172
|
+
|
|
173
|
+
追記2
|
|
174
|
+
|
|
175
|
+
===
|
|
176
|
+
|
|
177
|
+
これでこの質問への回答は最後にします。
|
|
178
|
+
|
|
179
|
+
どうしてもif文にしたいならこうですね。元のif文を維持するのは統一感がなくて正直嫌なので変えます。
|
|
180
|
+
|
|
181
|
+
もし、部下からのレビューで来たらキレる自信があります。
|
|
182
|
+
|
|
183
|
+
|
|
184
|
+
|
|
185
|
+
```js
|
|
186
|
+
|
|
187
|
+
const showTab = (selector) => {
|
|
188
|
+
|
|
189
|
+
// 引数selectorの中身をコンソールで確認する
|
|
190
|
+
|
|
191
|
+
console.log(selector);
|
|
192
|
+
|
|
193
|
+
|
|
194
|
+
|
|
195
|
+
$(".tabs-menu > div").removeClass("active");
|
|
196
|
+
|
|
197
|
+
$(".tabs-content > div").hide();
|
|
198
|
+
|
|
199
|
+
|
|
200
|
+
|
|
201
|
+
if (selector === "tab-menu-a") {
|
|
202
|
+
|
|
203
|
+
$("#tab-menu-a").addClass('active');
|
|
204
|
+
|
|
205
|
+
$("#tabs-a").show();
|
|
206
|
+
|
|
207
|
+
}
|
|
208
|
+
|
|
209
|
+
else if (selector === "tab-menu-b") {
|
|
210
|
+
|
|
211
|
+
$("#tab-menu-b").addClass('active');
|
|
212
|
+
|
|
213
|
+
$("#tabs-b").show();
|
|
214
|
+
|
|
215
|
+
}
|
|
216
|
+
|
|
217
|
+
else if (selector === "tab-menu-c") {
|
|
218
|
+
|
|
219
|
+
$("#tab-menu-c").addClass('active');
|
|
220
|
+
|
|
221
|
+
$("#tabs-c").show();
|
|
222
|
+
|
|
223
|
+
}
|
|
224
|
+
|
|
225
|
+
};
|
|
226
|
+
|
|
227
|
+
|
|
228
|
+
|
|
229
|
+
|
|
230
|
+
|
|
231
|
+
// タブがクリックされたらコンテンツを表示
|
|
232
|
+
|
|
233
|
+
$(".tabs-menu div").on("click", (e) => {
|
|
234
|
+
|
|
235
|
+
const selector = $(e.target).attr('id');
|
|
236
|
+
|
|
237
|
+
showTab(selector);
|
|
238
|
+
|
|
239
|
+
});
|
|
240
|
+
|
|
241
|
+
|
|
242
|
+
|
|
243
|
+
// 初期状態として1番目のタブを表示
|
|
244
|
+
|
|
245
|
+
showTab('tab-menu-a');
|
|
246
|
+
|
|
247
|
+
|
|
248
|
+
|
|
249
|
+
// $(".tabs-menu div").on("click", (e) => {…}) でselectorはtab-menuだからこの指定はダメ
|
|
250
|
+
|
|
251
|
+
// 関数もmenuを取るようにしか作っていないでしょう？
|
|
252
|
+
|
|
253
|
+
// showTab('#tabs-a');
|
|
254
|
+
|
|
255
|
+
```
|
1
コメントとかでどう解決したよってのが無いし魚が欲しかったっぽいのでレビューがてらに追記
test
CHANGED
|
@@ -49,3 +49,121 @@
|
|
|
49
49
|
動作コードは裏で書いていますがまずはヒントに留めます。
|
|
50
50
|
|
|
51
51
|
頑張ってください。
|
|
52
|
+
|
|
53
|
+
|
|
54
|
+
|
|
55
|
+
追記
|
|
56
|
+
|
|
57
|
+
=====
|
|
58
|
+
|
|
59
|
+
こう解決したとかなかったんで、単に中身が欲しかったと解釈したのでコードを追記します。
|
|
60
|
+
|
|
61
|
+
|
|
62
|
+
|
|
63
|
+
## jsfiddleで確認する
|
|
64
|
+
|
|
65
|
+
[https://jsfiddle.net/uLmyc3br/1/](https://jsfiddle.net/uLmyc3br/1/)
|
|
66
|
+
|
|
67
|
+
|
|
68
|
+
|
|
69
|
+
## レビューしたコード
|
|
70
|
+
|
|
71
|
+
```js
|
|
72
|
+
|
|
73
|
+
const showTab = (selector) => {
|
|
74
|
+
|
|
75
|
+
// 引数selectorの中身をコンソールで確認する
|
|
76
|
+
|
|
77
|
+
console.log(selector);
|
|
78
|
+
|
|
79
|
+
|
|
80
|
+
|
|
81
|
+
// TIPS: 同じ個所を操作する実装はなるべく固めよう
|
|
82
|
+
|
|
83
|
+
// tab-menuの操作をまとめる。分散させると全体に神経を使うので疲れる。
|
|
84
|
+
|
|
85
|
+
$(".tabs-menu > div").removeClass("active");
|
|
86
|
+
|
|
87
|
+
$(`.tabs-menu div[id="${selector}"]`).addClass('active');
|
|
88
|
+
|
|
89
|
+
|
|
90
|
+
|
|
91
|
+
// 同様にtabsの操作をまとめる
|
|
92
|
+
|
|
93
|
+
$(".tabs-content > div").hide();
|
|
94
|
+
|
|
95
|
+
$(`.tabs-content div[id="tabs-${selector.slice(-1)}"]`).show();
|
|
96
|
+
|
|
97
|
+
|
|
98
|
+
|
|
99
|
+
// なぜtabs-menuは動的にselector変数で処理しているのにtabsはifなのか？
|
|
100
|
+
|
|
101
|
+
// 同じように実装した方が統一感があってキレイだ
|
|
102
|
+
|
|
103
|
+
// if (selector === "tab-menu-a") {
|
|
104
|
+
|
|
105
|
+
// $("#tabs-a").show();
|
|
106
|
+
|
|
107
|
+
// }
|
|
108
|
+
|
|
109
|
+
// else if (selector === "tab-menu-b") {
|
|
110
|
+
|
|
111
|
+
// $("#tabs-b").show();
|
|
112
|
+
|
|
113
|
+
// }
|
|
114
|
+
|
|
115
|
+
// else if (selector === "tab-menu-c") {
|
|
116
|
+
|
|
117
|
+
// $("#tab-c").show();
|
|
118
|
+
|
|
119
|
+
// }
|
|
120
|
+
|
|
121
|
+
|
|
122
|
+
|
|
123
|
+
// TIPS: 無駄なコードは書かない
|
|
124
|
+
|
|
125
|
+
// これの有無で動作に違いはない。大体hide()している箇所ないし
|
|
126
|
+
|
|
127
|
+
// $("#" + selector).show();
|
|
128
|
+
|
|
129
|
+
};
|
|
130
|
+
|
|
131
|
+
|
|
132
|
+
|
|
133
|
+
// タブがクリックされたらコンテンツを表示
|
|
134
|
+
|
|
135
|
+
$(".tabs-menu div").on("click", (e) => {
|
|
136
|
+
|
|
137
|
+
const selector = $(e.target).attr('id');
|
|
138
|
+
|
|
139
|
+
showTab(selector);
|
|
140
|
+
|
|
141
|
+
});
|
|
142
|
+
|
|
143
|
+
|
|
144
|
+
|
|
145
|
+
// 初期状態として1番目のタブを表示
|
|
146
|
+
|
|
147
|
+
showTab('tab-menu-a');
|
|
148
|
+
|
|
149
|
+
```
|
|
150
|
+
|
|
151
|
+
|
|
152
|
+
|
|
153
|
+
## その他
|
|
154
|
+
|
|
155
|
+
ヒアドキュメント部分はこう直そうかどうか迷ったけど、clickイベント起動なのでパフォーマンスにそこまで影響しないだろうからそのままにした。基本的には"+"で連結した方が動作は早い。
|
|
156
|
+
|
|
157
|
+
```js
|
|
158
|
+
|
|
159
|
+
$(".tabs-menu > div").removeClass("active");
|
|
160
|
+
|
|
161
|
+
$('.tabs-menu div[id="' + selector + '"]').addClass('active');
|
|
162
|
+
|
|
163
|
+
|
|
164
|
+
|
|
165
|
+
$(".tabs-content > div").hide();
|
|
166
|
+
|
|
167
|
+
$('.tabs-content div[id="tabs-' + selector.slice(-1) + '"]').show();
|
|
168
|
+
|
|
169
|
+
```
|