Re: Template::Plugin::Tax

Template::Plugin::Tax - Unknown::Programming について,ネタにマジレスかもしれませんが

  1. int 関数ではなく POSIX::floor, POSIX::ceil を使うべき
  2. 税抜→税込価格(to_inc_tax)は(少なくとも日本では)四捨五入ではなくて切捨(逆に本体価格にするには切上)
  3. ってそもそも算出ロジックやデフォルト税率のことを考えると,Template::Plugin::Tax じゃなくて Template::Plugin::JapaneseConsumptionTax のほうがネーミングが妥当(長すぎだけど)
    • と思ったけど,Template::Plugin::Tax::Japan とかにして [% USE tax('japan') %] とかするのもいいかも
    • もっというと,税金計算部分は Business::Tax::* とかに外在化してそれを指定するようにすればより汎用的なのかも。ロジック部分でもそいつを使えばいいし。
      • そうすればビュー側に実装する意味がでてくるかな
    • とかいって重くしてくのが本意ではないんですよね,おそらく
  4. フィルタ名がいまいち伝わりにくい
    • でも何がいいかな。include_tax / exclude_tax とか,with_tax / wo_tax とか。
  5. 消費税 50% の時代が到来した場合にテンプレートの全ての to_inc_tax / to_enc_tax の引数を変えなきゃいけないのは大変
    • せめて new() 側でデフォルト税率を指定できるようにするべき
    • (もちろん $Template::Plugin::DEFAULT_TAX を書き換えれば対処はできますが)
  6. そもそも,消費税計算のようにもろロジック側のものをなぜビューである TT のプラグインとして作るのかわからない
    • グローバルなモジュールにするより,アプリ側のモジュールとして実装して TT インスタンス作成時にフィルタ等登録するほうが妥当だと思う

22:45 追記

あらためて読みかえしてみてなんか冷たげ・偉そうに見えたので追記をば。箇条書きにしたからそういう印象になっちゃったのかな(コーダーはコードで語れ?)

書きたかったことは id:fbis さんの真意を知りたかっただけで FUD とかではありません。