I just started learning Java streams and faced a problem. Please take a look at a the following example. This is part of a Node class:
private Map<String, Node> nodes; public Optional<Node> child(String name) { return Optional.<Node>ofNullable(nodes.get(name)); } private void findChildren(String name, List<Node> result) { child(name).ifPresent(result::add); nodes.values().stream() // .map(Node::findChildren(name, result)) // .forEach(Node::findChildren(name, result)) .forEach(node -> node.findChildren(name, result)); }
My intent was to call #findChildren with the name and result parameters on each node in the stream. I tried to use the method references Node::findChildren with no luck. I'd appreciate solutions other the the one with ->
operator.
Is it somehow possible to use the method reference together with a parameter? I like the idea of streams and I just want to make the code more readable.
Actually, I think there is a similar question Method references with a parameter which I read but cannot figure out how to use the bind2 method in my code. Is it the only solution?
The method references can only be used to replace a single method of the lambda expression. A code is more clear and short if one uses a lambda expression rather than using an anonymous class and one can use method reference rather than using a single function lambda expression to achieve the same.
You can’t use method references for this purpose. You have to resort to lambda expressions. The reason why the bind2
method of the linked question doesn’t work is that you are actually trying to bind two parameters to convert a three-arg function into a one-arg function. There is no similarly simple solution as there is no standard functional interface
for three-arg consumers.
It would have to look like
interface ThreeConsumer<T, U, V> { void accept(T t, U u, V v); } public static <T, U, V> Consumer<T> bind2and3( ThreeConsumer<? super T, U, V> c, U arg2, V arg3) { return (arg1) -> c.accept(arg1, arg2, arg3); }
Then .forEach(bind2and3(Node::findChildren, name, result));
could work. But is this really simpler than .forEach(node -> node.findChildren(name, result));
?
If you love us? You can donate to us via Paypal or buy me a coffee so we can maintain and grow! Thank you!
Donate Us With