smolcatgirl
Hi,
I am trying to get better at Elixir and I was looking for a reason to make a library. Because writing libraries seem to be a good way to learn Elixir, not just doing code but actually implementing low level tech and seeing what they return. So I found myself wanting to take a map and create a struct from it. The function should also tolerate if any keys are missing or if there are more keys than whats specified. So i wrote this library.
It has a function that takes a keyword list or a map and converts into a struct given a module name.
Here is the link for anyone interested: https://github.com/smolcatgirl/struct_create
I would like any feedback 
Thank you!
Where Next?
Trending in Announcing
Other Trending Topics
Latest on Elixir Forum
Sponsor Spotlight
Bringing learning, work and compliance together since 1987.
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
- #elixirconf-us
- #blog-post
- #ai
- #elixir-ls
- #phoenix_html
- #iex
- #graphql
- #genstage
- #websockets
- #supervisor
- #advent-of-code
- #distillery
- #processes
- #api
- #forms
- #hex
- #security
- #metaprogramming
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.
Error tracking, logs, uptime, and Just Enough APM. Built for developers.
Practical resources that improve the lives of professional developers.
Master modern tools, write cleaner code, and tackle real-world challenges.
Producing high quality Elixir screencasts since 2017.
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 7- Show Best Posts
- Show All (oldest first)
- Show All (newest first)
hauleth
Just in case, there is
Kernel.struct/2for that.ityonemo
As an exercise it’s nice! But you very likely don’t want to use this in production since it will cause an atom table leak if it’s passed user maps. This is a security regression as a malicious user can start spamming your server with strings that aren’t atoms yet and eventually cause your server to crash.
smolcatgirl
What if I change it to use String.to_existing_atom?
smolcatgirl
struct/2 fails if I include too many keys. I could filter the keys before but my library does it for me and it works with keyword lists too.
edit: wait it seems like i got it wrong, struct/2 works with this.
ityonemo
That’s slightly better but keep in mind that crashes the calling process. You should probably do a reverse check, which is to say, filter props in your proplist which don’t match Atom.to_string for the struct properties. Do you know how to obtain the list of fields in a struct?
smolcatgirl
Map.from_struct, Map.keys and then map them and convert them to strings?
Should I return a errror value in case the user provide a bad module name or a non keyword list or non map?
ityonemo
Yes. Also, modules with structs define a magic
__struct__/0function that you can use to get a hold of the default struct, bypassing @enforce_keys requirements (use with caution).