josefrichter
Hi,
I find myself doing something like this, mostly in tests:
{:ok, new_time} = Time.new(15, 00, 00)
valid_attrs = %{
body: "some body",
time: new_time
}
I wonder if it’s ok to extract the tuple value like this
valid_attrs = %{
body: "some body",
time: Time.new(15, 00, 00) |> elem(1)
}
I know I can do that, the question is whether I should be doing that this way? Thank you
Where Next?
Trending in Questions
Other Trending Topics
Latest on Elixir Forum
Sponsor Spotlight
Catch errors, track performance, monitor hosts and more.
Categories:
Sub Categories:
Forums
Popular Tags
- #ecto
- #liveview
- #troubleshooting
- #learning-elixir
- #library
- #deployment
- #erlang
- #testing
- #genserver
- #mix
- #absinthe
- #remote-other
- #otp
- #plug
- #how-to-question
- #macros
- #postgres
- #elixirconf
- #channels
- #exunit
- #discussion
- #code-sync
- #podcasts
- #javascript
- #onsite
- #dialyzer
- #docker
- #authentication
- #umbrella
- #full-time-contract
- #podcasts-by-brainlid
- #ecto-query
- #blog-post
- #elixirconf-us
- #elixir-ls
- #ai
- #phoenix_html
- #iex
- #graphql
- #genstage
- #websockets
- #supervisor
- #advent-of-code
- #distillery
- #processes
- #api
- #forms
- #hex
- #security
- #metaprogramming
Our Sponsors
Supporting innovation across the BEAM ecosystem.
We build reliable cloud platforms for business-critical systems.
Catch errors, track performance, monitor hosts and more.
Error tracking, logs, uptime, and Just Enough APM. Built for developers.
Elixir, Erlang & BEAM conferences from Code Sync - join us in 2026!
Practical resources that improve the lives of professional developers.
Master modern tools, write cleaner code, and tackle real-world challenges.
The deployment platform built for Elixir: PaaS ease, VPS control.
The team behind Membrane, Popcorn, LiveDebugger. Available for hire.
Courses that'll move you from confusion to "Aha, now I get it!"
Showing Posts 1 to 4- Show Best Posts
- Show All (oldest first)
- Show All (newest first)
trisolaran
The downside in this specific case is that you are ignoring possible errors. If
Time.newreturns{:error, :some_error_condition}thensome_error_conditionwill end up in yourvalid_attrsmap. This is in general not good :). But, if you are 100% sure thatTime.newwill always return a valid time, then it’s ok.josefrichter
Yeah I thought about that one. But since it’s only in tests, I guess it’s ok.
stefanchrobot
I think you’re looking for
Time.new!or~T:josefrichter
omg of course