From bbb42a45db2c380dc596d5be1730d878ad346511 Mon Sep 17 00:00:00 2001 From: Bas Leijdekkers Date: Sat, 19 Jul 2014 20:04:19 +0200 Subject: [PATCH] IDEA-126982 (Add an inspection to flag a nonfinal class that only has private constructors) --- .../src/META-INF/InspectionGadgets.xml | 4 + .../siyeh/InspectionGadgetsBundle.properties | 2 + ...OnlyPrivateConstructorsInspectionBase.java | 91 +++++++++++++++++++ ...WithOnlyPrivateConstructorsInspection.java | 33 +++++++ .../ClassWithOnlyPrivateConstructors.html | 9 ++ ...OnlyPrivateConstructorsInspectionTest.java | 49 ++++++++++ 6 files changed, 188 insertions(+) create mode 100644 plugins/InspectionGadgets/InspectionGadgetsAnalysis/src/com/siyeh/ig/classlayout/ClassWithOnlyPrivateConstructorsInspectionBase.java create mode 100644 plugins/InspectionGadgets/src/com/siyeh/ig/classlayout/ClassWithOnlyPrivateConstructorsInspection.java create mode 100644 plugins/InspectionGadgets/src/inspectionDescriptions/ClassWithOnlyPrivateConstructors.html create mode 100644 plugins/InspectionGadgets/testsrc/com/siyeh/ig/classlayout/ClassWithOnlyPrivateConstructorsInspectionTest.java diff --git a/plugins/InspectionGadgets/InspectionGadgetsAnalysis/src/META-INF/InspectionGadgets.xml b/plugins/InspectionGadgets/InspectionGadgetsAnalysis/src/META-INF/InspectionGadgets.xml index 890da15b546b..164c690d4da3 100644 --- a/plugins/InspectionGadgets/InspectionGadgetsAnalysis/src/META-INF/InspectionGadgets.xml +++ b/plugins/InspectionGadgets/InspectionGadgetsAnalysis/src/META-INF/InspectionGadgets.xml @@ -408,6 +408,10 @@ key="class.name.differs.from.file.name.display.name" groupBundle="messages.InspectionsBundle" groupKey="group.names.class.structure" enabledByDefault="false" level="WARNING" implementationClass="com.siyeh.ig.classlayout.ClassNameDiffersFromFileNameInspection"/> + #ref #loc +class.with.only.private.constructors.display.name=Class with only 'private' constructors should be declared 'final' +class.with.only.private.constructors.problem.descriptor=Class #ref with only 'private' constructors should be declared 'final' diff --git a/plugins/InspectionGadgets/InspectionGadgetsAnalysis/src/com/siyeh/ig/classlayout/ClassWithOnlyPrivateConstructorsInspectionBase.java b/plugins/InspectionGadgets/InspectionGadgetsAnalysis/src/com/siyeh/ig/classlayout/ClassWithOnlyPrivateConstructorsInspectionBase.java new file mode 100644 index 000000000000..6d091699ce82 --- /dev/null +++ b/plugins/InspectionGadgets/InspectionGadgetsAnalysis/src/com/siyeh/ig/classlayout/ClassWithOnlyPrivateConstructorsInspectionBase.java @@ -0,0 +1,91 @@ +/* + * Copyright 2000-2014 JetBrains s.r.o. + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ +package com.siyeh.ig.classlayout; + +import com.intellij.psi.PsiClass; +import com.intellij.psi.PsiMethod; +import com.intellij.psi.PsiModifier; +import com.siyeh.InspectionGadgetsBundle; +import com.siyeh.ig.BaseInspection; +import com.siyeh.ig.BaseInspectionVisitor; +import org.jetbrains.annotations.Nls; +import org.jetbrains.annotations.NotNull; + +import java.util.HashSet; +import java.util.Set; + +/** + * @author Bas Leijdekkers + */ +public class ClassWithOnlyPrivateConstructorsInspectionBase extends BaseInspection { + @Nls + @NotNull + @Override + public String getDisplayName() { + return InspectionGadgetsBundle.message("class.with.only.private.constructors.display.name"); + } + + @NotNull + @Override + protected String buildErrorString(Object... infos) { + return InspectionGadgetsBundle.message("class.with.only.private.constructors.problem.descriptor"); + } + + @Override + public BaseInspectionVisitor buildVisitor() { + return new ClassWithOnlyPrivateConstructorsVisitor(); + } + + private static class ClassWithOnlyPrivateConstructorsVisitor extends BaseInspectionVisitor { + + @Override + public void visitClass(PsiClass aClass) { + super.visitClass(aClass); + final PsiMethod[] constructors = aClass.getConstructors(); + if (constructors.length == 0) { + return; + } + for (PsiMethod constructor : constructors) { + if (constructor.hasModifierProperty(PsiModifier.FINAL)) { + return; + } + } + final PsiClass[] innerClasses = aClass.getInnerClasses(); + for (PsiClass innerClass : innerClasses) { + if (isExtendedByInnerClass(innerClass, aClass, new HashSet())) { + return; + } + } + registerClassError(aClass, aClass); + } + + private static boolean isExtendedByInnerClass(PsiClass innerClass, PsiClass superClass, Set visited) { + if (!visited.add(innerClass)) { + return false; + } + if (innerClass.isInheritor(superClass, false)) { + return true; + } + final PsiClass[] innerClasses = innerClass.getInnerClasses(); + for (PsiClass aClass : innerClasses) { + if (isExtendedByInnerClass(aClass, superClass, visited)) { + return true; + } + } + return false; + } + } +} diff --git a/plugins/InspectionGadgets/src/com/siyeh/ig/classlayout/ClassWithOnlyPrivateConstructorsInspection.java b/plugins/InspectionGadgets/src/com/siyeh/ig/classlayout/ClassWithOnlyPrivateConstructorsInspection.java new file mode 100644 index 000000000000..3aeebcd724e8 --- /dev/null +++ b/plugins/InspectionGadgets/src/com/siyeh/ig/classlayout/ClassWithOnlyPrivateConstructorsInspection.java @@ -0,0 +1,33 @@ +/* + * Copyright 2000-2014 JetBrains s.r.o. + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ +package com.siyeh.ig.classlayout; + +import com.intellij.psi.PsiClass; +import com.siyeh.ig.InspectionGadgetsFix; +import com.siyeh.ig.fixes.MakeClassFinalFix; +import org.jetbrains.annotations.Nullable; + +/** + * @author Bas Leijdekkers + */ +public class ClassWithOnlyPrivateConstructorsInspection extends ClassWithOnlyPrivateConstructorsInspectionBase { + + @Nullable + @Override + protected InspectionGadgetsFix buildFix(Object... infos) { + return new MakeClassFinalFix((PsiClass)infos[0]); + } +} diff --git a/plugins/InspectionGadgets/src/inspectionDescriptions/ClassWithOnlyPrivateConstructors.html b/plugins/InspectionGadgets/src/inspectionDescriptions/ClassWithOnlyPrivateConstructors.html new file mode 100644 index 000000000000..87122815ae53 --- /dev/null +++ b/plugins/InspectionGadgets/src/inspectionDescriptions/ClassWithOnlyPrivateConstructors.html @@ -0,0 +1,9 @@ + + +Reports classes with only private constructors that are not extended by any nested class. +Such classes can not be extended and should be declared final. + +

+New in 14 + + \ No newline at end of file diff --git a/plugins/InspectionGadgets/testsrc/com/siyeh/ig/classlayout/ClassWithOnlyPrivateConstructorsInspectionTest.java b/plugins/InspectionGadgets/testsrc/com/siyeh/ig/classlayout/ClassWithOnlyPrivateConstructorsInspectionTest.java new file mode 100644 index 000000000000..a177242537d2 --- /dev/null +++ b/plugins/InspectionGadgets/testsrc/com/siyeh/ig/classlayout/ClassWithOnlyPrivateConstructorsInspectionTest.java @@ -0,0 +1,49 @@ +/* + * Copyright 2000-2014 JetBrains s.r.o. + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ +package com.siyeh.ig.classlayout; + +import com.intellij.codeInspection.InspectionProfileEntry; +import com.siyeh.ig.LightInspectionTestCase; +import org.jetbrains.annotations.Nullable; + +public class ClassWithOnlyPrivateConstructorsInspectionTest extends LightInspectionTestCase { + + public void testSimple() { + doTest("class /*Class 'X' with only 'private' constructors should be declared 'final'*/X/**/ {" + + " private X() {}" + + " private X(int i) {}" + + "}"); + } + + public void testExtendingInnerClass() { + doTest("class X {\n" + + " private X() {}\n" + + " class Y {\n" + + " class Z extends X{}\n" + + " }\n" + + "}"); + } + + public void testNoConstructors() { + doTest("class X {}"); + } + + @Nullable + @Override + protected InspectionProfileEntry getInspection() { + return new ClassWithOnlyPrivateConstructorsInspection(); + } +} \ No newline at end of file