Corrected the reasoning in the 'Step into' command explanation and made the Making bound methods with class fields section easier to understand.#3972
Conversation
Corrected a section of the writeup on 'Step into' command pertaining to beginners. Instead of "as we don't have asynchronous calls yet", I suggest the words to be "as we haven't learnt about asynchronous calls yet". The former suggests that Javascript lacks the ability to make asynchronous calls, which is not the case, in my opinion.
javascript-translate-bot
added
review needed
|
@codex review |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d3e1443ba7
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| As demonstrated in the chapter <info:bind> functions in JavaScript have a dynamic `this`. It depends on the context of the call. | ||
|
|
||
| So if an object method is passed around and called in another context, `this` won't be a reference to its object any more. | ||
| Recall the **"losing `this`"** problem from the chapter <info:bind>. Functions in JavaScript have a dynamic `this` — it depends on how the function is called, not where it's defined. So, when an object method is passed around (e.g. as a callback to `setTimeout`, or as an event handler), `this` no longer points to the original object. |
There was a problem hiding this comment.
this rule
This newly added explanation is false for arrow functions: their this is lexical and therefore does depend on where they are defined. That distinction is essential here because the solution immediately below uses click = () => ... precisely to capture the instance's this; saying that JavaScript functions universally use call-site this makes the explanation contradict its own example. Please limit this rule to regular functions or methods.
Useful? React with 👍 / 👎.
3 participants
This pull request make the following improvements in the literature:
3.1 Debugging in the browser --> “Step into”, hotkey F11writeup.9.1 Class basic syntax --> Making bound methods with class fieldswriteup has been made easier to understand.