Phoenix, Does conn redirect остановить всю дальнейшую обработку?

Question

У меня есть контроллер, где я хотел бы добавить идентификатор страницы facebook и токен доступа к моей базе данных, чтобы я мог отправлять сообщения от имени пользователя.

Для этого мне нужно сделать GET-вызов в facebook, а затем с токеном доступа, который мне нужно вставить в мою базу данных.

Это означает, что у меня есть вложенный оператор случай, потому что есть 2 точки ошибок, от facebook вызова и вставлять в репозиторий:

case response_from_facebook do 
    {:ok, token} -> 
     case Repo.insert(token) do 
      {:ok, _} -> ... redirect the user 
      {:error, _} -> ... show an error message 
    {:error, _} -> redirect and show an error 

Это некрасиво имо поэтому мне было интересно, если я сделал что-то вроде охранник, если он прекратит дальнейшую обработку:

token = case response_from_facebook do 
    {:ok, token} -> token 
    {error,_ } -> conn |> redirect(to: user_path(conn, :show, user_id)) 
end 

# Does it ever try to insert into the repo if an error occurs? 

Repo.update ... etc. 

Это сделает мой код понятнее, чтобы понять/читать, но, я не совсем уверен, что если бы перенаправлять предотвратить попытку вставить в мой репозиторий. Если этот «защитник» не является решением, как я могу предотвратить многопользовательский аргумент case, который может быть затруднен, чтобы читать?

Answer 1

уборщик способ сделать это:

defmodule SomeApp.FacebookController do 
    ... 

    def create(conn, params) do 
    get_from_facebook 
    |> process_facebook(conn) 
    end 

    defp get_from_facebook do 
    # this is a function you implement 
    end 

    defp process_facebook({:ok, token}, conn) do 
    Repo.insert(token) 
    |> process_insert(conn) 
    end 

    defp process_facebook({:error, error}, conn) do 
    conn 
    |> redirect_with_error(error) # this is a function you will write 
    end 

    defp process_insert({:ok, _}, conn) do 
    conn 
    |> redirect(to: user_path(conn, :show, user_id)) 
    end 

    defp process_insert({:error, error}, conn) do 
    conn 
    |> redirect_with_error(error) # this is a function you will write 
    end 

    ... 
end 

никаких побочных эффектов не может произойти с этим факторинга логики, и это намного «симпатичнее».

Еще лучше, если вы нарушите большинство этой логики в другое модуль:

defmodule SomeApp.FacebookCommand do 

    def call do 
    make_request 
    |> process_response 
    end 

    defp make_request do 
    # whatever you are doing to request from Facebook 
    end 

    defp process_response({:ok, token}) do 
    Repo.insert(token) 
    |> process_insert(conn) 
    end 

    defp process_response({:error, error}) do 
    {:error, error} 
    end 

    defp process_insert({:ok, record}) do 
    {:ok, record} 
    end 

    defp process_insert({:error, error}) do 
    {:error, error} 
    end 

end 

, а затем использовать его в контроллере:

defmodule SomeApp.FcebookController do 
    ... 

    def create(conn, params) do 
    case SomeApp.FacebookCommand.call do 
     {:ok, user} -> 
     conn 
     |> redirect(to: user_path(conn, :show, user.id)) 
     {:error, error} -> 
     # redirect with error 
    end 
    end 

    ... 
end 

Очевидно, что если вы обновляете существующего пользователя, то вы можете выполнить команду, где вы передаете существующего пользователя в call в качестве параметра. '

Пожалуйста, помните, что мой модуль и функция назвали сосать, потому что я точно не знаю, что вы делаете. Лучшее именование - необходимость.