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

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

ただいまの
回答率

90.50%

  • アルゴリズム

    510questions

    アルゴリズムとは、定められた目的を達成するために、プログラムの理論的な動作を定義するものです。

肥大化したクラスをなるべく分割させてスッキリさせたい

解決済

回答 2

投稿

  • 評価
  • クリップ 2
  • VIEW 591

nyan_lia

score 30

こんにちは。よろしくお願いします。

前提・実現したいこと

肥大化したクラスを分割したい。

発生している問題

例えば、コンビニを題材にクラスを考えてみます。

コンビニでは客、店員、売品、仕入れ、売上高、売上目標、荷物受け取りなど様々な物があります。

なので上記に上げた各クラスを作ります。(客クラス、店クラス等)

私は店員と客が絡む関数を作ろうと思い考えた結果、各オブジェクトをまとめたクラスがあればスッキリするんじゃないかと思いました。

なのでオブジェクトを統括するクラスを作成してその中にListを作成。
上記に上げた各オブジェクトを生成したら直ぐにそのListに追加するようにしました。

その結果店員と客が絡む関数を作りたかったら引数にListで保持した店員オブジェクトと客オブジェクトを引数にして関数を定義してあげれば簡単にオブジェクト同士が絡む処理を作れるようになったので楽しく幸せにプログラミングをできていたのですが、いつの間にかListの長さが10個以上になり、4っつ以上の要素が絡むような関数を作らないといけなくなりました。

さすがに4っつも絡むと引数も4っつになり、処理の内容も増えて可読性が落ちるかなぁと思い、関数を2つずつに分けようと思ったのですが、その結果仕方がないことですが関数が増えてしまい、ソースコードの量が増えてしまいました。

このままオブジェクトが増え続けると、いつかこのオブジェクトを統括したクラスが色々なオブジェクトを管理した行数の多い読むのがめんどくさいコードになる気がするのでこれどうすればいいかなぁと考えたのですが、なかなかいい方法が思い浮かばないのでなんか良いやり方があるのであれば教えて頂けると助かります。

よろしくお願いします。
それでもしこのような問題にヒットするようなサイトがあれば教えてくれると助かります。

  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

質問への追記・修正、ベストアンサー選択の依頼

  • ozwk

    2016/08/13 16:09

    具体的なコードなりクラス図でも無いとふんわりとした回答しか来ないと思いますが…

    キャンセル

回答 2

checkベストアンサー

+2

コードがないので、なんとも言えませんが、

  • 共通化できるとこは共通化
  • 目的に応じたクラスの分割

ですかね。

 共通化

言わずもがな、全く同じ処理を何度も何度も書くなら、共通メソッドにしましょう。

 目的に応じたクラスの分割

店員クラスに、レジ打ちの処理を入れず、
レジスター(レジ)クラスを定義し、レジに関連する処理のみ記述するのが良いかと。

例えばRegisterクラス(一部)を定義しても、
レジの機能として

  • この会計の担当者の記録
  • 商品の金額の読み取り、合計、割引
  • ポイントカード関連の処理
  • 電子マネーでの金銭のやりとり
  • 現金での金銭のやり取り

などがありますが、これらの機能ごとにクラスを分割し、そのクラスをパッケージにまとめあげ、
店員が使用するときは、レジクラスに定義したメソッドを経由して各クラスの機能を呼び出すようにするとか。
register
├ レジ
├ 電子マネー
├ 現金
└ ポイントカード

 その他

メソッドの引数が増えるのなら、いっそObject(インスタンス)を引数に渡すとかもありですね。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/08/13 22:51

    返答ありがとうございます。

    機能ごとにクラスを分割させてみるを参考にしたらより良くなったと思います。ありがとうございました。

    キャンセル

+1

こんにちは。

店クラスや客クラスを同じListに入れるのでしょうか?
全く異なる振る舞いをするものを同じListに入れるのは強い違和感を感じます。

それぞれ専用のメンバーに入れた方がスマートにプログラム出来そうな気がします。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/08/13 22:57

    返答ありがとうございます。

    はい私も同じリストに入れるのは違和感を感じ、現在それぞれの専用のメンバーにリストで確保した後、更に専用クラスを作成して分割しております。

    キャンセル

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

  • アルゴリズム

    510questions

    アルゴリズムとは、定められた目的を達成するために、プログラムの理論的な動作を定義するものです。