リファクタリングと一時変数
リファクタリングでよく質問されるのが「結局一時変数って使っちゃダメなの？」という疑問です。
私は「後からコードを読む人が混乱する名前や使い方をせず、コードの可読性が上がるのであれば使用してよい」と答えています。
後からコードを読む人が混乱する名前を付けている場合は以下のリファクタリングを検討します。
●
後からコードを読む人が混乱する使い方をしている場合は以下のリファクタリングを検討します。
●
●
無駄な一時変数を使用している場合は以下のリファクタリングを検討します。
分類はあくまで目安としてお考えください。実際はこれら全てを複合してリファクタリングすることになります。
一時変数を「使え」というリファクタリングと、「使うな」というリファクタリングがあり最初は混乱しますが、目的は「コードの可読性を上げる」ですので、あなたの主観で読み易ければそこでリファクタリングを終了しても構いません。
全ての一時変数を除去することは不可能です。
コメントはまだありません。