リファクタリング第2章

Before（メインルーチン内で処理が重複している）

switch ($result) {
case self::TYPE_1:
$result[message] .= "({$result['message_for_type_1']})";
break;
case self::TYPE_2:
$result[message] .= "({$result['message_for_type_2']})";
break;
}

25.

[beta]

After（メソッドを定義することで、処理を分離）

$result['message'] = $this->getMessage($result['result_code'], $result['messages']);
// ----- 中略

-----

private function getMessage(int $resultCode, array $messages):string {
switch ($resultCode) {
case self::TYPE_1:
return "({$messages['message_for_type_1']})";
break;
case self::TYPE_2:
return "({$messages['message_for_type_2']})";
break;
}
}

26.

2. 理解のためのリファクタリング

27.

2. 理解のためのリファクタリングそこでどんな処理をしているかを理解するためのリファクタリング。【例】 ● 適切な変数名をつける ● 長いメソッドを分割する

28.

Before（変数名、メソッド名がわかりづらい） $data = fetchData($id); $isExists = false; $count = 0;

29.

After $orderDataArray = fetchOrderData($orderId); $isExistsInvalidOrder = false; $invalidOrderCount = 0; 注文データを取得し、想定通りの値かをチェックする処理を行う、と予想できる。

30.

3. ゴミ拾いのためのリファクタリング

31.

3. ゴミ拾いのためのリファクタリング手をつけたコードの周辺を少しでも綺麗にしておくためのリファクタリング。【例】 ● タイポ修正 ● 入り組んだロジックをシンプルに書き直す

32.

[beta]

Before（タイポあり。処理もわかりづらい）

$this->setConfition($conditions, $postedData);
private function setConfition(&$conditions, $postedData)
{
foreach ($postedData['orderIdArray'] as $orderId) {
$conditions[][$orderId] = $postedData['conditions'];
}
}

33.

[beta]

After（タイポ修正＆処理をわかりやすくした）

$conditions[] = $this->setCondition($postedData);
private function setCondition(array $postedData): array
{
foreach ($postedData['orderIdArray'] as $orderId) {
$conditions[$orderId] = $postedData['conditions'];
}
return $conditions;
}

34.

ここまでのまとめソフトウェアが「完成」することはない。いつか修正する日のために、変更しやすい状態を保つことが大事。

35.

リファクタリングの問題点

36.

リファクタリングは、(短期的な)開発時間とのトレードオフが発生する。

37.

リファクタリングを後回しにしても良いケース 1. 些細な機能追加 2. めったに触らない & 不便さを感じていない箇所かけた時間と効果が見合わないことが想定される。

38.

注意点リファクタリングは、純粋に経済的な基準で測られるものである。下記のような思考はNG ● 美しいコードを書くためにリファクタリングする ● リファクタリングはエンジニアの責務である

39.

リファクタリングとテスト