Hacker News Reader

Comment by sacnoradhq | Hacker News Reader

You're oversimplifying the use of continuation / builder pattern / "fluent" / monadic interfaces.

   SetReadTimeout(SetWriteTimeout(SetConnectTimeout(FactoryClass.factory(), 100), 2000), 4000);

vs.

   FactoryClass.factory().SetConnectTimeout(100).SetWriteTimeout(2000).SetReadTimeout(4000);

https://java-design-patterns.com/patterns/fluentinterface/

View on HN

Or, in my opinion, even worse... When you get mix-and-match because you want to extend a fluent API. Using your example, what if `SetWriteTimeout` was a static method you wrote, but all the others were provided by library as a fluent API? Then you end up with this kind of unholy mess:

    SetWriteTimeout(FactoryClass.factory().SetConnectTimeout(100), 2000).SetReadTimeout(4000);

Good luck correctly guessing the order of operations on that in a quick read...

This, having to remember which methods are implemented inside the class and which ones should have been is almost like the colored function problem.

Nice example. The extra parameters really drive the point home. Hope you don't mind if I steal it for the README.

Reply on news.ycombinator.com