A place for developers to talk about building on Farcaster.

/dev

Potentially hot take but early returns are horrible for readability and should be avoided

Potentially hot take but early returns are horrible for readability and should be avoided
https://x.com/t3dotgg/status/1799640178459656662?s=46

fortune cookie casts, memes, parenting, building mini apps, /framedl, /framedl-pro & /calmer

edge cases often no, validation / auth often yes, but some people prefer not to throw

Shouldn’t those checks throw errors normally?

depends. edge cases, validation, auth should be imo handled asap. people could argue that they should be outside of the function body, but that's often impractical

Building azura.xyz

Find me on the links ⛳️  slopes ⛷️ or base 🔵

icebreaker.xyz/sonofmosiah.fc

Mostly used to check for invalid/unwanted states. When this is done first you can separate that logic from your core logic and also get some type narrowing in most cases

I’m a big fan of early returns esp for js/ts

Yeah fs, but breaking it down further to avoid early returns makes it harder to read and parse imo

I can get behind no early returns for
other languages (particularly functional ones), but I like being able to immediately ‘throw out’ error cases with early returns in js

Gimme pattern matching and this changes…

Why? If there’s an opportunity for an early return there’s probably an opportunity to break down the function into smaller ones