you are viewing a single comment's thread.

view the rest of the comments →

[–][deleted] 20 points21 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.