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

サンプルコードの変数名をより具体的にすべきではないか #58

Open
dynamis opened this issue Oct 1, 2019 · 1 comment
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers
Milestone

Comments

@dynamis
Copy link
Contributor

dynamis commented Oct 1, 2019

変数名は GPIO の何番ポートでも I2C の何番ポートでも全て port 変数になっていたりする。JavaScript 経験者としては必要に応じて変数名を書き換えるのは当たり前で平気だが、JavaScript 初学者がコピペして組み合わせるときには分かりにくい (塩尻での WIMC にて gpio ポートを i2c ポートで上書きする人が続出した)。

なるべく初学者が困ることが少なくなるようにするためだけでなく、実際ちゃんとしたエンジニアが書くコードだとしても変数名からその中身が自明なコードにすべきものであり ledPort や switchPort など、複数同時利用時以外も用途の分かるコードを最初から利用するようコードを書き換えていくべきだと思った。

@dynamis dynamis added enhancement New feature or request good first issue Good for newcomers labels Oct 1, 2019
@dynamis
Copy link
Contributor Author

dynamis commented Oct 1, 2019

そもそも const 使ってればエラーが表示されるよって話がご尤もという感じで、変数名を変えると同時に const にした方が良さそう。

const と var と let との違いを教えることになるが、原因不明のエラーに長い間悩ませるよりマシな印象。1 人だけならともなく、20 数名の全員ではなく一部の人が組み合わせコードにトライしている中で 2-3 名が同じミスをしているところを見ると、2 割くらいの確率で変数名上書きしちゃうということなので。。。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants