refactor(textlint): add no-js-function-paren #143
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
`fn()`の表記を原則禁止にして、`fn`関数と書くように変更
lacolaco
reviewed
Sep 28, 2016
| @@ -67,10 +67,14 @@ function displayView(view) { | |||
| コールバック関数を使うと、ソースコードのネストが深くなったり、例外処理が複雑になったりします。 | |||
| Promiseを用いることで、可読性を保ちながらエラーハンドリングを簡単に行えます。 | |||
|
|
|||
| <!-- textlint-disable no-js-function-paren --> | |||
|
|
|||
| コールバック関数を使う形式のAPIをPromiseに置き換えるのは、次のコードのように`new Promise()`を用いるのが一般的です。 | |||
There was a problem hiding this comment.
書き直すとしたら "Promiseのコンストラクタを呼び出します"あたりですかね
There was a problem hiding this comment.
Promise() とも読めるから、Promiseをインスタンス化する or new するみたいな感じかなー。
コンストラクタって用語が扱いにくい…
new FooFooをインスタンス化する(Fooクラスとはいえないのが厄介…)Foo()Fooコンストラクタ関数を呼び出す
かなー
There was a problem hiding this comment.
ここでは、Promiseをインスタンス化したいってより、new Promise()というパターンで書くのが一般的ですみたいな話だからそのままでもいいかなと思ったりしました。
(別の書き方として Promise.resolve().then(() => ) で書くパターンもあるけど、あんまりみないかな)
azu
commented
Sep 28, 2016
| @@ -45,6 +45,6 @@ Console APIは原始的なプリントデバッグとして利用できるので | |||
| 「この値は何だろう」と思ったらコンソールに表示してみることで解決する問題は多いです。 | |||
|
|
|||
| また、JavaScriptの開発環境は高機能化が進んでいるため、デバッガーなどが用意されていることが多く、 | |||
| コードに`console.log()`と書かなくても値を見ることもできるようになっています。 | |||
| コードに`console.log`メソッドを書かなくても値を見ることもできるようになっています。 | |||
|
とりあえずマージします |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
fn()の表記を原則禁止にして、fn関数と書くように変更close #141
close #110
/cc @laco0416 こんな感じ