Is there any other method of stopping inheritance of a class apart from declaring it as final or by declaring its constructor as private?
-
What's wrong with final?– lc.Jan 16, 2009 at 17:03
-
could you try to explain why you are trying to avoid the use of the final keyword?– shsteimerJan 16, 2009 at 17:14
-
1Probably because there's a framework involved that creates subclasses.– krosenvoldJan 16, 2009 at 17:20
-
1@dtsazza There are things a framework could do that you'd normally not let user code do; a framework could change the protection level of a private constructor at run-time. AFIK you cannot mess with final though...– krosenvoldJan 16, 2009 at 17:30
-
6Why is that everyone is answering: "Use final" when the question it self says "Other than final..." I think they are right, but the point is, that is not the question. I guess we are getting use to "subjective & argumentative" questions that we are missing the point.– OscarRyzJan 16, 2009 at 17:34
8 Answers
A comment
//Do not inherit please
-
1I actually believe that's the best solution ever. You could even describe why it shouldn't to be inherited from.– vavaJan 16, 2009 at 17:09
Two more options:
make each method final, so people can't override them. You avoid accidental calling of methods from subclass this way. This doesn't stop subclassing though.
put check into constructor for class:
if (this.getClass() != MyClass.class) { throw new RuntimeException("Subclasses not allowed"); }
Then nobody will be able to instantiate subclass of your class.
(Not that I suggest using these techniques, it just came to my mind. I would use final class and/or private constructor)
-
-
Except if the finalize method is overriden, the instance can be recovered. Jan 16, 2009 at 17:17
-
1+1 I didn't think there would be a real answer to this question, but I guess this is as close as you're going to get.– Tim FreyJan 16, 2009 at 17:40
-
@Chris Lively: Constructor from super class must always be called. You can 'override' constructor (there is no such thing as overriding constructor though, as constructor is not inherited), but you cannot 'skip' constructor from super class. Jan 19, 2009 at 8:23
- Use final
- Use private constructors
Use a comment:
// do not inherit
Use a javadoc comment
- Make every method final, so people can't override them
Use a runtime check in the class constructor:
if (this.getClass() != MyClass.class) { throw new RuntimeException("Subclasses not allowed"); }
-
1The comment idea really does not make much sense. A JavaDoc comment makes a lot more sense, but is still not foolproof.– eleven81Jan 16, 2009 at 17:29
-
Final was created to solve this problem.
Make your constructors private and provide factory functions to create instances.
This can be especially helpful when you want to choose an appropriate implementation from multiple, but don't want to allow arbitrary subclassing as in
abstract class Matrix {
public static Matrix fromDoubleArray(double[][] elemens) {
if (isSparse(elements)) {
return new SparseMatrix(elements);
} else {
return new DenseMatrix(elements);
}
}
private Matrix() { ... } // Even though it's private, inner sub-classes can still use it
private static class SparseMatrix extends Matrix { ... }
}
Using final
is the canonical way.
public final class FinalClass {
// Class definition
}
If you want to prevent individual methods from being overridden, you can declare them as final instead. (I'm just guessing here, as to why you would want to avoid making the whole class final.)
I'd have to say it's typically bad form. Though there are almost always cases where something is valid, I'd have to saying stopping inheritance in an OO world is normally not a good idea. Read up on the Open-Closed Principle and here. Protect your functionality but don't make it impossible for the guy who comes in and supports it...
-
3Inheritance is way overrated. Unless you design your class for subclassing, you better avoid it. Jan 16, 2009 at 17:13
-
1+1 to Peter's comment. As I've said elsewhere (many times!) I wish classes were sealed (C#) / final (Java) by default. Inheritance is great, when you really need it - but it can be a pain if you just use it without thinking carefully about it. Jan 16, 2009 at 17:15
-
1"Design and document for inheritance or else prohibit it." -- Josh Bloch, Effective Java Item 17– Michael Myers ♦Jan 16, 2009 at 17:17
-
1Read Michael Feather's book "Working Effectively with Legacy Code". It's gotten me through a number of brownfield projects and things like classes and methods you can't touch are major pain points. Even if you didn't intend for it, feature XYZ will need it. All about future proofing...– Mark GJan 16, 2009 at 17:18
-
1@Jon Skeet: I guess in my experience I've just not seen it. Take any of the dialogs in the .NET BCL (Open/Save/Print), can't do a danged thing with them because they're sealed, major pain. But this discussion should probably be in a different question with the tag of 'Subjective'– Mark GJan 16, 2009 at 18:35
Without using a final class, you can basically make all the constructors private:
public class A {
private A() {} //Overriding default constructor of Java
}
Which although will also make this class abstract-ish by disallowing creating an object of this class, yet as any inheritance requires super();
in the constructor, and because the constructor is private, a compilation error will be the maximum you can get when one tries to inherit that class.
Yet, I would recommend using final instead as it is less code and includes the option of creating objects.