Skip to content

no use var #59

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 5 commits into from
Sep 14, 2015
Merged

no use var #59

merged 5 commits into from
Sep 14, 2015

Conversation

azu
Copy link
Owner

@azu azu commented Sep 14, 2015

close #45

letに統一

constに統一するのも意味的に意味があるけど、constがfreezeしてくれるわけではないというのはまだ周知の事実ではない気がする。
なので、定数的なモノに対して使うには誤解がなくていいけど、全てに付けるのはちょっと自分ルールという感じがまだある。

@azu
Copy link
Owner Author

azu commented Sep 14, 2015

Scalaのvarとvalみたいな文字数の違いがないかつそういう文化が形成されているといいのだけど、
デフォルトconstで、代入あるものはletというのだと好み的な問題に落ちてしまう。
この書籍を読む人がそのスタイルを読みやすいと感じるかがこのPRを入れるかの焦点な気がする。

あるスタイルを強要するのは読みやすい書籍ではないので、constが強要にみえるならconstは使わないほうがいいと思う。

@azu
Copy link
Owner Author

azu commented Sep 14, 2015

varじゃなくてletにしたかったのは、何か混在して気持ち悪かったからという理由。

azu added a commit that referenced this pull request Sep 14, 2015
@azu azu merged commit 079b15d into master Sep 14, 2015
@azu azu deleted the fix_45 branch September 14, 2015 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

no use var
1 participant