Jump in the discussion.

No email address required.

I was consulting at a place, there was a very bad programmer whose code looked sort of like this

const arrayIneed = [];

const arrayIdontNeed = firstArray.map(item => {

if(item.hasProp) { arrayIneed.push(item); }

});

return arrayIneed;

the above is very much a cleaned up and elegant version of what he would actually push into the repo.

he left for a competitor in the same industry, this was at the second biggest company for the industry in Denmark and he left for the biggest company - presumably he got a pay raise.

I asked the manager after he was gone, one time when I was refactoring some code of his - which in the end just meant throwing it all out and rewriting from scratch - why he had been kept on so long, and the manager said there were some layoffs coming up and he would have been out with those but because of the way things worked it didn't make sense to let him go earlier.

!codecels Check this guy out :marseyemojirofl:

Jump in the discussion.

No email address required.

:marseydisgust: I worked with a guy who would do C# code like

foreach (object obj in items)
{
    object MyNewObject = new object();
    MyNewObject.Property1 = obj.Property1;
    MyNewObject.Property2 = obj.Property2;
    MyNewObject.Property3 = obj.Property3;
    MyNewObject.Property4 = obj.Property4;
    MyNewObject.Property5 = obj.Property5;
    MyNewObject.Property6 = obj.Property6;
    MyNewObject.Property7 = obj.Property7;

    -- Do something with MyNewObject that you could have just done with obj
}

This fucker had been to college and spent 20 years working with C# and he has this flaky concept of how loop variables work. He also prefixed all variable names with My like an utter noob.


https://i.rdrama.net/images/17310210407557678.webp https://i.rdrama.net/images/17323179881945593.webp

Jump in the discussion.

No email address required.

Seems like he doesn't understand cloning or copying either

Jump in the discussion.

No email address required.

He also had an async method that calls a web service, saves results to database then fetches them back from the database and displays them on screen. He didn't know how to properly await the results of the DB operation so he just put sleep(5000) before querying the results from the DB and displaying them. If that ever wasn't long enough and people saw error messages from trying to display results that hadn't been saved yet, he'd add another 1000 to the sleep command and close the ticket.

:marseyspecial:

By the time I took over the code it had increased to an 8 second sleep and this was a web service call that data entry staff would be using constantly.


https://i.rdrama.net/images/17310210407557678.webp https://i.rdrama.net/images/17323179881945593.webp

Jump in the discussion.

No email address required.

If writing bad functions was the worst thing I saw I would never complain again. Our jeets will block entire projects because we don't integrate with their completely useless and r-slurred systems. One time I said "who asked for this?" and they reported me to my manager. They have been telling us for 2 years that their projects will solve our problems and the jeet management still actually believes them and gets mad if we try to make progress. "Why are you doing this? <Project we've been hearing about for years that doesn't exist> will do this. Please sync with Arjun" (Arjun lives in India and wants to meet at an IST friendly time. The meeting has 8 jeets I don't recognize)

Jump in the discussion.

No email address required.

The meeting has 8 jeets I don't recognize

Yeah, any outsourcing company that bills by the hour loves to send countless useless people to every call just to maximise billing.

Jump in the discussion.

No email address required.

They aren't outsourced. We work for the same global company and they don't make any extra money for doing this

Jump in the discussion.

No email address required.

Just time wasting then

Jump in the discussion.

No email address required.

Leave me alone

Jump in the discussion.

No email address required.

The meeting has 8 jeets I don't recognize

wtf is with that. I s2g they add like eight people to a group chat when I'm asking one person a question.

Jump in the discussion.

No email address required.

>Script crashes with this specific input...

Jeet solution:

def run():
    try:
        return get_results()
    except:
        return None

Does not crash any more, ticket closed

:#marseytunaktunak:

Jump in the discussion.

No email address required.

For a sec i thought your complaint was about using a new list to filter another (got me sweating, i guess you can do reduce?) but holy moly, why.. Chatgpt?

Jump in the discussion.

No email address required.

The only thing I can think of is that it was some debugging thing he forgot to rewrite after he fixed the issue.

Jump in the discussion.

No email address required.

Link copied to clipboard
Action successful!
Error, please refresh the page and try again.