922
top 50 comments
sorted by: hot top controversial new old
[-] aeki@slrpnk.net 94 points 11 months ago

I am really happy when people are quite strict in code reviews, it makes me feel safer and I get to learn more.

Nothing worse than some silent approvals with no real feedback. What if I missed something obvious... and now it's merged.

To be fair, I also enjoy getting my grammar corrected. I'm juggling 3 languages and things can get messy.

[-] jadero@programming.dev 33 points 11 months ago

In that spirit, I will call attention to your first sentence, specifically the comma. In my opinion, that can be improved. One of three other constructions would be more appropriate:

  • I am really happy when people are quite strict in code reviews. It makes me feel safer and I get to learn more.
  • I am really happy when people are quite strict in code reviews, because it makes me feel safer and I get to learn more.
  • I am really happy when people are quite strict in code reviews; it makes me feel safer and I get to learn more.

The first of my suggested changes is favoured by those who follow the school of thought that argues that written sentences should be kept short and uncomplicated to make processing easier for those less fluent. To me, it sounds choppy or that you've omitted someone asking "Why?" after the first sentence.

Personally, I prefer the middle one, because it is the full expression of a complete state of mind. You have a feeling and a reason for that feeling. There is a sense in which they are inseparable, so not splitting them up seems like a good idea. The "because" explicitly links the feeling and reason.

The semicolon construction was favoured by my grade school teachers in the 1960s, but, as with the first suggestion, it just feels choppy. I tend to overuse semicolons, so I try to go back and either replace them with periods or restructure the sentences to eliminate them. In this particular case, I think the semicolon is preferable to both comma and period, but still inferior to the "because" construction.

I've clearly spent too much time hashing stuff out in writers' groups. :)

[-] aeki@slrpnk.net 8 points 11 months ago

This is what I live for. :D

I agree with most of that. In formal settings, I prefer full sentences with conjunctions; however, choppy sentences are the ones that often end up in my Lemmy comments.

[-] jadero@programming.dev 8 points 11 months ago

That only makes sense. We are having a conversation, not creating literature.

[-] little_hermit@lemmus.org 6 points 11 months ago

Strange, I get a mild hostility vibe from colleagues if I review too ambitiously.

load more comments (1 replies)
[-] reflex@kbin.social 11 points 11 months ago* (last edited 11 months ago)

Like the other guy, I also read your comment twice looking for mistakes but found none.
You should of left something to fix!
😏

Edit: I'm glad there so many people who are as passionate about the correct spelling of "should've" as I am. I was testing you all, and you passed!

[-] ironhydroxide@partizle.com 15 points 11 months ago

Correcting the reviewer.
Notes: "should of" isn't valid, should implies a verb, of isn't a verb. I expect you meant "should have". Please recall this in future submissions.

[-] wandermind@sopuli.xyz 5 points 11 months ago

They should of course keep that in mind, but it's not that "should" should always be followed by a verb directly. The problem is that "of" in this context is a mishearing/spelling of "have", so they should in this case have written it like that instead.

[-] Huschke@programming.dev 5 points 11 months ago

I love that you used "should of" in a valid sentence.

[-] Doug@midwest.social 3 points 11 months ago

Except that it would be "they should, of course,".

load more comments (1 replies)
[-] reflex@kbin.social 3 points 11 months ago* (last edited 11 months ago)

Notes: “should of” isn’t valid, should implies a verb, of isn’t a verb. I expect you meant “should have”. Please recall this in future submissions.

😏

[-] interolivary@beehaw.org 2 points 11 months ago

should implies a verb, of isn't a verb

"should" and "of" should probably be in quotes here?

[-] LinuxSBC@lemm.ee 2 points 11 months ago* (last edited 11 months ago)

A question mark does not fit the sentence, which is a statement ("they should." rather than "should they?"). While question marks are commonly used to demonstrate a rising tone at the end of a sentence, its not considered correct for formal writing.

[-] interolivary@beehaw.org 2 points 11 months ago

A-ha, but this most decidedly not formal writing! UNO REVERSE CARD.

But on a more serious note, I did intend it as a sort of question because I'm not 100% sure, because the rules for quote use might well be different in English than my native language. I actually also don't know the rule for question mark usage in English; is it generally considered a crime against orthography to plonk a question mark on something that's a statement, or is it valid in some cases?

load more comments (2 replies)
[-] interolivary@beehaw.org 7 points 11 months ago* (last edited 11 months ago)

Notably, a good code review should also bring up the good parts of the submission, and not just concentrate on the errors. Not only does it make the recipient feel better to get positive feedback among the negative, but it helps them learn about good practices too. Just concentrating on the errors doesn't really tell them which things they're doing well.

Many reviewers concentrate on just finding mistakes, and while it's useful it's sort of the bare minimum; a good code review should be educational. Especially if the submitter's a more junior coder, in which case it'd also be a good idea to not just outright tell them how you'd fix some problem, but sort of lead them to a solution by asking them questions and pointing things out and letting them do the thinking themselves. But still, experienced coders will also benefit from well-structured feedback, it's not like we're "finished" and stopped learning.

load more comments (2 replies)
[-] Doug@midwest.social 5 points 11 months ago

I'm juggling 3 languages

We Americans like to forget that anyone might have any trouble understanding English especially in cases of polyglots.

I don't know which is your native tongue but from this comment it looks like you're doing a fine job.

[-] CoderKat@lemm.ee 5 points 11 months ago

Yeah, I learn so much from code reviews and they've saved me so much time from dumb mistakes I missed. I've also caught no shortage of bugs in other people's code that saved us all a stressful headache. It's just vastly easier to fix a bug before it merges than once it breaks a bunch of people.

[-] Anticorp@lemmy.ml 2 points 11 months ago

Assuming you have competent leadership, then it wouldn't be merged if you missed something obvious. I guess you're saying that you want more positive reinforcement.

[-] Sylvartas@lemmy.world 56 points 11 months ago

I will correct both + your spelling because it drives me fucking nuts when I can't find a function or variable due to it being severely misspelled

[-] Sonotsugipaa@lemmy.dbzer0.com 19 points 11 months ago

Me omw to tell the POSiX guys it's supposed to be "O_CREATE" instead of "O_CREAT":

[-] UndercoverUlrikHD@programming.dev 32 points 11 months ago
[-] QuazarOmega@lemy.lol 16 points 11 months ago
[-] marcos@lemmy.world 5 points 11 months ago* (last edited 11 months ago)

That's the Oracle way.

The Unix way: chexp.

[-] QuazarOmega@lemy.lol 2 points 11 months ago

Not gonna lie, I have no idea what chexp is supposed to mean

load more comments (2 replies)
[-] CanadaPlus@lemmy.sdf.org 3 points 11 months ago
load more comments (1 replies)
[-] Sonotsugipaa@lemmy.dbzer0.com 3 points 11 months ago

I stand corrected:

Me omw tell POSiX "O_CREATE" not "O_CREAT":

[-] explodicle@local106.com 42 points 11 months ago

Correcting my code is helpful. The machine didn't know what I even meant. Computers are interesting and changing rapidly.

Correcting my grammar is an unsolicited English lesson from someone who already knew what I meant. English is not interesting or changing quickly.

[-] lemmyingly@lemm.ee 11 points 11 months ago

What if your grammar is that bad that people struggle to understand you?

I know someone who is incomprehensible most of the time. I have to ask probing questions just to vaguely understand what they're trying to communicate. I've politely told them more than once about the issue but they never try; they're not mentally challenged or anything, just an ass.

load more comments (2 replies)
[-] MJBrune@beehaw.org 11 points 11 months ago

English changes very quickly. New words come out every year. Some programming languages takes years for updates.

[-] Doug@midwest.social 7 points 11 months ago

Also that person may have known what you meant, but another might not and may have any number of reasons for not asking.

Better communication skills are a worthwhile goal and there's no good reason to not learn and grow.

load more comments (7 replies)
[-] ipkpjersi@lemmy.ml 3 points 11 months ago

Except for front-end technologies like JavaScript where there's a new framework every week lol

[-] CanadaPlus@lemmy.sdf.org 2 points 11 months ago

Yeah, but what percentage of normal speech is made up by words under 20 years old?

load more comments (2 replies)
load more comments (1 replies)
[-] sj_zero@lotide.fbxl.net 2 points 10 months ago

fr fr no cap

[-] mojo@lemm.ee 16 points 11 months ago

It's actually "their'yre" dumby, learn ur words

[-] reflex@kbin.social 18 points 11 months ago* (last edited 11 months ago)

It’s actually “their’yre” dumby, learn ur words

SOme morans should of staid inn School!!

[-] interolivary@beehaw.org 3 points 11 months ago

get a brane morans

[-] PP_BOY_@lemmy.world 13 points 11 months ago
[-] milicent_bystandr@lemm.ee 3 points 11 months ago

No, you're grammerer

load more comments (1 replies)
[-] AnUnusualRelic@lemmy.world 9 points 11 months ago
[-] _Gandalf_the_Black_@feddit.de 5 points 11 months ago

However, real grammar and prescribed textbook grammar are two different things.

load more comments (3 replies)

Someone Renames your objects "because they sound better that way"...

[-] samus12345@lemmy.world 3 points 11 months ago

I use incorrect grammar all the time on Lemmy because I'm writing colloquially, comma splices are my biggest offense.

[-] Hotzilla@sopuli.xyz 3 points 11 months ago

Fuck me, never touch my code, it is perfect.

[-] hemko@lemmy.dbzer0.com 11 points 11 months ago

Brother you don't even remember what it does, how and why after 3 days

[-] Hotzilla@sopuli.xyz 4 points 11 months ago

Of course, but because it is perfect, no one, not even myself, needs to fix/modify/extend/understand

/s

load more comments
view more: next ›
this post was submitted on 26 Oct 2023
922 points (96.5% liked)

Programmer Humor

19208 readers
1230 users here now

Welcome to Programmer Humor!

This is a place where you can post jokes, memes, humor, etc. related to programming!

For sharing awful code theres also Programming Horror.

Rules

founded 1 year ago
MODERATORS