TheOdinProject / curriculum

The open curriculum for learning web development
https://www.theodinproject.com/
Other
9.99k stars 13.35k forks source link

JS Recursive Methods: Add a video as an additional resource #26497

Closed silenceinspace closed 11 months ago

silenceinspace commented 1 year ago

Describe your suggestion

Understanding recursion, especially the call stack, may be challenging for most folks. I faced the same problem - I couldn't get how exactly the recursion unwraps all its subcalls once it reaches the base case.

I found this video really helpful: https://www.youtube.com/watch?v=M-5nzjxtXpQ (starting from 9:44 there, the author shows how to visualize recursion better with debugger in Chrome. I have never really opened it in the browser before. However, it is honestly nice how one can walk through every step in the recursion and follow the logic)

I think others could benefit from it. Moreover, neither the article nor the already mentioned resources do not mention this way of visualizing recursion. Otherwise another resource I came across while trying to grasp the Fibonacci recursion was https://recursion.vercel.app/. It has really smooth animations and you can walk through every recursive call.

Path

Node / JS

Lesson Url

https://www.theodinproject.com/lessons/javascript-recursive-methods

Checks

(Optional) Discord Name

No response

(Optional) Additional Comments

No response

wise-king-sullyman commented 1 year ago

These resources both look good to me, the visualization app is especially neat!

If we remove that Medium article that we talked about in your open PR I'd be on board with adding both since they each bring something new to the lesson.

If we don't remove that Medium article we'll have to talk through which of these two would be better to include.

silenceinspace commented 1 year ago

Sounds solid! Then I will keep an eye on both the issue and PR. Thank you for your effort, Austin!

github-actions[bot] commented 11 months ago

This issue is stale because it has had no activity for the last 30 days.

wise-king-sullyman commented 11 months ago

@silenceinspace do you still want to replace that Medium article with the new resources you'd found here?

silenceinspace commented 11 months ago

@wise-king-sullyman hi Austin, glad to hear from you here again. Yes, absolutely! Can you make it happen? :D

wise-king-sullyman commented 11 months ago

@silenceinspace What is it you're needing on our end? Do you have a PR up already?

silenceinspace commented 11 months ago

@wise-king-sullyman well, I do have a PR #26495 which initially suggested to remove one already existing resource (medium article) in that lesson. So I think I could update the PR with removing the article and including the two resources I mentioned in this issue's description. Should I go for it?

wise-king-sullyman commented 11 months ago

Yep you can update that PR, or just open a new one, up to you!

silenceinspace commented 11 months ago

@wise-king-sullyman I was going to update my PR to include the alternative resources, but I noticed I was not the only one who had a problem with the medium article. Somehow, #26703 PR fixed what didn't sit right with me. Even though my resources were not included, I am totally cool with the new one. Anyways, thank you Austin for the willingness to work on my PR and issue. I will close the issue!:)

wise-king-sullyman commented 11 months ago

If you'd still like to add your resources please feel free to put a new PR up! Of course, if you no longer want to I totally understand, but IMO there's room for one more resource there and there could be a debate over which visualization site is better.