azu / JavaScript-Plugin-Architecture

JavaScriptプラグインアーキテクチャの本
https://azu.github.io/JavaScript-Plugin-Architecture/
MIT License
250 stars 12 forks source link

chore(ESLint): fix typo #42

Closed vzvu3k6k closed 9 years ago

vzvu3k6k commented 9 years ago

リンクされているissueの内容から考えると「ルールに渡して」となる気がします。

vzvu3k6k commented 9 years ago

この文章のポイントは、全てのルールの書き換えコマンドを一箇所に蓄積して最後に整理して適用するところだと思うので、SourceCodeオブジェクトをルールに渡すという説明は省略して、こんな感じにしてもいいかもしれないですね。

これはルールからの書き換えのコマンドをSourceCode抽象オブジェクトに集約して最後に実際の書き換えを行うという抽象レイヤーを設けています。

azu commented 9 years ago

これはルールからの書き換えのコマンドをSourceCodeというオブジェクトに集約して、最後に実際の書き換えを行うという抽象レイヤーを設けています。

という感じでも良いかもですね。(抽象オブジェクトと抽象レイヤーって二重に出てくると何か難しくなるかも)

これはルールからの書き換えのコマンドをSourceCodeというオブジェクトに集約して、最後に実際にASTを書き換えるという抽象レイヤーを設けています。

が実際の挙動に近いですが、ASTの書き換えという解説をしてない感じがするので。。

vzvu3k6k commented 9 years ago

開発中のautofixingの実装ではソースコードを文字列として直接書き換えてるようなので、azuさんの一番目の案がよさそうな感じがします。

azu commented 9 years ago

それが良さそうですね。

azu commented 9 years ago

@vzvu3k6k 微修正して取り込みました。ありがとうございます。