curriculum icon indicating copy to clipboard operation
curriculum copied to clipboard

Recursion: Merge sort solution in linked Wikipedia

Open thomasmcinnis opened this issue 1 year ago • 1 comments

Checks

Describe your suggestion

Problem

There is a version of merge sort in pseudocode on the linked Wikipedia page that makes solving the assignment trivial.

Context

There was recently a closed issue #27224 about the C-like version of merge sort on the linked Wikipedia page. But this issue and the change appears to be unaware there is also a highly verbose pseudocode example of merge sort further down the page which can be almost directly translate to JS.

In my own case reading through the wikipedia article and this pseudocode made writing a JS version trivial. I did not have to wrestle with the problem and figure out how to tackle it.

I am not sure if that matches the learning intent of the assignment, or perhaps this was added to the wikipedia page subsequent to this course module, or perhaps it is not the goal to have learners dig in to solve this assignment.

Action

There are two courses of action if this is worth addressing:

  1. Remove the link to wikipedia if the pseudocode version is judged to undermine the learning goal; or
  2. Refer learners to ignore the C versions in favour of the pseudocode if it is considered supporting the learning goal.

Path

Node / JS

Lesson Url

https://www.theodinproject.com/lessons/javascript-recursion

(Optional) Discord Name

latch909

(Optional) Additional Comments

No response

thomasmcinnis avatar Feb 18 '24 09:02 thomasmcinnis

Yeah I didn't notice that pseudo code last time I looked at this.

At this point I think we can probably just pull the wikipedia link because it's been kind of troublesome. People find the C/C++ code confusing, and then if we tell people to not read that, they may read the friendlier to translate pseudocode.

There's no shortage of explanations on that page (I really like the CS50/David Malan one), so I think it's probably fine to just take down the link.

I'll assign this to you. Make sure you fix it in the JavaScript course AND in the Ruby course.

JoshDevHub avatar Feb 19 '24 01:02 JoshDevHub

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

github-actions[bot] avatar Mar 20 '24 01:03 github-actions[bot]