binance APIから仮想通貨の種類一覧を取得し、DBにその情報を格納する処理を追加

[moromin]

変更点

Exchange

• 仮想通貨の種類一覧を取得するための処理は、以前と同様、exchange*にGetAllCurrency()的なメソッドを持たせて、mainからその処理を呼び出している。

DB

コード外の設定

• DB schemaを👇 のように設定した。これに合わせてOHLC, currency, symbolなどのentityも追加した（Intervalのentity完全に忘れてました）。

• 上記のDBを構成するために、Goのmigration toolを使用。migrationを実行するためのファイルをdb/migrationディレクトリに追加。

• Dockerコンテナ内にPostgreSQL環境を構築。そのための各処理（DBの初期化、テーブルの作成など）をMakefileに追加。

コード内の設計

• 各層の処理

  • infrastructure: sqlHandler を設定
  • interface
    • database: StoreCurrency(), FindAllCurrency()など、DB操作の具体的な部分を実装
    • presenter: ShowCurrency(), RegisterCurrency()など、usecase.Interacterのメソッドを用いて、ユーザーに操作結果などを返す。
  • usecase: interface/database で定義したDatabaseRepositoryのメソッドを呼び出し、操作を実行する。

• DIコンテナにおいて、APIとDBの処理を分割したかったため、NewAPIClient(), NewDB()というように各コンテナを返す関数にした。

• また、config パッケージ(.envファイル)からDBの各設定値を読み込めるようにした。

レビューで見てほしいポイント

• DBの設計やらDockerの構成やら、初めてのことが多かったんで、怪しいとこはバンバン突っ込んで欲しい笑
• テーブルの設計とかも話し合ってからOHLCデータの格納とかの実装したいから、今回はその処理にも必要になると思われる、仮想通貨の種類一覧を取得する処理までを追加した。
• 取得したデータをDBに格納した結果はこんな感じ👇

関連 Issue

• #

参考リンク

• テンプレートを拝借したリンク

[k-kohey]

詳細あとで見ます！ DB操作しているRepositoryのテストを書いてみて欲しいンゴ。 DBへの操作はランタイム時にクラッシュしやすいイメージがあるのでそれが起こってないのか検知したいのと、実際にいくつかInsertやselectする処理を書いてみて、使い勝手の良いDB構成になっているか確認したいです。

[moromin]

やってみるでぃ

[moromin]

現状のdatabase周りテスト一応できるけど、結構ガバいから多分描き直す

[moromin]

このパッケージ使うとDB周りの処理わかりやすそうだから、これ使って実装してみる https://github.com/kyleconroy/sqlc

[moromin]

DB周りの処理をするために、dabase*っていうファイルを今回のPRでいくつか追加したんだけど

このパッケージ使うとDB周りの処理わかりやすそうだから、これ使って実装してみる https://github.com/kyleconroy/sqlc

sqlc*っていうファイルに👆これ使った処理で書き直してみた。 テストもこっちで書こうと思いやす。

[moromin]

一旦、database*っていうファイルたちは無視してもらっても大丈夫です。 追加ファイルがクソ多いと思うんで笑

[moromin]

interfaces/databaseディレクトリ内のsqlc_test.goっていうファイルにテスト書いてみた。 Goのtesting パッケージ使って書いてるからVSCodeとかだと楽にテストできると思う。

[k-kohey]

差分1kのPR見るのキツイかもしれんｗｗ

[moromin]

テスト書く前後とかでPR分けた方が良かったねw 今回、database*.go、sqlc*.goっていう二つのDB実装パターン出してみたんだけど、sqlc*.goのパターンのほうが弄りやすいし、見やすいと思う。 次PR出すときは、sqlc*.goの方をDB処理の軸として出してみるね。