Skip to content

存在しない関数が記載されている? #361

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

Merged
merged 1 commit into from
Nov 13, 2024

Conversation

pan-no-mimi
Copy link
Contributor

javascript.info を通じて JavaScript の勉強をし始めた初心者なので、間違っていたら reject してください。

ドキュメント中に記載されている cancelInterval という関数ですが、
以下の観点から clearInterval の間違いではないかと推測し、修正プルリクを発行しました。

  • 同ドキュメントの他の部分では clearInterval と書かれている
  • リファレンスを見ても cancelInterval という関数は存在しなさそう

ただ、cancelInterval と書かれたサンプルコードを実行してみましたが、一応問題なく動くようです。

@javascript-translate-bot javascript-translate-bot requested a review from a team October 23, 2024 12:41
@KenjiI KenjiI merged commit 44bb200 into javascript-tutorial:master Nov 13, 2024
1 check passed
@KenjiI
Copy link
Collaborator

KenjiI commented Nov 13, 2024

ありがとうございます。おっしゃるとおり、cancelInterval は存在しません。
本家の最新では本タスクは削除されているようですが、過去の履歴を確認すると clearInterval で正しいです。
https://github.com/javascript-tutorial/en.javascript.info/blob/9c3ac133e3a6f777395dd8e564c2cbac48b827c2/1-js/06-advanced-functions/08-settimeout-setinterval/3-rewrite-settimeout/solution.md

@pan-no-mimi pan-no-mimi deleted the fix_unexist_function branch November 13, 2024 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants