Nice topic, @delbetu !
In the ActiveRecord example, you can also use ActiveRecord::Base#none as null-object pattern and remove the early return:
ActiveRecord
user = User.find_by(id: params[:person_id]) || User.none EmailSender.invite(user)
My personal preference is to avoid (if possible) guard clauses and assertions since those make it harder to read and reason about the code, but I agree that sometimes it's not feasible.
oh, that is cool, I didn't know about ActiveRecord::Base#none Thank you!
ActiveRecord::Base#none
Are you sure you want to hide this comment? It will become hidden in your post, but will still be visible via the comment's permalink.
Hide child comments as well
Confirm
For further actions, you may consider blocking this person and/or reporting abuse
We're a place where coders share, stay up-to-date and grow their careers.
Nice topic, @delbetu !
In the
ActiveRecord
example, you can also use ActiveRecord::Base#none as null-object pattern and remove the early return:My personal preference is to avoid (if possible) guard clauses and assertions since those make it harder to read and reason about the code, but I agree that sometimes it's not feasible.
oh, that is cool, I didn't know about
ActiveRecord::Base#none
Thank you!