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

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

ただいまの
回答率

87.96%

コンポジションのgetter連鎖

解決済

回答 4

投稿

  • 評価
  • クリップ 0
  • VIEW 1,415
退会済みユーザー

退会済みユーザー

よろしくお願いします。

現在、クラス設計を行っています。

has-aの関係が多かったので、コンポジションを多く使いました。
例えば、下記のようなクラス構成だったとします。

Houseクラス
 ┗Bathroomクラス
Bathクラス
     ┗Waterクラス
       ┗temperatureフィールド

この時、Houseインスタンスからtemperatureフィールドの値を取得しようとすると、Getterの嵐になってしまうのですが、これは設計としては普通でしょうか?
わかりにくくて申し訳ありません。

Getterは出来るだけ定義したくないという思想で設計をしたかったので、不安です。
ご教示をよろしくお願いします。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 4

+4

あまりプログラムの事情もわからずに設計の良し悪しは申し上げられませんが、お気付きの通り良くない設計です。
何が良くないかというと、Getterだらけになることそのものではありません。Getterだらけになっているのは結果です。

この例だと、「お風呂の水温」を家の外側で取得しなければいけないことがすでに設計の誤りです。
家を与えられたとき、その家のお風呂の水温を使って何かする処理、というのを家の外側で書かないといけないのが抽象化不足なのです。
処理そのものをお風呂の中に、家の中に、閉じ込められませんか?
もっと大きな処理要求を家にメソッドとして持たせるようにできませんか?

動作状況のログを取るような目的だとそうも言っていられなかったりしますが。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/07/15 22:16

    この設計ポリシーをデメテルの法則と呼びます。

    キャンセル

  • 2016/07/16 00:01

    デメテルの法則、初めて聞きました。心に留めておきます。
    確かに、家の外からお風呂の温度がわかったところで、利用することは出来ません。
    今一度、見なおしてみます。
    ありがとうございました。

    キャンセル

checkベストアンサー

+1

こんにちは。

Getterの嵐になってしまう

この嵐ですが、下記イメージでしょうか?

house.getBathroom().getBath().getWater().getTemplature();

もし、YESなら、これでは情報の隠蔽になりません。正にGetterの暗黒面に該当すると思います。
もし、houseの外から、風呂の温度を取得する必要があるのなら、

house.getBathTemplature();

だと思います。

そして、実は、風呂の温度を家の外から取得できると言うことは、風呂桶には温度計が付いていて、Bathroomもしくは、Houseの中にお風呂のコントローラが設置されていると思います。
つまり、温度は温度計が計測し、その温度情報は風呂の水の中にあるのではなく、お風呂のコントローラが保持している姿になるケースが多いように感じます。

つまり、house.getBathTemplature()は、Houseに含まれるBathControllerからお湯の温度を獲得するわけですね。
更にどちらかと言うと、getBathTemplature()で取得できる値は設定温度で、他に風呂の温度が規定範囲内か外かのフラグを返す関数が別途ある印象を受けます。

要するに、yubaさんの見解に賛成です。


【追記】
iwamoto_takaakiさんへのコメントを見て追記しています。

PNG画像データは構造を持っていますね。その各構造をオブジェクト指向的なクラスとして実装するのは間違いと私は考えています。
「PNG画像データに何かメッセージを送って何らかの処理をさせる」がオブジェクト指向的な発想ですね。
しかし、PNG画像データは単にデータに過ぎず、自律的に何かの処理を行うのは違和感が強いです。

そのようなデータを保持することが主目的な構造データは「構造体」です。
それは、全てのデータはpublicフィールドで保持し、メソッドはコンストラクタくらいしか持たないものをイメージしています。

そのように考えるとすっきり設計できるのではないでしょうか?
逆に、この概念を排除すると、今回悩まれているようにPNG画像データのような多少複雑な構造を持つデータのハンドリングが悩ましくなります。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/07/16 00:11

    house.getBathroom().getBath().getWater().getTemplature();
    まさしく、そのイメージです。

    BathControllerですね。まさしく、うちにも付いています。
    has-aの関係しか見ず、実際の相対関係が見えていませんでした。浅はかでした。
    今一度、設計を見なおしてみます。
    ありがとうございました。

    キャンセル

  • 2016/07/20 22:11

    遅くなり申し訳ございません。
    追記をいただきありがとうございます。
    お教え頂いたとおり、構造体を使って設計をやり直しております。
    なんとか、png.frameset(idx).GetBitmap(x,y)ぐらいまでに収めることができました。
    ツールの構造上、「どのPNGファイルの、どのフレームの、どこの座標の」までの
    紐付きが必要のようなので、ここらへんが限界のようです、、、
    最後までご教授をいただきありがとうございました。
    元々はHouseクラスについての質問だったので、
    誠に勝手な私見となってしまいますが、
    一番具体的に歩を進めることができましたので、ベストアンサーにさせていただきます。

    キャンセル

0

"この時、Houseインスタンスからtemperatureフィールドの値を取得しようとする" 用途が多ければ非常に使いづらいので、別途Utiltilyを作成して簡単に属性を取得できるクラスを設けるくらいでしょうか。

アクセサメソッドが多くなってしまう問題とクラス階層が深くなってしまう話はそこまで密接な関係ではないように思えますし、あまりに煩雑だと思うのであればCommons-OGNLのようなライブラリを使うなどで回避するでしょう。

DataTransferObjectのような、属性の出し入れしか提供しないクラスであれば、つけてもつけなくても一緒です。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/07/16 00:05

    Commons-OGNLやDataTransferObjectなど、
    初めて聞きました。
    申し訳ありません。心に留めておきます。
    実は、家の外からさらにWaterの量やBathの広さ等、
    色々取得しようとしています。
    Utilityも検討いたします。
    ありがとうございました。

    キャンセル

0

知りたいのは”お風呂の温度”ですよね。だったら、Bath.getWaterTemparature()で取得出来るべきかなと思います。当然お湯がなかったら、NoWaterExceptionが帰ってきます。

私は実際のコードが言語に近いほうが、良い設計だと思います。

オブジェクト指向を学ぶ際は、実行可能なコードを書くことを強くおすすめします。概念的になりすぎて、実際のコードに反映出来ないのであれば、オブジェクト指向のコードの勉強として意味が無いです。(実際のプログラムでは無いですよね?念のため・・・)

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/07/16 00:27 編集

    おっしゃるとおり、Waterにgetterがあるのはおかしいと感じます。
    また、Waterがない場合もありますね。それはBathが知っているべきでした。
    ありがとうございました。

    こちらは、実際のプログラムではないです。わかりやすい例えを考えて、
    辿り着いた先がHouseとtemperatureでした・・・

    実際のプログラムは、PNG画像の情報を表にまとめるツールを作っているのですが、

    PNG
     ┗Chunk
       ┗ColorData
         ┗RED(RGBのうちの1つ)

    このPNGインスタンスを生成して、外からREDの情報を取得しようとしたとき、
    どうしてもgetterが汚くなってしまったので、ご相談をさせていただきました。

    キャンセル

  • 2016/07/16 08:56

    それなら、getter自体は仕方が無いかもしれません。
    自然言語にすると”このPNGのこのchunkのREG”見たいな指定でしょうか。

    ただし階層が深すぎるときは呼び出し側でメソッドを分割することを考えた方がいいかもしれません。呼び出し側のコードがわからないのでどのように分割すると綺麗なのか提案出来ませんが、やはりわかりやすい名前がつく範囲に分割することをおすすめします。

    キャンセル

  • 2016/07/20 22:05

    遅くなり申し訳ございません。
    コメントをありがとうございます。
    下記でChironianさんからコメントいただきました、構造体を
    使っての設計を行っております。
    png.frameset(idx).GetBitmap(x,y)ぐらいまでに収めることができました^^;

    キャンセル

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

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

関連した質問

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