「負債返済」のない細い修正まで「リファクタリング」って言うな

プログラミング

リファクタリングについて、以前にも記事を書きました。

「リファクタリング」とは?
リファクタリングリファクタリングとは、すでに出来上がったプログラムに対し、プログラムの動きは変えずに読みやすく改善することです。(あってますか?)なぜ必要 ?たいていのプログラムの現場は 納期がタイト であり、時間的な制約の中...

私の解釈としては

_ 「プログラムの動作を変えずに、中身を改善する」 ことがリファクタリング_

だと思ってます。

「夫妻を返済している」 改修をしているかどうかが、リファクタリング だとみなすに妥当かどうか、という解釈です。

でも、他の人はどうやら違うらしい。

これを リファクタリング と言わないでほしい

ケース1

// php
public function hoge($user_id, $nickname)

私: 規約にのっとって、返り値を宣言してください

Sさん: はい、リファクタリングしますね。

// php
public function hoge(int $user_id, string $nickname)

ん、
コレってリファクタリングですか ?

ケース2

// php
$this->hoge(User $attacker_user_detail, User $defender_user_detail, Deck $battle_deck_attakker, Deck $battle_deck_defender);

私: 横にクソ長いので適当に改行してくれる ?

Sさん: はい、リファクタリングしますねー

// php
$this->hoge(
  User $attacker_user_detail,
  User $defender_user_detail,
  Deck $battle_deck_attakker,
  Deck $battle_deck_defender
);

ただ改行しただけですよね。


妥当なコミットメッセージを考えるのが面倒なので、 リファクタリングという言葉でイージーに片付けようとしてませんか ?

こんなもんをリファクタリングと言うな。

本当にリファクタリングしたいときに、マネージャーに「型を宣言したり、改行を足すだけなのに、なんで予算が必要になんのかゴルァ」と言われたら、返せないよね。

リファクタリング本、読んでないよね ?

言葉を選ぼうぜ、だってばよ