I always feel bad about putting little ifs at the top of functions. Is it not bad practice? I like them because they're simple to implement modify and read, but I have this voice telling me I need to make things more impressive.

[–] Jankatarch@lemmy.world 2 points 1 day ago* (last edited 1 day ago)

I started putting a helpful comment above the ifs as a seperator to cope with that.

public Value? Func(String arg) { 
  // Sanitize.
  if (arg.IsEmpty()) return null; 
  if (this.Bar == null) return null; 
  // Get [that] and/or do [this].
  var foo = this.baz.foo;
  ...
  return new Value();
}

[–] Lightfire228@pawb.social 2 points 21 hours ago

Never make things more "impressive"

Make them more comprehensible

Reduce the cognitive load required to understand and reason about a piece of code. Honestly, the more you can express complicated ideas simply, the more impressive you are