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

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

ただいまの
回答率

90.84%

  • Java

    12848questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

EffectiveJava項目14について

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 160

Attsu

score 6

effectiveJava項目14のサンプルコードが理解できませんのでどなたかご教授いただけますでしょうか。

継承の代わりにコンポジションを使用するとなっていますが結局Set<E>インターフェースをFowardingSet<E>が
継承しInstrumentedSet<E>がFowardingSet<E>を継承しているという三層構造になっていて継承の代わりにコンポジションを使用するという目的が達成されているように見えません。

サンプルコードの構造が理解できないのでどなたか教えていただけると幸いです。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • asahina1979

    2018/01/07 17:57

    どの版のEfect Java だろう・・ どの出版会社発行のエフェクトジャバだろう・・・・途中から 項目16になってるようだし

    キャンセル

  • asahina1979

    2018/01/07 18:07

    見つけた。。。。初版本「2001/12/3」・・・

    キャンセル

回答 2

+4

この件は、簡単に言えば、「責任の負えないクラスを直接継承するな」ということになります。
通常はHashSetのソースコードなんて誰も読んでないですよね。(私は読んでるという方には失礼)
まさかaddAll()のなかでadd()が呼ばれているなんて、、、(まあ予想はつきますが、、)
この内部仕様を知らずに、手軽にカウンターなんてつけちゃうと誤動作するわけです。
そこで、わが所有のインスタンス変数の制御権を取り戻すために、HashSetから切り離すのがこの事例なんです。
よく知らないクラスに振り回され、原因不明なバグに悩まされるクラスと、完全制御下に置いたクラスとの比較になっています。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/01/07 18:17 編集

    どうも質問の内容のは大きく改定している模様

    項目ーー(第三版 2018年出版:予定)
    項目16(第二版 2014年出版)
    項目14(第一版 2001年出版):質問者参考、古い言語仕様

    キャンセル

  • 2018/01/07 19:00

    すいません、項目16でした。申し訳ございません。

    キャンセル

  • 2018/01/08 00:09 編集

    私は訳本を以前持っていましたが、今は持っていませんでしたので、うろ覚えと経験で回答しただけです。しかしそのような理由があったとは。うん、パターンとアンチパターンに理由を求めるのはあまりよろしくないですね。ライブラリのクラスでもfinalになっているものがあると「こんちくしょー」とか思ったりするんですけど何かしらの理由があるということですね。

    キャンセル

checkベストアンサー

+1

Effective Javaのソースコードらしきものを見ましたが、確かにコンポジションを使用していますね。

https://github.com/marhan/effective-java-examples/tree/master/src/main/java/org/effectivejava/examples/chapter04/item16

まず、FowardingSetはSetを継承ではなく、実装しています。
ForwardingSetはコンストラクタでSetインターフェースの実装インスタンスをもらってますよね。で、保持してますよね。これはコンポジションです。

コンポジションしないで継承するやり方は、ForwardingSetクラスがコンストラクタにSetインターフェースの実装インスタンスを引数として取らずに、HashSetクラスやInstrumentedHashSetクラスを継承することです。このことにより、ForwardingSetクラスは間接的にSetインターフェースを実装することにもなります。

しかし、そうすると、ForwardingSetクラスは余計なメソッドやフィールドをもらってしまう訳なのです。それはForwardingSetクラスが本来やりたかったことでなく、ForwardingSetクラスのドキュメントも長大化します。

ForwardingSetクラスがやりたいことはSetインターフェースを実装しているクラスについて、Setインターフェース以外のメソッドやフィールドを隠すことです。その上で、InstrumentedSetクラスはカウント機能を追加しています。

なんでそんなことをするかというと、Setインターフェースの実装にはいろいろ種類があって、速度やメモリ使用に関する特性が異なるためです。だから、Setインターフェースの実装が具体的にどれなのかについては、あえて言及していない訳です。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/01/07 17:59

    最新原版では項目16であるものを項目14といってるから著しく中古書籍を譲り受けたか買ったひとみたいだねぇ・・・

    キャンセル

  • 2018/01/07 18:50

    丁寧なご回答ありがとうございます。ForwardingSetクラスの役割が見えてきました。
    重ねての質問申し訳ないのですがそもそもこんな二段構えにせずにInstrumentedSetクラスが直接Setインターフェースを実装してしまえばよいのではないかと思いました。何か理由があるのでしょうか。個人的には別の方法のAddメソッドを持つクラスがForwardingSetを継承するためとかかなと考えたりしています。

    キャンセル

  • 2018/01/08 00:38 編集

    はい、いいのではないでしょうか。しかしボトムアップな考え方ではそれが簡潔でも、トップダウンな考え方だと可説明的でなくなるときがあります。そもそもForwardingSetとはなんでしょうか。私にはわかりません。AbstructSetでもないですものね。ForwardingSetとInstrumentedSetというのが雑です。
    うーん、decorationパターンなのかな。違う気もするな。
    http://itpro.nikkeibp.co.jp/article/COLUMN/20060113/227227/

    一つ言えることはSetインターフェースにはコンストラクタを実装できませんが、ForwardingSetクラスにはコンストラクタを実装することができるということです。

    ぱちっ、コンポジションを使えばsuperで買い物をするひつようがないのか。しかしHashSetのaddAllでaddを呼び出すと、それを継承したクラスでカウント機能を付けた時にバグりますかね?カウントはInstrumentedSetクラスのaddメソッドとaddAllメソッドで行っているので、HashSetの実装がどうなっていようと、InstrumentedSetクラスのaddAllメソッドでダブルカウントすることはないと思うのです。ちょっと実装していないので何とも言えないですが、InstrumentedSetクラスのaddCountフィールドが汚されることはないと思います。ただ、protectedされたメソッドやフィールドなんかが絡んでくると頭が爆発してくるかもしれません。

    オブジェクト指向は関数型に比べて難しいと思います。ボトムアップとトップダウンという考え方が合流してしまって収拾がついてない感じがあります。とりあえず、品のよいプログラミングを目指されることをお勧めします。

    キャンセル

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

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

関連した質問

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

  • Java

    12848questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。