一つの関数は一つの仕事

一つの関数が複数の役割を担っている場合、

• 関数の汎用性が低くなり、部分的に処理内容が重複した関数が作成されることにつながる。
• 修正時の影響範囲が大きくなる
• 関数名から処理内容を判断するのが難しくなる

など、様々な悪影響が起こります。

複数の役割を持った関数を見分けるコツ

何気なく関数に多くの責務をもたせてしまい、あとあと困ることがよくあります。見分けるコツとして、

• 一つの関数が10行を超えている
• 命名しにくい
• 一部だけ異なる別の関数を作りたくなった

これらの特徴があるときは複数の役割を持っている可能性が高くなります。