利用好元编程重构下代码

xeruzo #1 2021年04月06日

算 1 个月的那坨代码抽出来
算季度半年一年的时传月数 3612 进去就好了
可以直接在里面写个 lambda 偷个懒
当然强烈赞同楼下补充的，多写注释/写测试也 ok

lyb124553153 #2 2021年04月06日

重构代码第一步添加注释

Awlter1 #3 2021年04月06日

对

lyb124553153 回复

or 测试

shiweifu #4 2021年04月07日

先写单元测试再重构比较好，万一重构后的代码拉下点什么功能就……

Rei #5 2021年04月07日

我愿意维护原先的代码。

14 个赞

ThxFly #6 2021年04月07日

建议大家不要模仿。普通业务中是真的用不到元编程。楼主这样改，真就从一坨还可以维护的代码往米田共方向奔跑。

spike76 #7 2021年04月08日

我刚学会元编程的时候也喜欢这样写骚代码，时间久了反而更偏爱朴实稳健的写法

2 个赞

xeruzo #8 2021年04月08日

对

Rei 回复

不能再同意了
lz 应该处于为赋新词强说愁的阶段？毕竟刚学完元编程，想秀一下
我之前刚学会新知识也这样，后面想了下还是少写点这些，是在要写都多写及行注释

crosspass #9 2021年04月08日

重构的姿势不对，先可以想想用常规的重构手法。你这个明显可以先可以将计算的那部分做一个方法，返回一个 hash 结果。

ywjno #10 2021年04月08日

采用元编程的话找该方法的实现只能用 [grep 关键字] 找。。。

mz2test #11 2021年04月08日

业务一旦修改，这一小块还会变成一坨

PaulChan1995 #12 2021年04月08日

所谓的重构前其实也比较简单明了吧，重构不只是代码写少点呀，我刚刚看完元编程那本也老想这样秀，但是后面发现没必要，秀着秀着发现别人看不懂的时候，是你最痛苦的时候，不能偷懒呀。

xinyifly #13 2021年04月08日

考虑提炼值对象，利用多态重构一下？

xinyifly #14 2021年04月08日

拿走不谢

spec/vip_money_spec.rb

require 'rspec/mocks'
require 'vip_money'

RSpec.describe 'vip_money' do
  subject { vip_money }

  let(:sale) do
    {
      month: {
        money: 1600, original_money: 2000, unit: 2000, discount: 0.8,
        saving: 0.2, saving_money: 400
      },
      quarter: {
        money: 4000, original_money: 6000, unit: 1666.67, discount: 0.67,
        saving: 0.33, saving_money: 2000
      },
      semi_annual: {
        money: 6400, original_money: 12_000, unit: 1333.34, discount: 0.54,
        saving: 0.46, saving_money: 5600
      },
      year: {
        money: 11_200, original_money: 24_000, unit: 1166.67, discount: 0.47,
        saving: 0.53, saving_money: 12_800
      }
    }
  end

  before do
    stub_const('Order::TYPE_MONEY', [2000, 5000, 8000, 14_000])
    allow(self).to receive(:test_money_discount).and_return(0.8)
  end

  it { is_expected.to eq sale }
end

lib/vip_money.rb

require 'active_support/all'
require 'order'
require 'sale_detail'
require 'test_money_discount'

def vip_money
  month_money, quarter_money, semi_annual_money, year_money = Order::TYPE_MONEY
  user_discount = test_money_discount

  sale_detail = SaleDetail.new(month_money, user_discount)

  %i[month quarter semi_annual year].index_with do |period|
    sale_detail.send("by_#{period}!", binding.local_variable_get("#{period}_money"))
    sale_detail.attributes
  end
end

lib/order.rb

class Order
  TYPE_MONEY = [2000, 5000, 8000, 14_000].freeze
end

lib/sale_detail.rb

require 'bigdecimal'
require 'active_support/all'

class SaleDetail
  attr_reader :monthly, :user_discount, :price, :months

  def initialize(monthly, user_discount)
    @monthly = BigDecimal(monthly.to_s)
    @user_discount = BigDecimal(user_discount.to_s)
  end

  def attributes
    {
      money: total.ceil(2),
      original_money: original.ceil(2),
      unit: unit.ceil(2),
      discount: discount.ceil(2),
      saving: saving.floor(2),
      saving_money: saved.floor(2)
    }
  end

  def by_month!(price, months = 1)
    @months = months
    @price = BigDecimal(price.to_s)
  end

  def by_quarter!(price)
    by_month!(price, 3)
  end

  def by_semi_annual!(price)
    by_month!(price, 6)
  end

  def by_year!(price)
    by_month!(price, 12)
  end

  def total
    price * user_discount
  end

  def original
    monthly * months
  end

  def unit
    price / months
  end

  def discount
    total / original
  end

  def saving
    1 - discount
  end

  def saved
    original - total
  end
end