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

名詞「草」を「ハーブ」に変換するルールを追加いたしましたわ #19

Closed
wants to merge 2 commits into from
Closed

Conversation

yamachoo
Copy link

@yamachoo yamachoo commented Jun 19, 2022

素敵なおツールの公開、ありがとうございますわ!

おコードを拝見しましたところ、簡単に「草」を「ハーブ」に変換できそうだったので対応してみましたわ。
これで「草」をお上品な「おハーブ」に変換できるはずですわ。

私も二次創作おガイドラインは一通り読みましたわ。
おコントリビュートが同ガイドラインに違反しないとは思うのですが、間違いや趣旨の違いがあったらRejectしてくださいまし。

@codecov
Copy link

codecov bot commented Jun 19, 2022

Codecov Report

Merging #19 (cb71711) into main (36afbf0) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #19   +/-   ##
=======================================
  Coverage   95.54%   95.54%           
=======================================
  Files           3        3           
  Lines         202      202           
=======================================
  Hits          193      193           
  Misses          6        6           
  Partials        3        3           
Impacted Files Coverage Δ
rule.go 100.00% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 36afbf0...cb71711. Read the comment docs.

@jiro4989
Copy link
Owner

ありがとうございますわ~~!!
貴方の素晴らしいおコードを大変嬉しく思いますわ💯

ですけれど、少しだけおレビューにお付き合いくださいまし・・・。

ひとつめ

名詞の草のみをおハーブに変換いたしますと、意味が変わってしまう場合がございますの。

例えば「三界草」は「三界」と「草」の2つのお単語に区切られますの。
こちらで変換いたしますと「三界おハーブ」となってしまいますわ😢

その他にも末尾が草になる言葉はたくさんございまして、前後のお単語のお品詞を確認しなくてはなりませんの。

参考: 「〇〇草」といえば? 言葉の種類や熟語一覧

ふたつめ

わたくしが思うに、貴方の書いてくださった「草」はおネットスラングとしての「草」だと想像しておりますの。

ですが、お植物の総称としての「草」まで「おハーブ」に変換してしまうと、
意味合いが変わってしまう場合がございますの。

例えば「(庭先の)草を刈る」といった文を変換して
「おハーブを刈る」としてしまうと、意味合いが全然異なってしまいますわ!

つまり、おネットスラングとしての「草」のみを「おハーブ」、
あるいは「おハーブですわ~~!」に変換するような、
変換ルールでないといけないんですの。

これは言い換えますと「(笑)」や「www」といった笑いを表現するおネットスラングを
すべて「おハーブですわ~~!」に変換するような変換ルールと考えてくださいまし。

実はわたくしも最初「草」を「おハーブ」に変換するルールを定義しておりまして、
単純な変換は適当ではないと判断して削除したんですの。

というのも、笑いのおネットスラングとしての「草」のみを「おハーブ」に変換するのは少し難しいんですの。
先に上げた「草を刈る」のように文中に含まれる「草」を変換しないで、
単独で出現した「草」や、「草のみの文」だけを変換する必要があるんですの。

わたくしが思うに、以下のような文が「おハーブですわ~~!」に変換するに適当な文ですわ。

草(前後に何も単語が存在しない)
A「〇〇でしょ」B「草」

ですが、他にも例外的に変換するべきでない文も存在しまして、
「土、草、水、火」
のように列挙する文中に「草」が単独で書かれている場合も変換してはなりませんの。

これらの難しい悩みをすぐには解決できそうになくて、
「草」を「おハーブ」に変換するルールを一旦は見送ってしまいましたの・・・。

おまとめ

以上の2つのお悩みポイントがございますの。
これらのお悩みポイントをもし解決できるのでしたら、貴方のPRをおマージいたしますわ!!

ですが、もし解決が難しそうでしたら、おっしゃってくださいまし。
わたくしも変換を諦めたわけではありませんので、わたくしの方でなんとか実装してみせますわ~~!!!

@yamachoo
Copy link
Author

yamachoo commented Jun 19, 2022

なるほど…。私、浅慮でございましたわ。また詳しい解説をご丁寧にしていただいて、ありがとうございますわ。
ただ、すぐに解決策が私の方で出せそうにはありませんので、一旦こちらのPRはCloseさせていただきますわ。

またもし何か良いアイデアが思いつけば、今度はissueで一度ご提案をさせていただきますわ〜!
今後も開発応援していますわ~~!!

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.

None yet

2 participants