service.impl/src/main/java/com/hack23/cia/service/impl/action/common/AbstractCommonBusinessServiceImpl.java
Change the visibility of this constructor to "protected". Open
Open
public AbstractCommonBusinessServiceImpl(final Class<T> clazz) {- Read upRead up
- Exclude checks
Abstract classes should not have public constructors. Constructors of abstract classes can only be called in constructors of their subclasses. So
there is no point in making them public. The protected modifier should be enough.
Noncompliant Code Example
public abstract class AbstractClass1 {
public AbstractClass1 () { // Noncompliant, has public modifier
// do something here
}
}
Compliant Solution
public abstract class AbstractClass2 {
protected AbstractClass2 () {
// do something here
}
}