エリクサーのネストされたif文を避けるには？

Question

エリクサーでこれをリファクタリングして読みやすくする方法はありますか？

def validate(params) do 
    Repo.start_link 

    if validate_oauth_params(params) === true do 
     oauth_client = Repo.get_by(OauthClient, random_id: params["client_id"], secret: params["secret"]) 

     if oauth_client != nil do 
     allowed_grant_types = Poison.Parser.parse!(oauth_client.allowed_grant_types) 
     if Map.has_key?(allowed_grant_types, params["grant_type"]) do 
      case params["grant_type"] do 
      "password" -> 
       process_password_grant(params, oauth_client) 
      "refresh_token" -> 
       process_refresh_token_grant(params["refresh_token"], oauth_client) 
      "client_credentials" -> 
       process_client_credentials_grant(oauth_client) 
      nil -> 
       %{message: "Invalid oauth credentials", code: 400} 
      end 
     end 
     else 
     %{message: "Invalid oauth credentials", code: 400} 
     end 
    else 
     %{message: "Invalid oauth credentials", code: 400} 
    end 
    end 

このコードはPHPのように見えるので、それを行うためのエリクサーの方法は何ですか。私はそれを書いていない。

Answer 1

あなたは正しく、PHPのようです。パターンマッチングのようなエリキシルの利点は使用されません。

これをリファクタリングするのは難しいです。他の方法もリファクタリングして、よりきれいにする必要があるようです。あなたはそれに対してパターンマッチするように、例えばvalidate_oauth_params機能は、代わりにブール値のタプルを返す「でした」、とあなたは同じように行うことになります。このような

def validate(params) do 
    case validate_oauth_params(params) do 
    {:ok, params} -> choose_oauth_method params 
    {:error} -> handle_error "Invalid params" 
    end 
end 

defp choose_oauth_method(%{"grant_type" => "password"} = params) do 
    process_password_grant(params) 
end 
defp choose_oauth_method(%{"grant_type" => nil}) do 
    handle_error "Method undefined" 
end 

defp handle_error(msg), do: %{message: msg, code: 400} 
defp handle_error(msg, code), do: %{message: msg, code: code} 
defp handle_error(), do: %{message: "Default error massage", code: 400} 

何かを、これは、すべてのコードを動作しません、ただのアイデア、どのようにパターンマッチング作品とその利点を得ることが

あなたはしかし、そのようなrefactiongは、他のタラでさらに変更が必要な場合があります

def validate(params) do 
    with 
    {:ok, p} <- validate_oauth_params(params), 
    {:ok, client} = get_client_for_params(p), 
    {:ok, allowed_grant_types} <- Poison.Parser.parse(oauth_client.allowed_grant_types) 
    do 
    case params["grant_type"] do 
     "password" -> 
     process_password_grant(params, oauth_client) 
     "refresh_token" -> 
     process_refresh_token_grant(params["refresh_token"], oauth_client) 
     "client_credentials" -> 
     process_client_credentials_grant(oauth_client) 
     nil -> :error 
    end 
    end |> case do 
    {:ok, something} -> something 
    _ -> %{message: "Invalid oauth credentials", code: 400} 
    end 
end 

のようなものにwith式を使用してこのコードを書き換えることを試みることができ

Answer 2

eまたは認識可能な値とエラーを返すためのヘルパー関数の導入withは現在パターンのガードをサポートしていませんが、1.3のガードをサポートします。

Answer 3

ここでは、ロジックが本当に複雑になり、with/doステートメントでは不十分な場合に使用する方法が少し異なります。

def validate(params) do 
    try do 
     if !validate_oauth_params(params), do: 
      throw({:error, %{message: "Invalid oauth credentials", code: 400}}) 

     if !oauth_client, do: 
      throw({:error, %{message: "Invalid oauth credentials", code: 400}}) 

     allowed_grant_types = Poison.Parser.parse!(oauth_client.allowed_grant_types) 
     if !Map.has_key?(allowed_grant_types, params["grant_type"]), do: 
      throw({:error, %{message: "No grant type", code: 400}}) 

     case params["grant_type"] do 
      "password" -> 
       process_password_grant(params, oauth_client) 
      "refresh_token" -> 
       process_refresh_token_grant(params["refresh_token"], oauth_client) 
      "client_credentials" -> 
       process_client_credentials_grant(oauth_client) 
      nil -> 
       throw({:error, %{message: "Invalid oauth credentials", code: 400}}) 
     end 
    catch 
     {:error, map} -> {:error, map} 
    else 
     result -> {:ok, result} 
    end 
end