質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.35%

  • Java

    16740questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

  • Eclipse

    2034questions

    Eclipseは、IBM社で開発された統合開発環境のひとつです。2001年11月にオープンソース化されました。 たくさんのプラグインがあり自由に機能を追加をすることができるため、開発ツールにおける共通プラットフォームとして位置づけられています。 Eclipse自体は、Javaで実装されています。

  • リファクタリング

    17questions

    リファクタリングとはコードの本体を再構築するための手法であり、外見を変更せずに内部構造を変更/改善させることを指します。

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

受付中

回答 4

投稿

  • 評価
  • クリップ 1
  • VIEW 3,775

ikenuma

score 9

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

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 4

+1

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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

+1

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

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

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

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

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

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

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

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

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

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

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

+1

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

0

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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.35%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

同じタグがついた質問を見る

  • Java

    16740questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

  • Eclipse

    2034questions

    Eclipseは、IBM社で開発された統合開発環境のひとつです。2001年11月にオープンソース化されました。 たくさんのプラグインがあり自由に機能を追加をすることができるため、開発ツールにおける共通プラットフォームとして位置づけられています。 Eclipse自体は、Javaで実装されています。

  • リファクタリング

    17questions

    リファクタリングとはコードの本体を再構築するための手法であり、外見を変更せずに内部構造を変更/改善させることを指します。