肥大化したJavaクラスを整理整頓する方法

Question

数千行のJavaで書かれたゲームプログラムに関わることになりました。
読み手への配慮はされているコードなのですが、全ての処理が一つのクラスに固まっており、コードの追加にあっちに行ったりこっちに行ったりと疲れてしまいます。
初めて関わる大規模プログラムのため、私にはソースの分割経験がありません。
こんな私でもどうにかしてファイルを分割する方法はないでしょうか。
開発環境はEclipseです。

Answer 1

大規模になっていながら単一クラスに処理がたくさん埋まっていつつ読み手に配慮があるということはOOPではない形で処理が関数分割されてることが予想されます。下手にクラス分割すると、今まで単一クラスでメソッド間で行き来してたのが、あなたのオレオレポリシーで散らかってしまい複数のクラスをさまようだけの変更にしてしまいかねないです。
次の人のことを考えるなら、最初の人のやっていた雰囲気を真似て、大きなクラスのまま同じになるようにするのがよいのではないかと思います。大規模なコードを読み修正する時には実はどんなポリシーであるかということより、同じポリシーが一貫しているかのほうが大事です。あそこはこんな書き方、こっちはこんな書き方というほうが困るのです。
他の方のお話もありますが、ソースはそのままのポリシーで変更を続け、むしろテストを整備するほうが適切であると思います。

Answer 2

リファクタリングは、小さく改善してテストの繰り返しが原則です。
その積み重ねで見違えるようになっていきます。
一気にやろうとすると、簡単なものでも動かなくなる可能性があります。

ですので、まず最初にクラスかメソッドごとにJunitなどで単体テストのコード（できれば自動）を書いてください。
こちらがリファクタリングがうまくいったか（もとのまま動くか）の指標になります。

次に大きなメソッドを小さなメソッドに分けてそのたびテストを実行してください。

そうしているうちに、クラス（ファイル）を分割できるぐらいほぐれてきます。

リファクタリングの具体的な手法は、
Martin Fowlerの本"リファクタリング 既存のコードを安全に改善する”
に詳しいです。
この本を読むとリファクタリングだけでなく、
オブジェクト指向プログラミングへの理解が大変深まるのでおすすめです。

2016/02/20 追記
この質問への回答の範囲を超えるのですが、ほかの人がポリシーについて答えられているので追記いたします。

リファクタリングをすべきか、すべきならどの程度やるべきかは
対象のコードの性質によります。

もし機能追加や改修、不具合修正が頻繁にあり、かつ長期に渡って使われるプロダクトやサービスならば
少しずつでもリファクタリングをしていくといいと思います。

それほど変更がないものなら、リファクタリングのコストがメリットを上回る可能性があります。

あと正直、業界や会社にもよりますね。
先輩社員にリファクタリングをやっているか、まず訊いてみるといいでしょう。

Answer 3

概念としては以下のページが簡単にまとめられています。
[リファクタリング]Extract Class

このページにも出てきますが**単一責任の原則（SRP）**という原則の観点を意識することで、どのように分割していけばいいかわかりやすくなると思います。

Answer 4

Eclipseにはリファクタリング用の機能があります。
他の回答にあるようなリファクタリングの概念を学んだ上で、実際の作業にはJDTの機能を使うとよいでしょう。

https://www.ibm.com/developerworks/jp/opensource/library/os-eclipse-refactoring/