Phoenix Elixir - 機能するリファクタリング機能

Question

私はElixirとPhoenixにはかなり新しいですが、私はそれが本当にクールだと思っています。私はRailsの背景から来た。私は、ユーザーがパスワード& password_confirmationせずにサインアップフォームを投稿することができませんフロントエンドのコードを持っていますが、実行するテストは、私が持っているしたい場合

def signup(struct, params \\ %{}) do 
    struct 
    |> cast(params, [:email, :password, :password_confirmation, :username]) 
    |> validate_required([:email, :password, :password_confirmation, :username]) 
    |> validate_length(:username, min: 3) 
    |> validate_length(:username, max: 20) 
    |> validate_length(:password, min: 6) 
    |> validate_format(:email, ~r/\A[\w+\-.]+@[a-z\d\-.]+\.[a-z]+\z/i) 
    |> password_match() 
    |> gen_hash_password() 
    |> set_defaults 
    end 

：

は、例えばこのチェンジを考えます空の値が渡されるかどうかを調べるテスト。私はそれのまわりで得た方法は、他の支店であれば非常に手続き方法のANに書き込むことによってました：bcryptのはnil文字列が与えられますし、ハッシュを計算することができなくなりますよう

defp gen_hash_password(signup) do 
    password = get_change(signup, :password) 
    if password == nil 
    signup 
    else 
    hash = Comeonin.Bcrypt.hashpwsalt(password) 
    signup |> put_change(:password_hash, hash) 
    end 
end 

そうでない場合、テストは失敗します。 これは非常にエレガントではないようですが、私はどのようにしてより良いバージョンを書くことができるのだろうと思っていました。

私の最初の考えは、バリデーションがパイプされた後、その時点でレコードが有効かどうかをチェックすることで早期復帰を行うことでしたが、他の言語と同じように早く戻ることはできません。それが有効だった場合

私の第二の考えを確認した、のようなもの：

if valid? do 
    struct 
    |> pipe the remaining changes 
else 
    struct 
end 

しかし、私はチェンジセット自体の内部構造体の妥当性を照会する方法を見つけるように見えるではないのですか？

おかげchangesetがあなたのgen_hash_passwordに渡され

Answer 1

、あなたは単にchangeset.valid?とチェンジセットの有効性を確認することができます。有効でない場合は、永続化しないレコードのパスワードをハッシングすることがないため、チェンジセットを戻します。それは頻繁にあなたの本来の意図に依存

defp encrypt_password(changeset) do 
    if changeset.valid? do 
    changeset 
    |> put_change(:password_hash, Comeonin.Bcrypt.hashpwsalt(changeset.changes.password)) 
    else 
    changeset 
    end 
end 

Answer 2

が、タイトルに記載された質問に答える：

は、ここでサンプル実装ですアーラン/エリクサーのいずれかではなく、if踊りは、パターンマッチングを行います。