this post was submitted on 07 Dec 2023
955 points (100.0% liked)

196

16437 readers
1493 users here now

Be sure to follow the rule before you head out.

Rule: You must post before you leave.

^other^ ^rules^

founded 1 year ago
MODERATORS
 
you are viewing a single comment's thread
view the rest of the comments
[–] [email protected] 41 points 11 months ago* (last edited 11 months ago) (4 children)

Definitely. Horror story time.

We had an outside contractor bring us some code once that was thousands of lines of Python to do a very simple job. I was perplexed. I dove in to figure out what the problem was, and somehow I was looking at the most Java-esque Python code I could imagine. What’s worse is that he implemented his own “Java style” property getters and setters for all the Python classes, which obviously aren’t needed because you can simply access properties directly. In the end I took an 80 line snippet of his code (which actually did the work we needed), swapped out all the getters and setters, and deleted all the rest.

[–] [email protected] 20 points 11 months ago

This is pretty common with outside contractors.

We just come in, say we'll pay them x dollars and they give us code that passes the test. But that code will not at all align with any prior patterns.

I absolutely know I'm guilty of it when I do freelancing. Sorry.

[–] [email protected] 14 points 11 months ago (1 children)

You can always tell when someone's been a career contractor because they never adhere to any of the established patterns/styles in the codebase.

[–] [email protected] 3 points 11 months ago

I disagree. Good career contractors should learn to write in the code style of the project. And the real pros do.

[–] [email protected] 8 points 11 months ago

That's not to say that python coding habits are the best either -- certainly they're terrible when translating outside of python (most of the time). And even within python, someone who is used to with only the base modules will write it differently than writing PyQt and still completely different than someone doing numpy code... because the styles of coding of the underlying system change your coding mode. Like, my variables are all CamelCase when doing user interfaces with Qt because it makes sense there, stylistically.

[–] [email protected] 6 points 11 months ago

Did you pay by the SLOC?