Q&A
具体的なコードなりクラス図でも無いとふんわりとした回答しか来ないと思いますが…
こんにちは。よろしくお願いします。
###前提・実現したいこと
肥大化したクラスを分割したい。
###発生している問題
例えば、コンビニを題材にクラスを考えてみます。
コンビニでは客、店員、売品、仕入れ、売上高、売上目標、荷物受け取りなど様々な物があります。
なので上記に上げた各クラスを作ります。（客クラス、店クラス等）
私は店員と客が絡む関数を作ろうと思い考えた結果、各オブジェクトをまとめたクラスがあればスッキリするんじゃないかと思いました。
なのでオブジェクトを統括するクラスを作成してその中にListを作成。
上記に上げた各オブジェクトを生成したら直ぐにそのListに追加するようにしました。
その結果店員と客が絡む関数を作りたかったら引数にListで保持した店員オブジェクトと客オブジェクトを引数にして関数を定義してあげれば簡単にオブジェクト同士が絡む処理を作れるようになったので楽しく幸せにプログラミングをできていたのですが、いつの間にかListの長さが１０個以上になり、４っつ以上の要素が絡むような関数を作らないといけなくなりました。
さすがに４っつも絡むと引数も４っつになり、処理の内容も増えて可読性が落ちるかなぁと思い、関数を２つずつに分けようと思ったのですが、その結果仕方がないことですが関数が増えてしまい、ソースコードの量が増えてしまいました。
このままオブジェクトが増え続けると、いつかこのオブジェクトを統括したクラスが色々なオブジェクトを管理した行数の多い読むのがめんどくさいコードになる気がするのでこれどうすればいいかなぁと考えたのですが、なかなかいい方法が思い浮かばないのでなんか良いやり方があるのであれば教えて頂けると助かります。
よろしくお願いします。
それでもしこのような問題にヒットするようなサイトがあれば教えてくれると助かります。
回答2件
0
ベストアンサー
コードがないので、なんとも言えませんが、
- 共通化できるとこは共通化
- 目的に応じたクラスの分割
ですかね。
共通化
言わずもがな、全く同じ処理を何度も何度も書くなら、共通メソッドにしましょう。
目的に応じたクラスの分割
店員クラスに、レジ打ちの処理を入れず、
レジスター（レジ）クラスを定義し、レジに関連する処理のみ記述するのが良いかと。
例えばRegisterクラス（一部）を定義しても、
レジの機能として
- この会計の担当者の記録
- 商品の金額の読み取り、合計、割引
- ポイントカード関連の処理
- 電子マネーでの金銭のやりとり
- 現金での金銭のやり取り
などがありますが、これらの機能ごとにクラスを分割し、そのクラスをパッケージにまとめあげ、
店員が使用するときは、レジクラスに定義したメソッドを経由して各クラスの機能を呼び出すようにするとか。
register
├ レジ
├ 電子マネー
├ 現金
└ ポイントカード
その他
メソッドの引数が増えるのなら、いっそObject（インスタンス）を引数に渡すとかもありですね。
投稿2016/08/13 07:06
総合スコア507
あなたの回答
tips
プレビュー
