回答編集履歴
5
<style/>のインデント修正
test
CHANGED
|
@@ -134,7 +134,7 @@
|
|
|
134
134
|
|
|
135
135
|
}
|
|
136
136
|
|
|
137
|
-
</style>
|
|
137
|
+
</style>
|
|
138
138
|
|
|
139
139
|
</head>
|
|
140
140
|
|
4
見出し修正
test
CHANGED
|
@@ -476,7 +476,7 @@
|
|
|
476
476
|
|
|
477
477
|
|
|
478
478
|
|
|
479
|
-
####
|
|
479
|
+
#### note
|
|
480
480
|
|
|
481
481
|
こうした、コードの整理の事を**リファクタリング**と呼びますが、
|
|
482
482
|
|
3
補足
test
CHANGED
|
@@ -473,3 +473,13 @@
|
|
|
473
473
|
}
|
|
474
474
|
|
|
475
475
|
```
|
|
476
|
+
|
|
477
|
+
|
|
478
|
+
|
|
479
|
+
#### appendix
|
|
480
|
+
|
|
481
|
+
こうした、コードの整理の事を**リファクタリング**と呼びますが、
|
|
482
|
+
|
|
483
|
+
ここで提示したコードは完全なリファクタリングを 10 と仮定した場合の 2 程度に相当します。
|
|
484
|
+
|
|
485
|
+
より高品質の**望ましいコード**は、最初期のコードの流れとは大きく異なってくるため、ここでは割愛し、本コードまでに留めました。
|
2
値バリデーションの不備を修正するため、ステップ数を若干冗長化
test
CHANGED
|
@@ -292,7 +292,7 @@
|
|
|
292
292
|
|
|
293
293
|
{
|
|
294
294
|
|
|
295
|
-
for ($i =
|
|
295
|
+
for ($i = 0; $i < $this->rows; $i++) {
|
|
296
296
|
|
|
297
297
|
$row = $arr[$this->row_keys[$i]];
|
|
298
298
|
|
1
文言修正
test
CHANGED
|
@@ -14,13 +14,13 @@
|
|
|
14
14
|
|
|
15
15
|
#### 設計
|
|
16
16
|
|
|
17
|
-
このコードでは、
|
|
17
|
+
このコードでは、logic と view を完全に分離しました。
|
|
18
|
-
|
|
18
|
+
|
|
19
|
-
index.php がメイン処理（Controller）
|
|
19
|
+
index.php がメイン処理（Controller相当）
|
|
20
|
-
|
|
20
|
+
|
|
21
|
-
Table.php が表の部品を管理するクラス（Model〈バリデーター内封型〉）
|
|
21
|
+
Table.php が表の部品を管理するクラス（Model相当〈バリデーター内封型〉）
|
|
22
|
-
|
|
22
|
+
|
|
23
|
-
View.php が描画を行う静的クラス（View）
|
|
23
|
+
View.php が描画を行う静的クラス（View相当）
|
|
24
24
|
|
|
25
25
|
となっています。
|
|
26
26
|
|