this post was submitted on 01 Sep 2023
320 points (96.0% liked)

Programming

17668 readers
160 users here now

Welcome to the main community in programming.dev! Feel free to post anything relating to programming here!

Cross posting is strongly encouraged in the instance. If you feel your post or another person's post makes sense in another community cross post into it.

Hope you enjoy the instance!

Rules

Rules

  • Follow the programming.dev instance rules
  • Keep content related to programming in some way
  • If you're posting long videos try to add in some form of tldr for those who don't want to watch videos

Wormhole

Follow the wormhole through a path of communities [email protected]



founded 2 years ago
MODERATORS
you are viewing a single comment's thread
view the rest of the comments
[–] [email protected] 28 points 1 year ago (6 children)

Shorter code is almost always better.

Should you use a class? Should you use a Factory pattern or some other pattern? Should you reorganize your code? Whichever results in the least code is probably best.

A nice thing about code length is it's objective. We can argue all day about which design pattern makes more sense, but we can agree on which of two implementations is shorter.

It takes a damn good abstraction to beat having shorter code.

[–] [email protected] 27 points 1 year ago (2 children)

I mostly agree with this but more than shorter code I value readability, I would rather take 3 lines to be clear to any developer than use some obscure or easy to misunderstand structure to write it in 1.

[–] [email protected] 7 points 1 year ago (2 children)

Yep. And three functions is better than one for legibility even if one would be fewer lines of code

[–] Feathercrown 5 points 1 year ago (1 children)

I think it really depends. Functions break up the visual flow, so if you need to look at multiple functions to visualize one conceptual process then it can be less efficient

[–] [email protected] 4 points 1 year ago

Yes. I learned this from Haskell. I like Haskell, but it has a lot of very granular functions.

Earlier comment said that breaking up 1 function into 3 improves readability? Well, if you really want readability then break it up into 30 functions using Haskell. Your single function with 25 lines will become 30 functions, so readable (/s).

In truth, there's a balance between the two. Breaking things up into function does have advantages, but, as you say, it makes it more likely that you'll have to jump around a lot to understand a single process.

[–] gornius 1 points 1 year ago

I specifically have a rule that if at the current abstraction layer, a step is more than one function call/assignment - I'm creating another function for that.

[–] [email protected] 1 points 1 year ago

I think they didn't mean how you structure your code but actually precision.

[–] [email protected] 13 points 1 year ago (1 children)

Here you go: https://codegolf.stackexchange.com/

But on a more serious note, I don't really agree. Writing more code needs to be a conscious choice, but going for the shortest code too often creates a mess. I know, since I was that junior dev who just wanted to get stuff done and I would ignore project architecture in order to have to implement less, like accessing the database in GUI code.

Shorter code with the same amount of coupling between components and with the same readability is always better though.

[–] [email protected] 2 points 1 year ago

That being said, if you access the database in GUI there is a high chance that you will repeat yourself making the whole program bigger.

[–] [email protected] 7 points 1 year ago

i used to think like this. nowadays I prefer readability, and even debugability.

sure, I could inline that expression, but if I assign it to a constant with a descriptive name instead, the next person reading that piece of code will not hate me.

[–] [email protected] 6 points 1 year ago

but we can agree on which of two implementations is shorter.

Shortness for the sake of being short sounds like optimizing for the wrong metric. Code needs to be easy to read, but it's more important that the code is easy to change and easy to test. Inline code and function calls are renowned to render code untestable, and introducing abstract classes and handles is a renowned technique to stub out dependencies.

[–] redempt 3 points 1 year ago

shorter code is not always better, especially when it comes to types. building in lots of guard rails by being verbose with the type system is a good thing. "shorter = better" is the python approach that starts off fun and easy but the codebase scales extremely poorly.

[–] corstian 1 points 1 year ago

I spent years writing an abstraction to be able to write shorter code... Not sure whether that was worth it 😅