Ruby on Rails: undefined method, checking boolean if true statement

Question

I have a page where I'm checking to see if the database column public is true or if a user is admin.

<% if @user.public || current_user.admin %>
  ... users page ...
<% end %>

The issue right now is that if I'm not logged in, then its checking if current_user.admin is true, so I get a NoMethodError

undefined method `admin' for nil:NilClass

What is the most logical syntax so I can check if its public or admin if I'm not logged in, so it should direct to a noticed page instead of getting this error.

Thanks

Answer 1

You need to check if current user is set before checking whether it's an admin :

if @user.public || (current_user && current_user.admin)

Updated answer

Ruby 2.3.0 introduced the Safe Navigation operator ( &. ).

This makes the code even nicer:

if @user.public || current_user&.admin