関数は何行まで？長い関数を分ける判断基準 | 仕組み解説

「関数は何行までならよいですか」と聞かれることがあります。

目安はありますが、行数だけで良し悪しは決まりません。

一言でいうと

長い関数を分ける基準は、行数ではなく、同じ抽象度で1つの目的を説明できるかです。

20行でも読みにくい関数はあります。80行でも、順番に処理を読むだけで理解できる場合もあります。

次の関数は短いですが、読みにくいです。

function f(u: User) {
 return u.a && u.b > 18 && !u.c && u.d?.includes("x");
}

行数は1行です。しかし、何を判定しているのか分かりません。

逆に、少し長くても、処理の流れが自然なら読みやすいことがあります。

特に、コメントが「ここから入力チェック」「ここから保存処理」のようになっている場合は、関数名に置き換えられることがあります。

長い関数を分ける代表的なリファクタリングが、Extract Functionです。

function checkout(cart: Cart) {
 validateCart(cart);
 const total = calculateTotal(cart.items);
 return createOrder(cart.userId, total);
}

外側の関数は流れを示し、詳細は小さな関数へ分けます。

関数分割にもやりすぎがあります。

関数分割の目的は、読む人の負担を減らすことです。ファイル数や関数数を増やすことではありません。

最初は次の基準で考えると十分です。

絶対ルールではなく、考えるきっかけとして使います。

関数の良し悪しは、行数だけでは決まりません。

長い関数は、抽象度、変更理由、ネスト、テストしやすさを見て分けるか判断します。