Trying to improve another if/else statement

Question

Is there a way to optimize the following if/else statement?

if current_user
  if current_user.id == @user.id
    render 'item'
  else
    render 'item_friend'
  end
else
  render 'item'
end

Answer 1

What about:

if !current_user || current_user.id == @user.id
  render 'item'
else
  render 'item_friend'
end

Answer 2

This is a little more dryer

render case 
       when current_user.nil?,current_user.id == @user.id; 'item'
       else 'item_friend'
       end

Answer 3

1. You can move the case when user is not signed in to another place (helper, base controller or something else).

2. You can move the logic for selecting a view template to a separate method.

---

before_filter :render_items_if_current_user_blank, :only => :foobar

# action
def foobar
  @user = User.find(params[:id]) # or something else
  render_user(@user)
end

private

def render_items_if_current_user_blank
  render 'item' unless current_user
end

def render_user
  if current_user.id == @user.id
    render 'item'
  else
    render 'item_friend'
  end
end