iiiiiiitttttttttttt

1493 readers

1 users here now

you know the computer thing is it plugged in?

A community for memes and posts about tech and IT related rage.

founded 8 months ago

MODERATORS

irelephant@programming.dev

irelephant@lemmy.dbzer0.com

Shivering@programming.dev

neidu3@sh.itjust.works

KingTootsie@programming.dev

AnarchistArtificer@slrpnk.net

101

My code review team is skeptical. (eviltoast.org)

submitted 1 month ago by dbtng@eviltoast.org to c/iiiiiiitttttttttttt@programming.dev

5 comments fedilink hide all child comments

Nixie and Larry ain't buying my bullshit. Not enough whitespace.

you are viewing a single comment's thread
view the rest of the comments

[–] luciferofastora@feddit.org 3 points 1 month ago (1 children)

Not enough whitespace.

I got to help a colleague debug a powershell script. It contained six SQL queries.

They were each written in a single line and featured double-nested queries and multi-clause conditionals. He didn't use any aliases either.

I'm sure there's a way to auto-format SQL in some tool, but in the moment, I couldn't remember.

Listen to your cats. Use more whitespace.

Also, if they like it, please pet them for me. Good code reviewers deserve pets.

[–] dbtng@eviltoast.org 1 points 1 month ago* (last edited 1 month ago)

Hehe. You are preachin to the choir.
In truth, most of my code, particularly my published pieces, looks like a textbook example of self-documentation.
This is because I'm inexpert enough at coding to need the pointers next time I look at my own code.
So if it looks like a kindergarten teacher authored it, I'll probly be able to understand it later.

And thanks! My kitties always want more scratches. :]