Q&A
前提・実現したいこと
JavaScriptでClassベースで開発をしています。
Classの中身が増えてきたら以下のように別のClassにしようと考えたのですが、Class内でいくつもnewを使うのがおかしいのではないかと考えました
該当のソースコード
js
1class d{ 2 ... 3} 4 5class a { 6 constructor(bb) { 7 this.bb = bb; 8 } 9 c(cc){ 10 new d(); 11 } 12}
試したこと
Classを複数使う or 単純な関数を使う
どちらもしっくりこなかったので何か良い実装を教えてほしいです。
補足情報（FW/ツールのバージョンなど）
node,ts
回答1件
0
ベストアンサー
new d(); という式は基本的には新たにオブジェクトを作成するときに使うものです.
new して作成したオブジェクトはメンバとして保持するのが正しいのでは？
もしくは extends する場合, super を使って継承元クラスの機能を呼びましょう.
また「中身が増えたから別のクラス」にするとしても, 必ずそこにはオブジェクト指向設計上の根拠が必要になります.
ざっくりと言うと, extends (A は B (の一種) である) すべき関係と, メンバとして保持 (A は B を持つ) すべき関係が考えられます.
どちらがふさわしいのかは, 「車と車種」「車とタイヤ」のような継承なのか保持なのか分かり易いものもあれば, そうでないものもあるので最終的には場面によります.
例えば「ユーザ」というクラスの中身が増えて別のクラスを作るとしても, 以下のような根拠付けがあり得ます:
- ユーザの「種別」によって保持すべきものが違うので, 共通部分をユーザ共通クラスで持ちそれを種別ごとに extends, 継承先で super を使って継承元のメソッドを呼ぶ.
- あれこれ沢山の要素を詰め込んでしまっているので要素を分類し, 分類ごとに別のクラスを作り, ユーザクラスで保持.
などなど...どんな分解をするにしても, そこにしっかりと根拠があるのであればメンバとして別の要素を new するのは問題ないのでは.
逆に無秩序に new するのであれば, 設計を見直すべきかと.
投稿2019/12/02 02:46
総合スコア354
あなたの回答
tips
プレビュー
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。
2019/12/02 02:53