質問編集履歴
3
誤字修正
test
CHANGED
|
File without changes
|
test
CHANGED
|
@@ -46,13 +46,13 @@
|
|
|
46
46
|
|
|
47
47
|
accordion、spNav、deviceJudgeという関数名がグローバルになることを防ぐために、
|
|
48
48
|
|
|
49
|
-
common.js中を
|
|
49
|
+
common.js中を即時関数で囲んだほうがいいのでしょうか？
|
|
50
50
|
|
|
51
51
|
|
|
52
52
|
|
|
53
53
|
```javascript
|
|
54
54
|
|
|
55
|
-
(function(){//ここで
|
|
55
|
+
(function(){//ここで即時関数で囲う必要があるのでしょうか？
|
|
56
56
|
|
|
57
57
|
|
|
58
58
|
|
2
誤字修正
test
CHANGED
|
File without changes
|
test
CHANGED
|
@@ -28,7 +28,7 @@
|
|
|
28
28
|
|
|
29
29
|
|
|
30
30
|
|
|
31
|
-
hoge(){ //グローバルな関数
|
|
31
|
+
function hoge(){ //グローバルな関数
|
|
32
32
|
|
|
33
33
|
var hogeLocal = 'hoge'; //ローカルな変数
|
|
34
34
|
|
1
質問、関数の中身を追記しました
test
CHANGED
|
File without changes
|
test
CHANGED
|
@@ -16,47 +16,103 @@
|
|
|
16
16
|
|
|
17
17
|
関数自体がグローバルになってしまっていることは問題ないのでしょうか？
|
|
18
18
|
|
|
19
|
+
```javascript
|
|
20
|
+
|
|
21
|
+
|
|
22
|
+
|
|
23
|
+
$(function () {
|
|
24
|
+
|
|
25
|
+
hoge();//hogeという関数名がグローバルになってしまっている？
|
|
26
|
+
|
|
27
|
+
});
|
|
19
28
|
|
|
20
29
|
|
|
30
|
+
|
|
31
|
+
hoge(){ //グローバルな関数
|
|
32
|
+
|
|
33
|
+
var hogeLocal = 'hoge'; //ローカルな変数
|
|
34
|
+
|
|
35
|
+
return hogeLocal;
|
|
36
|
+
|
|
37
|
+
}
|
|
38
|
+
|
|
39
|
+
```
|
|
40
|
+
|
|
41
|
+
|
|
42
|
+
|
|
43
|
+
|
|
44
|
+
|
|
45
|
+
例えば以下のように
|
|
46
|
+
|
|
47
|
+
accordion、spNav、deviceJudgeという関数名がグローバルになることを防ぐために、
|
|
48
|
+
|
|
21
|
-
|
|
49
|
+
common.js中を無名関数で囲んだほうがいいのでしょうか？
|
|
22
50
|
|
|
23
51
|
|
|
24
52
|
|
|
25
53
|
```javascript
|
|
26
54
|
|
|
27
|
-
(function(){
|
|
55
|
+
(function(){//ここで無名関数で囲う必要があるのでしょうか？
|
|
56
|
+
|
|
57
|
+
|
|
28
58
|
|
|
29
59
|
$(function () {
|
|
30
60
|
|
|
31
|
-
|
|
61
|
+
accordion();
|
|
32
62
|
|
|
33
|
-
|
|
63
|
+
spNav();
|
|
34
64
|
|
|
35
|
-
|
|
65
|
+
deviceJudge();
|
|
36
66
|
|
|
37
67
|
});
|
|
38
68
|
|
|
39
69
|
|
|
40
70
|
|
|
41
|
-
|
|
71
|
+
function accordion(){
|
|
42
72
|
|
|
43
|
-
//
|
|
73
|
+
//アコーディオン
|
|
74
|
+
|
|
75
|
+
var $accordion = $('.accordion');
|
|
76
|
+
|
|
77
|
+
$accordion.on("click", function() {
|
|
78
|
+
|
|
79
|
+
$(this).next().slideToggle();
|
|
80
|
+
|
|
81
|
+
});
|
|
44
82
|
|
|
45
83
|
}
|
|
46
84
|
|
|
47
85
|
|
|
48
86
|
|
|
49
|
-
|
|
87
|
+
function spNav(){
|
|
50
88
|
|
|
51
|
-
//
|
|
89
|
+
//スマホナビの開閉
|
|
90
|
+
|
|
91
|
+
var $spNavBtn= $('.sp-nav-btn');
|
|
92
|
+
|
|
93
|
+
$accordion.on("click", function() {
|
|
94
|
+
|
|
95
|
+
$('body').addClass('is-open');
|
|
96
|
+
|
|
97
|
+
});
|
|
52
98
|
|
|
53
99
|
}
|
|
54
100
|
|
|
55
101
|
|
|
56
102
|
|
|
57
|
-
|
|
103
|
+
function deviceJudge(){
|
|
58
104
|
|
|
59
|
-
//
|
|
105
|
+
//デバイス判定
|
|
106
|
+
|
|
107
|
+
if (navigator.userAgent.match(/(iPhone|iPod|Android.*Mobile)/i)) {
|
|
108
|
+
|
|
109
|
+
return 'SP';
|
|
110
|
+
|
|
111
|
+
} else {
|
|
112
|
+
|
|
113
|
+
return 'PC';
|
|
114
|
+
|
|
115
|
+
}
|
|
60
116
|
|
|
61
117
|
}
|
|
62
118
|
|