nmcalabroso

nmcalabroso

What is the idiomatic way for input validations?

I’m new to elixir and functional programming, in general. I want to ask what’s the idiomatic way to validate our input parameters? Say I want to do this in my UserController.create.

Check if password is valid by PasswordManager.valid?/1
If not, render error
else, continue

This is what I’ve come up with:

defmodule MyApp.UserController do
  def create(conn, %{"data" => data}) do
    case PasswordManager.valid? user_attrs["password"] do
      {:ok, raw_password} ->
        # make changeset with hashed password
        case Repo.insert(changeset) do
          {:ok, user} ->
            # success
          {:error, invalid_changeset} ->
            # render failure  
      {:error, reason} ->
        render(conn, :errors, data: reason)
    end
  end
end

The issue here is the “ugly” nesting seems unnecessary. I’m looking for a “return early” statements but I guess elixir wants us to write pure functions as much as possible and utilize pattern matching accordingly. How do I refactor this?

Coming from python, I would do this:

def create(user_attrs):
  if not PasswordManager.is_valid(user_attrs['password']):
    # display error
    return
  
  # continue

But what’s the best practice in elixir?

Feel free to do some nitpicks to my code so I can improve writing in this language.

Also, please do note that I’m asking this as a generic question. I hope you don’t dwell too much with the password validation as an example. Thank you.

Marked As Solved

NobbZ

NobbZ

Nope, I think its meant like this:

with {:ok, data} <- validate_one(data),
     {:ok, data} <- validate_two(data) do
  {:ok, data}
else
  {:error, reason} -> {:error, reason}
end

Also Liked

wmnnd

wmnnd

Hello @nmcalabroso,

in Elixir and other functional languages, you cannot »return early« as you would do in Python or Ruby.
In your particular case, you could use the with keyword in order to avoid nesting and chain pattern-matched conditions together:

with {:ok, raw_password} <- PasswordManager.valid?(user_attrs["password"]),
     {:ok, changeset} <- create_changeset(user_attrs, raw_password),
     {:ok, user} <- Repo.insert(changeset) do
     #whatever you want to do if everything went okay
else
  {:error, reason} -> #whatever you want to do if there was an {:error, reason} tuple
end

You can read more about the with keyword in the Elixir Getting Started Guide: http://elixir-lang.org/getting-started/mix-otp/docs-tests-and-with.html#with

And here is a nice blog article about the with keyword and what’s called »Railway-oriented programming« in the F# world: http://www.scottmessinger.com/2016/03/25/railway-development-in-elixir-using-with/

michalmuskala

michalmuskala

I would say the most natural way of emulating “early return” would be to split the logic into multiple functions each one returning or continuing to the next one. The with construct is kind of a “get out of jail free card” - it allows you to encode those patterns less verbosely in a single expression. I’d say that’s the most “common” way to do this.

In case of ecto, the most idiomatic way of doing validations is to use, as mentioned, changesets.

nmcalabroso

nmcalabroso

Definitely reading this. Thanks a lot.

Where Next?

Popular in Questions Top

9mm
I am constructing a JSON object (map) and I need to conditionally set a field. I’m trying to write proper elixir-way code… and I’m at a l...
New
senggen
Erlang/OTP 25 [erts-13.2.2] [source] [64-bit] [smp:8:8] [ds:8:8:10] [async-threads:1] 15:22:35.803 [error] gen_event {lager_file_backend...
New
Harrisonl
We have an ECS cluster with 4 services, where each task joins a single cluster, via discovery ECS discovery service. Currently when I de...
New
marius95
Hello everyone, I try to use an Javascript Event Handler in my root.html.leex file. Therefore I created a function in the app.js file: ...
New
qwerescape
Is there a way to get the call stack or stack trace at any point in the code? Not from exceptions, but an expression that returns how the...
New
chrisalley
ExUnit now has describe blocks which is a welcome addition coming from RSpec. In the docs, it states that nested hierarchies of describe ...
New
Lily
In templates/appointment/index.html.eex: &lt;%= for appointment &lt;- @appointments do %&gt; &lt;tr&gt; &lt;td&gt;&lt;%= appoi...
New
joeerl
Hello again - after a longish gap I’ve decided I really must dig into Elixir and see what’s been happening here - so I have a few questio...
New
Emily
I have VueJS GUIs with the project generated using Webpack. I have Elixir modules that will need to be used by the VueJS GUIs. I forese...
New
bsollish-terakeet
Credo is smart enough to check for (something like) this: assert length(the_list) == 0 with this response: Checking if an enum is empt...
New

Other popular topics Top

skosch
To my knowledge, put_in, Map.update etc. all have the one limitation of not automatically creating intermediate keys when needed (for exa...
New
gshaw
What is the idiomatic way of matching for not nil in Elixir? E.g., First way: defp halt_if_not_signed_in(conn, signed_in_account) when...
New
minhajuddin
I have seen a lot of code which picks the first element from a list using Enum.at(0) instead of List.first. Is there a reason why people ...
New
Fl4m3Ph03n1x
About me? ( if you have nothing better to do than reading about some random guy in the internet :stuck_out_tongue: ) Hello all, this is ...
New
msaraiva
Surface is an experimental library built on top of Phoenix LiveView and its new LiveComponent API that aims to provide a more declarative...
564 43757 214
New
jerry
Good day to you all. I have been struggling to get a query involving like and ilike to work. Can anyone assist me on this, please? pro...
New
Lily
In templates/appointment/index.html.eex: &lt;%= for appointment &lt;- @appointments do %&gt; &lt;tr&gt; &lt;td&gt;&lt;%= appoi...
New
komlanvi
Hi everyone, I was playing with phoenix liveView but I run into an issue. I have a form and want to validate each input text when the te...
New
joaquinalcerro
Hi there, I am working with Ecto-Postgresql and I need to call all of the records from a specific table but the table has 40,000 records...
New
vegabook
I’m brand new to Phoenix and I have stripped one of the demo applications to the bone. I just want to get an svg up on the screen. Here i...
New

Latest on Elixir Forum

Elixir Forum

We're in Beta

About us Mission Statement