Phoenix，conn重定向停止所有进一步处理？

Question

我有一个控制器，我想添加Facebook页面ID和访问令牌到我的数据库，以便我可以代表用户发布。

为此，我需要对Facebook进行GET调用，然后使用我必须插入到数据库中的访问令牌。

这意味着我有一个嵌套的case语句，因为有错误的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作为参数传递给现有用户。

请记住我的模块和函数命名吸吮，因为我不知道你到底在做什么。更好的命名是必须的。