tags:

views:

249

answers:

6

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()
+4  A: 

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.

Chandra Patni
+7  A: 

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.

Binoj Antony
+3  A: 

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

Alberto Zaccagni
+1  A: 

Here is an oldie but goodie about using the final keyword in Java:

http://www.ibm.com/developerworks/java/library/j-jtp1029.html

Dougman
Man I wish C# had a final keyword. Instead it's got const, readonly and sealed. bleh.
Matt Greer
A: 

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.

giri
It will not hit the performance, compilers are smarter than this
Alberto Zaccagni
+2  A: 

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.

Tom Hawtin - tackline
+1 for the whole and especially the last sentence.
Pascal Thivent