10
Wisward
5y

Starting a project with someone that programs like an idiot on steroids. Like, why are you using 20 switch statements to set a variable instead of of just a single line linear expression. I removed hundreds of lines from one of this ass's files. I've never seen such a complicated mess of garbage.

Comments
  • 4
    Could you share some of his sins with us? I'm curious.

    Have you tried talking to him/her/it?
  • 1
    Maybe because there are 20 ways to set the variable?

    You are not giving us data to judge here.
  • 1
    @mundo03 @mundo03 he could simply use "double levelMultiplier = multiplierSeed * 0.025 + 0.875;"

    This reduces the code by 12 times that of my replacement. On top of that, you could provide documentation of what levelMultiplier does in the first place....(this project has been rewritten and is now half the size as it originally was just from his files alone.)
  • 0
    @Wisward I'm forgetting to mention the lack of breaks and default case. There are more of these exact same cases.
  • 0
    @Wisward ah well, yes this is shit.
    I was thinking each expression was complete un related to each other.
Add Comment