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

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

ただいまの
回答率

90.33%

  • Java

    14473questions

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

Optional,Streamについて。どちらが正しいのでしょうか。

解決済

回答 2

投稿 編集

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

ponkotate

score 5

Java8で追加されたOptionalとStreamについていくつか実装について迷った点を質問させていただきます。

Optional.ofの使い所

以下のようなnullを許容しないフィールドxxxを受け取る関数を実装しようと考えます。Optional.ofはnullを許容しないものでありますが、そうであれば、そのOptionalインスタンスについてorElse等を使える状況を作り出す必要はないと思います。Optional.ofを使ってOptionalを受け取る利点はあるのでしょうか。

final T xxx;

T getXxx() {
  return Objects.requireNonNull(xxx);
}


Optional<T> getXxxAsOptional() {
 return Optional.of(xxx);
}

Collectionの駆逐

Java8になって、for文からforEachへの移行が行われました。その際、Collectionではstream()を利用してStreamへの変換を行うわけですが、毎回そうする必要はあるのかと疑問に思いました。そこで、Streamをフィールドとして持たせようと考えました。これについて正しいのかどうか教えていただきたいです。
以下について、再代入せずに済む方法もあれば教えていただきたいです。

List<T> list = new ArrayList();
Stream<T> stream = Stream.empty();

void addToList(T t) {
  list.add(t);
}

void addToStream(T t) {
  stream = Stream.concat(stream, Stream.of(t));
}

Streamが空であるかどうかの判定

Streamが空であるかどうかを以下のように実装するのは正しいのでしょうか(1はcount()が不定なため良くないのかとは思いますが)。正しい方法を教えてください。

Stream stream;

boolean isEmpty1() {
  return stream.count() <= 0;
}

boolean isEmpty2() {
  return stream.findAny().isPresent();
}
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 2

checkベストアンサー

+2

Optional.ofの使い所

Optional.ofメソッドは「非nullが保証されるT型の変数」からOptional<T>への変換を行います。質問中のgetXxxAsOptional()メソッド実装はその目的通りですから、特に問題無いと思います。


Collectionの駆逐

Streamオブジェクトをクラスフィールドとして持つのは不適切です。Streamオブジェクトは「使い切り」、つまり利用のたびにstream()メソッドにて生成する必要があります。

Java 8 インタフェースStream<T>より引用:

(中間または終端ストリーム操作を呼び出して)ストリームを操作するのは、一度だけにすべきです。このため、たとえば同じソースが2つ以上のパイプラインに対してデータを供給する「フォークされた」ストリームや、同じストリームを複数回トラバースすることなどは、禁止されます。ストリーム実装は、ストリームが再利用されているのを検出すると、IllegalStateExceptionをスローします。ただし、一部のストリーム操作からは新しいストリーム・オブジェクトではなくレシーバが返されることがあるため、場合によっては再利用を検出できない可能性があります。 


Streamが空であるかどうかの判定

いずれも正しい実装です。要素数が多いときは、短絡評価(short-circuit)を行う後者isEmpty2()メソッドの方が効率が良いと思います。

追記:対象ストリームが無限ストリーム(例:java.util.Random#ints())の場合、前者isEmpty()メソッドは正常に機能しなくなります。安全性と効率の両面から後者実装の方が好ましいと思いました。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/03/10 11:28

    回答中では「Optional.ofを使ってOptionalを受け取る利点」の部分が意図不明のため無視しました。

    キャンセル

  • 2017/03/10 13:37 編集

    Optionalについて、言葉足らずで申し訳ありません。Optional.ofで生成されたOptionalインスタンスを返すメソッド(ここでいうgetXxxAsOptional)を利用される際、当然中身は空でないことがわかっているため、
    getXxxAsOptional().get()
    で中身を取り出す等の動作を期待するわけですが、利用する側に立つとブラックボックスとして扱うことになるため、orElse等で本来実行されることのない処理を記述されるかもしれません。そのような危険性のあるOptional.ofに使い所はあるのかという質問でした。

    CollectionとStreamに関しては、試行もしくは正確に情報を得ていれば理解できたはずな愚問で申し訳ありません。

    空の判定は安心しました。もし使う必要が出た場合は後者を利用するようにします。

    キャンセル

  • 2017/03/10 17:07 編集

    > 利用する側に立つとブラックボックスとして扱うことになるため、orElse等で本来実行されることのない処理を記述されるかもしれません。そのような危険性のあるOptional.ofに使い所はあるのかという質問でした。

    どちらかと言えば、クラス外部設計の観点で考えるべきでしょう。あるメソッドがOptional<T>を返すとき、クラス利用者からは「何らかの有効値または無効値のいずれかが返ってくるメソッド」と認識します。Java 8のOptional<T>型は、まさにこのような使い方のために導入されたクラスです。
    → 参考:http://stackoverflow.com/a/23464794/684921 Stuart Marks氏による回答(同氏はJava仕様策定をリードしている人)

    クラス提供者が「Optional<T>を返すが常に有効値だからorElseされると無駄だ」と考えること自体が間違っています。そのような無用な心配をするのではなく、クラス提供者の責任でメソッドの戻り値を単にT型を返すべきです(もちろんnullを返したらどうなるという古典的な問題は残りますが)。言い換えると、メソッドの戻り値型はクラスの“設計意図”を表しているべきです。

    キャンセル

  • 2017/03/10 17:13

    orElseを「危険」と捉えていることに違和感を覚えますが…。
    ヌルポが発生するのはプログラムが強制終了してしまうため危険ですが、意味のないorElseはただ無駄なだけで危険ではありません。

    キャンセル

+2

nullでないことが保証されているものにOptionalは確かに向きませんね。使い方としては、メソッドの引数などでOptionalを要求するところに渡す際、対象の値がnullでないことを確認することでしょうか…。

Streamは1回使い切りです。一度終端操作を始めたStreamをもう一度使うことはできません。

Streamはデータ集合に対する操作を提供するものなので、「空かどうか」をチェックするのはStreamっぽくないともいえます。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/03/10 12:03

    ご回答ありがとうございます。

    やはり、Optional.ofはあまり用いることがないのですね。

    Streamについてはこちらの勉強不足でした。認識を改めるようにします。ありがとうございました。

    キャンセル

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

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

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

  • Java

    14473questions

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