Đây có phải là một antipotype để sử dụng peek () để sửa đổi một phần tử luồng không?

Giả sử tôi có một luồng điều và tôi muốn "làm phong phú" chúng giữa dòng, tôi có thể sử dụng peek()để làm điều này, ví dụ:

streamOfThings.peek(this::thingMutator).forEach(this::someConsumer);

Giả sử rằng việc thay đổi Điều ở thời điểm này trong mã là hành vi đúng - ví dụ: thingMutatorphương thức có thể đặt trường "LastProcessed" thành thời điểm hiện tại.

Tuy nhiên, peek()trong hầu hết các bối cảnh có nghĩa là "nhìn, nhưng đừng chạm vào".

Là sử dụng peek()để đột biến các yếu tố dòng một antipotype hoặc không khuyến khích?

Chỉnh sửa:

Cách tiếp cận khác, thông thường hơn sẽ là chuyển đổi người tiêu dùng:

private void thingMutator(Thing thing) {
    thing.setLastProcessed(System.currentTimeMillis());
}

đến một hàm trả về tham số:

private Thing thingMutator(Thing thing) {
    thing.setLastProcessed(currentTimeMillis());
    return thing;
}

và sử dụng map()thay thế:

stream.map(this::thingMutator)...

Nhưng điều đó giới thiệu mã chiếu lệ ( returnvà) và tôi không tin nó rõ ràng hơn, bởi vì bạn biết peek()trả về cùng một đối tượng, nhưng với map()một cái nhìn thậm chí không rõ ràng rằng đó là cùng một loại đối tượng.

Hơn nữa, với peek()bạn có thể có một lambda đột biến, nhưng với map()bạn phải xây dựng một con tàu đắm. Đối chiếu:

stream.peek(t -> t.setLastProcessed(currentTimeMillis())).forEach(...)
stream.map(t -> {t.setLastProcessed(currentTimeMillis()); return t;}).forEach(...)

Tôi nghĩ rằng peek()phiên bản rõ ràng hơn và lambda rõ ràng đang biến đổi, vì vậy không có tác dụng phụ "bí ẩn". Tương tự, nếu một tham chiếu phương thức được sử dụng và tên của phương thức ngụ ý rõ ràng đột biến, thì điều đó cũng rõ ràng và rõ ràng.

Về lưu ý cá nhân, tôi không ngại sử dụng peek()để biến đổi - tôi thấy nó rất tiện lợi.

Bạn đúng, "peek" theo nghĩa tiếng Anh của từ này có nghĩa là "nhìn, nhưng đừng chạm vào."

Tuy nhiên , JavaDoc tuyên bố:

nhìn trộm

Stream peek (Hành động của người tiêu dùng)

Trả về một luồng bao gồm các phần tử của luồng này, ngoài ra thực hiện hành động được cung cấp trên mỗi phần tử khi các phần tử được tiêu thụ từ luồng kết quả.

Từ khóa: "thực hiện ... hành động" và "tiêu thụ". JavaDoc rất rõ ràng rằng chúng ta sẽ peekcó khả năng sửa đổi luồng.

Tuy nhiên , JavaDoc cũng tuyên bố:

Lưu ý API:

Phương thức này tồn tại chủ yếu để hỗ trợ gỡ lỗi, nơi bạn muốn xem các phần tử khi chúng chảy qua một điểm nhất định trong đường ống

Điều này chỉ ra rằng nó được dự định nhiều hơn để quan sát, ví dụ: các phần tử ghi nhật ký trong luồng.

Những gì tôi nhận được từ tất cả những điều này là chúng ta có thể thực hiện các hành động bằng cách sử dụng các phần tử trong luồng, nhưng nên tránh làm biến đổi các phần tử trong luồng. Ví dụ, hãy tiếp tục và gọi các phương thức trên các đối tượng, nhưng cố gắng tránh các hoạt động đột biến trên chúng.

Ít nhất, tôi sẽ thêm một nhận xét ngắn gọn vào mã của bạn dọc theo những dòng này:

// Note: this peek() call will modify the Things in the stream.
streamOfThings.peek(this::thingMutator).forEach(this::someConsumer);

Ý kiến ​​khác nhau về tính hữu ích của những bình luận như vậy, nhưng tôi sẽ sử dụng một nhận xét như vậy trong trường hợp này.

Nó có thể dễ dàng bị hiểu sai, vì vậy tôi sẽ tránh sử dụng nó như thế này. Có lẽ tùy chọn tốt nhất là sử dụng hàm lambda để hợp nhất hai hành động bạn cần vào lệnh gọi forEach. Bạn cũng có thể muốn xem xét trả lại một đối tượng mới thay vì thay đổi đối tượng hiện tại - nó có thể kém hiệu quả hơn một chút, nhưng nó có thể dẫn đến mã dễ đọc hơn và giảm khả năng vô tình sử dụng danh sách đã sửa đổi cho một hoạt động khác nên đã nhận được bản gốc.

Lưu ý API cho chúng ta biết rằng phương thức này đã được thêm vào chủ yếu cho các hành động như gỡ lỗi / ghi nhật ký / bắn chỉ số, v.v.

@apiNote Phương thức này tồn tại chủ yếu để hỗ trợ gỡ lỗi, nơi bạn muốn * xem các phần tử khi chúng chảy qua một điểm nhất định trong đường ống: *

       {@code
     * Stream.of ("một", "hai", "ba", "bốn")
     * .filter (e -> e.length ()> 3)
     * .peek (e -> System.out.println ("Giá trị được lọc:" + e))
     * .map (Chuỗi :: toUpperCase)
     * .peek (e -> System.out.println ("Giá trị đã ánh xạ:" + e))
     * .collect (Collector.toList ());
     *}