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

そろそろリファクタリングしたほうが良くない? #25

Open
Raclett3 opened this issue Aug 10, 2020 · 7 comments
Open

Comments

@Raclett3
Copy link
Collaborator

でもそんなんじゃだめ もうそんなんじゃほら
コードは進化するよ もっともっと

@yuta0801
Copy link
Contributor

そもそも それ 本当に 必要 ミドルウェアパターン?それは 増やす 複雑性、違う それ?
少なくとも 私 感じる それは ほぼ ない 便利

@Raclett3
Copy link
Collaborator Author

Raclett3 commented Aug 10, 2020

Думаю, я смогу улучшить библиотеку Redis implementing a simple mapper, もしくはそんな感じのこと

@yuta0801
Copy link
Contributor

那需要用middleware pattern吗?不可以像现在也在使用的command函数是的方法吗?
関数で包んで行く方法のほうがシンプルじゃない?

@Raclett3
Copy link
Collaborator Author

いくつかのモジュールはcommand()がさせてくれることより複雑なことを必要としているのよ

@yuta0801
Copy link
Contributor

yuta0801 commented Aug 10, 2020

モジュールごとの処理結果によってnextを呼ぶかどうかの分岐しか見つけられなかったけど
これだとして特別多くの影響を与えてない割には複雑だし、全部個別に実行されたほうがシンプルではない?

@Raclett3
Copy link
Collaborator Author

なるほどね

@Raclett3
Copy link
Collaborator Author

やってます

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

No branches or pull requests

2 participants