-
Notifications
You must be signed in to change notification settings - Fork 331
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
Higher Order Functions should come before Options #101
Comments
Thanks for reporting this. Can you please be more specific about which reference disturbs you and what kind of explanation is missing? What would you propose? Do you want to move the higher order function section? Or remove the higher order function reference? |
Sure. In the Options exercise, we find this piece of text:
This makes reference to Similarly, the next piece of instruction references
I would suggest moving the higher order function section, to come right after Classes, or perhaps even before Classes. Scala is primarily oriented towards object oriented programming and functional programming, so it makes sense for both Classes and HOFs to be introduced very early on. I also wonder if it would be worth adding just a section for "Functions,"which provides an introduction to the concept and syntax of functions in Scala, perhaps after Classes but before HOFs. Again, functions are obvious for people with prior programming knowledge, but are not for people without programming knowledge. Further, even if someone has prior programming knowledge, if they come from a procedural/imperative/otherwise non-functional background, they may not be familiar with the concept of a "pure" function, and it would be nice to introduce that somewhere. |
@FRosner any thoughts on what I wrote? |
Sorry @Ubadub I'm very busy at the moment and I couldn't find the time. Maybe another maintainer can take over. |
@FRosner Ok, thank you. |
The Options exercises makes reference to higher order functions, with very little explanation, which can be confusing for beginners. It should come before Options. The explanations should be buffed up too.
I'm willing to fork it and make the necessary changes if no one objects.
The text was updated successfully, but these errors were encountered: