14 get clean data for specified period - Githubissues

42association / 42ActivityAPI

1 stars 1 forks source link

14 get clean data for specified period #18

Closed Sawawa42 closed 5 months ago

Sawawa42 commented 5 months ago

This PR is to resolve https://github.com/42association/server_with_m5/issues/14 .

kotto5 commented 5 months ago

db 関係のmethod は gorm 使うので要らなくなりそう? ただ、sql の勉強にはなるのでtry 自体はいいと思います!

Sawawa42 commented 5 months ago

db 関係のmethod は gorm 使うので要らなくなりそう? ただ、sql の勉強にはなるのでtry 自体はいいと思います!

Gormを使うとしても流れは一緒なのでコードを流用するつもりです

Sawawa42 commented 5 months ago

944d8b792f0ca1468343b04a4bcb562982adeeb6

変更点

Preload()を用いて先に関連テーブルを読み込むことでデータを取得できるように修正
Find()を用いて構造体の配列に直接スライスを入れ、rowsでのループを廃止した
Joinsでテーブルを結合して読み込む前に時間で絞り込みを行うように順序を変更し、負荷を減らした。

備考

すべてをプリロードするPreload(clause.Associations)などでもう少しコードを縮められるが、Gorm v2でしかできない模様