stiegi
Most elegant way for simple conditions an deconstructions
I would like to have your opinions about a very simple example, because I worked with imperative languages so far only, and I would like to get the mindset of declarative programming.
So i have a function that is called parse_index/1. During runtime we don’t know if we receive an integer or a string as argument, but if it is a string, we want to convert it to a number.
I wrote it like this:
def parse_index(index) do
is_number(index)
|> case do
true -> index
false -> Integer.parse(index)
end
|> case do
{number, _} -> number
number -> number
end
end
These are a lot of lines for a pretty simple task, and I was wondering: how small and elegant can you rewrite that by taking advantage of more language features.
It’s really just to learn ‘how to think’ about problems why I ask that, because it kind of feels like I force the imperative way on Elixir when I write it like that…
Thanks!
Marked As Solved
kokolegorille
Pipe is nice, but too much is ugly 
def parse_index(string) when is_binary(string) do
case Integer.parse(string) do
{number, _} -> number
:error -> 0
end
end
… and if You really wanted to, You should have started with
def parse_index(string) when is_binary(string) do
string
|> Integer.parse()
|> case do
{number, _} -> number
:error -> 0
end
end
Also Liked
kokolegorille
Hello and welcome,
You could pattern match on function header…
like this
def parse_index(string) when is_binary(string) do
...
end
def parse_index(number), do: number
LostKobrakai
I generally tend to use ecto wherever data enters the system and converting it to a known format/type. E.g. using schemaless changesets:
def parse_something_with_an_index(data) do
{%{index: nil}, %{index: :integer}}
|> Ecto.Changeset.cast(data, [:index])
|> Ecto.Changeset.validate_required([:index])
|> …
end
changeset = parse_something_with_an_index(input)
case Ecto.Changeset.apply_action(changeset, :validate) do
{:ok, data} -> # correct data
{:error, changeset} -> # some errors
end
After that you can go on and do whatever you need to do with the data knowing it’s in the format type you expect it to be in.
stiegi
I see, thanks! Prefer without pipes here too
Popular in Questions
Other popular topics
Latest on Elixir Forum
Sponsor Spotlight
The deployment platform built for Elixir: PaaS ease, VPS control.
Categories:
Sub Categories:
Forums
Our Sponsors
Build Elixir applications with speed and confidence.
Supporting innovation across the BEAM ecosystem.
We build reliable cloud platforms for business-critical systems.
Catch errors, track performance, monitor hosts and more.
Develop your skills with books, videos, and courses.
Producing high quality Elixir screencasts since 2017.
The deployment platform built for Elixir: PaaS ease, VPS control.
Courses that'll move you from confusion to "Aha, now I get it!"