refactoring

Author: tam315

books/refactoring.md

@@ -244,30 +244,30 @@ const main = () => {
 
 ### リファクタリングの問題点
 
-#### 新機能の実装が遅くなる
-
-- リファクタの目的はプログラミングの速度を上げること
-- よって実装は遅くならない
-- ただしトレードレードオフもある。このあたりは経験がモノを言う。
-  - リファクタする場合の例
-    - 以後の作業が楽になるとき
-    - 何度も同じ問題が起きているとき
-    - 同じような醜いコードに何度も出くわしているとき
-  - リファクタしない場合の例
-    - 新機能が非常に些細なもので、リファクタを後回しにできるとき
-    - めったに触らない箇所であるとき
-- 道徳的な理由でリファクタしようとするのは危険
-  - 「コードが美しくなる」とか
-  - 「(しばしば原理主義的な)すばらしいプラクティスにコードを近づける」とか
-- 経済的な基準でのみ判断せよ
+#### 新機能の実装が遅くなる？
+
+- 実装は遅くならない
+  - なぜならリファクタの目的はプログラミングの速度を上げることだから
+  - ただしトレードレードオフはあるので、このあたりの判断は経験がモノを言う。
+- リファクタする場合の例
+  - 以後の作業が楽になるとき
+  - 何度も同じ問題が起きているとき
+  - 同じような醜いコードに何度も出くわしているとき
+- リファクタしない場合の例
+  - 新機能が非常に些細なもので、リファクタを後回しにできるとき
+  - めったに触らない箇所であるとき
+  - 道徳的な理由でリファクタしてはならない
+    - 「コードが美しくなる」だの
+    - 「(しばしば原理主義的な)すばらしいプラクティスにコードを近づける」だの
+    - 経済的な基準でのみ判断せよ
 
 #### コードの所有権
 
 - コードの部分ごとにオーナーを設定し、変更を承認するようなやり方が良いのでは
-- これは、以下の 2 つの間を取ったもの
+- これは以下の 2 つのあいだを取ったもの
   - 所有権が強すぎて、リファクタが困難な状態
-    - e.g. 関数名を変えたときに、呼び出し元のコードを変更できない場合など。こういったときは古い関数から新しい関数を呼び出すようにして、古い関数には deprecated を指定するなどするものの、最悪の場合、古い関数は永遠に残り続けることになる。
-  - 所有権がなくて、誰もが好き勝手に変更を加えてカオスになった状態
+    - e.g. 関数名を変えたときに、呼び出し元のコードを変更できない、など。こういったときは古い関数から新しい関数を呼び出すようにして、古い関数には deprecated を指定するなどするものの、最悪の場合、古い関数は永遠に残り続けることになる。
+  - 所有権がなくて、誰もが好き勝手に変更を加えてカオスになる状態
 
 #### ブランチ
 
@@ -290,3 +290,26 @@ const main = () => {
     - 新たに埋め込んでしまったバグにすぐ気づくから
 - テストがない or 少ない場合は、IDE に安全が保証されている自動実行できる種類のリファクタであれば、実施できるかも
   - ただし注意深く手順に従う必要があるし、言語仕様にも依存するので比較的高度なやり方
+
+#### レガシーコード
+
+- レガシーコード = テストのないコード、テストの追加も困難なコード
+- 「レガシーコード改善ガイド」を読めとのこと
+
+#### データベース
+
+- 個々の変更は小さく、かつ完結したものにする
+- 正式リリースに至るまでに複数のリリースに分割する
+  - 例えばフィールド名の変更であれば：
+    - 1. 新規フィールドの追加（まだ使わない）
+    - 2. 新旧両方のフィールドに書き込みを行うようコードを変更
+    - 3. 古いフィールドを削除（使われていないことを確認した上で）
+
+### リファクタリング、アーキテクチャ、Yagni
+
+- リファクタリングという武器を手にすると:
+  - 現時点で判明している最低限の要件を満たすだけの、最もシンプルなコードを書きさえすればよい
+  - 将来のために変に柔軟性を持たせたりしない
+    - e.g. 関数に無駄にたくさん引数持たせるなど
+    - YAGNI = You ain't goint to need it = どうせ使わねえよ
+    - リファクタリングできるんだから、必要になった段階で変更すれば足りる