Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

オプションの振る舞いを HKD? で畳み込んで構築する #23

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

matsubara0507
Copy link
Member

@matsubara0507 matsubara0507 commented Jan 16, 2020

なんか色々試行錯誤してたら思いついたので。実験的に実装してみた。

かなり冗長になってますけど利点としては

  1. CLI オプションの振る舞いをオプションごとに定義できる
    • 普通はメインの処理に渡してどこかで使われている
  2. オプションの役割を型で表現できる
    • 今回は処理全体の分岐(CmdEnchantments
    • グローバル変数の構築(LogOptEnchantemnts
    • 前後処理の追加(ActionEnchantments)の3つ

割と大きな欠点として、処理全体を俯瞰しにくいところがあげられる気がする。
個別のオプションの振る舞いの順番が割と重要だったりするが、それがちょっとわかりにくい。

まぁ面白いアプローチな気がするのでこのままでもいいかなって思ってる笑

@matsubara0507 matsubara0507 mentioned this pull request Jan 16, 2020
5 tasks
@matsubara0507 matsubara0507 self-assigned this Jan 16, 2020
@matsubara0507 matsubara0507 marked this pull request as ready for review January 17, 2020 03:04
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant