重构:对于复杂的if-else的手法

访问量: 765

基本是几个:

1. 在逻辑上简化

2. 使用guard statement ( 守护语句,  xxx if ooo )

3. 使用case 表意更明确。

4. 对于  if ... else ... end , 使用  三元表达式:     true ?  state1 : state2  

下面是个例子: (有三层 if-else )

    def show
      if current_user_can_view_page?
        if should_skip_to_first_child?
          redirect_to refinery.url_for(first_live_child.url)
        elsif page.link_url.present?
          redirect_to page.link_url
        else
          if requested_friendly_id != page.friendly_id
            redirect_to refinery.url_for(page.url), :status => 301 
          else
            render_with_templates?
          end 
        end 
      else
        error_404
      end 
    end 

第一步, 简化最外层的分支。 因为 else 中的内容只有一行。

if ..
else
   error_404    
end 
改成:

error_404 unles ...
if ... end 

#  经历了各种步骤之后,看起来应该是:

      error_404 and return unless current_user_can_view_page?
      redirect_to refinery.url_for(first_live_child.url) and return if should_skip_to_first_child?
      redirect_to page.link_url and return if page.link_url.present?
      requested_friendly_id != page.friendly_id ?
        redirect_to refinery.url_for(page.url), :status => 301   :   
        render_with_templates?

订阅/RSS Feed

Subscribe

分类/category