注: 記述する上で面倒なので、以下で「関数」と単に書いてある場合、メソッドやプロシージャといったものをまとめて指すこととする。

TL;DR

「分割」しようとするな。一つ一つ、別の処理に「分解」しろ。

そもそもなぜ「分割」しろといわれるのか

短くしたいから。

人間、長い文章は途中で混乱するものだ。
そしてプログラムも結局は文章である。

多くの日本人が英語の本を読んだら混乱するように、人類は長いプログラムを読むと混乱する。これは当然だ。

なにしろプログラムは「プログラミング言語」という言語で記述されている。この言語は困ったことに、なんと外国人どころかコンピュータと対話するための言語である。
そりゃあ、慣れないうちは（あるいは慣れたとしても）混乱して当然だ。

重要なのは「短い」ことではない

「This is a pen」は日本人を混乱させるだろうか。
もちろん、させないだろう。「これはペンです」。簡単だ。

では「Er ist wieder da」はどうだろう (いや、これは人によっては通じると思うが) 。
答えは「彼、再び」といった具合になるらしい。ちなみにドイツ語で、小説のタイトルである。彼再び！って、なんのこっちゃ？¹

そう、仮に十分に短い場合であっても、それ単独で意味が通じなければ、結局多方面を調べ回るハメになる。当然時間がかかる。
重要なのは、わかりやすいことだ。もし混乱しないのであれば、長くてもその方がよい。
短くしろと言われるのは、短ければ「マシ」になりやすいからであって、短くすることそれ自体は解決策たり得ない。

例

ここにクソ長いメソッドがあり、これを短くしろと命じられたとする。

public void hoge(List<Hoge> h, List<Fuga> f){

    ...(なんかいっぱい)...

}

駄目な例

「切り離しました！」

public void hoge(List<Hoge> h, List<Fuga> f){

    part1(h, f);
    part2(h, f);
    part3(h, f);

}

private void part1(List<Hoge> h, List<Fuga> f){

    ...(なんか)...

}

(以下略)

ぶん殴られても文句言えない。

何が駄目か

単純に、結果どうなるのかを見通すためには全部読まなければならない。
part1, part2, part3 はそれぞれ独立していないので、本当にただ切り離しただけなのだ。
こうなってくると単に連続した処理が別の場所に書かれただけで、いろいろなところを読まなければならない分却ってめんどくさい。

どうすべきか

こうすると少し改善する。
(他にもいろいろやるべきことはあるが、今回の記事でいいたいのはここ)

public void hoge(List<Hoge> h, List<Fuga> f){

    List<Pair<Hoge, Fuga>> result1 = part1(h, f);
    List<Pair<Hoge, Fuga>> result2 = part2(result1);
    List<Pair<Hoge, Fuga>> result3 = part3(result2);

}

protected List<Pair<Hoge, Fuga>> part1(List<Hoge> h, List<Fuga> f){

    ...(なんか)...

}
protected List<Pair<Hoge, Fuga>> part2(List<Pair<Hoge, Fuga>> result1){

    ...(なんか)...

}

(以下略)

どう違うのか

先ほどと違い、part1, part2, part3 はそれぞれ「結果」を返している。
乱暴な言い方だが、結果が出るということは、その処理一つだけでも意味が通り、検証が可能だということである。
このため、自動ユニットテストも掛けやすくなる。手動のテストもいくらかやりやすくなるだろう。
(protected になっていることに気づいたらなかなか鋭い。普段ならリフレクションで private を触らせるが、今回はそういう意味合いも込めて protected にした)

結構面白かった。日本語訳も出てるので興味があれば調べるとよい。そう、調べないとわからんだろ。 ↩