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.

[–] Potatos_are_not_friends@lemmy.world 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.

[–] BaskinRobbins@sh.itjust.works 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.

[–] gribodyr@lemmy.ml 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.

[–] troyunrau@lemmy.ca 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.

[–] jasondj@ttrpg.network 6 points 11 months ago

Did you pay by the SLOC?