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

04_removeFromArray: Hint about array.forEach and/or array.includes() in some way #520

Open
4 of 5 tasks
schikulski opened this issue Feb 2, 2025 · 0 comments
Open
4 of 5 tasks
Labels
Status: Needs Review This issue/PR needs an initial or additional review

Comments

@schikulski
Copy link

Prerequisites

  • I have thoroughly read and understand The Odin Project Contributing Guide
  • I have searched the existing issues to ensure this improvement hasn't been suggested before (please have a look through our open issues list to make sure)
  • The title of this issue follows the <Location of the improvement>: <Brief description of the improvement> format, e.g. Exercises: Add exercise on XYZ
  • (Optional) I am interested in working on this issue and would like to be assigned to it

Improvement Description

As a TOP student I successfully did this exercise by using two for loops – though it was not pretty.

After checking the solution, I see that both array.forEach and array.Includes() would make this a whole lot easier, and I'm thankful I learned that.

But I can't help but think that it would be a better learning experience if I would get a hint about those methods beforehand in the hints section.

I'm pretty sure I wasn't exposed to the either forEach or Includes earlier in the course, but I might be wrong.

Acceptance Criteria

  • Include a subtle hint about the forEach and Includes methods in the hints section.

(Optional) Additional Comments

No response

@schikulski schikulski added the Status: Needs Review This issue/PR needs an initial or additional review label Feb 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Needs Review This issue/PR needs an initial or additional review
Projects
None yet
Development

No branches or pull requests

1 participant