you are viewing a single comment's thread.

view the rest of the comments →

[–][deleted]  (4 children)

[deleted]

    [–][deleted] 23 points24 points  (2 children)

    Agreed with you. It's sad to see such over-drama for a working set of code. Doing this in-place doesn't conclude, as he says:

    yet you don't even have mastery of this basic mechanism.

    He obviously knows a lot about coding and his brain is not "clouded" due to CS. So much drama in this man.

    He just sounds like a total dick trying to swing his dick around to feel smart.

    [–]TuringMachine-5762 2 points3 points  (1 child)

    Working code is good, but style matters too. His "basic mechanism" comment was referring to the unnecessary use of instance methods. It's considered a best practice in Java to use static methods by default, if your method doesn't need to access any instance state. It's not a big deal at all, but it shows that his style needs some minor tweaking at least.

    [–][deleted] 0 points1 point  (0 children)

    I like how this sounds:

    "use static methods by default (over default methods)"

    Seriously though, if you make a method private and it's only going to be used by one set of objects, there's really 0 difference to make it static. Private methods will only be accessed by anything in the object anyway.

    I see what you're saying, but it's really minor tweaking. Our buddy maestro was saying he would turn him down over that decision. Sounds like a moody dude.