110

So I was code reviewing this guy's code before merging into the master branch.... and then I FOUND THIS FUCKING CODE, WHAT THE FUCK!!!!!!!!!!!!!!!! What the fuck is this ???

Comments
  • 26
    *Screaming in terror and confusion*
  • 10
    @SoldierOfCode yea , I can feel this too...and I already screaming...
  • 4
    could be an override.
  • 13
    I've seen enough async shit for one day, and you go and throw this... at me.

    Fuck me dead @rutee07, I don't want to live anymore.
  • 4
    @C0D4 Bro, chillax..... we are on the same boat. I begin to question my life when I saw this in the PR.
  • 10
    @johnmelodyme

    Hahaha, sorry, it's been one of those days where nothing you do, no matter what the fuck you do, breaks anyway.

    Seriously wtf is Y.R.T ๐Ÿ˜ง

    some people don't deserve to write code when there's absolutely no way in knowing what it's doing.
  • 5
    @C0D4 Exactly What I was thinking about what the F is Y.R.T as he declare it in .gitignore. which I am confused even more .

    Technically he ignored it, I can't read the file because it doesn't exist.
  • 5
    @johnmelodyme wait what!

    So we call this random Y class... well I'm assuming it's a class, and prevent it from being added to the code base ๐Ÿคจ

    Am I missing something or does this guy not know how things work?
  • 4
    @C0D4 I also think Y must be a public class. I am confused too, why he declare the module in .gitignore . he add this line K/ in there, I assume how many modules are ignored.

    I believe the K is a folder if I am not wrong.
  • 9
    @johnmelodyme if K/ is in the git ignore then yea it's a folder somewhere.

    But how you get from K to Y.R.T.t(), I pray for you

    Hmm.

    So, K is a folder
    Y is a module or namespace
    R is a class
    T is a method
    Leaving us with that little t
    Hmm ๐Ÿค” a setter maybe?

    But either way, fuck this guy!
  • 7
    @C0D4 I told him to refer to the documentation I wrote about the company's coding guideline.

    I clear explain that the delaration of the variable must be clear something like let customer_id = 'something; ' for methods :

    function get_customer_details(param1, param2) {}

    I don't know how he understand his code.

    I told him to redo it.
  • 35
    I love the "// this is a function".
  • 5
    @atheist that's unnecessary though
  • 13
    @atheist
    // good luck

    Would have been more accurate to what this is trying to do ๐Ÿ˜
  • 9
    @johnmelodyme That's why I love it๐Ÿ˜‰
  • 5
    Did he put the brackets around k just to make things clearer ๐Ÿ˜ณ
  • 13
    "What the fuck is this ??"
    It is a function.
  • 7
    Why bother with 'payment_successful', just replace it with '๐Ÿ˜'
  • 6
    Did they fucking made a patch for the minified version of something? Wtf
  • 3
    Bro do you even jsdoc
  • 3
    Damn bro! When did you go to Coding Hell?
  • 8
    that dev be humming the alphabet song while coding
  • 3
    It’s d clean code mortal enemy for sure haha
  • 11
    Ah yes, the You aRe Truly trash design pattern, used to annotate the programmer himself as a retard.

    It's not very popular, but it has its uses
  • 4
    fire him :D unless he is a junior and did this mistake first time.
  • 5
    “This is a function”
  • 2
    But what if not i?
    No return then?
    ๐Ÿค”
  • 2
    @atheist that's how you know that this code is actually a work of art
  • 4
    @brrrTrrrDrrr I guess gore is a form of art.

    Nice username btw.
  • 5
    Copy/paste from an optimized code I presume.
  • 2
    @wf3d If this is the case , im even more confused...
  • 4
    Looks like minified code, what the hell
  • 7
    Code is obviously copy-pasted from some minified script

    But... That comment. That was likely added by him. Which is scary.
  • 3
    And that is why a good mathematician isn't neccessarily also a good coder.

    Get HR to stop using algorithm-related interview questions!
  • 6
    Felt this, there was a guy at a company I worked at who took his revenge for the team not writing code in x86 by writing C like this. Single letter vars and methods, wasn't his fault or problem if the others weren't keen on playing detective every time they wanted to work on his code.

    Dunno the context but I'd report this or give the guy a chewing out yourself, if you can. There is zero benefit to code that no fucker can even read, especially if it's dealing with actual money.
  • 3
    @MM83 even he will forget
  • 11
    Is this part of some minified code?
    If not, you might want to minify your dev team a bit.
  • 3
    I'm intrigued and confused.
  • 2
    I don't get the confusion?
    Like others noted this is a function and if i then yrtt payment succesful.
    Yolo right?
  • 6
    @Root it wasn't a minified code. He explained to me he was lazy to name the function and class full name and also save space.

    Weird.
  • 6
    @johnmelodyme I'd like to punch that person
  • 4
    @melezorus34 If you are in my country, I will fetch you all the way to his home. and gladly be my guest.
  • 5
    I’d be too lazy to pay him, and also save space on the team for someone better suited. :)
  • 5
    @Root Wow will be my excuse, maybe I can propose to my manager, hahaha
  • 5
    @Root The funny thing is he frequently told us that he worked for Facebook previously. (Well , I don't see any company named Facebook in Malaysia and in order to work with Facebook , he need to work in the US, and he doesn't have a passport. )
  • 5
  • 4
    @Root Yeap that's what @TeachMeCode will react too.
  • 3
  • 5
    Looks like something GitHub Copilot would come up with.
  • 4
    I'd simply add a comment to this line of code, "Fix variable/method naming, write cleaner code" and not approve the PR
  • 3
    @Angry-dev he is junior actually .
  • 2
    @vergil and GitHub Copilot reaction will be like ๐Ÿ˜ฑ๐Ÿ˜ฑ๐Ÿ˜ฑ๐Ÿ˜ฑ๐Ÿ˜จ๐Ÿ˜จ๐Ÿ˜จ๐Ÿ˜จ๐Ÿ˜ฐ๐Ÿ˜ฐ๐Ÿ˜ฐ๐Ÿ˜ฐ๐Ÿ˜จ๐Ÿ˜จ๐Ÿ˜จ๐Ÿ˜ฑ๐Ÿ˜ฑ๐Ÿ˜ฐ๐Ÿ˜ฐ๐Ÿ˜จ๐Ÿ˜จ๐Ÿ˜ฑ๐Ÿ˜จ๐Ÿ˜ฐ๐Ÿ˜ฐ๐Ÿ˜จ๐Ÿ˜จ๐Ÿ˜ก๐Ÿ˜ก๐Ÿ˜ก๐Ÿคฌ๐Ÿคฌ๐Ÿคฌ๐Ÿคฌ๐Ÿ˜ก๐Ÿ˜ก๐Ÿ˜ ๐Ÿ˜ฑ๐Ÿ˜ฑ๐Ÿคฏ๐Ÿคฏ๐Ÿคฏ๐Ÿ˜ฑ๐Ÿ˜ฑ๐Ÿ˜ฑ๐Ÿ˜ฐ๐Ÿ˜ฐ๐Ÿ˜–๐Ÿ˜–๐Ÿ˜ž๐Ÿ˜ž๐Ÿ˜“๐Ÿ˜“๐Ÿ˜ฅ๐Ÿ˜ฅ๐Ÿ˜ข๐Ÿ˜ข๐Ÿฅฑ๐Ÿคฎ๐Ÿคฎ๐Ÿคฎ๐Ÿฅต๐Ÿคข๐Ÿคฏ๐Ÿ’€
  • 3
    Normally I try to contribute something constructive to threads. Not this time.

    Fuckity fuckity fuck
  • 3
    @C0D4 there is no reason for this to be async as there is no internally awaited code. ๐Ÿ˜ถ

    @johnmelodyme just reject the PR with "insufficiently documented". ๐Ÿคท‍โ™‚๏ธ
  • 3
    @kwilliams I rejected the PR. And asked him directly wtf was going on.
  • 2
  • 3
    This actually looks like production code. Did this genius build the project and copy the minified version of the function over the original?

    What in god's name is this nonsense lmao
  • 0
    @jfgilmore yes I agree.
  • 0
    @RexGalilae I genuinely have no idea what he do.
  • 1
    @johnmelodyme maybe he's doing this to get around not being able to use his favourite library due to it but being approved by the tech lead
  • 1
    Dude you are acting very sus,
    `git blame @RexGalilae+noreply@devrant.com`
  • 1
    @melezorus34
    Lmao nah. Just trying to add some method to this madness
  • 1
    Writing in obfuscated code directly
  • 2
    You sure the guy's name doesn't end in Webpack or something?
  • 1
    Fuckin procrastination
  • 1
    "Payment successful"
    Maybe he just had it and wrote a function to pay himself
Add Comment