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

refactor: console.log(式); //=> 評価結果 に表記を統一 #212

Merged
merged 7 commits into from
Mar 28, 2017

Conversation

azu
Copy link
Collaborator

@azu azu commented Mar 23, 2017

追記: 2017-03-28

コメントのところに評価コメントの書式についてをちゃんと説明を書く
    comment: => についてを説明する · Issue #215 · asciidwango/js-primer
不要なところについてはconsole.logは省いても良い
    場合によって使いわけ
    明確な使い分けは見えず

    https://github.com/asciidwango/js-primer/blob/master/meetings/2017-03-24/README.md#%E7%B5%90%E8%AB%96-1

となった。
のでconsole.logは一部使い分け。
基本的に利用する。


console.log(a); // => x
a; // => x

が混在してるので、揃えてみようという試み。

実際にやってみると、演算子やリテラルそのものの評価結果みたいのは

console.log(1 + 1); // => 2

より

1 + 1; // => 2

の方が分かりやすい気がする。

ざっくり統一できてればいいので、演算子の章とかはファイルごと無視してみてる。

ちゃんとやるなら、ASTをみて //=>してる評価対象に変数があるならconsole.logを必須にするとかをやると上手く行くのかも。

fix #195


```js
.123; // => 0.123
console.log(.123); // => 0.123
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

この変も微妙なのかな?

@azu
Copy link
Collaborator Author

azu commented Mar 23, 2017

試しに operator も console.log で統一してみた d99f3cc

@azu azu force-pushed the fix-console-comment branch from d99f3cc to e222f1d Compare March 24, 2017 01:48
@@ -68,7 +68,7 @@ multiple(10); // => 20
function noop() {
}

noop(); // => undefined;
console.log(noop()); // => undefined;
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

これも不要っぽい

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

と思ったけど値として返ってくるという話なのであったほうがいいか

@azu azu merged commit 1e90b91 into master Mar 28, 2017
@azu azu deleted the fix-console-comment branch March 28, 2017 12:52
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.

console.logと// => の使い分け
1 participant