[AlanGit-debug2604]

Learners, PR Template

Self checklist

• I have titled my PR with Region | Cohort | FirstName LastName | Sprint | Assignment Title
• My changes meet the requirements of the task
• I have tested my changes
• My changes follow the style guide

Task code

CYF-1066

Changelist

Remove redundant and unreachable codes in both exercise 1 and exercise 2.

Questions

Exercise 1: With regards to name as a parameter inside the function, the function sayHello was called and executed by greetingMessage, therefore the function as a whole not considered as redundant. But one of its parameter name was not given an argument to print, and the code rather assigned a value to testName, is 'name' somehow can be considered as dead-code? If so, as Unused code according to devopedia?

[cjyuan]

Dead code deleted.

Note: In this exercise, the goal is supposed to only delete the dead code. Refactoring code is unnecessary.

[cjyuan]

Regarding your question, I would suggest showing the code to AI and ask it to use the code to explain these concepts

1. What is variable scope?
2. How JS resolves conflicting names -- greeting.
3. How arguments are passed to parameters during a function call.

---

Note: If this is the case

function sayHello(greeting, name) {
  return `${greeting} John`;
}

then the second parameter, name, would be considered a dead code because it is not used in the function.