गार्ड के बजाय एक सशर्त अभिव्यक्ति के अंदर कोड लपेटकर के खंड रेल

Question

निम्नलिखित कोड है:

# API controller for authentication 
class Api::V1::SessionsController < Api::V1::ApplicationController 
    skip_before_action :authorize 

    def create 
    @user = User.find_by(email: params[:user][:email]) 
    unless @user && @user.authenticate(params[:user][:password]) 
     @error_message = 'Invalid username or password' 
     render 'shared/error', status: :unauthorized 
    end 
    end 
end 

मैं अगर यह रूबी दिशा निर्देशों के मेल खाता है Rubocop मेरी कोड की जांच करने का उपयोग करें। मैं निम्नलिखित त्रुटि मिली:

Use a guard clause instead of wrapping the code inside a conditional expression. 
    unless @user && @user.authenticate(params[:user][:password]) 

तो, मुझे समझ नहीं आता कि कैसे मैं बेहतर गार्ड खंड का उपयोग कर इस कोड को बना सकते हैं। अग्रिम में धन्यवाद!

Answer 1

बाद rubocops विशिष्ट जानकारी: http://www.rubydoc.info/github/bbatsov/rubocop/Rubocop/Cop/Style/GuardClause

कुछ की तरह ...

return if @user && @user.authenticate(params[:user][:password]) 
@error_message = 'Invalid username or password' 
render 'shared/error', status: :unauthorized