Make private methods final?

JavaFinal

Java Problem Overview


Is it beneficial to make private methods final? Would that improve performance?

I think "private final" doesn't make much sense, because a private method cannot be overridden. So the method lookup should be efficient as when using final.

And would it be better to make a private helper method static (when possible)?

What's best to use?

  private Result doSomething()
  private final Result doSomething()
  private static Result doSomething()
  private static final Result doSomething()

Java Solutions


Solution 1 - Java

Adding final to methods does not improve performance with Sun HotSpot. Where final could be added, HotSpot will notice that the method is never overridden and so treat it the same.

In Java private methods are non-virtual. You can't override them, even using nested classes where they may be accessible to subclasses. For instance methods the instructoin to call privates is different from that used for non-privates. Adding final to private methods makes no odds.

As ever, these sort of micro-optimisations are not worth spending time on.

Solution 2 - Java

private static Result doSomething(), if this method is not using any instance variables. In any case making them final makes no sense since the accessor is private.

Solution 3 - Java

No. It will not. private methods are not inherited. So making them final is a moot point. Also note that you should not make methods final for performance. JVM is smarter than that. This kind of optimization is not much useful. You should make things final, private, private, protected, private, etc based on the semantics and design.

Solution 4 - Java

Marking a private method as final does not change anything but it might confuse junior developers looking at your code. Keep it simple.

Solution 5 - Java

The IBM Developer works: Java theory and practice: Is that your final answer? article is an oldie but goodie about using the final keyword in Java:

Solution 6 - Java

By making method final we can not override the method. Since Private methods cant be accessed outside the class. There is no sence of making the method final and private. It may hit the performance.

Attributions

All content for this solution is sourced from the original question on Stackoverflow.

The content on this page is licensed under the Attribution-ShareAlike 4.0 International (CC BY-SA 4.0) license.

Content TypeOriginal AuthorOriginal Content on Stackoverflow
QuestiondeamonView Question on Stackoverflow
Solution 1 - JavaTom Hawtin - tacklineView Answer on Stackoverflow
Solution 2 - JavaBinoj AntonyView Answer on Stackoverflow
Solution 3 - JavaChandra PatniView Answer on Stackoverflow
Solution 4 - JavaAlberto ZaccagniView Answer on Stackoverflow
Solution 5 - JavaDoug PorterView Answer on Stackoverflow
Solution 6 - JavagiriView Answer on Stackoverflow