Skip to content
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

The vec2 exercise is inappropriate #2102

Open
senekor opened this issue Sep 10, 2024 · 1 comment
Open

The vec2 exercise is inappropriate #2102

senekor opened this issue Sep 10, 2024 · 1 comment
Labels
A-exercises Area: Exercises C-enhancement Category: Enhancement
Milestone

Comments

@senekor
Copy link
Contributor

senekor commented Sep 10, 2024

The following commit added significant complexity to the exercise: 8e1f617

I understand the desire to show people functional programming, I love it as much as the next Rust developer. But this exercise comes pretty much right after primitive types.

Even using vectors is a stretch here, the book itself covers vectors only much later. But I think that's withing reason here, because vectors are necessary to do almost anything and most programmers will be familiar with something similar. So I think it's justified to cover vectors a bit earlier than the book.

However... iterators? map? collect? lambda syntax? I find this completely unjustified. If this content is not covered in the dedicated section for iterators, it can be moved there.

Please consider removing the vec_map part from that exercise so as not to overwhelm new learners too early.

@mo8it
Copy link
Contributor

mo8it commented Sep 10, 2024

I agree. I will remove it from the exercise in v7. But I will keep it in the solution as one of the possible solutions. I will mention that we will deal with iterators later.

@mo8it mo8it added C-enhancement Category: Enhancement A-exercises Area: Exercises labels Sep 10, 2024
@mo8it mo8it added this to the v7 milestone Sep 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-exercises Area: Exercises C-enhancement Category: Enhancement
Projects
None yet
Development

No branches or pull requests

2 participants