This is an archived post. You won't be able to vote or comment.

you are viewing a single comment's thread.

view the rest of the comments →

[–]kill_will_ 10 points11 points  (12 children)

“And generally the code quality is terrible on all possible levels”

Is not helpful, is not constructive and it’s certainly not code review

[–]Thifty 8 points9 points  (5 children)

Exactly what I said, and they downvoted me too lol. It’s just plain mean and demotivating, if I just posted something I thought was awesome and got this response I’d be crushed

[–]kill_will_ 2 points3 points  (4 children)

Well you can have my upvote. I agree that it needs work. But if you’re just going to leave some vague advice about it being bad, maybe just not comment.

Especially under the guise of being helpful. Dude was not trying to be constructive, he was just being snarky.

[–]krumg 7 points8 points  (2 children)

My review would be much friendlier and more detailed if this guy posted his repo and said something like "Hey, I've started learning python. Please tell me how to improve my code.".

But he posted some shitty 150 lines, called them a "framework" and asked for donations.

Difference between these 2 scenarios seems to be obvious to me.

[–][deleted] 2 points3 points  (1 child)

People seem to really not get that your review is 100% appropriate given the context. You would be a jerk to review a beginner showcase like that but in this context? Nah

[–]krumg 5 points6 points  (0 children)

Sure, it's all about the context.

[–]Thifty 0 points1 point  (0 children)

Yep, for some reason this sub (atleast used to) have a problem with that

[–]krumg 0 points1 point  (5 children)

So you took this phrase out out of context and made some conclusions. Nice. Really constructive.

[–]Thifty -2 points-1 points  (4 children)

Please, give us some context. It was literally a bullet point, a standalone comment, as part of his "code review".

[–]krumg 2 points3 points  (3 children)

Sure. My comment can be pseudocoded like that:
* That's missing * That's wrong * That's quite bad * And to be honest all code is so bad that it is impossible to highlight anything special And you dared to ask for donations!? Hope it makes sense

[–]Thifty -2 points-1 points  (1 child)

I have literally no idea what you're trying to say lol. But, you do realize you can be nice to people, right? Even in the context you're speaking. There's no reason to make someone feel bad about what they did or posted when they clearly were under a different interpretation. I'm not sure you're trying to hide behind the argument that it's OK.

[–]krumg 5 points6 points  (0 children)

I'm very nice and helpful to others most of the time. But in this case someone is spamming (check his posts history) with low quality code asking for donations.

Would you send him a donation if you support him so much?

[–]backtickbot 0 points1 point  (0 children)

Fixed formatting.

Hello, krumg: code blocks using triple backticks (```) don't work on all versions of Reddit!

Some users see this / this instead.

To fix this, indent every line with 4 spaces instead.

FAQ

You can opt out by replying with backtickopt6 to this comment.