12、重构基础登录功能

重构基础登录功能

在编写用户控制器的操作时，我们积累了不少冗余代码。在多个地方暴露了用户登录状态的实现细节，并且在操作和测试中都存在重复代码。重构可以在不改变代码功能的前提下，让代码更美观、更易维护和扩展，同时减少潜在的错误。

1. 重构的重要性与安全性

• 重要性 ：组织良好、美观的代码对程序员来说至关重要，它更易于维护和扩展，并且往往更少出现错误。
• 安全性 ：虽然重构存在风险，但我们通过为模型、控制器和视图编写了大量测试，只要新代码能通过测试套件，就可以确信其基本行为未发生改变。

2. 是否已登录？

在检查用户是否登录时，我们之前直接使用会话变量，这暴露了实现细节。我们可以通过定义一个 logged_in? 函数来构建一个小的抽象层。

# app/helpers/application_helper.rb
# Return true if some user is logged in, false otherwise.
def logged_in?
  not session[:user_id].nil?
end

将该函数定义在 ApplicationHelper 模块中，然后在 UserController 中引入：

# app/controller