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

DBクラスをもう少し良い感じに共通化する #132

Open
massongit opened this issue Jun 6, 2020 · 4 comments
Open

DBクラスをもう少し良い感じに共通化する #132

massongit opened this issue Jun 6, 2020 · 4 comments

Comments

@massongit
Copy link
Member

https://github.com/nakkaa/hato-bot/blob/develop/library/database.py はpylintに怒られない程度に共通化しているだけですが、SQLを実行する部分をexecute_sqlに寄せるなど、もう少し良い感じに共通化します。

@massongit massongit moved this to In Progress in task list Sep 24, 2022
@massongit massongit removed the status in task list Sep 24, 2022
@fono09
Copy link
Collaborator

fono09 commented Feb 25, 2024

ORM使っちゃう説はある(トリッキーなことをしない場合は特に)

SQLAlchemy: ド定番、日本語での知見が多い
peewee: 我々がやりたいことを考えるとこちらのほうが最短で手が届きそう

SQLAlchemy: https://www.sqlalchemy.org/
peewee: https://docs.peewee-orm.com/en/latest/

@massongit
Copy link
Member Author

massongit commented Feb 25, 2024

良さそう。
peeweeの方がライトに書けるって感じなのかな?

@fono09
Copy link
Collaborator

fono09 commented Feb 25, 2024

peeweeのほうがシンプルに書けそうだと思ってる。

@massongit
Copy link
Member Author

なるほどね。
DB書き込みの処理そこまであるわけじゃないしよさそう。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

No branches or pull requests

2 participants