回答編集履歴
2
指摘を参考に修正
test
CHANGED
|
@@ -12,35 +12,13 @@
|
|
|
12
12
|
|
|
13
13
|
```javascript
|
|
14
14
|
|
|
15
|
-
|
|
15
|
+
{
|
|
16
16
|
|
|
17
|
-
|
|
17
|
+
let _n = 1;
|
|
18
|
-
|
|
19
|
-
}
|
|
20
|
-
|
|
21
|
-
|
|
22
18
|
|
|
23
19
|
Object.defineProperty(
|
|
24
20
|
|
|
25
|
-
|
|
21
|
+
this,
|
|
26
|
-
|
|
27
|
-
'_n',
|
|
28
|
-
|
|
29
|
-
{
|
|
30
|
-
|
|
31
|
-
value: 1,
|
|
32
|
-
|
|
33
|
-
writable: true
|
|
34
|
-
|
|
35
|
-
}
|
|
36
|
-
|
|
37
|
-
);
|
|
38
|
-
|
|
39
|
-
|
|
40
|
-
|
|
41
|
-
Object.defineProperty(
|
|
42
|
-
|
|
43
|
-
window,
|
|
44
22
|
|
|
45
23
|
'n',
|
|
46
24
|
|
|
@@ -52,15 +30,11 @@
|
|
|
52
30
|
|
|
53
31
|
newVal = Number(newVal);
|
|
54
32
|
|
|
55
|
-
if (isNaN(newVal) || newVal < 1 || newVal > 6)
|
|
33
|
+
if (isNaN(newVal) || newVal < 1 || newVal > 6) throw new RangeError('The value to asign must be number between 1 and 6.');
|
|
56
34
|
|
|
57
|
-
|
|
35
|
+
|
|
58
36
|
|
|
59
|
-
} else {
|
|
60
|
-
|
|
61
|
-
|
|
37
|
+
_n = newVal;
|
|
62
|
-
|
|
63
|
-
}
|
|
64
38
|
|
|
65
39
|
},
|
|
66
40
|
|
|
@@ -71,6 +45,8 @@
|
|
|
71
45
|
}
|
|
72
46
|
|
|
73
47
|
);
|
|
48
|
+
|
|
49
|
+
}
|
|
74
50
|
|
|
75
51
|
|
|
76
52
|
|
|
@@ -83,3 +59,13 @@
|
|
|
83
59
|
console.log(n);
|
|
84
60
|
|
|
85
61
|
```
|
|
62
|
+
|
|
63
|
+
|
|
64
|
+
|
|
65
|
+
think49さん >
|
|
66
|
+
|
|
67
|
+
度々のご指摘、大変にありがとうございます。
|
|
68
|
+
|
|
69
|
+
自分自身、大変に勉強になりました。
|
|
70
|
+
|
|
71
|
+
もっと自身も精進します 苦笑
|
1
指摘を受けて、コード改良
test
CHANGED
|
@@ -4,11 +4,21 @@
|
|
|
4
4
|
|
|
5
5
|
|
|
6
6
|
|
|
7
|
-
ただし、ブラウザ環境限定かと。
|
|
7
|
+
~~ただし、ブラウザ環境限定かと。~~
|
|
8
|
+
|
|
9
|
+
ブラウザでなくてもいけるようにしました。
|
|
8
10
|
|
|
9
11
|
|
|
10
12
|
|
|
11
13
|
```javascript
|
|
14
|
+
|
|
15
|
+
if (typeof window != 'object') {
|
|
16
|
+
|
|
17
|
+
var window = this;
|
|
18
|
+
|
|
19
|
+
}
|
|
20
|
+
|
|
21
|
+
|
|
12
22
|
|
|
13
23
|
Object.defineProperty(
|
|
14
24
|
|
|
@@ -40,9 +50,11 @@
|
|
|
40
50
|
|
|
41
51
|
set(newVal){
|
|
42
52
|
|
|
43
|
-
|
|
53
|
+
newVal = Number(newVal);
|
|
44
54
|
|
|
55
|
+
if (isNaN(newVal) || newVal < 1 || newVal > 6) {
|
|
56
|
+
|
|
45
|
-
throw new Error('
|
|
57
|
+
throw new RangeError('The value to asign must be number between 1 and 6.');
|
|
46
58
|
|
|
47
59
|
} else {
|
|
48
60
|
|