-
Notifications
You must be signed in to change notification settings - Fork 170
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
why Filtered and not predictable, why a for each, and not streams? #1
Comments
@pgermosen I don't think Streams is a good idea in OOP, in general |
@pgermosen as for "predictable" -- I didn't understand. Can you give an example? |
Of course, streams are not oop , but is easy of use. this example ?
jdk 9 forEach in a collection I understand better this, than the oop approached
|
@pgermosen from my experience streams and other message driving architectures motivate split data and methods of its processing. |
Yes that is a motive of use. But what i want to make a point is the following points for a new api to take in account
the example of To sort a list of words in the file
for me has the same readability than
|
@pgermosen I'm not sure traditional declarative code will be more clear and shorter for this specific task. Moreover, I'm not sure |
I think api that can be read like this, is clearer(not necessary shorted, sometimes extra text for clarity is a better)
Example 2
|
@pgermosen this "fluent" interface ( |
@yegor256 it's quite declarative. The problem is different: it's always hard to support these fluent things combining smth new from individual objects and using inheritance. |
What is more important |
@pgermosen fluent interface is easy to use only because it's your habit to use it. Also, if you want it, there are plenty of libraries that do exactly the same as Cactoos, but in a traditional way. |
@pgermosen Do you agree with this solution? If yes, could you close this issue? If not, could you explain why not? Thanks for your attention! |
Don't you like the regular foreach yet present in java language? No lambdas needed. Dont make things too complex. |
@pgermosen can we close this issue? Our README states the motivations and design principles behind |
@pgermosen closing due to inactivity |
The job is not in WBS, won't close the order |
Correct the code to avoid stylechecker violation
No description provided.
The text was updated successfully, but these errors were encountered: