`
hideto
  • 浏览: 2687425 次
  • 性别: Icon_minigender_1
  • 来自: 北京
社区版块
存档分类
最新评论

Rails宝典之第十一式: 重构用户名-p2

    博客分类:
  • Ruby
阅读更多
继续重构。

我们先来写User类的单元测试,定义3个测试方法:
require File.dirname(__FILE__) + '/../test_helper'

class UserTest < Test::Unit::TestCase
  fixtures :users

  def test_full_name_without_middle_initial
    user = User.new(:first_name => 'John', :last_name => 'Doe')
    assert_equal 'John Doe', user.full_name
  end

  def test_full_name_with_middle_initial
    user = User.new(:first_name => 'John', :middle_initial => 'H', :last_name => 'Doe')
    assert_equal 'John H. Doe', user.full_name
  end

  def test_full_name_with_blank_middle_initial
    user = User.new(:first_name => 'John', :middle_initial => '', :last_name => 'Doe')
    assert_equal 'John Doe', user.full_name
  end
end

我们再看看我们的full_name方法,显然是无法满足所有的测试的:
class User < ActiveRecord::Base
  def full_name
    name = first_name
    name += "#{middle_initial}." unless middle_initial.nil?
    name += last_name
    name
  end
end

而且middle_initial后面跟一个"."也让我们的代码"风水"不好,比如我们突然有一天把"."改成"-"呢?好,我们继续重构吧:
class User < ActiveRecord::Base
  def full_name
    [first_name, middle_initial_with_period, last_name].compact.join(' ')
  end

  def middle_initial_with_period
    "#{middle_initial}." unless middle_initial.blank?
  end
end

运行测试,ok,通过了!
分享到:
评论

相关推荐

Global site tag (gtag.js) - Google Analytics