this post was submitted on 21 Jan 2024
1293 points (99.5% liked)

Programmer Humor

19594 readers
1021 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
 
top 50 comments
sorted by: hot top controversial new old
[–] marcos 84 points 10 months ago (3 children)

As long as their salary keeps increasing, I'd say go for it!

[–] [email protected] 16 points 10 months ago (2 children)

You only hurt yourself down the line. My last job had not improved their own product, processes, tools or frameworks, so everything was still stuck in the 90s. Their product was build on an discontinued an proprietary database and server system you never heard about, jQuery UI from 10 years ago and other BS.

However if you don't upskill yourself in this situation you will be unemployable in the future, because all other employers demand modern technologies, git, docker, unit testing etc., which I was yelled at in meetings for suggesting it.

[–] [email protected] 24 points 10 months ago (6 children)

git is considered modern?

I don't mean to bash git but I'd have assumed git is utilized in some capacity in every dev environment.

[–] [email protected] 14 points 10 months ago* (last edited 10 months ago) (2 children)

The Lead Dev/team Lead was quite arrogant and in his own mind the worlds best developer who had all the answers. If some technology or software was not written by him or already existed in the 90s it was "useless" and not fit for the company (without him having looked at it or the docs). If asked why we would not use X which was out for years, well maintained, had no critical bugs would solve problem Z we where having, he would reply "because i said so" and insist in writing out own variant - which ended up having 10% of the features, 10 times the bugs, terrible UI and would take months to develop.

When support repeatetly told him that users had issues with feature X because the only error message on a 10 fields forms page was "Error", he would respond that this is a user problem, the end user is clearly stupid (despide used in a field where you need to study for years) and that support must hold training sessions so the users can "learn" how to use his product.

As such, the company would reject git and instead email each other files and changes.

Each meeting felt like living inside a Dilbert cartoon.

[–] [email protected] 9 points 10 months ago* (last edited 10 months ago)

And he can still do that, because? Friend in management or what?

load more comments (1 replies)
[–] fsr1967 10 points 10 months ago (1 children)

There are other version control systems out there, and have been for decades. So yes, I would consider git to be modern.

[–] ChickenLadyLovesLife 8 points 10 months ago

It's more modern than Visual Source Safe, that's for sure. I kind of miss the days of coworkers leaving for two-week vacations and forgetting to check their shit in first. It was a built-in excuse for the rest of us to not do anything and blame it all on vacation boy.

[–] vaknin 8 points 10 months ago (1 children)

I don’t mean to bash git

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

pun wasnt intented.
But now I kinda have to commit...

[–] [email protected] 4 points 10 months ago (1 children)

Git wasn't used all that much in the 2000s. As far as I know it became popular in the 2010s (though it was always a thing in some circles I think) and then just supplanted almost everything else.

Also keep in mind some shops tend to follow larger tech companies (microsoft, etc.) and their product offering. So even new products might not have been on git until MS went in that direction.

[–] [email protected] 4 points 10 months ago (1 children)
[–] [email protected] 4 points 10 months ago (1 children)

Takes time to become ubiquitous.

load more comments (1 replies)
load more comments (2 replies)
[–] [email protected] 7 points 10 months ago (1 children)

all other employers demand modern technologies

There are a lot of employers that'll throw good money at you for maintaining and extending their outdated crap. Have you ever considered learning COBOL?

[–] [email protected] 11 points 10 months ago* (last edited 10 months ago) (1 children)

No wonder COBOL programmers are paid a lot, because what would be a 1-liner for "hello world" in other languages looks like this in Cobol:

IDENTIFICATION DIVISION.
PROGRAM-ID. IDSAMPLE.
ENVIRONMENT DIVISION.
PROCEDURE DIVISION.
    DISPLAY 'HELLO WORLD'.
    STOP RUN.

This is already $6000 worth of code right there!

[–] [email protected] 5 points 10 months ago (1 children)

I wonder how much open source COBOL is out there for LLMs to harvest.

[–] [email protected] 6 points 10 months ago* (last edited 10 months ago)

The thing people always overlook is that these legacy systems are only still running because they're super important. Nobody's hiring a junior COBOL dev to maintain NORAD, and hopefully nobody's contemplating putting ChatGPT in charge either.

The move if you want this kind of job is to learn a language that's not quite a dinosaur yet, and have 20 years experience in 20 years. Perl or PHP maybe.

[–] [email protected] 5 points 10 months ago

My first thought!

load more comments (1 replies)
[–] AeonFelis 59 points 10 months ago (2 children)

After so many years in this company, lots of the unmaintainable code I have to deal with is either my own fault, or the fault of someone I used to work with but and now they left and I'm the one who has to apologize for their code.

If I move to a different company, 100% of the unmaintainable code I'll have to deal with there will be someone else's fault.

[–] [email protected] 28 points 10 months ago

In the industry we call this responsibility load balancing

[–] [email protected] 20 points 10 months ago (1 children)

And managers don't like it when you explain that the code is a unmanageable mess because they put a deadline on every goddamn thing and never pay off technical debt.

At a new place you can honestly say "the code is kinda a mess, it needs a bunch of work" and the manager can just assume it was because the last guy didn't know what he was doing and not because of their own shitty management.

[–] [email protected] 8 points 10 months ago (1 children)

To be honest, sometimes shit code is 100% the Devs fault. I've witnessed it happen with other teams in my own company.

Let's just say it was unavoidable to report it

[–] [email protected] 4 points 10 months ago (1 children)

Management could implement a code review process to avoid this.

Software development isn't a brand new field anymore. Most problems are well known and therefore have well known solutions. So it pretty much always comes down to management not wanting to implement the known solutions to the problems because its easier to blame the devs.

load more comments (1 replies)
[–] [email protected] 58 points 10 months ago (7 children)

Sometimes a fresh pair of eyes on a code base can reveal some opportunities for big improvements in maintainability 😜

[–] [email protected] 34 points 10 months ago (1 children)

Ahahhahhahha. Ha.....ha. Ahem.

Sorry. The idea that any of the opportunities for improvement at my last "job A" code base might need "revealed" struck me as really funny.

[–] [email protected] 9 points 10 months ago

Sometimes there's an opportunity to delete it and start again 😜

[–] [email protected] 23 points 10 months ago* (last edited 10 months ago)

Sometimes it takes a new dev coming in for management to give the greenlight for a major overhaul. It’s shitty, but it’s true.

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

Also new people are still motivated to change stuff. They are not yet worn down by bureaucracy.

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

Opportunity is my managers favorite word. I even got him an ink stamp that said “certified opportunity”

load more comments (3 replies)
[–] [email protected] 42 points 10 months ago (1 children)

Recently switched jobs from maintaining a 15 year old Windows Forms .NET Framework legacy codebase.

At the new job we stick to Clean Architecture, use unit and integration tests, have a code generation tool, actually make nice use of generics and use dependency injection. Also agile processes, automatic build tools, whatever. The difference is night and day and I'm so glad my ex boss fired me because I told him he's an asshole and his codebase is shit.

[–] LeafOnTheWind 12 points 10 months ago

My first job out of college I have been able to see a steady improvement in the codebase. A little while ago I had to go back to an old tag and was horrified with what it used to be and impressed how much it improved.

[–] [email protected] 32 points 10 months ago* (last edited 10 months ago) (2 children)

You have to listen to your heart, at least once in your career, to learn that grass on the other side is covered in just as much dog shit as it is over here.

[–] tym 39 points 10 months ago

"Maybe the grass is greener on the other side because you're not over there fucking it up."

-Abraham Lincoln

[–] Smoogs 13 points 10 months ago* (last edited 10 months ago) (1 children)

I’ve known people who do this several times in a year. One even came back to his old job, just to leave it within months to go to a new one, brag about how much better it is. He moved on from that job too within a year.

Might just be the entire industry has reached enshittification in more than one way.

[–] [email protected] 11 points 10 months ago* (last edited 10 months ago) (1 children)

To me, a corporation cannot maintain quality code because requirements are ill defined, and there is no “done” state. With those two conditions present, unable to be changed, it’s not possible to form a coherent codebase. Those who try will make things worse, because their abstractions won’t fit in a year or two.

This is exactly the “messy code” people then leave behind. Bad code can come about for other reasons too, of course, but this is one of the more annoying reasons, because someone wrote it with self-righteousness, as if they were the only people to truly SEE the problem. Sigh.

It’s fine, this is how enterprise works. You can learn to navigate and make a living from it. You MUST internalize and accept that it is NOT the same as maintaining code for an open source library or whatever people think it’s going to be.

[–] Smoogs 4 points 10 months ago

because someone wrote it with self-righteousness

Usually a call sign of someone who hasn’t been really entrenched with bad code to understand their foolishness in comparison.

I’ve only seen people hold that idea if :

  1. New and amateurish, I give them a chance cuz they might learn. But let them learn.
  2. Someone who’s only ever worked in maybe two places for very long lengths of time, given way too much power too early, people threw around ‘genius’ too eagerly and these people guard their code like a watch dog likely because it’s so fragile a simple ‘()’ in a string will bust everything . No one else can work on it and the only way you can fix it is the moment they leave. They will not learn. You can only hope the eye of Sauron will stop looking in your direction.
[–] [email protected] 28 points 10 months ago (1 children)

Just rewrite it with 80% functionality and force migrations on the users. Once the remaining 20% "edge cases" that require serious effort hop to the next job - where you where hired to "maintain" such a system and "just add a small feature here and there". Ooops.

[–] [email protected] 13 points 10 months ago

Reddit: You're hired!

[–] [email protected] 25 points 10 months ago (2 children)

Project A: Has 6 different implementations of the same complex business logic.

Project B: Has one implementation of the complex business logic... But it's ALL in one function with 17 arguments and 1288 lines of code.

"The toast always lands the buttered side down."

[–] [email protected] 7 points 10 months ago (2 children)

Only 1288 lines? Can I raise you a 6000+ lines stored procedure that calls to multiple different sql functions that each implements a slightly different variation of the same logic?

[–] [email protected] 4 points 10 months ago (2 children)

Are there triggers in the sql database? It's too easy otherwise

load more comments (2 replies)
load more comments (1 replies)
[–] [email protected] 6 points 10 months ago (6 children)

Project B is just called neural network

load more comments (6 replies)
[–] [email protected] 19 points 10 months ago (2 children)

Learning to deal with "unmaintanable" codebases is a pretty good skill. It taught me good documentation and refactoring manners. It's only a problem for you if management does not accept that their velocity has gone down as a result of tech debt pilling up.

Code should scream it's intent (business-wise) so as to be self-documenting as much as possible As much as possible is not 100%, so add comments when needed. Comments should be assumed to be relevant when written, at best. Git comment should be linked to your work ticket so that we can figure out why the hell you would do that, when looking at the code file itself. I swear some people seem to think we only read them in PRs (we don't). Overall concepts used everyday, if they need to be reexplained, should probably be written down (at least today's version). Tests are documentation. Often the only up to date one?

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

This right here. Get good at navigating code of questionable quality that you didn’t write. If you can’t do it, start questioning your tools, and mastery of those tools. For the big boy jobs, you should be working with existing code much more than writing new code. Learn to get excited by tweaking existing systems with a few well placed, well researched changes, instead of being The Asshole that adds a new abstraction wart.

[–] Smoogs 4 points 10 months ago

I’ve known influential assholes who poopood commentating as if it’s only a superficial job.

I hate those people.

[–] [email protected] 13 points 10 months ago

Switching jobs can be worth it just for the change up.

[–] [email protected] 12 points 10 months ago

Can we arrange some swaps? I'm not getting paid enough and neither are you.

[–] [email protected] 9 points 10 months ago* (last edited 10 months ago)

I left a company when management decided to discontinue a product right after we finally made its code more maintainable.

Had a look at the product that would replace it, and it was a bigger mess from what we started with.

[–] IndustryStandard 9 points 10 months ago

Some intern writing spaghetti which a company is going to be based off

[–] [email protected] 4 points 10 months ago* (last edited 10 months ago)

I see it as a win, I love refactoring bad code . Just feels so nice afterwards.

Especially when you can remove thousands of lines of duplicated ui code.

That's so much better than writing new code

load more comments
view more: next ›