xpjp / xpfiat-bot

XP JPのディスコード上で動作しているボットです。 XPをJPYに変換したりする機能があります。
MIT License
20 stars 5 forks source link

時価総額を返すbotを追加 #98

Closed mario2331 closed 6 years ago

mario2331 commented 6 years ago

何を解決するのか

?mcap ?時価総額 ?戦闘力 とコマンドを入力するとXPの現在の時価総額の日本円換算した金額を知ることができる。 issue https://github.com/xpjp/xpfiat-bot/issues/49

レビューポイント

・coinmarketcapのapiから時価総額の日本円換算を取得しread_urlメソッドに追記しました。 ・時価総額の日本円換算で桁数が大きくなり分かりづらかったため、数値を日本語表記する gemを導入しました。 https://github.com/xpjp/xpfiat-bot/issues/96 https://github.com/siman-man/yen

・時価総額をXPちゃんが「私の戦闘力は○○億○○万○○円です」と表示するように仮設定しました。 問題があれば、「XPの時価総額は---」に変更します。

@xpjp/ruby-reviewer ご確認をお願い致します。

mc-chinju commented 6 years ago

便利な機能ですね! 👍 yen ですが、Star の数が少ないのとメンテナンス全くされていなそうな gem なので、導入せずに直書きしたいなーという気持ちが強いです。

mario2331 commented 6 years ago

@mc-chinju san http://simanman.hatenablog.com/entry/2013/03/15/224727 ご確認ありがとうございます。ここでgemの作者さんが直書きして下さっているので、直にメソッドを書いてみます。

mc-chinju commented 6 years ago

@mario2331 さん リンク先みてみましたが、あまり綺麗なコードじゃないので、今回実装する上で最低限のメソッドを定義して変換してあげるだけがいいかと思いますー。引数に日本円そのまま渡してあげるイメージです。

(このあたりは多少意見分かれる気もしますし、絶対ではないです)

mario2331 commented 6 years ago

@mc-chinju san 承知しました。シンプルに4桁区切りで日本語表記に変換するメソッドに致します。

mario2331 commented 6 years ago

@mc-chinju san

https://teratail.com/questions/51557 こちらの記事を参考に、to_sした時価総額に、日本語の万・億・兆の単位を4桁区切りでinsertするメソッドを追加しました。ご確認をお願い致します。

mario2331 commented 6 years ago

@mc-chinju san 一度to_iでintegerに変換後、ケタ数チェックを追加しました。 ご確認をお願い致します。

mario2331 commented 6 years ago

@mc-chinju san @Asuforce san NumberToYenをmarcket_cap.rbでextendするように修正しました。ご確認をお願い致します。

mario2331 commented 6 years ago

@Asuforce san ご確認ありがとうございました!

mario2331 commented 6 years ago

@p-suke san お疲れ様です。ready_to_mergeのラベルを貼っていただきましたため、 よきタイミングでマージをお願い致します。