電気料金シュミレーション機能の追加 #313
電気料金シュミレーション機能の追加 #313
Conversation
|
Sider has detected 1 error and 2 warnings on analyzing the commit 79a1285.
We recommend fixing them as possible by updating the dependencies, configuring the analysis tool, configuring If you have problems or questions still, feel free to ask us via chat. 💬 You can turn off such notifications if unnecessary. |
challengeA/katayama/plan.rb
Outdated
| def calc_as_pay_charge(input_amount) | ||
|
|
||
| # 入力された電気使用量に対応する従量料金の単価を算出する | ||
| target_range = self.as_pay.find {|a| (a["to"] == AS_PAY_CHARGE_RANGE_MAX ) ? true : (a["from"] <= input_amount && input_amount <= a["to"])} |
There was a problem hiding this comment.
従量料金の計算方法も、改めて正しいかご確認いただけますでしょうか？
challengeA/katayama/plan.rb
Outdated
| def calc_base_charge(input_amp) | ||
|
|
||
| # 入力された契約アンペア数に対応する基本料金を算出する | ||
| target_amp = self.base.find {|b| b["amp"] == input_amp} |
There was a problem hiding this comment.
基本料金の導出について、改めて正しいか確認いただきたいです・・・！
|
@ityusuke |
@yuyasat また、基本料金の算出について質問なのですが、対応外のアンペアが入力された場合どのように算出するべきでしょうか？ お忙しい中申し訳ありませんが対応いただけると幸いです |
|
@ityusuke
こちら、私の方から言ってしまえば実装の難易度は高くないものとなります。 のところで、target_rangeを一つ取得してますが、250kWhのとき、計算で用いる従量料金単価は何になりますか？
とてもいい指摘ですが、 @ityusuke さんが考えるあるべき形はどうでしょうか？enechange.jpやお渡ししたchallenge Aの資料、資料から貼られているリンク（約款等）、いろいろ総合して調べてみていただき、@ityusuke さんの仮定で構いませんので実装してみてください！ |
・電気を全く使用しない場合電気料金を半額にするよう修正 ・対応外のアンペアが入力された場合のロジックを追加 ・従量料金の計算方法を見直し(従量料金単価をそれぞれの単価において個別で計算したものを合算するように修正) ・月額使用料が最低料金を割る場合のロジックを追加
こちら失念しておりました。それぞれの単価毎に計算したものを合算するべきでした。
アドバイスありがとうございます！最新資材では従量料金の計算ロジック修正に加え下記も修正を行っておりますので、お手すきの際にご確認いただけると幸いです。
|
challengeA/katayama/simulator.rb
Outdated
| as_pay_charge = plan.calc_as_pay_charge(self.kwh) | ||
|
|
||
| # 電気を全く使用していない場合、基本料金が半額になる | ||
| if as_pay_charge == 0 |
There was a problem hiding this comment.
使用料金が0なのか使用量が0なのかでいうと...!
There was a problem hiding this comment.
こちら、後者であるべきですね。失礼しました。
最新コミット(79a1285)で修正しました。
challengeA/katayama/simulator.rb
Outdated
| end | ||
|
|
||
| # 電気料金 = 基本料金 + 従量料金(少数切り捨て) | ||
| total_charge = (base_charge + as_pay_charge).floor |
There was a problem hiding this comment.
total_chargeに対して割引かけたり、下の方にある最低月額料金の適用だったりはプランによるので、total_chargeの算出もプランが持っていい責務かなと思いました。
There was a problem hiding this comment.
ありがとうございます！
こちら最新コミット(79a1285)で修正いたしました。
・電気料金計算ロジックをPlanクラスに移行
No description provided.