関数の引数がbooleanだった場合は気をつけよう

2021年6月7日

関数の引数がフラグ(boolean)だった場合は、その関数をリファクタリングするチャンスかもしれません。

というのも、フラグを引数にとっているケースでは、そのフラグによって処理を分岐している可能性があるからです。

trueの場合はこの処理、falseの場合はこの処理、というようになっているということは、この関数は二つのことをやってしまっています。

それぞれの処理は分けて別々の関数に抽出してみると良いです。