xpjp / xpfiat-bot

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

Feature/add img generate #58

Closed s9rt closed 6 years ago

s9rt commented 6 years ago

何を解決するのか

XPちゃんを使ったノベルゲーム風の画像を生成する

問題点:書き出し、送信した後のファイルを削除するたびに謎の”1”が送信される

レビューポイント

問題点についての改善案などいただけると嬉しいです

toshikidoi commented 6 years ago

xpchan_crypto_ruby_1513669116

s9rt commented 6 years ago

@toshikidoi レビューありがとうございます!

細かいんですが、タイミングが被るかもしれないので、以下の方が良いかもです〜😄 このコマンドを呼んだDiscordユーザー名とUNIXタイムスタンプの組み合わせです。

確かにこっちならばかぶり無しになりそうですね。あとWindows環境だと:が入ったファイル名ということでエラーが吐かれていたのですが同時に解決しそうです。

この1は、File.deleteメソッドの戻り値です。 このメソッドは削除したファイル数を戻り値で返すようになっているので😅

こちらも解決案の通りnilを使うことで無事解決しました!戻り値だったんですね…

あとrmagickに関しては他の方の意見もお聞きしたいということで一旦置いておきます。

Diaboro87 commented 6 years ago

rmagickについては良くわかりません

chipstar commented 6 years ago

XPFiat-BOT動かしているサーバーでrmagick環境整えました。 gem install rmagickで入ったので大丈夫だと思います。

toshikidoi commented 6 years ago

@p-suke おお、ありがとうございます! さすが😄

@s9rt ということで、お待たせいたしました〜🙇 LGTMです🎉

s9rt commented 6 years ago

@Diaboro87 @mc-chinju レビューありがとうございます。遅くなりましたがご指摘いただいた点を修正しました。

@p-suke 環境設定ありがとうございます

@toshikidoi LGTMいただいたのですがまた少しコードを変えたためこんな感じで大丈夫かというところの評価をお願いします(手間かけさせてすみません…)。

toshikidoi commented 6 years ago

@s9rt 素晴らしい! 皆さんのご指摘に対応してくださったんですね〜😄

改めてLGTMです〜🎉