Q&A
individualClassX とは何をするクラスで、sharedClass とは何をするクラスなのかという具体的な情報がないと、具体的なアドバイスをすることは難しいと思います
似たような処理をするindividualClassAとindividualClassBをコーディングしたいと考えています。
今後もクラスを追加していきたいので、できるだけクラス間の結合度を弱めつつ重複を排除するためにオブジェクト指向について説明した書籍を読みつつコーディングしました。（実用と勉強半々です。）
以下のコード群ような構成にしてみましたが、下のような問題があると思います。
- var1、var2のようなindividualClass内で共通的に使用するパラメータが増えると、パラメータを使用するメソッド（individualMethod）の引数が長くなる。（２つだとあまり実感がないが、５つ以上になると汚くみえる）
- 引数とすることを避けるために、各メソッドからパラメータを直接参照できるようグローバル変数にすると、integrityを保証できない（予期せず変更されてしまう可能性がある）
こうした問題を解消するようなうまいプログラム構造設計の方法はあるのでしょうか。
そもそも以下のようなコーディングはオブジェクト指向プログラミングとして適切かつ効率的ないのでしょうか。（何かツッコミどころがないでしょうか）
やや漠然とした質問ですみません。
なお、individualClassは２つだけではなく、さらに増やしていくつもりでいます。individualClassの固有パラメータも２つだけではなく増やすつもりです。
python
1class sharedClass: 2 3 def sharedMethodA(): #実装は省略 4 5 def sharedMethodB(): #実装は省略 6 7 def sharedMethodC(): #実装は省略
python
1class individualSuperClass: #実装は省略
python
1class individualClassA(individualSuperClass): 2 3 def __init__(self): 4 # ClassA 固有のパラメータ 5 self.__var1 = '' 6 self.__var2 = '' 7 8 def individualMethod(self): 9 # ClassA 固有の処理を実行 10 hogehoge 11 # ClassA 固有の処理を実行後、ClassAのパラメータを使って共通処理を行う 12 sc = sharedClass() 13 sc.sharedMethodA(self.__var1, self.__var2) 14 sc.sharedMethodB(self.__var1) 15 sc.sharedMethodC(self.__var2)
python
1class individualClassB(individualSuperClass): 2 3 def __init__(self): 4 # ClassB 固有のパラメータ 5 self.__var1 = '' 6 self.__var2 = '' 7 8 def individualMethod(self): 9 # ClassB 固有の処理を実行 10 hogehoge 11 # ClassB 固有の処理を実行後、ClassBのパラメータを使って共通処理を行う 12 sc = sharedClass() 13 sc.sharedMethodA(self.__var1, self.__var2) 14 sc.sharedMethodB(self.__var1) 15 sc.sharedMethodC(self.__var2)
python
1def main(): 2 ic = individualClassA() 3 ic.individualMethod() 4 ic = individualClassB() 5 ic.individualMethod() 6 7if __name__ == "__main__": 8 main()
用途によってクラスの構成が変わるということでしょうか。
sharedClassはindividualClassで共通的に行う処理を独立させたものです。
individualClassは軽自動車やトラックのクラスで、sharedClass（のメソッド）はタイヤをつけるとかエンジンを搭載するとか、そういうイメージだとお伝えすれば大丈夫でしょうか。
つけるタイヤやエンジンを変えつつ軽自動車やトラックを生成するということです。
それは sharedClass, IndividualClass の具体的な説明ではなく、「プログラミング言語におけるクラスという機能は何か」の説明ですよね？
重要なのは、そのような機能を使って、 sharedClass, IndividualClass がプログラムの中でどのようなことを実現したいのか、表現したいのかということです。
finok さんの作っているプログラムは、軽自動車やトラックやタイヤやエンジンを扱うプログラムなんでしょうか？
回答2件
0
ベストアンサー
var1、var2のようなindividualClass内で共通的に使用するパラメータが増えると、パラメータを使用するメソッド（individualMethod）の引数が長くなる。（２つだとあまり実感がないが、５つ以上になると汚くみえる）
単純に method( a, b, c, d ... ) とやるのはアレだからってことですか?
それなら私なら「クラスとして抽出する」ですかね。抽出っていうか、method( a, b, c ... ) でいう a～z のような引数を ArgumentEx とかみたいなクラスとして定義し、それを渡す。
質問にあるコードであれば、individualMethodメソッドの処理部を見ると処理内容がほぼ一致している。
その場合も、やはり引数を別のクラスとして切り出すかな。
たとえば、ArgumentExとしてクラスにし、
def __init__(self, argumentEx): // <- これと # ClassB 固有のパラメータ argumentEx_ = argumentEx // <- これと def individualMethod(self): # ClassB 固有の処理を実行 hogehoge # ClassB 固有の処理を実行後、ClassBのパラメータを使って共通処理を行う argumentEx.commonMethod() # <- これ!
のように。
引数とすることを避けるために、各メソッドからパラメータを直接参照できるようグローバル変数にすると、integrityを保証できない（予期せず変更されてしまう可能性がある）
これも上記のオブジェクトにすればいいはず。
[追記1]
この場合、新しいindividualClassを追加しようとすると、パラメータクラスも同時に追加することとなり、２つクラスを作らなければいけないのが少々気になっていました
A,Bが持つフィールドってまったく別物ですか? たとえば Aでは "攻撃力"と"守備力", Bでは "回復力"と"ダメージ力"とかみたいに。
まったく別物であればクラスごとに作ることになりますが、私の最初の所見では、A,Bともにほぼ同じフィールド(個別のものがいくつかあるぐらい)だと考えていたのですが。
ほぼ同じフィールドであればまとめるためのクラスは一個でいいです。
...commonMethodをそれらの中に定義することとなると、これは共通処理なので、クラス間で重複コードが生じることとならないでしょうか。
まあ、でしょうね。ただ、上で述べたように私の解釈ではA,Bは同じフィールドを持っているっていうことなので。(もちろん中身自体は別物だけど)
インターフェースみたいなものを用意してこれを実装して引数をまとめたやつを定義する。
単純に 上でいう ArgumentExは commonInitメソッドみたいな共通の初期化をするようなメソッドを持ち、そのメソッドをA,Bがそれぞれ動かす。
def __init__(self, argumentEx): // <- これと # ClassB 固有のパラメータ argumentEx_ = argumentEx // <- これと
に関しては、単純に argumentExっていう引数としてのオブジェクトを受け取って フィールドとしての argumentEx_ にセットしているだけです。
別にget/set系でやってもいいですが。
単純に「クラスにまとめればいいじゃね?」ってことが言いたかっただけです。
投稿2022/02/19 09:02
編集2022/02/20 13:56
総合スコア4958
ありがとうございます。クラスとしてまとめるのも手ですよね。
この場合、新しいindividualClassを追加しようとすると、パラメータクラスも同時に追加することとなり、２つクラスを作らなければいけないのが少々気になっていました。各パラメータクラス各individualClassと１対１対応になるなら、individualClass内に書いてしまうのが自然であり、実装上の都合でクラスを分割することに少し違和感があったということです。
ただ、それを割り切ればイメージしていたものとだいぶ近いものです。
ちなみに、ArgumentExクラスってindividualClassA、individualClassBごとに定義する必要があると思いますので、ArgumentExAとかArgumentExBとして定義することとなると思いますが、commonMethodをそれらの中に定義することとなると、これは共通処理なので、クラス間で重複コードが生じることとならないでしょうか。
それとも、そうではなくてArgumentExを共有のパラメータクラスとして１つだけとすることとし、individualClass内のコンストラクタでsetter経由でパラメータの値を設定するということでしょうか。
argumentEx_ = argumentEx の部分が勉強不足でよく理解できませんでした。
追記のご説明、ありがとうございました。
パラメータクラスを１つ作る方法で実装してみたいと思います。
0
共通で利用したい変数をindividualSuperClassにprivateで持たせて、取得はgetterのみという形にするというのはどうでしょう。
python
1class parent: 2 def __init__(self): 3 self.__ver1 = 1 4 self.__ver2 = 2 5 def ver1(self): return self.__ver1 6 def ver2(self): return self.__ver2 7 8class child(parent): 9 def myprint(self): 10 print(self.ver1()) 11 print(self.ver2()) 12 13a = child() 14a.myprint()
これであればパラメータが増えてきたなく見えるのを避け、かつ外部からの変更を防ぐことができます。
投稿2022/02/18 20:09
退会済みユーザー
総合スコア0
ありがとうございます。
privateにした上でgetterで値を取得するなら、意図せず値が変更されてしまうことを防ぎつついろんな場所から参照できますね。
今回はindividualClassAとindividualClassBで異なるパラメータを持たせたいのですが、その場合はsharedClassの各メソッドの実行前にsetterで値を設定しておくこととなるのでしょうか。
また、パラメータはどちらかというとsharedClassのメソッド内で利用したいのですが、sharedClassからindividualSuperClassの変数をgetterで取得することはできるのでしょうか。
私もいろいろ構成考えてみましたが、あちらを立てるとこちらが立たずな状況で行き詰ってしまいました。
getter自体はpublicなので特に問題なく参照可能です。
ClassAとClassBで異なるパラメータを持たせたいという部分はコンストラクタで実現させるのがよいかと思います。
(setterを用意したらprivateで隠ぺいした意味がなくなりそうです)
sharedClassは独立したクラスなので、例えばindividualClassAから呼び出された場合、var1を参照するにはindividualClassA().var1と書かないといけないと思います。
すると、individualClassBから呼び出された場合もindividualClassAのパラメータを参照することとなってしまわないでしょうか。if文で参照するクラスを分岐させるのはオブジェクト指向ではないと思いますし・・・。
個別に引数指定するか、パラメータを含むオブジェクトを引数にするか、BeatStarさんの言うように共有のパラメータクラスを用意しないといけないのでしょうね。
> individualClassAから呼び出された場合、var1を参照
individualClassA内のメソッドでsharedClassのメソッドにvar1を渡す場合はself.var1()という形でgetter経由になるかと思いますが、あくまでもreadonlyなのでかまわないのではないでしょうか。
> individualClassBから呼び出された場合もindividualClassAのパラメータを参照する
すみません、これはちょっと意味が分かりませんでした。
クラスが異なっているので関係ないかと思いますが…
あなたの回答
tips
プレビュー
