Kürzere Code, verhindern duplizierten Anruf

Question

Im Moment habe ich einen Code, der wie folgt aussieht:

def work_with_access_token_or_request_one 
    if is_valid_access_token?(params[:code]) 
     request_client_with_access_token(params[:code]) 
    else 
     request_access_token! 
    end 
end 

Bei diesem Verfahren I erste Prüfung, wenn der Code, den ich in den params bekommen gültig ist. Wenn nicht, fordere ich eins an.

Mein Problem ist, dass gerade jetzt die Methode get_access_token ist zweimal genannt. Einmal wenn ich überprüfe ob das access_token gültig ist und das andere mal wenn ich den Client anfrage.

def get_access_token(code) 
    return Okto.request_access_token_from_code(code) 
end 

Ich habe, dies zu tun, weil die Ausgabe von get_access_token(code) kehrt zurück, wenn erfolgreich {access_code: 89283} oder wenn ein Fehler {error: "not valid"}

def is_valid_access_token?(code) 
    result = get_access_token(code) 
    return result[:access_code] 
end 

def request_client_with_access_token(code) 
    result = get_access_token(code) 
    client = Okto.get_client_from_access_code(result[:acess_code]) 
end 

aufgetreten Wie kann ich meinen Code in work_with_access_token_or_request_one umschreiben, so dass nur ein einziges Mal die Methode get_access_token wird genannt? Danke

Answer 1

Es sieht so aus, als ob diese Methoden zu einer einzigen Klasse gehören. Dann könnten Sie Ihre get_access_token Methode Cache die Ergebnisse wie so umschreiben:

def get_access_token(code) 
    @access_token_cache ||= {} 
    @access_token_cache[code] ||= Okto.request_access_token_from_code(code) 
    @access_token_cache[code] 
end 

Wenn die code ablaufen soll, dann können Sie diese Logik innerhalb Ihrer Methode als auch hinzufügen.

Answer 2

def access_token(code) 
    h = get_access_token(code) 
    h.key?(:access_code) ? h[:access_code] : nil 
end 

def work_with_access_token_or_request_one 
    result = access_token(@params[:code]) 
    result ? request_client_with_access_token(result) : request_access_token! 
end 

Wenn Sie get_access_token und es gibt Effekt keine unerwünschten Neben ändern können, könnten Sie vereinfachen:

def get_access_token(code) 
    h = Okto.request_access_token_from_code(code) 
    h.key?(:access_code) ? h[:access_code] : nil 
end 

def work_with_access_token_or_request_one 
    result = get_access_token(@params[:code]) 
    result ? request_client_with_access_token(result) : request_access_token! 
end 

Beachten Sie, dass die Variable zeigt auf dem Hash muss eine Instanz variabel sein (@params) für den Hash innerhalb der Methode sichtbar sein.