dee*_*dee 9 ruby refactoring ruby-on-rails
Railscasts发布了一篇关于重构的精彩集.一种方法是将复杂的控制器逻辑移动到服务对象中,而不是将其推送到模型中.在一个服务对象中,使用以下代码:
class PasswordReset
attr_reader :user
def self.from_email(email)
new User.find_by_email(email)
end
def self.from_token(token)
new User.find_by_password_reset_token!(token)
end
...
end
Run Code Online (Sandbox Code Playgroud)
new
两个方法体中关键词的作用是什么?new User.find_by_
.那有什么不同User.find_by_
?
这是调用代码:
def create # controller
password_reset = PasswordReset.from_email(params[:email])
if password_reset.user
password_reset.send_email
redirect_to root_url, notice: "Email sent with password reset instructions."
else
redirect_to new_password_reset_url, alert: "Email address does not match a user account."
end
end
Run Code Online (Sandbox Code Playgroud)
还有,为什么attr_reader :user
需要?
Jef*_*tte 11
classname隐含在self方法中.代码可以写成:
def self.from_email(email)
PasswordReset.new User.find_by_email(email)
end
Run Code Online (Sandbox Code Playgroud)
要回答问题的后半部分,请attr_reader
定义实例变量和读取器方法(如果您来自java或c#,则称为getter方法).把它们放在一起,你可以把它写成:
class PasswordReset
def user
@user
end
def self.from_email(email)
PasswordReset.new User.find_by_email(email)
end
def self.from_token(token)
PasswordReset.new User.find_by_password_reset_token!(token)
end
...
end
Run Code Online (Sandbox Code Playgroud)
这假设PasswordReset#initialize将User作为参数,并相应地设置@user