回答編集履歴
3
誤ったコメントを削除
test
CHANGED
@@ -100,7 +100,7 @@
|
|
100
100
|
|
101
101
|
|
102
102
|
|
103
|
-
return true;
|
103
|
+
return true;
|
104
104
|
|
105
105
|
}
|
106
106
|
|
2
typo修正、参照透過性が成り立つコードの説明を修正
test
CHANGED
@@ -46,7 +46,7 @@
|
|
46
46
|
|
47
47
|
|
48
48
|
|
49
|
-
しかしながら、構造化プログラミングには入口/出口が狭い都合上、コードの階層が深くなってしまう欠点があり、最近の考え方としては「例外処理は先に返
|
49
|
+
しかしながら、構造化プログラミングには入口/出口が狭い都合上、コードの階層が深くなってしまう欠点があり、最近の考え方としては「例外処理は値を先に返す事で追い出してしまい、後続コードを正常系処理のみにする事ですっきりしたアルゴリズムにしよう」が主流なようです。
|
50
50
|
|
51
51
|
実際にJavaScriptの基本仕様である ECMAScript でも同様のアルゴリズムが組まれています。
|
52
52
|
|
@@ -120,7 +120,7 @@
|
|
120
120
|
|
121
121
|
|
122
122
|
|
123
|
-
余談ですが、入口
|
123
|
+
余談ですが、入口を一つにする原則は、「**参照透過性が成り立つコード**」を組む時にも通用する考え方となります。
|
124
124
|
|
125
125
|
参照透過性が成り立たないコードは「引数とは別に入口がある」と考えられるからです。
|
126
126
|
|
1
「構造化プログラミング」の節を追記。それに伴い、「複数の出口を持つコード」の節に補足説明を追記。
test
CHANGED
@@ -1,3 +1,135 @@
|
|
1
|
+
### 構造化プログラミング(structured programming)
|
2
|
+
|
3
|
+
|
4
|
+
|
5
|
+
構造化プログラミング(structured programming)と呼ばれる思想があり、その原則に「**関数は単一の入口と単一の出口を持つべきである**」というものがあります。
|
6
|
+
|
7
|
+
|
8
|
+
|
9
|
+
```JavaScript
|
10
|
+
|
11
|
+
'use strict';
|
12
|
+
|
13
|
+
function getHeightClamped (text) { // 仮引数 (単一の入口)
|
14
|
+
|
15
|
+
const minHeight = 5;
|
16
|
+
|
17
|
+
const maxHeight = 10;
|
18
|
+
|
19
|
+
const textHeight = text.split("\n").length; // 返り値となる変数を作る
|
20
|
+
|
21
|
+
|
22
|
+
|
23
|
+
if (textHeight < minHeight) {
|
24
|
+
|
25
|
+
textHeight = minHeight; // 返り値に最小値を代入する
|
26
|
+
|
27
|
+
} else if (textHeight > maxHeight) {
|
28
|
+
|
29
|
+
textHeight = maxHeight; // 返り値に最大値を代入する
|
30
|
+
|
31
|
+
}
|
32
|
+
|
33
|
+
|
34
|
+
|
35
|
+
return textHeight; // 返り値を返す (return 節を一度だけ使う事で単一の出口となる)
|
36
|
+
|
37
|
+
}
|
38
|
+
|
39
|
+
```
|
40
|
+
|
41
|
+
|
42
|
+
|
43
|
+
この原則の利点として、アルゴリズムを書きだした時に**すっきりした構造となること**が挙げられます。
|
44
|
+
|
45
|
+
入口/出口が複数あると、考えるパターンが複雑化してしまい、人間の目で見て追いきれない事があります。
|
46
|
+
|
47
|
+
|
48
|
+
|
49
|
+
しかしながら、構造化プログラミングには入口/出口が狭い都合上、コードの階層が深くなってしまう欠点があり、最近の考え方としては「例外処理は先に返して追い出してしまい、後続コードを正常系処理のみにする事ですっきりしたアルゴリズムにしよう」が主流なようです。
|
50
|
+
|
51
|
+
実際にJavaScriptの基本仕様である ECMAScript でも同様のアルゴリズムが組まれています。
|
52
|
+
|
53
|
+
一例として、ECMAScript 5.1 規定の `Array.isArray` の処理を読んでみましょう。
|
54
|
+
|
55
|
+
|
56
|
+
|
57
|
+
- [15.4.3.2 Array.isArray ( arg ) - ECMA-262 Edition 5.1](http://ecma-international.org/ecma-262/5.1/#sec-15.4.3.2)
|
58
|
+
|
59
|
+
|
60
|
+
|
61
|
+
> 1. If Type(arg) is not Object, return false.
|
62
|
+
|
63
|
+
> 2. If the value of the [[Class]] internal property of arg is "Array", then return true.
|
64
|
+
|
65
|
+
> 3. Return false.
|
66
|
+
|
67
|
+
|
68
|
+
|
69
|
+
1. 引数 `arg` が Object 型でない場合は `false` を返す
|
70
|
+
|
71
|
+
2. 引数 `arg` の内部プロパティ `[[Class]]` が `"Array"` ならば、`true` を返す
|
72
|
+
|
73
|
+
3. `false` を返す
|
74
|
+
|
75
|
+
|
76
|
+
|
77
|
+
1., 2., 3. のstepで**「そうでなければ(else)」の文言がないのは、前stepで値を返す事で例外処理が終わっている為**です。
|
78
|
+
|
79
|
+
互換コード(Polyfillコード)は次のように書けます。
|
80
|
+
|
81
|
+
|
82
|
+
|
83
|
+
```JavaScript
|
84
|
+
|
85
|
+
Object.defineProperty(Array, 'isArray', {writable: true, enumerable: false, configurable: true,
|
86
|
+
|
87
|
+
value: function isArray (arg) { // 1. If Type(arg) is not Object, return false.
|
88
|
+
|
89
|
+
|
90
|
+
|
91
|
+
if (Object(arg) !== arg) {
|
92
|
+
|
93
|
+
return false;
|
94
|
+
|
95
|
+
}
|
96
|
+
|
97
|
+
|
98
|
+
|
99
|
+
if (Object.prototype.toString.call(arg) === '[object Array]') { // 2. If the value of the [[Class]] internal property of arg is "Array", then return true.
|
100
|
+
|
101
|
+
|
102
|
+
|
103
|
+
return true; // 3. Return false.
|
104
|
+
|
105
|
+
}
|
106
|
+
|
107
|
+
|
108
|
+
|
109
|
+
return false; // 3. Return false.
|
110
|
+
|
111
|
+
}
|
112
|
+
|
113
|
+
});
|
114
|
+
|
115
|
+
```
|
116
|
+
|
117
|
+
|
118
|
+
|
119
|
+
このコードは「入口が一つ、出口は複数」ですが、正常系処理に限っては「入口が一つ、出口が一つ」とする事で構造化プログラミングの理念の則ってすっきりしたコードを書けます。
|
120
|
+
|
121
|
+
|
122
|
+
|
123
|
+
余談ですが、入口と出口を制限する原則は、「**参照透過性が成り立つコード**」を組む時にも通用する考え方となります。
|
124
|
+
|
125
|
+
参照透過性が成り立たないコードは「引数とは別に入口がある」と考えられるからです。
|
126
|
+
|
127
|
+
|
128
|
+
|
129
|
+
### 複数の出口を持つコード
|
130
|
+
|
131
|
+
|
132
|
+
|
1
133
|
私は基本的に汎用性重視でコードを組みます。
|
2
134
|
|
3
135
|
|
@@ -6,8 +138,6 @@
|
|
6
138
|
|
7
139
|
- 引数無しは無制限です
|
8
140
|
|
9
|
-
- if -> return のコードは後続コードが実行されないので、else-if で繋ぎませんでした
|
10
|
-
|
11
141
|
|
12
142
|
|
13
143
|
```JavaScript
|
@@ -58,4 +188,18 @@
|
|
58
188
|
|
59
189
|
|
60
190
|
|
191
|
+
`return` 節が複数存在する事から、このコードは「構造化プログラミング」の原則に反しています。
|
192
|
+
|
193
|
+
このコードは「例外処理系」と「正常処理系」を分割する事に利点があると考える為、else-if 節で繋がず、空行を開ける事で「それぞれの例外処理」を細かく分けるように書いています。
|
194
|
+
|
195
|
+
|
196
|
+
|
197
|
+
### 更新履歴
|
198
|
+
|
199
|
+
|
200
|
+
|
201
|
+
- 2017/10/29 16:37 「構造化プログラミング」の節を追記。それに伴い、「複数の出口を持つコード」の節に補足説明を追記。
|
202
|
+
|
203
|
+
|
204
|
+
|
61
205
|
Re: nabettu さん
|