fix(dict2): fixで"[助詞]"を省略するのは「が」のみに限定する #17
Merged
Conversation
This file contains hidden or 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
azu
commented
Jan 4, 2019
src/dictionary.js
Outdated
| return "は"; | ||
| return ""; | ||
| } | ||
| return ""; |
There was a problem hiding this comment.
これ結果的にすべて "" を返すことになるのですが、どうするのが意図した感じですかね?
他に来るパターンの助詞があるのかな。
やは一応これなのかな。(このtokenだとマッチしないけど)
必要なら解析することや聞くことができます
actualToken.surface_formを返すのが意図した感じですかね?
There was a problem hiding this comment.
当初の意図
これは，単純にその助詞を削除して，元々意図していた意味を変えてしまうのを防ぐのを意図しました．
a: 期間は延長することができます
b: 期間は短縮できます．延長することもできます
c 期間は短縮できません．延長することはできます
a,b,cの助詞を単純に削除すると
期間は延長できます
となってしまいますが，そうすると元々の意図していた意味が変わってしまいます．
そこで，「も」「は」の場合は存置して
b': 延長もできます
c': 延長はできます
となるようにしたわけです．
他の副助詞も同様ですね．
延長することくらいできます
（なお，「や」は「できます」の 前 には来ないと思います）
問題
しかし，前文脈によっては不自然なケースがありますね．
オブジェクトを作成することもできます -> オブジェクトを作成もできます
は少々変で，「オブジェクトの作成もできます」にしたほうが自然ですね．
「が」の場合は自動fixできるようにして，それ以外の助詞の場合は自動fixしない，というのがベターではないかと思いました．
There was a problem hiding this comment.
_capture_to_messageでメッセージには助詞を含める- 自動修正は 助詞が
がに限定する
という感じにしてみます。
azu
commented
Jan 5, 2019
| return ""; | ||
| } | ||
| return ""; | ||
| return ExpectedType.STOP_REPLACE; |
azu
changed the title
azu
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
とメッセージが出ているのに対して
--fixでは [助詞] が修正されてないのを修正してみました。asciidwango/js-primer#614 で--fixしてみた結果-> #17 (comment) 誤修正を防ぐために、「が」のみを自動修正するように変更した。