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

继续重构。 

我们先来写User类的单元测试,定义3个测试方法: 
Java代码   收藏代码
  1. require File.dirname(__FILE__) + '/../test_helper'  
  2.   
  3. class UserTest < Test::Unit::TestCase  
  4.   fixtures :users  
  5.   
  6.   def test_full_name_without_middle_initial  
  7.     user = User.new(:first_name => 'John', :last_name => 'Doe')  
  8.     assert_equal 'John Doe', user.full_name  
  9.   end  
  10.   
  11.   def test_full_name_with_middle_initial  
  12.     user = User.new(:first_name => 'John', :middle_initial => 'H', :last_name => 'Doe')  
  13.     assert_equal 'John H. Doe', user.full_name  
  14.   end  
  15.   
  16.   def test_full_name_with_blank_middle_initial  
  17.     user = User.new(:first_name => 'John', :middle_initial => '', :last_name => 'Doe')  
  18.     assert_equal 'John Doe', user.full_name  
  19.   end  
  20. end  

我们再看看我们的full_name方法,显然是无法满足所有的测试的: 
Java代码   收藏代码
  1. class User < ActiveRecord::Base  
  2.   def full_name  
  3.     name = first_name  
  4.     name += "#{middle_initial}." unless middle_initial.nil?  
  5.     name += last_name  
  6.     name  
  7.   end  
  8. end  

而且middle_initial后面跟一个"."也让我们的代码"风水"不好,比如我们突然有一天把"."改成"-"呢?好,我们继续重构吧: 
Java代码   收藏代码
  1. class User < ActiveRecord::Base  
  2.   def full_name  
  3.     [first_name, middle_initial_with_period, last_name].compact.join(' ')  
  4.   end  
  5.   
  6.   def middle_initial_with_period  
  7.     "#{middle_initial}." unless middle_initial.blank?  
  8.   end  
  9. end  

运行测试,ok,通过了!

评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值