"So let's simplify code a bit today..."

237 points · 16 comments · view on lemmy.world

16 Comments

Wolfizen@pawb.social · 50 pts · 2y (1 reply)

Simplifying doesn't have to mean fewer lines. LGTM.

Rooki@lemmy.world · 28 pts · 2y

Yeah, too simple is sometimes bad too. I had to refactor for better readability and it added 3k lines. But it was easier for everyone to understand the code. :)

EpicFailGuy@kbin.social · 22 pts · 2y (5 replies)

@Matriks404

I pray that all those extra lines are comments.

mb_@lemm.ee · 12 pts · 2y

Removed unreadable // Added /* */ on their own lines with fancy design above and below comments

Matriks404@lemmy.world · 3 pts · 2y

Some of them, yes.

fushuan@lemm.ee · 1 pts · 2y

ctrl+P > format.

Git explodes.

Rooki@lemmy.world · -8 pts · 2y (1 reply)

Then the code is noisy. And that isnt simplyfied it is made noisy/clustered up.

Zron@lemmy.world · 9 pts · 2y

Your username is very fitting

tias@discuss.tchncs.de · 15 pts · 2y

Eufalconimorph@discuss.tchncs.de · 12 pts · 2y

If GitHub's UI isn't saying "infinity files changed" you're not trying hard enough.

iHUNTcriminals@lemm.ee · 10 pts · 2y

By simply I thought you meant delete the project.

CommunicationOk3492@feddit.de · 9 pts · 2y

Ahh, yes, tracked eight hours of refactoring today…

fiah@discuss.tchncs.de · 6 pts · 2y

I've done worse

RIP_Cheems@lemmy.world · 5 pts · 2y

Pov: you find an image of a coconut in your code and try to remove it

boletus@sh.itjust.works · 5 pts · 2y

Brought in a feature from a fork of our engine at work the other day. 8000 additions and a bunch of deletions and fix commits.

And apparently that's a pretty small merge...

Took me days just to review it.

DmMacniel@feddit.de · 1 pts · 2y

It's just code formatting, isn't it?