Code Review

This `if`/`else` is totally fine, but a more common implementation might be:

```
if session[:user_id] == nil
 redirect('/login')
 return
end

# rest of code
```