Skip to content
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

変数名に使える名前のルール: varではなくletに変更する #1388

Closed
azu opened this issue Feb 7, 2022 Discussed in #1387 · 2 comments · Fixed by #1404
Closed

変数名に使える名前のルール: varではなくletに変更する #1388

azu opened this issue Feb 7, 2022 Discussed in #1387 · 2 comments · Fixed by #1404
Labels
good first issue Status: PR Welcome Pull Requestを歓迎する状態

Comments

@azu
Copy link
Collaborator

azu commented Feb 7, 2022

Discussed in #1387

Originally posted by UMA0626 February 7, 2022
以下のページの「変数名に使える名前のルール」セクションで、変数として利用できる文字列の例の部分で「var」が使用されています。
https://jsprimer.net/basic/variables/#variable-name

var $; // OK: $が利用できる
var _title; // OK: _が利用できる
var jquery; // OK: 小文字のアルファベットが利用できる
var TITLE; // OK: 大文字のアルファベットが利用できる
var es2015; // OK: 数字は先頭以外なら利用できる
var 日本語の変数名; // OK: 一部の漢字や日本語も利用できる
var 1st; // NG: 数字から始まっている
var 123; // NG: 数字のみで構成されている
var var; // NG: `var`は変数宣言のために予約されているので利用できない
var if; // NG: `if`はif文のために予約されているので利用できない

その上のセクションでvarはあまり使わないようにという話をしたばかりなので、上記についても「let」で記載する方が良いかと思いますが、いかがでしょうか?

@azu azu added Status: PR Welcome Pull Requestを歓迎する状態 good first issue labels Feb 7, 2022
@azu
Copy link
Collaborator Author

azu commented Feb 7, 2022

全部 letで問題ないのでletに変更する。

おそらく最初に書いたときにvarで書いた名残り。
#15
#46

やること

  • let を使うように変更
  • var varlet let に変更する

予約語とは、varのように構文として意味を持つキーワードのことです。 予約語の一覧は予約語 - JavaScript | MDNで確認できますが、基本的には構文として利用される名前が予約されています。

@azu
Copy link
Collaborator Author

azu commented Feb 7, 2022

var var; // NG: varは変数宣言のために予約されているので利用できない は 両方letかな

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Status: PR Welcome Pull Requestを歓迎する状態
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant