this post was submitted on 28 Nov 2024
520 points (98.7% liked)

Programmer Humor

32720 readers
385 users here now

Post funny things about programming here! (Or just rant about your favourite programming language.)

Rules:

founded 5 years ago
MODERATORS
520
Lean vs. fast (lemmy.ml)
submitted 3 weeks ago* (last edited 3 weeks ago) by MonkderVierte@lemmy.ml to c/programmerhumor@lemmy.ml
 
all 32 comments
sorted by: hot top controversial new old
[–] Skullgrid@lemmy.world 82 points 3 weeks ago (4 children)

TFW you want to do things good, slow and expensive, but management makes you do them fast, cheap and crap.

For your entire career.

Please kill me

[–] NocturnalMorning@lemmy.world 63 points 3 weeks ago (1 children)

Please kill me

No, you still have a PR to review.

[–] _____@lemm.ee 24 points 3 weeks ago* (last edited 3 weeks ago) (3 children)

> someone nitpicks word you used in a variable declaration

> you change it

> someone more senior says the former made more sense

> this goes on for far longer than it should

> eventually you get a real review from someone in your team that identified something actually needs to change

> you change it and re request reviews

rinse and repeat

[–] projectmoon@lemm.ee 8 points 3 weeks ago (2 children)

Had a team lead that kept requesting nitpicky changes, going in a FULL CIRCLE about what we should change or not, to the point that changes would take weeks to get merged. Then he had the gall to say that changes were taking too long to be merged and that we couldn't just leave code lying around in PRs.

Jesus fucking Christ.

There's a reason that team imploded....

[–] lord_ryvan@ttrpg.network 4 points 3 weeks ago (1 children)

Had a colleague who would comment things like “add a newline here” as well as things that were fully his own preference.

That was the only time I closed comments without replying to them or fixing them, without feeling bad.

[–] projectmoon@lemm.ee 4 points 3 weeks ago (1 children)

For stuff like that, it's best to have an auto formatter like checkstyle or something.

[–] lord_ryvan@ttrpg.network 1 points 3 weeks ago

My point exactly! But naw, several others on the team insisted this guy policing others manually is better than putting a linter in the pipeline.

I don't work there anymore, this is one of the (minor) reasons.

[–] UnrepententProcrastinator@lemmy.ca 1 points 3 weeks ago (1 children)

People need to lameduck their code more

[–] zalgotext@sh.itjust.works 2 points 3 weeks ago

People need to reply to those comments with "out of scope" and a link to a new issue that will get buried in the backlog more often

[–] mac@lemm.ee 4 points 3 weeks ago

Don't forget get questioned by your manager/scrum lead as to why its taking so long to get out.

Well, I've had the PR ready for 3 days and the team asked me to make changes today

[–] Skullgrid@lemmy.world 2 points 3 weeks ago* (last edited 3 weeks ago)

you forgot the part where you have to rebase your branch and that causes merge conflicts that were resolved later but somehow still persist.

[–] RagingToad@feddit.nl 7 points 3 weeks ago (1 children)

Change management! :-)

Or, if possible, change employer.

(And I know we're in meme-land, but I always see it as a developer's task to inform of the trade-off between fast and good)

[–] Bougie_Birdie@lemmy.blahaj.zone 7 points 3 weeks ago (1 children)

Every other skilled trade just says "Fast, Right, or Cheap: pick two."

It's not my fault if they always pick fast and cheap

[–] RagingToad@feddit.nl 4 points 3 weeks ago

Okay, sure, do fast. Then:

  • I'll add a refactor task to the backlog filed under "tech debt"
  • please confirm again that you know we are still behind on security updates and that you're ok with it because you are responsible for how I spend my hours
  • I'd like more time to spend on bugs before we lose customers.
  • Also I won't touch that buggy part without taking the time for a rewrite because we did it "fast" per your request and it's so hard to maintain now that it becomes a time sink on every minor change
  • I know we are under time pressure right now, but as a stakeholder I request we plan a few sprints for improving reliability of our product
  • It's not "fun" to work on our code. We might lose developers if we do not address this. We both know the good coders will have no problem finding a new job and you'll end up with the bad ones.
  • Either that, or plan for loss of personnel and the extra time we need for the hiring process and the loss of developer hours
[–] eldavi@lemmy.ml 5 points 3 weeks ago

TFW you want to do things good, slow and expensive, but management makes you do them fast, cheap and crap.

For your entire career.

Please kill me

i got the sense that some people wanted to when i made this same point about this industry in this same community about a week or so ago.

i love the duality of lemmy sometimes. lol

[–] perviouslyiner@lemmy.world 53 points 3 weeks ago* (last edited 3 weeks ago)

TFW "npm install somePackage" adds hundreds of names to your supplier list, some of whom aren't even adults let alone companies, and the policy says that each new supplier needs to go through a thorough vetting process.

[–] hakunawazo@lemmy.world 23 points 3 weeks ago
[–] devfuuu@lemmy.world 20 points 3 weeks ago (2 children)

It's funny because it's true. All those little guys will fuck you over eventually and likely are real trojans.

[–] bitwolf@sh.itjust.works 11 points 3 weeks ago

For that reason I see why companies maintain private mirrors. Heck I want one myself in case any more get lost or removed.

[–] slazer2au@lemmy.world 5 points 3 weeks ago

And each in their own way. Like leftpad or openssl

[–] fossphi@lemm.ee 14 points 3 weeks ago

Nice to meet you, Jia Tan

[–] Earflap@reddthat.com 7 points 3 weeks ago (1 children)

If they wanted me to use a specific tool or lack thereof they should have said that. Instead they said "fix this problem" and instead of writing the entire codebase from the ground up I used the tools that were available to me so I could focus on fixing the problem instead of fixing the fix to fix the fix for the fix of the problem.

[–] aidan@lemmy.world 1 points 3 weeks ago (2 children)

I can't relate to this feeling at all, writing code using a library I've found is almost always the source of bugs. Miscommunication between the library developer and their documentation, or my ability to read the documentation. And that's on top of how many big libraries I've seen with extremely simple exploits. Sadly I have to use a few, but I wince every time I install a package.

[–] Earflap@reddthat.com 3 points 3 weeks ago* (last edited 3 weeks ago) (1 children)

I am NOT writing a database connector unless you add an additional three months to your projects expectations.

I am NOT writing an LDAP connector.

I am NOT writing code to execute shell processes safely.

And I'm sure as hell not writing an XML parser just so I can say I did it without libraries.

JS devs that import libraries for every stupid thing (lpad comes to mind) are bad programmers, but libraries are useful and have their place.

And if my boss doesn't want me using those libraries, they need to specify that in advance or there needs to be a company policy to that effect. Otherwise, I'm solving the problem my way since that's what I'm getting paid to do.

[–] aidan@lemmy.world 1 points 2 weeks ago

Yeah I absolutely agree, my issue is with libraries that do trivial or not particularly useful things.

[–] Boomkop3@reddthat.com 2 points 3 weeks ago

And then a requirement changes, and good luck duct taping it to the darn framework

[–] SanndyTheManndy@lemmy.world 3 points 3 weeks ago

Never run npm audit. You won't like what you see.

[–] aidan@lemmy.world 2 points 3 weeks ago (1 children)

I absolutely cannot relate to using a ton of libraries in production code.

[–] MonkderVierte@lemmy.ml 1 points 3 weeks ago (1 children)

Node, and you'd be surprised.

[–] aidan@lemmy.world 1 points 2 weeks ago

I've written Node code, I just try to limit the number of libraries I use