philm

joined 1 year ago
[–] philm 10 points 1 year ago

Don't get me wrong comments != documentation (e.g. doc-comments above function/method).

I probably was a bit unprecise, as others here summed up well, it's the why that should be commented.

[–] philm 3 points 1 year ago

Yeah that's a good summary

[–] philm 12 points 1 year ago (43 children)

Yeah, but unironic...

If your code needs comments, it's either because it's unnecessarily complex/convoluted, or because there's more thought in it (e.g. complex mathematic operations, or edge-cases etc.). Comments just often don't age well IME, and when people are "forced" to read the (hopefully readable) code, they will more likely understand what is really happening, and the relevant design decisions.

Good video I really recommend: https://www.youtube.com/watch?v=Bf7vDBBOBUA

[–] philm 3 points 1 year ago

Indeed, and the maintainer opened this RFC (I think because of it)

[–] philm 1 points 1 year ago (16 children)

SUUUUUUUUURE!!!11 I"M oN ITTTTTTTT

[–] philm 1 points 1 year ago (18 children)

We're at 22.8̅2̅8̅7̅8̅4̅1̅1̅9̅1̅0̅6̅6̅9̅9̅7̅5̅1̅8̅6̅1̅0̅4̅2̅1̅8̅3̅6̅2̅2̅% slowly gaining rainbow ground

[–] philm 3 points 1 year ago (22 children)

I just calculated exact subpixel accuracy, for me it's exactly 20.5̅9̅5̅5̅3̅3̅4̅9̅8̅7̅5̅9̅3̅0̅5̅2̅1̅0̅9̅1̅8̅1̅1̅4̅1̅4̅3̅9̅2̅0̅ % that is still missing to fill the whole comment body with rainbows, way to go!

[–] philm 3 points 1 year ago* (last edited 1 year ago) (25 children)

Let's start the sixth rainbow!

[–] philm 3 points 1 year ago (27 children)

Plenty of space for me still (browser version on desktop)

[–] philm 3 points 1 year ago (33 children)

Rookie numbers, it's probably 15% on my screen, There's space for a lot more rainbows

[–] philm 4 points 1 year ago (1 children)

"easily" solve it.

FTFY

[–] philm 8 points 1 year ago

I mean if you have a super nice working environment (team etc.), I don't see an issue with staying at the company.

But yeah as you say, if the new company is better in every single way, of course you should move.

view more: ‹ prev next ›