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

看这个显示用户Profile的页面:

<h1>Profile</h1>
<p>
  Name:
  <%= @user.first_name %>
  <%= "#{@user.middle_initial}." unless @user.middle_initial.nil? %>
  <%= @user.last_name %>
</p>

<% link_to 'Users List', users_path %>

当别的页面需要显示用户名时，我们又得将上述代码copy一遍，敏感的开发人员可能已经闻到“风水”或bad smell了吧。

好，我们来重构:

class User < ActiveRecord::Base
  def full_name
    name = first_name
    name += "#{middle_initial}." unless middle_initial.nil?
    name += last_name
    name
  end
end

页面就可以这样写了:

<h1>Profile</h1>
<p>
  Name:
  <%= @user.full_name %>
</p>

<% link_to 'Users List', users_path %>

评论

3 楼 volnet 2007-08-02  

呵呵，Copy非正道

2 楼 jonlove 2007-07-30  

哦！对了！自己失误了！

1 楼 jonlove 2007-07-30  

我在controller创建了

  @user = User.find("1")

但在

   <h1>Profile</h1>  
<p>  
  Name:   
  <%= @user.full_name %>（报了错！说的undefined method `full_name' ）  
</p>  
  
<% link_to 'Users List', users_path %>  

我就不能理解为什么@user能点出full_name方法！